Skip to content

docs(azure-batch): correct credential field labels and managed identity guidance#1396

Open
justinegeffen wants to merge 6 commits into
masterfrom
justine-azure-cloud-batch-fixes
Open

docs(azure-batch): correct credential field labels and managed identity guidance#1396
justinegeffen wants to merge 6 commits into
masterfrom
justine-azure-cloud-batch-fixes

Conversation

@justinegeffen
Copy link
Copy Markdown
Contributor

Summary

Cloud-side companion to the Azure Batch get-well docs work. Sourced from the engineering Confluence page on Azure Batch Entra Credentials & Managed Identity. The same fixes are being applied separately to the enterprise doc on the get-well branch.

Tier 1 (correctness)

  • Fix swapped Tenant ID / Client ID field labels in the Entra credential instructions. Tenant ID maps to Directory (tenant) ID in Azure; Client ID maps to Application (client) ID.
  • Add Managed Identity Operator role to the service principal role list. Without this role, Seqera cannot attach the managed identity to a Batch pool.
  • Rename Azure Batch Data Contributor to Azure Batch Contributor — the built-in role sufficient for management-plane pool creation, narrower than general Contributor.
  • Consolidate the storage role to Storage Blob Data Contributor (the previously-listed Storage Blob Data Reader was redundant — Contributor is a superset).

Tier 2 (conceptual gaps)

  • Document why managed identities require Entra credentials: pool creation with MI uses the Azure Batch management plane, which only accepts AAD tokens. Shared-key credentials cannot create pools with managed identities.
  • Add AcrPull role guidance on the managed identity for Azure Container Registry image pulls.
  • Document the four managed identity fields in the compute environment form (a client ID and a resource ID for both head and worker pool):
    • Resource IDs are used to attach the managed identity to the pool VMs at pool-create time.
    • Client IDs are used by Nextflow, Fusion, and AzCopy on the pool VM to tell the Azure Instance Metadata Service which managed identity to mint a token for.
  • Document how the four fields work for single-pool vs dual-pool topologies.

Test plan

  • Verify Azure Batch Contributor is the correct built-in role name with engineering (Jon Martí or Forge team)
  • Confirm removing Storage Blob Data Reader from the SP role list doesn't break any documented setup
  • Visual review of the rendered Managed identity section in the deploy preview

🤖 Generated with Claude Code

…ty guidance

- Fix swapped Tenant ID / Client ID field labels in the Entra credential
  instructions (Tenant ID maps to Directory (tenant) ID; Client ID maps to
  Application (client) ID).
- Add Managed Identity Operator role assignment for the service principal,
  required to attach a managed identity to a Batch pool.
- Rename Azure Batch Data Contributor to Azure Batch Contributor (the
  built-in role sufficient for management-plane pool creation).
- Add AcrPull role guidance on the managed identity for ACR image pulls.
- Document why managed identities require Entra credentials (management-plane
  pool creation only accepts AAD tokens).
- Document the four managed identity fields (resource ID vs client ID, head
  vs worker pool) and how they work in single-pool and dual-pool topologies.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented May 6, 2026

Deploy Preview for seqera-docs ready!

Name Link
🔨 Latest commit 541437d
🔍 Latest deploy log https://app.netlify.com/projects/seqera-docs/deploys/6a19cd9d7e1d1d000809c188
😎 Deploy Preview https://deploy-preview-1396--seqera-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@justinegeffen justinegeffen added the 1. Dev/PM/SME Needs a review by a Dev/PM/SME label May 6, 2026
@justinegeffen
Copy link
Copy Markdown
Contributor Author

fix formatting

@justinegeffen
Copy link
Copy Markdown
Contributor Author

@jonmarti, I opened this PR based on the Confluence pages you shared. Curious as to whether these changes are accurate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1. Dev/PM/SME Needs a review by a Dev/PM/SME

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants