Skip to content

Commit

Permalink
check that batchrun ins't a pipeline job (Azure#30840)
Browse files Browse the repository at this point in the history
  • Loading branch information
TajiHarrisMicrosoft authored Jul 6, 2023
1 parent c86ca6a commit 8b753de
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
1 change: 1 addition & 0 deletions sdk/ml/azure-ai-ml/azure/ai/ml/constants/_job/pipeline.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ class PipelineConstants:
REUSED_FLAG_FIELD = "azureml.isreused"
REUSED_FLAG_TRUE = "true"
REUSED_JOB_ID = "azureml.reusedrunid"
PIPELINE_JOB_TYPE = "azureml.pipelinejob"


class ValidationErrorCode:
Expand Down
5 changes: 4 additions & 1 deletion sdk/ml/azure-ai-ml/azure/ai/ml/operations/_job_operations.py
Original file line number Diff line number Diff line change
Expand Up @@ -714,7 +714,10 @@ def download(
error_category=ErrorCategory.USER_ERROR,
)

is_batch_job = job_details.tags.get("azureml.batchrun", None) == "true"
is_batch_job = (
job_details.tags.get("azureml.batchrun", None) == "true"
and job_details.tags.get("azureml.jobtype", None) != PipelineConstants.PIPELINE_JOB_TYPE
)
outputs = {}
download_path = Path(download_path)
artifact_directory_name = "artifacts"
Expand Down

0 comments on commit 8b753de

Please sign in to comment.