Add new genesis vars set command to simplify vars managment

The command `genesis vars set` allows to create a couple of var/value
to simplify vars management.

Signed-off-by: Anton Kremenetsky <anton.kremenetsky@gmail.com>

Author: akremenetsky

docs/cli/vars_set.md

@@ -0,0 +1,74 @@
+# vars_set
+
+Create variable if missing and set its value by creating a new value record
+
+## Usage
+
+```console
+Usage: genesis vars set [OPTIONS] VAR_UUID_OR_NAME VALUE
+```
+
+## Options
+
+- `var_uuid_or_name` (REQUIRED):
+    - Type: text
+    - Default: `sentinel.unset`
+    - Usage: `var_uuid_or_name`
+
+- `value` (REQUIRED):
+    - Type: text
+    - Default: `sentinel.unset`
+    - Usage: `value`
+
+- `project_id` (REQUIRED):
+    - Type: uuid
+    - Default: `sentinel.unset`
+    - Usage: `-p, --project-id`
+
+  UUID of the project in which to deploy the variable
+
+- `name`:
+    - Type: text
+    - Default: `none`
+    - Usage: `--name`
+
+  Name of the variable to create if it does not exist
+
+- `description`:
+    - Type: text
+    - Default: `none`
+    - Usage: `--description`
+
+  Description of the variable to create if it does not exist
+
+- `rotate`:
+    - Type: boolean
+    - Default: `false`
+    - Usage: `--rotate`
+
+  Delete all existing values for the variable before creating the new one
+
+- `help`:
+    - Type: boolean
+    - Default: `false`
+    - Usage: `--help`
+
+  Show this message and exit.
+
+## CLI Help
+
+```console
+Usage: genesis vars set [OPTIONS] VAR_UUID_OR_NAME VALUE
+
+  Create variable if missing and set its value by creating a new value record
+
+Options:
+  -p, --project-id UUID  UUID of the project in which to deploy the variable
+                         [required]
+  --name TEXT            Name of the variable to create if it does not exist
+  --description TEXT     Description of the variable to create if it does not
+                         exist
+  --rotate               Delete all existing values for the variable before
+                         creating the new one
+  --help                 Show this message and exit.
+```

genesis_devtools/cmd/secret/certificate/commands.py

