Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add openstack infra provider event blacklist #14369

Conversation

tzumainn
Copy link
Contributor

@miq-bot
Copy link
Member

miq-bot commented Mar 16, 2017

Checked commit tzumainn@dc9110d with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 0 offenses detected
Everything looks good. 🍪

@@ -64,6 +64,12 @@ def self.description
@description ||= "OpenStack Platform Director".freeze
end

def self.default_blacklisted_event_names
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lfu weren't we capturing blacklisted events elsewhere?

We used to have a list of filtered event types. But, I don't know how that's done now.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@blomquisg It used to be in config/event_handling.tmpl.yml. But this PR has moved them into DB.
Each provider model needs to maintain a list of the filtered events in default_blacklisted_event_names.

@blomquisg blomquisg merged commit 09eecb5 into ManageIQ:master Mar 24, 2017
@blomquisg blomquisg added this to the Sprint 57 Ending Mar 27, 2017 milestone Mar 24, 2017
simaishi pushed a commit that referenced this pull request Apr 13, 2017
…klisted-events

Add openstack infra provider event blacklist
(cherry picked from commit 09eecb5)

https://bugzilla.redhat.com/show_bug.cgi?id=1436237
@simaishi
Copy link
Contributor

Euwe backport details:

$ git log -1
commit b773ba484aef42e982a4148f86152c20f154c434
Author: Greg Blomquist <[email protected]>
Date:   Fri Mar 24 10:15:58 2017 -0400

    Merge pull request #14369 from tzumainn/openstack-infra-provider-blacklisted-events
    
    Add openstack infra provider event blacklist
    (cherry picked from commit 09eecb577667bd5535b74dfa6d490de5944e27cf)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1436237

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants