Skip to content

Commit

Permalink
tempest: Deduce min_compute_nodes from Juju model.
Browse files Browse the repository at this point in the history
At present this value is hard coded, which may cause test failures
on deployment with different number of compute nodes.

Signed-off-by: Frode Nordahl <[email protected]>
  • Loading branch information
fnordahl committed Oct 11, 2024
1 parent 115c4e7 commit 9511aae
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion zaza/openstack/charm_tests/tempest/templates/tempest_v3.j2
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ image_ref = {{ image_id }}
image_ref_alt = {{ image_alt_id }}
flavor_ref = {{ flavor_ref }}
flavor_ref_alt = {{ flavor_ref_alt }}
min_compute_nodes = 3
min_compute_nodes = {{ min_compute_nodes }}

# TODO: review this as its release specific
# min_microversion = 2.2
Expand Down
1 change: 1 addition & 0 deletions zaza/openstack/charm_tests/tempest/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -241,6 +241,7 @@ def _add_nova_config(ctxt, keystone_session, missing_fatal=True):
ctxt['flavor_ref'] = flavor.id
if flavor.name == TEMPEST_ALT_FLAVOR_NAME:
ctxt['flavor_ref_alt'] = flavor.id
ctxt['min_compute_nodes'] = len(model.get_units('nova-compute'))


def _add_neutron_config(ctxt, keystone_session, missing_fatal=True):
Expand Down

0 comments on commit 9511aae

Please sign in to comment.