Inventory collection default for infra manager #15082
Conversation
| end | ||
|
|
||
| def clusters(extra_attributes = {}) | ||
| attributes = { |
There was a problem hiding this comment.
in the parser you use uid_ems as manager ref, so in this case you should have here
:manager_ref => [:uid_ems]
| def clusters(extra_attributes = {}) | ||
| attributes = { | ||
| :model_class => ::EmsCluster, | ||
| :attributes_blacklist => [:ems_children] |
There was a problem hiding this comment.
missing :association => :clusters
pkliczewski
force-pushed
the
master
branch
2 times, most recently
from
b5759d7 to
372c9a3
Compare
|
Checked commit pkliczewski@3498e00 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0 |
|
This PR provides not finished default manager but it enables ManageIQ/manageiq-providers-ovirt#35 to run so it would be great to have it in as is and improve as we implement full refresh. |
pkliczewski
changed the title
|
@matobet please take a look |
|
@agrare should be ready to merge, there are few TODOs, but it's best to do them incrementally. @pkliczewski is focusing on having targeted refresh of the Vm done first. |
Infra manager containing defaults for inventory collection. Needed for:
ManageIQ/manageiq-providers-ovirt#35