Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Watcher zed #1178

Open
wants to merge 9 commits into
base: stable/zed
Choose a base branch
from
Open

Watcher zed #1178

wants to merge 9 commits into from

Conversation

wolsen
Copy link
Contributor

@wolsen wolsen commented Jan 4, 2024 •
edited
Loading

freyes added 9 commits January 4, 2024 15:47
@freyes @wolsen
Add Watcher client support
32308a1 
Add get_watcher_session_client() helper function to build a
watcherclient.v1.Client instance authenticated with a keystone session
that uses admin credentials by default.

(cherry picked from commit fd82476)
(cherry picked from commit 03875e1)
@freyes @wolsen
Use None as get_nova_session_client() version default
3d17c1e 
This allows callers to pass None and let get_nova_session_client() to
use a sane default API, specifically this allows intermediate users
(e.g. launch_guest() ) to proxy values passed by the caller.

(cherry picked from commit 3283ed4)
(cherry picked from commit dd72af4)
@freyes @wolsen
launch_instance: expose nova api version and target host
e0cf425 
This change exposes two new parameters in the launch_instance()
function:

- nova_api_version: Set the microversion the novaclient should use.
- host: Request to launch the instance on a specific hypervisor host.

(cherry picked from commit be6ba2a)
(cherry picked from commit 41d4e42)
@freyes @wolsen
resource_reaches_status: add new parameter stop_status
cfc0fb0 
The stop_status parameter allows callers to ask stop retrying based on a
list of statuses that are known to be final (and error) states, this
saves time failing earlier.

Usage example for fail early when an instance reaches to ERROR status:

    openstack_utils.resource_reaches_status(self.nova_client.servers,
                                            instance_uuid,
                                            resource_attribute='state',
                                            expected_status='ACTIVE',
                                            stop_status='ERROR')

(cherry picked from commit 11c3f80)
(cherry picked from commit b7c38ed)
@freyes @wolsen
Log available attributes on AttributeError
b129d57 
This simplifies when developing new tests and the programmer is not too
familiar with the different attributes an object has.

(cherry picked from commit 55b3145)
(cherry picked from commit 42e9404)
@freyes @wolsen
Add watcher test.
8d600ff 
This test launches 1 instance per hypervisor, and then launches a new
audit to optimize the use of hypervisors and consolidate the instances
in a single hypervisor, but also disabling the nova-compute service to
avoid new instances get allocated.

(cherry picked from commit 6787c84)
(cherry picked from commit 35741f1)
@freyes @wolsen
Add 'watcher' to tempest list of services
88f07b1 
(cherry picked from commit 1d42b64)
(cherry picked from commit 9771dd2)
@freyes @wolsen
Fix version and nova_api_version data types declared in docstring
72457ba 
(cherry picked from commit 4491642)
(cherry picked from commit b2a8f5a)
@freyes @wolsen
Retry check to avoid live migration unstability.
851ac24 
(cherry picked from commit b087c0c)
(cherry picked from commit e093e1b)
openstack-mirroring pushed a commit to openstack/charm-watcher that referenced this pull request Apr 3, 2024
@wolsen @freyes
Backport watcher support to zed release.
3610c78 
Backport the watcher support to zed. This involves the following:

* Update .gitreview file
* Update osci jobs in osci.yaml
* Update src/metadata.yaml to specify jammy/kinetic support, fix
  descriptions
* Update src/test-requirments.yaml for stable/zed zaza branches
* Update test bundles to use zed
* Add override to specify the common cpu-model for migrations

Func-Test-Pr: openstack-charmers/zaza-openstack-tests#1178
Change-Id: Ie9ae372b63d17c7b9087ebd7674fa58e7bc8f634
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@wolsen @freyes