diff --git a/llama_stack/cli/stack/_build.py b/llama_stack/cli/stack/_build.py index 1b2470918..3887bf4f9 100644 --- a/llama_stack/cli/stack/_build.py +++ b/llama_stack/cli/stack/_build.py @@ -39,7 +39,7 @@ from llama_stack.distribution.resolver import InvalidProviderError from llama_stack.distribution.utils.config_dirs import DISTRIBS_BASE_DIR from llama_stack.distribution.utils.dynamic import instantiate_class_type from llama_stack.distribution.utils.exec import formulate_run_args, run_with_pty -from llama_stack.distribution.utils.image_types import ImageType +from llama_stack.distribution.utils.image_types import LlamaStackImageType from llama_stack.providers.datatypes import Api TEMPLATES_PATH = Path(__file__).parent.parent.parent / "templates" @@ -170,7 +170,7 @@ def run_stack_build_command(args: argparse.Namespace) -> None: ) sys.exit(1) - if build_config.image_type == ImageType.container.value and not args.image_name: + if build_config.image_type == LlamaStackImageType.CONTAINER.value and not args.image_name: cprint( "Please specify --image-name when building a container from a config file", color="red", @@ -226,7 +226,7 @@ def _generate_run_config( """ apis = list(build_config.distribution_spec.providers.keys()) run_config = StackRunConfig( - container_image=(image_name if build_config.image_type == ImageType.container.value else None), + container_image=(image_name if build_config.image_type == LlamaStackImageType.CONTAINER.value else None), image_name=image_name, apis=apis, providers={}, @@ -279,16 +279,16 @@ def _run_stack_build_command_from_build_config( template_name: Optional[str] = None, config_path: Optional[str] = None, ) -> str: - if build_config.image_type == ImageType.container.value: + if build_config.image_type == LlamaStackImageType.CONTAINER.value: if template_name: image_name = f"distribution-{template_name}" else: if not image_name: raise ValueError("Please specify an image name when building a container image without a template") - elif build_config.image_type == ImageType.conda.value: + elif build_config.image_type == LlamaStackImageType.CONDA.value: if not image_name: raise ValueError("Please specify an image name when building a conda image") - elif build_config.image_type == ImageType.venv.value: + elif build_config.image_type == LlamaStackImageType.VENV.value: if not image_name and os.environ.get("UV_SYSTEM_PYTHON"): image_name = "__system__" if not image_name: diff --git a/llama_stack/distribution/build.py b/llama_stack/distribution/build.py index 3d808a4a4..0e990d129 100644 --- a/llama_stack/distribution/build.py +++ b/llama_stack/distribution/build.py @@ -16,7 +16,7 @@ from termcolor import cprint from llama_stack.distribution.datatypes import BuildConfig, Provider from llama_stack.distribution.distribution import get_provider_registry from llama_stack.distribution.utils.exec import run_command, run_with_pty -from llama_stack.distribution.utils.image_types import ImageType +from llama_stack.distribution.utils.image_types import LlamaStackImageType from llama_stack.providers.datatypes import Api log = logging.getLogger(__name__) @@ -95,7 +95,7 @@ def build_image( normal_deps, special_deps = get_provider_dependencies(build_config.distribution_spec.providers) normal_deps += SERVER_DEPENDENCIES - if build_config.image_type == ImageType.container.value: + if build_config.image_type == LlamaStackImageType.CONTAINER.value: script = str(importlib.resources.files("llama_stack") / "distribution/build_container.sh") args = [ script, @@ -104,7 +104,7 @@ def build_image( container_base, " ".join(normal_deps), ] - elif build_config.image_type == ImageType.conda.value: + elif build_config.image_type == LlamaStackImageType.CONDA.value: script = str(importlib.resources.files("llama_stack") / "distribution/build_conda_env.sh") args = [ script, @@ -112,7 +112,7 @@ def build_image( str(build_file_path), " ".join(normal_deps), ] - elif build_config.image_type == ImageType.venv.value: + elif build_config.image_type == LlamaStackImageType.VENV.value: script = str(importlib.resources.files("llama_stack") / "distribution/build_venv.sh") args = [ script, diff --git a/llama_stack/distribution/utils/exec.py b/llama_stack/distribution/utils/exec.py index aae6b35d8..86613dc9c 100644 --- a/llama_stack/distribution/utils/exec.py +++ b/llama_stack/distribution/utils/exec.py @@ -20,14 +20,14 @@ import importlib import json from pathlib import Path -from llama_stack.distribution.utils.image_types import ImageType +from llama_stack.distribution.utils.image_types import LlamaStackImageType def formulate_run_args(image_type, image_name, config, template_name) -> list: env_name = "" - if image_type == ImageType.container.value or config.container_image: + if image_type == LlamaStackImageType.CONTAINER.value or config.container_image: env_name = f"distribution-{template_name}" if template_name else config.container_image - elif image_type == ImageType.conda.value: + elif image_type == LlamaStackImageType.CONDA.value: current_conda_env = os.environ.get("CONDA_DEFAULT_ENV") env_name = image_name or current_conda_env if not env_name: diff --git a/llama_stack/distribution/utils/image_types.py b/llama_stack/distribution/utils/image_types.py index 1a43b092f..403c91ca6 100644 --- a/llama_stack/distribution/utils/image_types.py +++ b/llama_stack/distribution/utils/image_types.py @@ -4,10 +4,10 @@ # This source code is licensed under the terms described in the LICENSE file in # the root directory of this source tree. -from enum import Enum +import enum -class ImageType(Enum): - container = "container" - conda = "conda" - venv = "venv" +class LlamaStackImageType(enum.Enum): + CONTAINER = "container" + CONDA = "conda" + VENV = "venv"