-
Notifications
You must be signed in to change notification settings - Fork 897
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
Additions for Amazon tag mapping in graph refresh #16734
Conversation
2001fef
to
be7d05c
Compare
This looks great @cben. |
@miq-bot assign @blomquisg |
Checked commits cben/manageiq@ebe2008~...be7d05c with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0 app/models/manageiq/providers/cloud_manager.rb
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great. 👍
Additions for Amazon tag mapping in graph refresh (cherry picked from commit cd17a20) https://bugzilla.redhat.com/show_bug.cgi?id=1531291
Gaprindashvili backport details:
|
ManageIQ#16734 plus additional methods required
For ManageIQ/manageiq-providers-amazon#382
cc @juliancheal @Ladas
https://bugzilla.redhat.com/show_bug.cgi?id=1506404
ems.vm_and_template_taggings
resulting query: