Skip to content

Commit

Permalink
Merge pull request #7 from harshad16/fix-validation
Browse files Browse the repository at this point in the history
fix: rebuild subclass map with runtime class
  • Loading branch information
harshad16 authored Mar 15, 2024
2 parents a00930a + 5134c69 commit ce67607
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
1 change: 1 addition & 0 deletions elyra/cli/pipeline_app.py
Original file line number Diff line number Diff line change
Expand Up @@ -297,6 +297,7 @@ def validate(pipeline_path: str, runtime_config: Optional[str] = None):
except Exception:
raise click.ClickException("Pipeline validation FAILED.")

print_info("Pipeline validation SUCCEEDED.", [])

def validate_timeout_option(ctx, param, value):
"""Callback for monitor-timeout parameter validation"""
Expand Down
3 changes: 3 additions & 0 deletions elyra/pipeline/pipeline_definition.py
Original file line number Diff line number Diff line change
Expand Up @@ -265,6 +265,9 @@ def convert_pipeline_parameters(self, runtime_type_name: str) -> None:
if parameter_class is None:
return None # runtime type does not support parameters, skip

if not ElyraProperty.subclass_exists_for_property(parameter_class.property_id):
ElyraProperty.build_property_map()

# Convert pipeline parameters to runtime-specific instances
converted_value = ElyraProperty.create_instance(parameter_class.property_id, self.pipeline_parameters)
if converted_value is not None:
Expand Down

0 comments on commit ce67607

Please sign in to comment.