-
Notifications
You must be signed in to change notification settings - Fork 898
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 host_guest_devices association and inv_collection #17505
Conversation
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.
👍
cc @slemrmartin
Remove after ManageIQ/manageiq#17505 is merged
Remove after ManageIQ/manageiq#17505 is merged
Remove after ManageIQ/manageiq#17505 is merged
Remove after ManageIQ/manageiq#17505 is merged
2693c24
to
e556b0a
Compare
@@ -24,6 +24,14 @@ def guest_devices | |||
) | |||
end | |||
|
|||
def host_guest_devices |
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.
if this PR will be merged before https://github.com/ManageIQ/manageiq-providers-vmware/pull/299/files#diff-d1d21103659304033655618595a1b7feR85, it should be removed from vmware's PR
Add the host_guest_devices association and inventory collection for graph refresh.
e556b0a
to
9a5ec30
Compare
ping @blomquisg |
Checked commits agrare/manageiq@b528ade~...9a5ec30 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0 |
@@ -32,6 +40,14 @@ def host_hardwares | |||
) | |||
end | |||
|
|||
def host_system_services |
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.
@agrare 👍 - I'll wait with my PR
Add the host_guest_devices association and inventory collection for
graph refresh.