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
1.18.0 rebase #351
1.18.0 rebase #351
Conversation
6c8e339
to
75219c7
Compare
75219c7
to
a2bc3e5
Compare
926128e
to
e257c4a
Compare
e257c4a
to
df148f5
Compare
Requires openshift/origin#24765 to get merged to resolve failures in |
df148f5
to
d50edf3
Compare
/test verify-deps |
d50edf3
to
ec828a5
Compare
/retest |
/test e2e-cmd |
This is stuck waiting for openshift/origin#24769 to land |
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.
Split the Adjust code on top of 1.18.0-beta.2
commit into the following commits, for readability:
- context changes
- dry-run changes
- other 1.18 changes, although I'm not seeing any other
The rest looks good.
ec828a5
to
32db407
Compare
/test e2e-cmd |
/test e2e-aws |
/test e2e-cmd |
/test unit |
/retest |
2 similar comments
/retest |
/retest |
…try/pkg/appregistry to set context for client's Secrets().Get method
494ea7d
to
2acf792
Compare
2acf792
to
2ad4ef9
Compare
/retest |
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: ingvagabund, soltysh 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 |
/test e2e-aws-builds |
/retest Please review the full test history for this PR and help us cut down flakes. |
3 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
Previous rebase: https://github.com/openshift/oc/pull/218/commits
There are still some consumers of --dry-run[bool] option:
They define the flag on they own (not through
k8s.io/kubectl/pkg/cmd/util.AddDryRunFlag
) or they don't set it at all./hold