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.

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

Functional tests for instance-count action on nova-compute charm. #535

Merged

Conversation

mkalcok
Copy link
Contributor

@mkalcok mkalcok commented Mar 31, 2021

Test for `remove_from_cloud_actions` waits at the end until
the nova-compute service is available again. However there's a few
seconds lag until that host shows up also in `hypervisor-list` which
can cause race conditions with other tests.
lourot
lourot previously requested changes Apr 7, 2021
Copy link
Contributor

@lourot lourot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great, thanks! I'm just blocking it right now because we're in code freeze but I intend to land it right after freeze.

@ajkavanagh ajkavanagh dismissed lourot’s stale review May 13, 2021 10:47

The reason for holding has now passed; so we can now land this. Thanks @AurelienLourot for holding during freeze.

@ajkavanagh ajkavanagh merged commit 3aacd5c into openstack-charmers:master May 13, 2021
openstack-mirroring pushed a commit to openstack/charm-nova-compute that referenced this pull request May 13, 2021
Closes-Bug: #1911011
Change-Id: If9ae646bc91b6389876830843bbf905596963054
func-test-pr: openstack-charmers/zaza-openstack-tests#535
openstack-mirroring pushed a commit to openstack/openstack that referenced this pull request May 13, 2021
* Update charm-nova-compute from branch 'master'
  to 5eea014dd5e56c3205e1defbdc6253c5fbd8953a
  - Add action that returns number of running VMs on the unit.
    
    Closes-Bug: #1911011
    Change-Id: If9ae646bc91b6389876830843bbf905596963054
    func-test-pr: openstack-charmers/zaza-openstack-tests#535
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants