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 cloud tenant events #59

Merged
merged 1 commit into from
Feb 23, 2017
Merged

Conversation

tzumainn
Copy link
Contributor

@tzumainn
Copy link
Contributor Author

@miq-bot add_label bug

@miq-bot miq-bot added the bug label Feb 23, 2017
@miq-bot
Copy link
Member

miq-bot commented Feb 23, 2017

Checked commit tzumainn@2d86806 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
0 files checked, 0 offenses detected
Everything looks good. 👍

@gmcculloug gmcculloug self-assigned this Feb 23, 2017
@gmcculloug
Copy link
Member

@lfu Please review

@lfu
Copy link
Member

lfu commented Feb 23, 2017

@tzumainn Please make sure the steps in Creating a Pull Request for enhancements to ManageIQ Automate Domain has been followed to export the automate modeling.
Otherwise it looks good to me.

@tzumainn
Copy link
Contributor Author

@lfu Yep! I followed the path in Option 2, and also verified that catching the relevant events caused a refresh to trigger.

@gmcculloug gmcculloug merged commit 5686030 into ManageIQ:master Feb 23, 2017
@gmcculloug gmcculloug added this to the Sprint 55 Ending Feb 27, 2017 milestone Feb 23, 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.

4 participants