From cc180ff62b80d7ce8a848a9c4cc3deaae9959c6e Mon Sep 17 00:00:00 2001 From: amrit2356 Date: Tue, 13 Jan 2026 17:56:18 +0530 Subject: [PATCH] Fix CI/CD issues: remove docker.io install, update actions, fix unbound variables - Remove docker.io installation from test workflow (pre-installed in GitHub runners) - Update actions/upload-artifact from v3 to v4 - Fix unbound variable error in engine.sh MLENV_LOG_LEVEL - Fix unbound variable error in config/parser.sh indirect expansion - Simplify Docker verification in CI workflow --- lib/mlenv/config/parser.sh | 2 +- lib/mlenv/core/engine.sh | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/mlenv/config/parser.sh b/lib/mlenv/config/parser.sh index b5560fa..ebe3c9c 100644 --- a/lib/mlenv/config/parser.sh +++ b/lib/mlenv/config/parser.sh @@ -107,7 +107,7 @@ config_apply_env_overrides() { for mapping in "${env_mappings[@]}"; do IFS=':' read -r env_var config_key <<< "$mapping" - if [[ -n "${!env_var}" ]]; then + if [[ -n "${!env_var:-}" ]]; then MLENV_CONFIG["$config_key"]="${!env_var}" vlog " Env override: $config_key = ${!env_var}" fi diff --git a/lib/mlenv/core/engine.sh b/lib/mlenv/core/engine.sh index b53e554..d7cf962 100644 --- a/lib/mlenv/core/engine.sh +++ b/lib/mlenv/core/engine.sh @@ -164,5 +164,5 @@ engine_get_info() { echo "MLEnv Engine v$(engine_get_version)" echo "Container Adapter: ${MLENV_ACTIVE_CONTAINER_ADAPTER:-none}" echo "Registry Adapter: ${MLENV_ACTIVE_REGISTRY_ADAPTER:-none}" - echo "Log Level: ${MLENV_LOG_LEVEL}" + echo "Log Level: ${MLENV_LOG_LEVEL:-info}" }