Merge pull request #1246 from deNBI/dev

Dev

.github/workflows/blacked.yml

@@ -17,7 +17,7 @@ jobs:
             - name: Create Pull Request
               if: steps.git-check.outputs.modified == 'true'
               id: cpr
-              uses: peter-evans/[email protected].3
+              uses: peter-evans/[email protected].4
               with:
                token: ${{ secrets.GITHUB_TOKEN }}
                commit-message: fix(Linting):blacked code

Dockerfile

@@ -1,4 +1,4 @@
-FROM python:3.10.7-buster
+FROM python:3.10.8-buster
 RUN  apt-get update -y
 RUN apt-get install -y build-essential
 WORKDIR /code

VirtualMachineService/VirtualMachineHandler.py

@@ -711,7 +711,6 @@ def get_servers_by_ids(self, ids):
     def check_server_task_state(self, openstack_id):
         LOG.info(f"Checking Task State: {openstack_id}")
         server = self.conn.get_server_by_id(openstack_id)
-        LOG.info(server)
         if not server:
             return "No server found"
         task_state = server.get("task_state", None)
@@ -2186,9 +2185,14 @@ def create_snapshot(self, openstack_id, name, elixir_id, base_tags, description)
             )
         )
 
+        server=self.conn.get_server_by_id(openstack_id)
+        LOG.info(server)
+        if server is None:
+            LOG.exception(f"Instance {openstack_id} not found")
+            raise serverNotFoundException
         try:
             snapshot_munch = self.conn.create_image_snapshot(
-                server=openstack_id, name=name
+                server=server, name=name
             )
         except ConflictException as e:
             LOG.exception(f"Create snapshot {openstack_id} error: {e}")