Skip to content

Commit a71ccf6

Browse files
committed
Pipeline fixes
1 parent 7a34092 commit a71ccf6

File tree

1 file changed

+9
-8
lines changed

1 file changed

+9
-8
lines changed

scripts/release/pipeline_main.py

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,6 @@ def image_build_config_from_args(args) -> ImageBuildConfiguration:
9292
raise ValueError(f"Image '{image}' is not defined in the build info for scenario '{build_scenario}'")
9393

9494
# Resolve final values with overrides
95-
# TODO: cover versions for agents and OM images
9695
version = args.version or image_build_info.version
9796
registry = args.registry or image_build_info.repository
9897
platforms = get_platforms_from_arg(args.platform) or image_build_info.platforms
@@ -113,17 +112,19 @@ def image_build_config_from_args(args) -> ImageBuildConfiguration:
113112

114113

115114
def get_scenario_from_arg(args_scenario: str) -> BuildScenario | None:
116-
if args_scenario:
117-
try:
118-
return BuildScenario(args_scenario)
119-
except ValueError as e:
120-
raise ValueError(f"Invalid scenario '{args_scenario}': {e}")
115+
if not args_scenario:
116+
return None
121117

122-
return None
118+
try:
119+
return BuildScenario(args_scenario)
120+
except ValueError as e:
121+
raise ValueError(f"Invalid scenario '{args_scenario}': {e}")
123122

124123

125124
def get_platforms_from_arg(args_platforms: str) -> list[str] | None:
126-
"""Parse and validate the --platform argument"""
125+
if not args_platforms:
126+
return None
127+
127128
platforms = [p.strip() for p in args_platforms.split(",")]
128129
if any(p not in SUPPORTED_PLATFORMS for p in platforms):
129130
raise ValueError(

0 commit comments

Comments
 (0)