Revert "Add nova-manage ironic-compute-node-move"
This reverts commit 9068db09e4.
Change-Id: Ibf9a426a90c73bfc921f807e617a25647fc49400
This commit is contained in:
Sylvain Bauza
parent
36a5740e2a
commit
3491b945b9
@ -465,47 +465,6 @@ command completed successfully with exit code 0.
|
||||
* - 127
|
||||
- Invalid input was provided.
|
||||
|
||||
db ironic_compute_node_move
|
||||
---------------------------
|
||||
|
||||
.. program:: nova-manage db ironic_compute_node_move
|
||||
|
||||
.. code-block:: shell
|
||||
|
||||
nova-manage db ironic_compute_node_move --ironic-node-uuid <uuid> --destination-host <host>
|
||||
|
||||
Move ironic nodes, along with any associated instances,
|
||||
between nova-compute services.
|
||||
|
||||
This is useful when migrating away from using peer_list and multiple
|
||||
hash ring balanced nova-compute servers to the new ironic shard system.
|
||||
|
||||
First you must turn off the nova-compute service that currently manages
|
||||
the Ironic host. Second you mark that nova-compute service as forced down
|
||||
via the Nova API. Third, you ensure the new nova-compute service is
|
||||
correctly configured to target the appropriate shard (and optionally
|
||||
also a conductor group). Finally, most Ironic nodes should now move to
|
||||
the new service, but any Ironic nodes with instances on them
|
||||
will need to be manually moved to their new Ironic service
|
||||
by using this nova-manage command.
|
||||
|
||||
.. versionadded:: 28.0.0 (2023.2 Bobcat)
|
||||
|
||||
.. rubric:: Options
|
||||
|
||||
.. option:: --ironic-node-uuid <uuid>
|
||||
|
||||
Ironic node uuid to be moved (which is also the Nova compute node uuid
|
||||
and the uuid of corresponding resource provider in Placement).
|
||||
|
||||
The Nova compute service that currently manages this Ironic node
|
||||
must first be marked a "forced down" via the Nova API, in a similar
|
||||
way to a down hypervisor that is about to have its VMs evacuated to
|
||||
a replacement hypervisor.
|
||||
|
||||
.. option:: --destination-host <host>
|
||||
|
||||
Destination ironic nova-compute service CONF.host.
|
||||
|
||||
API Database Commands
|
||||
=====================
|
||||
|
||||
@ -623,47 +623,6 @@ class DbCommands(object):
|
||||
# "there are more migrations, but not completable right now"
|
||||
return ran and 1 or 0
|
||||
|
||||
@args('--ironic-node-uuid', metavar='<uuid>', dest='compute_node_uuid',
|
||||
help='UUID of Ironic node to be moved between services')
|
||||
@args('--destination-host', metavar='<host>',
|
||||
dest='destination_service_host',
|
||||
help='Destination ironic nova-compute service CONF.host')
|
||||
def ironic_compute_node_move(self, compute_node_uuid,
|
||||
destination_service_host):
|
||||
ctxt = context.get_admin_context()
|
||||
|
||||
destination_service = objects.Service.get_by_compute_host(
|
||||
ctxt, destination_service_host)
|
||||
if destination_service.forced_down:
|
||||
raise exception.NovaException(
|
||||
"Destination compute is forced down!")
|
||||
|
||||
target_compute_node = objects.ComputeNode.get_by_uuid(
|
||||
ctxt, compute_node_uuid)
|
||||
source_service = objects.Service.get_by_id(
|
||||
ctxt, target_compute_node.service_id)
|
||||
if not source_service.forced_down:
|
||||
raise exception.NovaException(
|
||||
"Source service is not yet forced down!")
|
||||
|
||||
instances = objects.InstanceList.get_by_host_and_node(
|
||||
ctxt, target_compute_node.host,
|
||||
target_compute_node.hypervisor_hostname)
|
||||
if len(instances) > 1:
|
||||
raise exception.NovaException(
|
||||
"Found an ironic host with more than one instance! "
|
||||
"Please delete all Nova instances that do not match "
|
||||
"the instance uuid recorded on the Ironic node.")
|
||||
|
||||
target_compute_node.service_id = destination_service.id
|
||||
target_compute_node.host = destination_service.host
|
||||
target_compute_node.save()
|
||||
|
||||
for instance in instances:
|
||||
# this is a bit like evacuate, except no need to rebuild
|
||||
instance.host = destination_service.host
|
||||
instance.save()
|
||||
|
||||
|
||||
class ApiDbCommands(object):
|
||||
"""Class for managing the api database."""
|
||||
|
||||
@ -163,9 +163,7 @@ class NovaManageDBIronicTest(test.TestCase):
|
||||
user_id=self.context.user_id,
|
||||
project_id=self.context.project_id,
|
||||
flavor=flavor,
|
||||
node=cn.hypervisor_hostname,
|
||||
host=cn.host,
|
||||
compute_id=cn.id)
|
||||
node=cn.hypervisor_hostname)
|
||||
inst.create()
|
||||
self.insts.append(inst)
|
||||
|
||||
@ -175,57 +173,6 @@ class NovaManageDBIronicTest(test.TestCase):
|
||||
if i.node == self.cn4.hypervisor_hostname]
|
||||
|
||||
|
||||
class TestIronicComputeNodeMove(NovaManageDBIronicTest):
|
||||
"""Functional tests for "nova-manage db ironic_compute_node_move" CLI."""
|
||||
api_major_version = 'v2.1'
|
||||
|
||||
def setUp(self):
|
||||
super(TestIronicComputeNodeMove, self).setUp()
|
||||
self.enforce_fk_constraints()
|
||||
self.cli = manage.DbCommands()
|
||||
self.output = StringIO()
|
||||
self.useFixture(fixtures.MonkeyPatch('sys.stdout', self.output))
|
||||
|
||||
def test_ironic_compute_node_move_success(self):
|
||||
self.service1.forced_down = True
|
||||
self.service1.save()
|
||||
self.assertEqual(self.service1.id, self.cn1.service_id)
|
||||
# move cn1 on service1 to service2
|
||||
node_uuid = self.cn1.uuid
|
||||
dest_host = self.service2.host
|
||||
|
||||
self.commands.ironic_compute_node_move(node_uuid, dest_host)
|
||||
|
||||
# check the compute node got moved to service 2
|
||||
updated_cn1 = objects.ComputeNode.get_by_id(self.context, self.cn1.id)
|
||||
self.assertEqual(self.service2.id, updated_cn1.service_id)
|
||||
self.assertEqual(self.service2.host, updated_cn1.host)
|
||||
# check the instance got moved too
|
||||
updated_instance = objects.Instance.get_by_id(
|
||||
self.context, self.insts[0].id)
|
||||
self.assertEqual(self.service2.host, updated_instance.host)
|
||||
|
||||
def test_ironic_compute_node_move_raise_not_forced_down(self):
|
||||
node_uuid = self.cn1.uuid
|
||||
dest_host = self.service2.host
|
||||
|
||||
self.assertRaises(exception.NovaException,
|
||||
self.commands.ironic_compute_node_move,
|
||||
node_uuid, dest_host)
|
||||
|
||||
def test_ironic_compute_node_move_raise_forced_down(self):
|
||||
self.service1.forced_down = True
|
||||
self.service1.save()
|
||||
self.service2.forced_down = True
|
||||
self.service2.save()
|
||||
node_uuid = self.cn1.uuid
|
||||
dest_host = self.service2.host
|
||||
|
||||
self.assertRaises(exception.NovaException,
|
||||
self.commands.ironic_compute_node_move,
|
||||
node_uuid, dest_host)
|
||||
|
||||
|
||||
class NovaManageCellV2Test(test.TestCase):
|
||||
def setUp(self):
|
||||
super(NovaManageCellV2Test, self).setUp()
|
||||
|
||||
@ -10,8 +10,3 @@ features:
|
||||
which ironic nodes are managed by each nova-compute service.
|
||||
Note that when you use ``[ironic]shard`` the ``[ironic]peer_list``
|
||||
is hard coded to a single nova-compute service.
|
||||
|
||||
There is a new nova-manage command ``db ironic_compute_node_move`` that
|
||||
can be used to move ironic nodes, and the associated instances, between
|
||||
nova-compute services. This is useful when migrating from the legacy
|
||||
hash ring based HA towards the new sharding approach.
|
||||
|
||||
Loading…
Reference in New Issue
Block a user