Merge pull request #323 from rackerlabs/fix_dedault_token

fix: Moving default token filename outside of class

python/neutron-understack/neutron_understack/argo/workflows.py

@@ -6,25 +6,26 @@
 urllib3.disable_warnings()
 
 
-class ArgoClient:
-    DEFAULT_TOKEN_FILENAME = "/run/secrets/kubernetes.io/serviceaccount/token"
+DEFAULT_TOKEN_FILENAME = "/run/secrets/kubernetes.io/serviceaccount/token"
+
 
+class ArgoClient:
     def __init__(
         self,
-        token: str = None,
+        token: str | None = None,
         namespace="default",
         api_url="https://argo-server.argo.svc.cluster.local:2746",
         logger=None,
     ):
+        if token is None:
+            with open(DEFAULT_TOKEN_FILENAME) as token_file:
+                token = token_file.read()
         self.token = token
         self.namespace = namespace
         self.api_url = api_url
         self.headers = {"Authorization": f"Bearer {self.token}"}
         self.logger = logger
 
-        if token is None:
-            with open(DEFAULT_TOKEN_FILENAME) as token_file:
-                token = token_file.read()
 
     def submit(
         self,