Factor out Hypervisor compute-service enabling/disabling to own controller#328
Factor out Hypervisor compute-service enabling/disabling to own controller#328fwiesel wants to merge 2 commits into
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
4c09880 to
9d785cb
Compare
332a5dc to
6b3ca9d
Compare
The instance ha controller will eventually move out to the instance-ha service itself. But for now, we keep it as a stand in to reflect the spec value over the https api.
6b3ca9d to
8a3eb91
Compare
|
Merging this branch will increase overall coverage
Coverage by fileChanged files (no unit tests)
Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code. Changed unit test files
|
Extracts the Nova compute-service enable/disable logic from
HypervisorMaintenanceControllerinto a dedicatedHypervisorComputeServiceController. This separates the concerns of HA-state management (maintenance) from compute-service lifecycle.Includes full unit tests for the new controller.
Depends on:
ssa(#327).