-
Notifications
You must be signed in to change notification settings - Fork 373
Clarify production environment secret setup #753
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||
|---|---|---|---|---|---|---|---|---|
|
|
@@ -59,6 +59,15 @@ passes `production_environment: production`; the upstream reusable workflow runs | |||||||
| its production job in that environment, and GitHub injects the production token | ||||||||
| only after approval. | ||||||||
|
|
||||||||
| If promotion fails with | ||||||||
| `CPLN_TOKEN_PRODUCTION is not set. Add it as a secret on the 'production' GitHub Environment.`, | ||||||||
| the token is missing from the environment scope. Configure it with: | ||||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fixed in f76c3a1 by adding the missing repository/organization-secret warning to |
||||||||
|
|
||||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The other two files (
This is the key clarification that prevents confusion. It's missing here, leaving team members without the explicit warning.
Suggested change
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fixed in f76c3a1 by adding the missing repository/organization-secret warning to |
||||||||
| ```sh | ||||||||
| gh secret set CPLN_TOKEN_PRODUCTION --repo shakacode/react-webpack-rails-tutorial --env production | ||||||||
| gh secret list --repo shakacode/react-webpack-rails-tutorial --env production | ||||||||
| ``` | ||||||||
|
|
||||||||
| Generated caller workflows pass only the named secrets each upstream workflow | ||||||||
| needs. They do not use `secrets: inherit`; `CPLN_TOKEN_PRODUCTION` is supplied | ||||||||
| only by the protected `production` Environment after approval. | ||||||||
|
|
||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gh secret setwithout--bodyor piped input will prompt interactively for the secret value, which is the right behaviour — but worth noting for readers who aren't familiar with the CLI, otherwise the command looks incomplete. A short comment like# (prompts for the token value)or a preceding sentence would help.