-
Notifications
You must be signed in to change notification settings - Fork 502
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
Minor improvements to krel promote-images #2320
Minor improvements to krel promote-images #2320
Conversation
7c307fa
to
5583534
Compare
When `krel promote-images` is called without passing a GitHub token, the error message didn't say what env var should be set. This commit adds the same to the error message. Signed-off-by: Nabarun Pal <[email protected]>
5583534
to
f61457d
Compare
/test pull-release-verify |
This will be helpful to verify the changes, if required to do so. Signed-off-by: Nabarun Pal <[email protected]>
7f058d3
to
f032d32
Compare
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.
/lgtm
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: palnabarun, xmudrii The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/kind documentation
What this PR does / why we need it:
When
krel promote-images
is called without passing a GitHub token, theerror message didn't say what env var should be set. This commit adds
the same to the error message.
The promotion PR body now includes the command that was run to create the PR.
Which issue(s) this PR fixes:
Follow up on kubernetes/sig-release#1734
Special notes for your reviewer:
Does this PR introduce a user-facing change?
/cc @kubernetes/release-engineering
/area release-eng
/priority important-soon