openstack
/
ironic-tempest-plugin
Code Issues Proposed changes

Merge "Vmedia advanced ops: Clarify config drive and don't ping after rebuild"

This commit is contained in:
Zuul 2024-05-15 18:17:24 +00:00 committed by Gerrit Code Review
parent 926db2e70a 56b1631c56
commit 316fed10a2
2 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
ironic_tempest_plugin/config.py
View File

@ -242,7 +242,9 @@ BaremetalGroup = [
default=True,
help="If we should issue a rebuild request when testing "
"dhcpless virtual media deployments. This may be useful "
"if bug 2032377 is not fixed in the agent ramdisk."),
"if bug 2032377 is not fixed in the agent ramdisk. This "
"test is functionally incompatible with Temporary URLs "
"as they may timeout while the test is running."),
cfg.StrOpt("public_subnet_id",
help="The public subnet ID where routers will be bound for "
"testing purposes with the dhcp-less test scenario."),

5
ironic_tempest_plugin/tests/scenario/ironic_standalone/test_advanced_ops.py
View File

@ -137,8 +137,9 @@ class BaremetalRedfishDHCPLessDeploy(bsm.BaremetalStandaloneScenarioTest):
self.wait_provisioning_state(self.node['uuid'], 'active',
timeout=CONF.baremetal.active_timeout,
interval=30)
# Assert we were able to ping after rebuilding.
self.assertTrue(self.ping_ip_address(self.node_ip))
# NOTE(TheJulia): We explicitly don't ping in this test as
# what we are testing is that we are able to return to active.
# i.e. That the agent is operating as designed.
# Force delete so we remove the vifs
self.terminate_node(self.node['uuid'], force_delete=True)