diff options
author | Mark Goddard <mark@stackhpc.com> | 2020-09-30 15:03:36 +0100 |
---|---|---|
committer | Mark Goddard <mark@stackhpc.com> | 2020-09-30 15:03:59 +0100 |
commit | 42d83a2bff1e91abd961c817cd70e0e0ecfc82bc (patch) | |
tree | 466b24c7e1b3b8e11e909602dff099ddbde76e56 /ironic/tests/unit/db/test_api.py | |
parent | 2151d660d2156c5f75986fbebe2fc7f35b0de81b (diff) | |
download | ironic-42d83a2bff1e91abd961c817cd70e0e0ecfc82bc.tar.gz |
Don't migrate away from iscsi if it is the default
We hit an issue in Kayobe CI where we have the following config:
enabled_deploy_interfaces: iscsi,direct
default_deploy_interface: iscsi
In practice, direct does not work out of the box in Kayobe currently.
The online migration added recently to move nodes from iscsi to direct
caused issues in our upgrade jobs, since it migrated nodes to direct,
without the config in place to make this work.
This change avoids migrating nodes when iscsi is the default deploy
interface.
Story: 2008114
Task: 40996
Change-Id: I5cf0684dc53a2d89fac12e578b28e73e7697a97e
Diffstat (limited to 'ironic/tests/unit/db/test_api.py')
-rw-r--r-- | ironic/tests/unit/db/test_api.py | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/ironic/tests/unit/db/test_api.py b/ironic/tests/unit/db/test_api.py index 9252e1d59..701d1afc4 100644 --- a/ironic/tests/unit/db/test_api.py +++ b/ironic/tests/unit/db/test_api.py @@ -290,6 +290,14 @@ class MigrateFromIscsiTestCase(base.DbTestCase): self.assertEqual( (0, 0), self.dbapi.migrate_from_iscsi_deploy(self.context, 0)) + def test_migration_impossible3(self): + self.config(default_deploy_interface='iscsi') + for _i in range(3): + uuid = uuidutils.generate_uuid() + utils.create_test_node(uuid=uuid, deploy_interface='iscsi') + self.assertEqual( + (0, 0), self.dbapi.migrate_from_iscsi_deploy(self.context, 0)) + def test_force_migration(self): self.config(enabled_deploy_interfaces='iscsi') utils.create_test_node(deploy_interface='direct') |