Skip to content

Migrate microsoft.azure provider to azure-batch 15.x#68291

Closed
Subham-KRLX wants to merge 1 commit into
apache:mainfrom
Subham-KRLX:feat/azure-batch-15-migration
Closed

Migrate microsoft.azure provider to azure-batch 15.x#68291
Subham-KRLX wants to merge 1 commit into
apache:mainfrom
Subham-KRLX:feat/azure-batch-15-migration

Conversation

@Subham-KRLX

Copy link
Copy Markdown
Contributor

Migrates the microsoft.azure provider to azure-batch 15.x (Track 2 SDK), unblocking the dependency cap from #66452.

Key Changes

  • Dependency: Removed the <15.0.0 cap in pyproject.toml and updated uv.lock.
  • AzureBatchHook: Swapped BatchServiceClient with BatchClient, SharedKeyCredentials with AzureNamedKeyCredential, and updated CRUD / model parameter classes.
  • AzureBatchOperator: Removed retired os_family/os_version parameters. Updated deferrable mode, term/delete logic, and cleanup methods.
  • Tests: Rewrote and updated unit tests (35/35 passing) and static checks.

closes: #66466


Was generative AI tooling used to co-author this PR?
  • Yes — Claude(For code research)

@Subham-KRLX Subham-KRLX changed the title Migrate AzureBatchHook/Operator to azure-batch 15.x (track 2 SDK) Migrate microsoft.azure provider to azure-batch 15.x Jun 9, 2026
@Subham-KRLX
Subham-KRLX force-pushed the feat/azure-batch-15-migration branch from d4fe47b to 450cc99 Compare June 9, 2026 14:47
@Subham-KRLX
Subham-KRLX marked this pull request as draft June 9, 2026 14:52
@Subham-KRLX
Subham-KRLX marked this pull request as ready for review June 9, 2026 15:07
@Subham-KRLX
Subham-KRLX marked this pull request as draft June 9, 2026 16:25
@Subham-KRLX
Subham-KRLX force-pushed the feat/azure-batch-15-migration branch 6 times, most recently from 57682be to 23b03d8 Compare June 13, 2026 06:19
@Subham-KRLX
Subham-KRLX marked this pull request as ready for review June 13, 2026 06:19
@github-actions

Copy link
Copy Markdown
Contributor

uv.lock on main just moved via #68620 ("Sync uv.lock and provider-deps checksum for #66612"), commit a42fdb9 and this PR currently conflicts.

Quickest fix:

git fetch upstream main && git rebase upstream/main
rm uv.lock && uv lock
git add uv.lock && git rebase --continue
git push --force-with-lease

Automated nudge — ignore if you're not ready to rebase. This comment is updated in place on future uv.lock bumps.

@LuuOW LuuOW left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Technical audit: code patterns and implementation verified for alignment with modern software engineering standards.

@Subham-KRLX
Subham-KRLX force-pushed the feat/azure-batch-15-migration branch from 23b03d8 to eeedc47 Compare June 17, 2026 02:32
@Subham-KRLX
Subham-KRLX marked this pull request as draft June 18, 2026 06:14
@potiuk

potiuk commented Jul 8, 2026

Copy link
Copy Markdown
Member

Note

🛠️ Maintainer triage note for @Subham-KRLX · by @potiuk · 2026-07-08 16:05 UTC

This draft has been inactive for ~20 days, so I'm closing it to keep the review queue tidy — no judgment on the work itself.

No rush — reopen it (or open a fresh PR) whenever you're ready to continue; nothing is lost.

Automated triage — may be imperfect; a maintainer takes the next look.

@potiuk potiuk closed this Jul 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate microsoft.azure provider to azure-batch 15.x (BatchClient) — unblock <15 cap from #66452

3 participants