Drop AQ manage step

DavidFair · DavidFair · commit 8acc703e25cf · 2023-07-04T11:31:52.000+01:00
The AQ Make step implies all the details, such as arch / personality /
sandbox or domain. Doing a manage step after is redundant and caused us
to revert back to the domain type instead of the sandbox
diff --git a/OpenStack-Rabbit-Consumer/rabbit_consumer/message_consumer.py b/OpenStack-Rabbit-Consumer/rabbit_consumer/message_consumer.py
@@ -160,8 +160,6 @@ def handle_create_machine(rabbit_message: RabbitMessage) -> None:
 
     # Manage host in Aquilon
     aq_api.create_host(image_meta, network_details, machine_name)
-
-    aq_api.aq_manage(network_details, image_meta)
     aq_api.aq_make(network_details, image_meta)
 
     add_aq_details_to_metadata(vm_data, network_details)
diff --git a/OpenStack-Rabbit-Consumer/test/test_message_consumer.py b/OpenStack-Rabbit-Consumer/test/test_message_consumer.py
@@ -248,7 +248,6 @@ def test_consume_create_machine_hostnames_good_path(
     aq_api.create_host.assert_called_once_with(
         image_metadata, network_details, machine_name
     )
-    aq_api.aq_manage.assert_called_once_with(network_details, image_metadata)
     aq_api.aq_make.assert_called_once_with(network_details, image_metadata)
 
     # Metadata

Original file line number	Diff line number	Diff line change
`@@ -248,7 +248,6 @@ def test_consume_create_machine_hostnames_good_path(`
`248`	`248`	`aq_api.create_host.assert_called_once_with(`
`249`	`249`	`image_metadata, network_details, machine_name`
`250`	`250`	`)`
`251`		`- aq_api.aq_manage.assert_called_once_with(network_details, image_metadata)`
`252`	`251`	`aq_api.aq_make.assert_called_once_with(network_details, image_metadata)`
`253`	`252`
`254`	`253`	`# Metadata`