docs(cloud): port Azure get-well cloud changes#1323
Open
justinegeffen wants to merge 8 commits intomasterfrom
Open
docs(cloud): port Azure get-well cloud changes#1323justinegeffen wants to merge 8 commits intomasterfrom
justinegeffen wants to merge 8 commits intomasterfrom
Conversation
Contributor
justinegeffen
commented
Apr 17, 2026
- Fix trailing whitespace on Bitbucket heading and add missing EOF newline in git/overview.md
- Reword Studios note about head vs worker queue in studios/add-studio.md
- Fix trailing whitespace on Bitbucket heading and add missing EOF newline in git/overview.md - Reword Studios note about head vs worker queue in studios/add-studio.md Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
❌ Deploy Preview for seqera-docs failed. Why did it fail? →
|
adamrtalbot
reviewed
Apr 17, 2026
Contributor
adamrtalbot
left a comment
There was a problem hiding this comment.
One part is a little bit awkward, and I don't like the phrase "dual-pool mode"
| - **Active jobs and schedules**: Each Nextflow process will require an active Azure Batch job per pipeline while running, so increase this number to a high level. See [here][az-learn-jobs] to learn more about jobs in Azure Batch. | ||
| - **Pools**: Each platform compute environment requires one Azure Batch pool. Each pool is composed of multiple machines of one virtual machine size. | ||
| :::note | ||
| To use separate pools for head and compute nodes, see [this FAQ entry](../troubleshooting_and_faqs/azure_troubleshooting). |
Contributor
There was a problem hiding this comment.
Do we want to remove the old docs?
| - Service principals enable role-based access control with more precise permissions. | ||
| - Service principals map to a many-to-many relationship with Azure Batch and Azure Storage accounts. | ||
| - Some Azure Batch features are only available when using a service principal. | ||
| - Some Azure Batch features, such as VNet/subnet configuration, are only available when using a service principal. |
Contributor
There was a problem hiding this comment.
Suggested change
| - Some Azure Batch features, such as VNet/subnet configuration, are only available when using a service principal. | |
| - Some Azure Batch features, such as VNet/subnet configuration, are only available when using Microsoft Entra. |
@MichaelTansiniSeqera is this true?
Contributor
Author
There was a problem hiding this comment.
Following up on this, @MichaelTansiniSeqera
Comment on lines
+149
to
+152
| Therefore, you must create both an Entra service principal and a managed identity. You add the service principal to your Seqera credentials and attach the managed identity to your Azure Batch node pools. | ||
|
|
||
| :::info | ||
| Batch Forge compute environments must use access keys for authentication. Service principals are only supported in manual compute environments. | ||
| - In dual-pool mode (where separate pools are used for head and compute jobs), each pool is assigned only its relevant managed identity. | ||
| - In Forge mode, you must also provide the managed identity resource ID for each pool, as Seqera uses it to assign the identity when provisioning the pool. |
Contributor
There was a problem hiding this comment.
The wording here is a bit clunky. Some suggestions:
- You add the service principal detailas as credentials to Seqera Platform
- You attach the managed identity to your Azure Batch pools
- Assign the managed identity to each Azure Batch node pool with the relevant permissions.
- When using Batch Forge, provide the managed identity resource ID for each managed identityy. Seqera Platform will assign this managed identity to each pool during creation.
| 1. When you set up the Seqera compute environment, select the Azure Batch pool by name and enter the managed identity client ID in the specified field as instructed above. | ||
|
|
||
| :::note | ||
| In dual-pool mode, you can assign separate managed identities to the head and compute pools. Each pool receives only the managed identity relevant to its role. |
Contributor
There was a problem hiding this comment.
Not sure I like the term "dual pool mode", is this the official name?
Contributor
Author
There was a problem hiding this comment.
| |--------|---------|-------------| | ||
| | **Delete jobs on completion** | Off | Permanently deletes all jobs and their tasks from Azure Batch when the workflow finishes. | | ||
| | **Delete tasks on completion** | On | Deletes individual tasks from jobs when they complete successfully. Failed tasks are preserved for debugging. | | ||
| | **Terminate jobs on completion** | Off | Sets jobs to terminate when all their tasks complete. Jobs remain in "completed" state but are no longer active. | |
Contributor
There was a problem hiding this comment.
This should be default on!
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Co-authored-by: Adam Talbot <12817534+adamrtalbot@users.noreply.github.com> Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.