chore: fix deployment test

Deployment tests were broken due to conflicting strings.
Adding unique ids to the name of the instance, security groups, etc
should solve this.

src/concrete/ml/deployment/deploy_to_aws.py

@@ -14,6 +14,7 @@
 import json
 import subprocess
 import time
+import uuid
 import zipfile
 from contextlib import closing
 from datetime import datetime
@@ -126,7 +127,11 @@ def create_instance(
     with closing(boto3.client("ec2", region_name=region_name)) as client:
         resources = boto3.resource("ec2", region_name=region_name)
         str_now = datetime.now().strftime(DATE_FORMAT)
-        name = f"deploy-cml-{str_now}" if instance_name is None else f"{instance_name}-{str_now}"
+        name = (
+            f"deploy-cml-{str_now}-{uuid.uuid4()}"
+            if instance_name is None
+            else f"{instance_name}-{str_now}"
+        )
 
         # Get VPC
         vpc_id: str = client.describe_vpcs().get("Vpcs", [{}])[0].get("VpcId", "")

tests/deployment/test_deployment.py

@@ -1,6 +1,7 @@
 """Test deployment."""
 import io
 import time
+import uuid
 import warnings
 from pathlib import Path
 
@@ -42,7 +43,7 @@ def test_instance_management():
     with AWSInstance(
         instance_type="t3.nano",
         region_name="eu-west-3",
-        instance_name="cml_test_aws_deploy",
+        instance_name=f"cml_test_aws_deploy-{uuid.uuid4()}",
         verbose=True,
     ) as metadata:
         time.sleep(1)
@@ -94,7 +95,7 @@ def test_deploy(load_data, tmp_path):  # pylint: disable=too-many-locals,too-man
 
     with AWSInstance(
         instance_type="c5.large",
-        instance_name="cml_test_aws_deploy",
+        instance_name=f"cml_test_aws_deploy-{uuid.uuid4()}",
         open_port=5000,
         region_name="eu-west-3",
         verbose=True,