@@ -83,7 +83,9 @@ def delete_certificate_cmd(
     certificate_lib.delete_certificate(client, uuid)
 
 
-@certificates_group.command("add", help="Add a new certificate to the Genesis installation")
+@certificates_group.command(
+    "add", help="Add a new certificate to the Genesis installation"
+)
 @click.pass_context
 @click.option(
     "-u",

genesis_devtools/cmd/values/commands.py

@@ -15,7 +15,6 @@
 #    under the License.
 from __future__ import annotations
 
-import json
 import typing as tp
 import uuid as sys_uuid
 
@@ -146,7 +145,7 @@ def add_value_cmd(
         "project_id": str(project_id),
         "name": name,
         "description": description,
-        "value": _convert_to_nearest_type(value),
+        "value": utils.convert_to_nearest_type(value),
     }
 
     # Validate variable UUID if provided
@@ -221,7 +220,7 @@ def update_value_cmd(
     if description is not None:
         data["description"] = description
     if value is not None:
-        data["value"] = _convert_to_nearest_type(value)
+        data["value"] = utils.convert_to_nearest_type(value)
     if variable is not None:
         data["variable"] = variable
     value_resp = value_lib.update_value(client, uuid, data)
@@ -254,44 +253,3 @@ def _print_values(values: tp.List[dict]) -> None:
         )
 
     click.echo(table)
-
-
-def _convert_to_nearest_type(value: str) -> bool | int | float | list | dict | str:
-    """
-    Convert input string to the nearest appropriate type.
-
-    Args:
-        value: String value to convert
-
-    Returns:
-        Converted value in appropriate type (bool, int, float, list, dict, or str)
-    """
-    # Try boolean
-    if value.lower() in ("true", "false"):
-        return value.lower() == "true"
-
-    # Try integer
-    try:
-        return int(value)
-    except ValueError:
-        pass
-
-    # Try float
-    try:
-        return float(value)
-    except ValueError:
-        pass
-
-    # Try list (JSON array or object)
-    if (value.startswith("[") and value.endswith("]")) or (
-        value.startswith("[") and value.endswith("]")
-    ):
-        try:
-            parsed = json.loads(value)
-            if isinstance(parsed, (list, dict)):
-                return parsed
-        except (ValueError, TypeError):
-            pass
-
-    # Return as string if no other type matches
-    return value

genesis_devtools/cmd/vars/commands.py

@@ -20,9 +20,12 @@
 
 import click
 import prettytable
+from bazooka import exceptions as bazooka_exc
 
 from gcl_sdk.clients.http import base as http_client
 
+from genesis_devtools import constants as c
+from genesis_devtools.clients import value as value_lib
 from genesis_devtools.clients import variable as variable_lib
 from genesis_devtools.common import utils
 
@@ -62,6 +65,123 @@ def show_variable_cmd(
     _print_variables([variable])
 
 
+@variables_group.command(
+    "set",
+    help="Create variable if missing and set its value by creating a new value record",
+)
+@click.argument(
+    "var_uuid_or_name",
+    type=str,
+    required=True,
+)
+@click.argument(
+    "value",
+    type=click.STRING,
+    required=True,
+)
+@click.option(
+    "-p",
+    "--project-id",
+    type=click.UUID,
+    required=True,
+    help="UUID of the project in which to deploy the variable",
+)
+@click.option(
+    "--name",
+    type=str,
+    default=None,
+    help="Name of the variable to create if it does not exist",
+)
+@click.option(
+    "--description",
+    type=str,
+    default=None,
+    help="Description of the variable to create if it does not exist",
+)
+@click.option(
+    "--rotate",
+    is_flag=True,
+    default=False,
+    help="Delete all existing values for the variable before creating the new one",
+)
+@click.pass_context
+def set_variable_cmd(
+    ctx: click.Context,
+    var_uuid_or_name: str,
+    value: str,
+    project_id: sys_uuid.UUID | None,
+    name: str | None,
+    description: str | None,
+    rotate: bool,
+) -> None:
+    client: http_client.CollectionBaseClient = ctx.obj.client
+
+    var_uuid: str | None = None
+    variable: dict[str, tp.Any] | None = None
+
+    if utils.is_valid_uuid(var_uuid_or_name):
+        var_uuid = var_uuid_or_name
+        try:
+            variable = variable_lib.get_variable(client, sys_uuid.UUID(var_uuid))
+        except bazooka_exc.NotFoundError as exc:
+            raise click.ClickException(
+                f"Variable with UUID {var_uuid} not found"
+            ) from exc
+    else:
+        variables = variable_lib.list_variables(client, name=var_uuid_or_name)
+        if len(variables) > 1:
+            raise click.ClickException(
+                f"Multiple variables with name {var_uuid_or_name} found; use UUID"
+            )
+        if len(variables) == 1:
+            variable = variables[0]
+            var_uuid = variable["uuid"]
+
+    if variable is None or var_uuid is None:
+        root_ctx = ctx.find_root()
+        project_id = project_id or root_ctx.params.get("project_id")
+        if project_id is None:
+            raise click.ClickException(
+                "Unable to create variable: project id is not set. "
+                "Provide it via '--project-id' (or set 'project_id' in config)."
+            )
+
+        variable_name = name if name is not None else var_uuid_or_name
+        variable_description = description if description is not None else ""
+
+        created = variable_lib.add_variable(
+            client,
+            {
+                "uuid": str(sys_uuid.uuid4()),
+                "project_id": str(project_id),
+                "name": variable_name,
+                "description": variable_description,
+                "setter": {"kind": "selector", "selector_strategy": "latest"},
+            },
+        )
+        variable = created
+        var_uuid = created["uuid"]
+
+    if rotate:
+        variable_ref = f"{c.VARIABLE_COLLECTION}{var_uuid}"
+        existing_values = value_lib.list_values(client, variable=variable_ref)
+        for existing in existing_values:
+            value_lib.delete_value(client, sys_uuid.UUID(existing["uuid"]))
+
+    data = {
+        "uuid": str(sys_uuid.uuid4()),
+        "project_id": variable["project_id"],
+        "name": f"{variable['name']}_value",
+        "description": "",
+        "value": utils.convert_to_nearest_type(value),
+        "variable": f"{c.VARIABLE_COLLECTION}{var_uuid}",
+    }
+    value_lib.add_value(client, data)
+
+    variable = variable_lib.get_variable(client, sys_uuid.UUID(var_uuid))
+    _print_variables([variable])
+
+
 @variables_group.command("delete", help="Delete variable")
 @click.argument(
     "uuid",

genesis_devtools/common/utils.py

@@ -13,7 +13,9 @@
 #    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
 #    License for the specific language governing permissions and limitations
 #    under the License.
+from __future__ import annotations
 
+import json
 import os
 import typing as tp
 import uuid
@@ -32,4 +34,32 @@ def get_project_path() -> str:
     return os.sep.join(__file__.split(os.sep)[:-3])
 
 
+def convert_to_nearest_type(value: str) -> bool | int | float | list | dict | str:
+    lower_value = value.lower()
+    if lower_value in ("true", "false"):
+        return lower_value == "true"
+
+    try:
+        return int(value)
+    except ValueError:
+        pass
+
+    try:
+        return float(value)
+    except ValueError:
+        pass
+
+    if (value.startswith("[") and value.endswith("]")) or (
+        value.startswith("{") and value.endswith("}")
+    ):
+        try:
+            parsed = json.loads(value)
+            if isinstance(parsed, (list, dict)):
+                return parsed
+        except (ValueError, TypeError):
+            pass
+
+    return value
+
+
 PROJECT_PATH = get_project_path()