Skip to content

Sync eng/common directory with azure-sdk-tools for PR 17393 - #10381

Merged
Praveen Kuttappan (praveenkuttappan) merged 2 commits into
mainfrom
sync-eng-common-fix/shared-release-completion-sdk-pr-17393
Oct 27, 2026
Merged

Praveen Kuttappan (praveenkuttappan) merged 2 commits into
mainfrom
sync-eng-common-fix/shared-release-completion-sdk-pr-17393

Conversation

@azure-sdk-automation

Copy link
Copy Markdown
Contributor

Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#17393 See eng/common workflow

@azure-sdk-automation
azure-sdk-automation Bot requested review from a team as code owners October 26, 2026 22:25
@azure-sdk-automation
azure-sdk-automation Bot requested review from Helen Gao (helen229) and a balanced review from Copilot October 26, 2026 22:25
@azure-sdk-automation azure-sdk-automation Bot added EngSys This issue is impacting the engineering system. Central-EngSys This issue is owned by the Engineering System team. labels Oct 26, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
1 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

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.

🟢 Approval recommended

The focused changes preserve explicit and populated runtime values while handling missing optional values, with no blocking issues identified.

No open issues 🎉

What changed in this PR

Syncs shared engineering files from Azure/azure-sdk-tools PR #17393 to fix SDK pull request handling during release-plan completion.

Changes:

  • Uses the runtime AutoReleaseSdkPullRequestUrl when no explicit SdkPullRequest is supplied.
  • Initializes an absent or empty runtime value for manual runs while preserving populated values.
  • Documents the fix in the common engineering changelog.
File Description
eng/​common/​pipelines/​templates/​steps/​mark-release-completion.yml Adds runtime initialization and the SDK PR fallback.
eng/​common/​CHANGELOG.md Documents the corrected release-completion behavior.

Review effort: Balanced

@helen229 Helen Gao (helen229) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approving the exact eng/common sync from Azure/azure-sdk-tools#17393. The two changed files match the source completion template and changelog.

@praveenkuttappan
Praveen Kuttappan (praveenkuttappan) merged commit 27bcd61 into main Oct 27, 2026
17 of 18 checks passed
@praveenkuttappan
Praveen Kuttappan (praveenkuttappan) deleted the sync-eng-common-fix/shared-release-completion-sdk-pr-17393 branch October 27, 2026 21:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants