search

ManageIQ / manageiq-providers-kubevirt

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

Move RefreshMemory under InfraManager #252

Closed agrare closed 1 month ago

agrare commented 1 month ago

Better support subclassed providers (aka Openshift) which have the Kubevirt InfraManager in the same vendor namespace as other manager types like a ContainerManager.

Related:

Fryguy commented 1 month ago

Does this require a PR in the openshift plugin side as well?

agrare commented 1 month ago

Does this require a PR in the openshift plugin side as well?

Yes, I mentioned in this comment that we should move this under the manager namespace and added this PR as a dependency of that PR

Fryguy commented 3 weeks ago

Backported to radjabov in commit 0a7b4bce1b84720693663df07e8f0ad7614cf04c.

commit 0a7b4bce1b84720693663df07e8f0ad7614cf04c
Author: Jason Frey <fryguy9@gmail.com>
Date:   Wed Sep 25 14:22:47 2024 -0400

    Merge pull request #252 from agrare/move_refresh_memory_under_infra_manager

    Move RefreshMemory under InfraManager

    (cherry picked from commit 51ffdf268b30bbf4ab354bb8eb1ed0cced2b10a7)