From 91f6219d302a9a04d0ad70e7745d1d8ee04edffb Mon Sep 17 00:00:00 2001
From: sean <sean.berry@cudoventures.com>
Date: Fri, 4 Oct 2024 12:55:34 +0100
Subject: [PATCH] possibly better

---
 helpers/cudo_api.py | 62 +++++++++++++++++----------------------------
 1 file changed, 23 insertions(+), 39 deletions(-)

diff --git a/helpers/cudo_api.py b/helpers/cudo_api.py
index 231987d..6e7f59d 100644
--- a/helpers/cudo_api.py
+++ b/helpers/cudo_api.py
@@ -1,3 +1,5 @@
+from asyncio import timeout
+
 import cudo_compute as cudo
 import os
 from time import sleep
@@ -7,6 +9,9 @@
 import atexit
 import threading
 
+from cudo_compute.models.create_vm_response import CreateVMResponse
+from cudo_compute.models.vm import VM
+
 home = os.path.expanduser("~")
 
 
@@ -68,45 +73,31 @@ def project_id_throwable():
 
 
 # APIs
+c, err = client()
+if err:
+    raise Exception(err)
+
 def api_keys():
-    c, err = client()
-    if err:
-        raise Exception(err)
     return cudo.APIKeysApi(c)
 
 
 def disks():
-    c, err = client()
-    if err:
-        raise Exception(err)
     return cudo.DisksApi(c)
 
 
 def networks():
-    c, err = client()
-    if err:
-        raise Exception(err)
     return cudo.NetworksApi(c)
 
 
 def object_storage():
-    c, err = client()
-    if err:
-        raise Exception(err)
     return cudo.ObjectStorageApi(c)
 
 
 def permissions():
-    c, err = client()
-    if err:
-        raise Exception(err)
     return cudo.PermissionsApi(c)
 
 
 def projects():
-    c, err = client()
-    if err:
-        raise Exception(err)
     return cudo.ProjectsApi(c)
 
 
@@ -118,16 +109,10 @@ def ssh_keys():
 
 
 def search():
-    c, err = client()
-    if err:
-        raise Exception(err)
     return cudo.SearchApi(c)
 
 
 def user():
-    c, err = client()
-    if err:
-        raise Exception(err)
     return cudo.UserApi(c)
 
 
@@ -145,13 +130,14 @@ def create_vm(self, project_id, create_vm_body, **kwargs):
         self.start_queue()
         self.task_queue.put((project_id, create_vm_body))
         self.start_workers()
-        return {"id": create_vm_body.vm_id}
+        return CreateVMResponse(id=create_vm_body.vm_id, vm=VM())
 
     def worker(self):
         while self.workers_active:
             if not self.task_queue:
                 break
-            req = self.task_queue.get() # block true, timeout none
+            req = self.task_queue.get(timeout=1)
+            print(req)
             create_vm_body = None
             try:
                 project, create_vm_body = req
@@ -164,6 +150,7 @@ def worker(self):
                     print(f"Error creating VM: {e}")
 
             self.task_queue.task_done()
+            print("Task done")
 
     def start_queue(self):
         if not self.task_queue:
@@ -178,23 +165,20 @@ def start_workers(self):
                 self.executor.submit(self.worker)
 
     def stop_workers(self):
-        if not self.shutdown_event.is_set():
-            try:
-                self.workers_active = False
-                self.shutdown_event.set()
+        print("stop workers")
+        # if not self.shutdown_event.is_set():
+        #     try:
+        #         self.workers_active = False
+        #         self.shutdown_event.set()
+        #
+        #         self.executor.shutdown(wait=False)
+        #
+        #     except Exception as e:
+        #         print(f"Error shutting down: {e}")
 
-                if self.executor:
-                    self.executor.shutdown(wait=False)
-
-            except Exception as e:
-                print(f"Error shutting down: {e}")
 
 
-c, err = client()
-if err:
-    raise Exception(err)
 pool = PooledVirtualMachinesApi(c)
 
-
 def virtual_machines():
     return pool