-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
fix bad url #55300
fix bad url #55300
Conversation
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: stewart-yu, thockin Associated issue requirement bypassed by: thockin The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
/retest |
@stewart-yu: you can't request testing unless you are a kubernetes member. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/test pull-kubernetes-e2e-gce |
@stewart-yu: you can't request testing unless you are a kubernetes member. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/ok-to-test |
@stewart-yu This PR needs to be opened against the master branch and not the |
This pr is to fix bad url in release-1.7 branch, does to be opened against the master branch? @nikhita |
@stewart-yu Typically, when something needs to be merged into a previous release branch, a PR is created against the master first. Then it is cherry-picked to the release-1.7 branch, if the branch manager (@wojtek-t for 1.7) approves. But like you said, it looks like the file does not exist on master in this case. @wojtek-t What must be done here? |
/retest Review the full test history for this PR. |
/retest |
Automatic merge from submit-queue. |
What this PR does / why we need it:
The link url is bad, fix it.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #
Special notes for your reviewer:
Release note: