Skip to content

Add support for multiple Azure Container Registries - #7318

Open
adamrtalbot wants to merge 3 commits into
masterfrom
azure-multi-registry
Open

Add support for multiple Azure Container Registries#7318
adamrtalbot wants to merge 3 commits into
masterfrom
azure-multi-registry

Conversation

@adamrtalbot

Copy link
Copy Markdown
Collaborator

Why

Azure Batch pools may need to pull container images from more than one private Azure Container Registry, but azure.registry only supported a single set of credentials.

What

  • azure.registry now accepts either a single registry map (unchanged, backward-compatible) or a list of registry maps
  • AzConfig.registries() returns the parsed List<AzRegistryOpts>
  • AzBatchService builds one ContainerRegistryReference per configured registry when creating a pool's VM config
  • Added tests covering none/single/multiple/mixed-configured registries
  • Updated docs/azure.mdx and docs/reference/config.mdx

🤖 Generated with Claude Code

Allow `azure.registry` to accept a list of registry definitions, in
addition to the existing single registry map, so images can be pulled
from more than one private Azure Container Registry.

Generated by Claude Code

Signed-off-by: adamrtalbot <12817534+adamrtalbot@users.noreply.github.com>
@adamrtalbot
adamrtalbot requested a review from a team as a code owner July 9, 2026 17:57
@netlify

netlify Bot commented Jul 9, 2026

Copy link
Copy Markdown

Deploy Preview for nextflow-docs canceled.

Name Link
🔨 Latest commit d7e93f4
🔍 Latest deploy log https://app.netlify.com/projects/nextflow-docs/deploys/6a68c0811f5c4f000811e02d

@adamrtalbot

Copy link
Copy Markdown
Collaborator Author

@copilot resolve the merge conflicts in this pull request

Signed-off-by: GitHub <noreply@github.com>

Copilot AI commented Jul 28, 2026

Copy link
Copy Markdown

@copilot resolve the merge conflicts in this pull request

Resolved by merging origin/master into this branch and fixing the conflicts in AzBatchService.groovy and AzBatchServiceTest.groovy in commit 60a5c440.

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.

3 participants