Merge "Vmedia advanced ops: Clarify config drive and don't ping after rebuild"
diff --git a/ironic_tempest_plugin/config.py b/ironic_tempest_plugin/config.py
index 5a71b84..a728bef 100644
--- a/ironic_tempest_plugin/config.py
+++ b/ironic_tempest_plugin/config.py
@@ -242,7 +242,9 @@
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."),
diff --git a/ironic_tempest_plugin/tests/scenario/ironic_standalone/test_advanced_ops.py b/ironic_tempest_plugin/tests/scenario/ironic_standalone/test_advanced_ops.py
index 7db9ae1..0a7116a 100644
--- a/ironic_tempest_plugin/tests/scenario/ironic_standalone/test_advanced_ops.py
+++ b/ironic_tempest_plugin/tests/scenario/ironic_standalone/test_advanced_ops.py
@@ -137,8 +137,9 @@
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)