Skip to content

Publish techdocs via OIDC instead of static AWS credentials - #125

Open
psi wants to merge 1 commit into
masterfrom
oidc-techdocs
Open

Publish techdocs via OIDC instead of static AWS credentials#125
psi wants to merge 1 commit into
masterfrom
oidc-techdocs

Conversation

@psi

@psi psi commented Aug 11, 2026

Copy link
Copy Markdown
Member

Summary

Migrates this repo's techdocs publishing off the static TRUSS_AWS_ACCESS_KEY_ID / TRUSS_AWS_SECRET_ACCESS_KEY org secrets and onto GitHub OIDC role assumption.

The shared Bridge Backstage Techdocs action (get-bridge/developer_portal/.github/actions/techdocs) now assumes the github-truss IAM role via OIDC by default (see get-bridge/developer_portal#105). Consumers only need to:

  1. Grant permissions: id-token: write on the job.
  2. Stop passing the aws_access_key_id / aws_secret_access_key inputs.

No AWS credentials are handled by this workflow anymore.

⚠️ Merge ordering

This depends on get-bridge/developer_portal#105 being merged first. This workflow tracks the action at ref: master, and until #105 lands the action still marks those credential inputs as required. Please merge #105 before merging this PR.

Reference migration pattern: get-bridge/bridge-instilled@a3545c8.

Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com

The techdocs workflow no longer passes TRUSS_AWS_ACCESS_KEY_ID /
TRUSS_AWS_SECRET_ACCESS_KEY to the Bridge Backstage Techdocs action. The
action now assumes the github-truss IAM role via GitHub OIDC by default, so
the workflow only needs to grant id-token: write.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

@jamesh-gomo jamesh-gomo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hi @psi ,

The change looks good. I don't think we can't really test this until it's merged into Master - so keeping an eye on Spinnaker staging and edge env's after merging is probably the safest way to confirm it's working

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants