Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 1448811 - Container FailedSync events no longer supported but still present in UI
Container FailedSync events no longer supported but still present in UI
Status: CLOSED ERRATA
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: Control (Show other bugs)
5.8.0
Unspecified Unspecified
medium Severity medium
: GA
: 5.9.0
Assigned To: Greg McCullough
Shalom Naim
container
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2017-05-08 04:57 EDT by Beni Paskin-Cherniavsky
Modified: 2018-04-09 04:51 EDT (History)
6 users (show)

See Also:
Fixed In Version: 5.9.0.1
Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2018-03-01 08:12:04 EST
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: Container Management


Attachments (Terms of Use)


External Trackers
Tracker ID Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2018:0380 normal SHIPPED_LIVE Moderate: Red Hat CloudForms security, bug fix, and enhancement update 2018-03-01 13:37:12 EST

  None (edit)
Description Beni Paskin-Cherniavsky 2017-05-08 04:57:03 EDT
Description of problem:
FailedSync events proved too spammy and were blacklisted on master, 5.8, 5.7 by
https://github.com/ManageIQ/manageiq/pull/14633
However UI still shows them, and lets you create policies reacting to them.  Such policies will never fire because the event is no longer collected.

How reproducible: 100%

Steps to Reproduce:
1. Control -> Explorer -> Policies accordion -> create a Pod Control policy
2. assign Pod Failed Sync event
3. assign an action (eg. Generate Log) to the event
4. create a pod in kubernetes with a non-existant image name

Actual results:
`oc get events` will show a stream of FailedSync events.
Yet the policy will never run, can be seen in CFME's policy.log.

Expected results:
If you can create a policy, it should work.
Assuming we don't want this event, we should remove it from UI.

- I think removing from miq_event_definitions.csv seed won't delete it,
  we'll need a migration.  This means unfixable in 5.7 & 5.8?
- What to do with existing policies?
  A migration would have to unassign the event from existing policies,
  and it can add a note to such policies' notes field.
Comment 2 Federico Simoncelli 2017-05-08 16:44:49 EDT
Beni, implementation-wise let's split this in backportable (remove from UI) to be targeted for 5.8.z and non-backportable that we'll keep for 5.9.

I am assigning this to Zahi because I suppose you haven't started working on this. In case Beni you have a PR feel free to re-assign to you.
Comment 3 Beni Paskin-Cherniavsky 2017-05-09 04:19:44 EDT
haven't started any work.
Comment 5 zakiva 2017-06-07 04:33:10 EDT
(In reply to zakiva from comment #4)
> https://github.com/ManageIQ/manageiq/pull/15089

PR was merged to master. 
I don't think there is a way to remove the event from the UI without a migration, so we may have a problem with backporting this.
Moving to POST.
Comment 14 errata-xmlrpc 2018-03-01 08:12:04 EST
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2018:0380

Note You need to log in before you can comment on or make changes to this bug.