Files
watcher/releasenotes/notes/remove-unused-nova-helper-methods-3ee84f57eaab458d.yaml
jgilaber c90af937fa Remove unused methods from NovaHelper
Remove three methods from the NovaHelper class that were only used in
unit tests:

- get_aggregate_detail
- wait_for_volume_status
- _check_nova_api_version

This cleanup reduces code complexity and maintenance burden by
removing dead code that provides no value to the current codebase.

The corresponding test cases for these methods have also been
removed from test_nova_helper.py, and test_compute.py has been
updated to remove the now-unnecessary mock for get_aggregate_detail.
Unused imports (novaclient.api_versions and novaclient.exceptions)
have been removed from the test file.

Generated-By: claude-code (sonnet-4.5)

Change-Id: Id631997f2da7ff729502c86afe27ec9ee0b62fa5
Signed-off-by: jgilaber <jgilaber@redhat.com>
2026-02-26 11:24:56 +01:00

16 lines
585 B
YAML

---
upgrade:
- |
Three unused methods have been removed from the ``NovaHelper`` class in
``watcher.common.nova_helper``:
* ``get_aggregate_detail``
* ``wait_for_volume_status``
* ``_check_nova_api_version``
These methods were only used in unit tests and provided no value to the
production codebase. If you have custom code or plugins that depend on
these methods, you will need to implement equivalent functionality
directly. The methods were not part of the stable public API and their
removal should not affect normal Watcher operations.