search

ManageIQ / manageiq-providers-kubevirt

ManageIQ plugin for the Kubevirt provider.
https://kubevirt.io/
Apache License 2.0
4 stars 29 forks source link

Fix parent_manager class name #240

Closed agrare closed 3 months ago

agrare commented 3 months ago

The parent_manager class had Kubernetes::ContainerManager specified which would result in parent_manager being nil when kubevirt was added to an Openshift::ContainerManager due to ActAsStiLeafClass.

Since the base ContainerManager doesn't have an infra_manager we have to drop the inverse_of.

Fixes https://github.com/ManageIQ/manageiq-providers-kubevirt/issues/239

miq-bot commented 3 months ago

Checked commit https://github.com/agrare/manageiq-providers-kubevirt/commit/301726d03f77daa644ea957c35cc78ef499927e2 with ruby 3.1.5, rubocop 1.56.3, haml-lint 0.51.0, and yamllint 1 file checked, 0 offenses detected Everything looks fine. :trophy:

Fryguy commented 2 months ago

Backported to radjabov via merge of master into radjabov branch