feat: GCP get-well [COMP-793]#1316
feat: GCP get-well [COMP-793]#1316justinegeffen wants to merge 29 commits intoenterprise-26.1-documentationfrom
Conversation
✅ Deploy Preview for seqera-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
The main thing that is missing here is the addition env var required to configure Platform when WIF is required. The user needs to
openssl genrsa -out private.pem 4096 Then set:
In your GCP WIF pool, set the issuer URL to: |
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Added OIDC PEM path configuration details. Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
christopher-hakkaart
left a comment
There was a problem hiding this comment.
I think the links should be updated/checked. I've made some suggestions to remove the em dashes. Otherwise looking good.
Co-authored-by: Chris Hakkaart <chris.hakkaart@seqera.io> Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Co-authored-by: Chris Hakkaart <chris.hakkaart@seqera.io> Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Co-authored-by: Chris Hakkaart <chris.hakkaart@seqera.io> Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Co-authored-by: Chris Hakkaart <chris.hakkaart@seqera.io> Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Co-authored-by: Chris Hakkaart <chris.hakkaart@seqera.io> Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Co-authored-by: Chris Hakkaart <chris.hakkaart@seqera.io> Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Co-authored-by: Chris Hakkaart <chris.hakkaart@seqera.io> Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
Co-authored-by: Chris Hakkaart <chris.hakkaart@seqera.io> Signed-off-by: Justine Geffen <justinegeffen@users.noreply.github.com>
christopher-hakkaart
left a comment
There was a problem hiding this comment.
Editorially, this looks good 👍
I can't speak to the accuracy of the content changed.
A few things worth clarifying before we finalize:
@MichaelTansiniSeqera, yes, we can add |
|
|
||
| * **Service Account Email**: The email address of the Google Cloud service account that WIF will impersonate. | ||
| * **Workload Identity Provider**: The full resource path of the Workload Identity Provider (e.g., `projects/PROJECT_NUMBER/locations/global/workloadIdentityPools/POOL_ID/providers/PROVIDER_ID`). | ||
| * **Token Audience** (optional): The intended audience for the OIDC token. Configure this if your Workload Identity Provider requires a specific audience value. |
There was a problem hiding this comment.
@MichaelTansiniSeqera the comment is anchored to the Token Audience bullet but your proposed text is about user roles. I think the comment may have landed on the wrong line during review. Did you mean to comment on the User permissions list above (roles/batch.jobsEditor, roles/iam.serviceAccountUser, roles/iam.serviceAccountViewer)?
If that's the case, +1 on framing it as "minimum set of roles required", making that intent explicit in the doc would help admins justify the list to their security teams.
The Token Audience description as written is accurate. It's an optional field that's only needed when the WIF provider was created with a custom allowed audience; by default, Platform derives the audience from the provider path, so most setups can leave it blank. We could optionally tighten the wording to make the "leave blank unless..." default explicit, but no factual change needed.
updated with spot reclamation settings Signed-off-by: MichaelTansiniSeqera <michael.tansini@seqera.io>
Signed-off-by: MichaelTansiniSeqera <michael.tansini@seqera.io>
Signed-off-by: MichaelTansiniSeqera <michael.tansini@seqera.io>
|
I've updated this with a section to show on Enterprise what a user needs to do in the GCP Console to set up WIF for integration with Seqera. |
Fixes EDU-1057
Fixes https://seqera.atlassian.net/browse/COMP-793