Skip to content

Fix batch-score.sh: use defaults.deployment instead of deprecated def… - #4035

Merged
yetamsft merged 1 commit into
mainfrom
yeta-microsoft/fix-batch-score-deployment-name
Jun 10, 2026
Merged

Fix batch-score.sh: use defaults.deployment instead of deprecated def…#4035
yetamsft merged 1 commit into
mainfrom
yeta-microsoft/fix-batch-score-deployment-name

Conversation

@yetamsft

@yetamsft yetamsft commented Jun 9, 2026

Copy link
Copy Markdown
Member

Description

Replace --set-default with an explicit endpoint update using defaults.deployment_name=$DEPLOYMENT_NAME

Checklist

  • I have read the contribution guidelines.
  • I have coordinated with the docs team (mldocs@microsoft.com) if this PR deletes files or changes any file names or file extensions.
  • Pull request includes test coverage for the included changes.
  • This notebook or file is added to the CODEOWNERS file, pointing to the author or the author's team.

@yetamsft
yetamsft force-pushed the yeta-microsoft/fix-batch-score-deployment-name branch 5 times, most recently from 4a42dff to d2c2014 Compare June 9, 2026 22:03
The --set-default flag on batch-deployment create fails with a RequestInvalid
error because the API can no longer find 'deployment_name' on BatchEndpointDefaults
when sent through that code path.

Replace --set-default with a two-step approach: create the deployment, then
explicitly update the endpoint with --set defaults.deployment_name to set the
default deployment.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@yetamsft
yetamsft force-pushed the yeta-microsoft/fix-batch-score-deployment-name branch from d2c2014 to 34c2687 Compare June 9, 2026 22:28
@yetamsft
yetamsft requested review from ekmalkan and trimicrosoft June 9, 2026 23:35

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the cli/batch-score.sh example to avoid using az ml batch-deployment create --set-default (which relies on a deprecated BatchEndpointDefaults property) by explicitly setting the batch endpoint’s default deployment via az ml batch-endpoint update.

Changes:

  • Remove --set-default from the initial az ml batch-deployment create.
  • Add an explicit az ml batch-endpoint update --set ... call to set the endpoint default deployment.

Comment thread cli/batch-score.sh
@yetamsft
yetamsft merged commit 8729cf3 into main Jun 10, 2026
8 checks passed
@yetamsft
yetamsft deleted the yeta-microsoft/fix-batch-score-deployment-name branch June 10, 2026 04:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants