Skip to content

Requiring customer to set runner SA - #5197

Open
seanbollin wants to merge 1 commit into
mainfrom
sean/runner-sa
Open

Requiring customer to set runner SA#5197
seanbollin wants to merge 1 commit into
mainfrom
sean/runner-sa

Conversation

@seanbollin

Copy link
Copy Markdown
Contributor

What does this PR do?

Requires the customer to set the Runner SA so it doesn't fall-back to an insecure default

Notes to reviewers

Copilot AI balanced review requested due to automatic review settings August 27, 2026 21:20
@seanbollin
seanbollin requested a review from a team as a code owner August 27, 2026 21:20
@vercel

vercel Bot commented Aug 27, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
temporal-documentation Ready Ready Preview Aug 27, 2026 9:26pm

Request Review

@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

📖 Docs PR preview links

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Requires an explicit Cloud Run runner service account to avoid an insecure default.

Changes:

  • Removes documentation of the default service-account fallback.
  • Marks runner_service_account_email as required.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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