Skip UUID resolution for image URLs in conductor config (#1542)

The conductor configuration now preserves image URLs instead of attempting
to resolve them to OpenStack image UUIDs. This allows using external image
sources like HTTP URLs for Ironic deployments.

- Add validators dependency for robust URL/UUID validation
- Replace custom is_uuid() function with validators.uuid()
- Skip image resolution when value is already a URL or UUID
- Affects image_source, deploy_kernel, and deploy_ramdisk parameters

AI-assisted: Claude Code

Signed-off-by: Christian Berendt <[email protected]>

Author: berendt

Pipfile

@@ -35,4 +35,5 @@ setuptools = "==80.9.0"
 sqlmodel = "==0.0.24"
 sushy = "==5.6.0"
 transitions = "==0.9.2"
+validators = "==0.35.0"
 watchdog = "==6.0.0"

Pipfile.lock

@@ -1,22 +1,12 @@
 # SPDX-License-Identifier: Apache-2.0
 
-import uuid
-
 from loguru import logger
+import validators
 import yaml
 
 from osism.tasks import Config, openstack
 
 
-def is_uuid(value):
-    """Check if a string is a valid UUID."""
-    try:
-        uuid.UUID(value)
-        return True
-    except (ValueError, AttributeError):
-        return False
-
-
 def get_configuration():
     with open("/etc/conductor.yml") as fp:
         configuration = yaml.load(fp, Loader=yaml.SafeLoader)
@@ -39,7 +29,9 @@ def get_configuration():
                 image_source = configuration["ironic_parameters"]["instance_info"][
                     "image_source"
                 ]
-                if not is_uuid(image_source):
+                if not validators.uuid(image_source) and not validators.url(
+                    image_source
+                ):
                     result = openstack.image_get(image_source)
                     if result:
                         configuration["ironic_parameters"]["instance_info"][
@@ -53,7 +45,9 @@ def get_configuration():
                 deploy_kernel = configuration["ironic_parameters"]["driver_info"][
                     "deploy_kernel"
                 ]
-                if not is_uuid(deploy_kernel):
+                if not validators.uuid(deploy_kernel) and not validators.url(
+                    deploy_kernel
+                ):
                     result = openstack.image_get(deploy_kernel)
                     if result:
                         configuration["ironic_parameters"]["driver_info"][
@@ -68,7 +62,9 @@ def get_configuration():
                 deploy_ramdisk = configuration["ironic_parameters"]["driver_info"][
                     "deploy_ramdisk"
                 ]
-                if not is_uuid(deploy_ramdisk):
+                if not validators.uuid(deploy_ramdisk) and not validators.url(
+                    deploy_ramdisk
+                ):
                     result = openstack.image_get(deploy_ramdisk)
                     if result:
                         configuration["ironic_parameters"]["driver_info"][

osism/tasks/conductor/config.py

@@ -30,4 +30,5 @@ sushy==5.6.0
 tabulate==0.9.0
 transitions==0.9.2
 uvicorn[standard]==0.35.0
+validators==0.35.0
 watchdog==6.0.0