fix(compute): reuse an existing node key when registering a Node

Node.insert() unconditionally created a NodeEncryptionKey for its
uuid, so registering a compute Node for a machine that was already a
local hypervisor's node (which provisions its own key the same way)
crashed on the unique node uuid. Route through get_or_create like
MachinePool already does.

Author: VasilyStepanov

exordos_core/tests/functional/restapi/compute/test_hypervisor_api.py

@@ -376,3 +376,50 @@ def test_hypervisors_add_local_hyper_provisions_a_node_key(
         client.delete(
             client.build_resource_uri(["compute", "hypervisors", hypervisor["uuid"]])
         )
+
+    def test_node_reuses_a_local_hyper_pools_node_key(
+        self,
+        pool_factory: tp.Callable,
+        node_factory: tp.Callable,
+        user_api_client: iam_clients.GenesisCoreTestRESTClient,
+        auth_user_admin: iam_clients.GenesisCoreAuth,
+    ):
+        # A machine can be both a local hypervisor's node and a plain
+        # compute Node - both provision a NodeEncryptionKey for the same
+        # uuid, so registering the Node after the hypervisor must reuse
+        # the existing key instead of conflicting on insert.
+        node_uuid = sys_uuid.uuid4()
+        client = user_api_client(auth_user_admin)
+
+        hypervisor = pool_factory(
+            driver_spec={
+                "kind": "exordos_local_hyper",
+                "connection_uri": "qemu:///system",
+                "node": str(node_uuid),
+            },
+        )
+        hypervisor.pop("status", None)
+        response = client.post(
+            client.build_collection_uri(["compute", "hypervisors"]), json=hypervisor
+        )
+        assert response.status_code == 201
+
+        key_before = ua_models.NodeEncryptionKey.objects.get_one(
+            filters={"uuid": dm_filters.EQ(node_uuid)}
+        )
+
+        node = node_factory(uuid=node_uuid)
+        response = client.post(
+            client.build_collection_uri(["compute", "nodes"]), json=node
+        )
+        assert response.status_code == 201
+
+        key_after = ua_models.NodeEncryptionKey.objects.get_one(
+            filters={"uuid": dm_filters.EQ(node_uuid)}
+        )
+        assert key_after.private_key == key_before.private_key
+
+        client.delete(client.build_resource_uri(["compute", "nodes", str(node_uuid)]))
+        client.delete(
+            client.build_resource_uri(["compute", "hypervisors", hypervisor["uuid"]])
+        )