fix(k8sworker): tags

Signed-off-by: Chris Snow <[email protected]>

Author: snowch

hpecp/cli/k8sworker.py

@@ -55,7 +55,7 @@ def create_with_ssh_key(
         ip=None,
         ssh_key=None,
         ssh_key_file=None,
-        tags=[],
+        tags=None,
         ephemeral_disks=None,
         persistent_disks=None,
         wait_for_operation_secs=0,
@@ -73,7 +73,7 @@ def create_with_ssh_key(
         ssh_key_file : str, optional
             The SSH key file path, by default None
         tags : list, optional
-            Tags to use, e.g. "{ "tag1": "foo", "tag2": "bar"}", by default []
+            Tags to use, e.g. /api/v2/tag/1:foo,/api/v2/tag/1:bar, by default None
         ephemeral_disks : str
             Comma separated string containing ephemeral disks.
             e.g: "/dev/nvme2n1,/dev/nvme2n2"
@@ -123,10 +123,14 @@ def create_with_ssh_key(
             )
             sys.exit(1)
 
+        tags_parsed=[]
+        if tags is not None:
+            tags_parsed=[dict(item.split(":") for item in tags.split(','))]
+
         worker_id = base.get_client().k8s_worker.create_with_ssh_key(
             ip=ip,
             ssh_key_data=ssh_key,
-            tags=tags,
+            tags=tags_parsed,
         )
 
         if wait_for_operation_secs > 0:

tests/k8s_worker_test.py

@@ -173,7 +173,7 @@ def test_key_or_keycontent_provided(
         self.assertEqual(cm.exception.code, 1)
 
         actual_err = self.err.getvalue().strip()
-        expected_err = "Either ssh_key or ssh_key_file must be provided"
+        expected_err = "At least one of ssh_key or ssh_key_file must be provided"
 
         self.assertEqual(self.out.getvalue(), "", "stdout should be empty")
 
@@ -197,7 +197,7 @@ def test_key_and_keycontent_provided(
         self.assertEqual(cm.exception.code, 1)
 
         actual_err = self.err.getvalue().strip()
-        expected_err = "Either ssh_key or ssh_key_file must be provided"
+        expected_err = "Either ssh_key or ssh_key_file must be provided, but not both."
 
         self.assertEqual(self.out.getvalue(), "", "stdout should be empty")