diff options
author | Thiago Paiva <thiagop@lsd.ufcg.edu.br> | 2016-08-15 15:23:30 -0300 |
---|---|---|
committer | Jim Rollenhagen <jim@jimrollenhagen.com> | 2016-11-15 11:52:05 -0500 |
commit | fac0a5fa8755d3dd5b7c59780f80b52c43b24c90 (patch) | |
tree | 92323782751fbf464310d0d9931f5b404ee77117 | |
parent | 3a1b3e44b0aae27130f9321fce37544d5cd0e778 (diff) | |
download | ironic-fac0a5fa8755d3dd5b7c59780f80b52c43b24c90.tar.gz |
Bring configurations from tempest to ironic_tempest_plugin
To remove our tests from tempest first we need to bring the configurations that
we need to the tempest plugin.
Closes-Bug: #1614516
Change-Id: Iad6e16c28cf2a35cc6bf040f57224d32c085b7bc
(cherry picked from commit ab6f73f7a06a9c9e497762e1dc917a9626cccd26)
-rw-r--r-- | devstack/lib/ironic | 3 | ||||
-rw-r--r-- | ironic_tempest_plugin/clients.py | 8 | ||||
-rw-r--r-- | ironic_tempest_plugin/config.py | 60 | ||||
-rw-r--r-- | ironic_tempest_plugin/tests/scenario/baremetal_manager.py | 4 | ||||
-rw-r--r-- | ironic_tempest_plugin/tests/scenario/test_baremetal_basic_ops.py | 1 |
5 files changed, 40 insertions, 36 deletions
diff --git a/devstack/lib/ironic b/devstack/lib/ironic index 84880b741..3be88cb80 100644 --- a/devstack/lib/ironic +++ b/devstack/lib/ironic @@ -1026,6 +1026,9 @@ function ironic_configure_tempest { die_if_not_set $LINENO bm_flavor_id "Failed to get id of baremetal flavor" iniset $TEMPEST_CONFIG compute flavor_ref $bm_flavor_id iniset $TEMPEST_CONFIG compute flavor_ref_alt $bm_flavor_id + + iniset $TEMPEST_CONFIG auth create_isolated_networks False + iniset $TEMPEST_CONFIG network-feature-enabled port_admin_state_change False } # Restore xtrace + pipefail diff --git a/ironic_tempest_plugin/clients.py b/ironic_tempest_plugin/clients.py index 3a098eecd..051bfe9ea 100644 --- a/ironic_tempest_plugin/clients.py +++ b/ironic_tempest_plugin/clients.py @@ -31,9 +31,15 @@ class Manager(clients.Manager): service=None, api_microversions=None): super(Manager, self).__init__(credentials, service) + default_params_with_timeout_values = { + 'build_interval': CONF.compute.build_interval, + 'build_timeout': CONF.compute.build_timeout + } + default_params_with_timeout_values.update(self.default_params) + self.baremetal_client = BaremetalClient( self.auth_provider, CONF.baremetal.catalog_type, CONF.identity.region, endpoint_type=CONF.baremetal.endpoint_type, - **self.default_params_with_timeout_values) + **default_params_with_timeout_values) diff --git a/ironic_tempest_plugin/config.py b/ironic_tempest_plugin/config.py index 1f9ba51a5..5fc1062e5 100644 --- a/ironic_tempest_plugin/config.py +++ b/ironic_tempest_plugin/config.py @@ -28,42 +28,36 @@ baremetal_group = cfg.OptGroup(name='baremetal', 'live_migration, pause, rescue, resize, ' 'shelve, snapshot, and suspend') -# NOTE(maurosr): Until liberty-eol we need to keep config options and tests -# on tempest's tree to test stable branches and thus we have to comment the -# options bellow to avoid duplication. Only new options should live here. BaremetalGroup = [ - # cfg.StrOpt('catalog_type', - # default='baremetal', - # help="Catalog type of the baremetal provisioning service"), - # cfg.BoolOpt('driver_enabled', - # default=True, - # help="Whether the Ironic nova-compute driver is enabled"), - # cfg.StrOpt('driver', - # default='fake', - # help="Driver name which Ironic uses"), - # cfg.StrOpt('endpoint_type', - # default='publicURL', - # choices=['public', 'admin', 'internal', - # 'publicURL', 'adminURL', 'internalURL'], - # help="The endpoint type to use for the baremetal provisioning" - # " service"), + cfg.StrOpt('catalog_type', + default='baremetal', + help="Catalog type of the baremetal provisioning service"), + cfg.StrOpt('driver', + default='fake', + help="Driver name which Ironic uses"), + cfg.StrOpt('endpoint_type', + default='publicURL', + choices=['public', 'admin', 'internal', + 'publicURL', 'adminURL', 'internalURL'], + help="The endpoint type to use for the baremetal provisioning" + " service"), cfg.IntOpt('deploywait_timeout', default=15, help="Timeout for Ironic node to reach the " "wait-callback state after powering on."), - # cfg.IntOpt('active_timeout', - # default=300, - # help="Timeout for Ironic node to completely provision"), - # cfg.IntOpt('association_timeout', - # default=30, - # help="Timeout for association of Nova instance and Ironic " - # "node"), - # cfg.IntOpt('power_timeout', - # default=60, - # help="Timeout for Ironic power transitions."), - # cfg.IntOpt('unprovision_timeout', - # default=300, - # help="Timeout for unprovisioning an Ironic node. " - # "Takes longer since Kilo as Ironic performs an extra " - # "step in Node cleaning.") + cfg.IntOpt('active_timeout', + default=300, + help="Timeout for Ironic node to completely provision"), + cfg.IntOpt('association_timeout', + default=30, + help="Timeout for association of Nova instance and Ironic " + "node"), + cfg.IntOpt('power_timeout', + default=60, + help="Timeout for Ironic power transitions."), + cfg.IntOpt('unprovision_timeout', + default=300, + help="Timeout for unprovisioning an Ironic node. " + "Takes longer since Kilo as Ironic performs an extra " + "step in Node cleaning."), ] diff --git a/ironic_tempest_plugin/tests/scenario/baremetal_manager.py b/ironic_tempest_plugin/tests/scenario/baremetal_manager.py index 1f4d78f51..8fe95784a 100644 --- a/ironic_tempest_plugin/tests/scenario/baremetal_manager.py +++ b/ironic_tempest_plugin/tests/scenario/baremetal_manager.py @@ -56,8 +56,8 @@ class BaremetalScenarioTest(manager.ScenarioTest): @classmethod def skip_checks(cls): super(BaremetalScenarioTest, cls).skip_checks() - if not CONF.baremetal.driver_enabled: - msg = 'Ironic not available or Ironic compute driver not enabled' + if CONF.network_feature_enabled.port_admin_state_change: + msg = "Port state change feature isn't supported by Ironic." raise cls.skipException(msg) @classmethod diff --git a/ironic_tempest_plugin/tests/scenario/test_baremetal_basic_ops.py b/ironic_tempest_plugin/tests/scenario/test_baremetal_basic_ops.py index 0564c1049..63831ce07 100644 --- a/ironic_tempest_plugin/tests/scenario/test_baremetal_basic_ops.py +++ b/ironic_tempest_plugin/tests/scenario/test_baremetal_basic_ops.py @@ -42,6 +42,7 @@ class BaremetalBasicOps(baremetal_manager.BaremetalScenarioTest): * Monitors the associated Ironic node for power and expected state transitions """ + def rebuild_instance(self, preserve_ephemeral=False): self.rebuild_server(server_id=self.instance['id'], preserve_ephemeral=preserve_ephemeral, |