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

Event switchboard entries for Amazon EBS events. #15500

Closed

Conversation

bronaghs
Copy link

@bronaghs bronaghs commented Jul 3, 2017

This PR will configure entries in the event switchboard for AWS EBS events. The specific events are listed below and will trigger an EMS refresh to keep the AWS inventory up to date.

Links:

https://bugzilla.redhat.com/show_bug.cgi?id=1449235
https://www.pivotaltracker.com/story/show/146650341

Testing:

  • Add an AWS provider
  • Take note of the number of EBS cloud volumes
  • Add a new EBS cloud volume through the AWS Console
  • Ensure the new EBS cloud volume appears under: Storage / Block Storage / Volumes. This will indicate an event was received and went through the event switchboard where an EMS refresh was kicked off.
  • Follow the same steps for:
  • DeleteVolume
  • AttachVolume
  • CreateSnapshot
  • DeleteSnapshot
  • CopySnapshot

@bronaghs
Copy link
Author

bronaghs commented Jul 3, 2017

@miq-bot assign @lfu
@miq-bot add_label fine/yes

@bronaghs
Copy link
Author

bronaghs commented Jul 3, 2017

@miq-bot add_label blocker, enhancement

@bronaghs
Copy link
Author

bronaghs commented Jul 3, 2017

The blocker label applies to the Fine release (5.8.1)

@miq-bot
Copy link
Member

miq-bot commented Jul 3, 2017

Checked commit bronaghs@03c675c with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks fine. 🍰

@gmcculloug
Copy link
Member

@bronaghs Modeling changes should be applied to the ManageIQ-content repo.

@@ -0,0 +1,125 @@
---
Copy link
Member

Choose a reason for hiding this comment

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

This file is not needed.

@@ -0,0 +1,10 @@
---
Copy link
Member

Choose a reason for hiding this comment

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

This file is not needed.

@@ -0,0 +1,10 @@
---
Copy link
Member

Choose a reason for hiding this comment

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

Same here.

@@ -0,0 +1,10 @@
---
Copy link
Member

Choose a reason for hiding this comment

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

Same here.

@@ -0,0 +1,13 @@
---
Copy link
Member

Choose a reason for hiding this comment

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

Same here.

@lfu
Copy link
Member

lfu commented Jul 3, 2017

@bronaghs You only need to add those event instance yaml files.

Please export these instance yaml files from automate following steps here: http://talk.manageiq.org/t/creating-a-pull-request-for-enhancements-to-manageiq-automate-domain/1041

As @gmcculloug said, this PR should be moved to manageiq-content repo.

@bronaghs
Copy link
Author

bronaghs commented Jul 3, 2017

Closing. Superseded by ManageIQ/manageiq-content#137 in the manageiq-content repo.

@bronaghs bronaghs closed this Jul 3, 2017
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.

None yet

4 participants