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
remove deprecated validEgressSelectorNames 'master' #102242
remove deprecated validEgressSelectorNames 'master' #102242
Conversation
This PR may require API review. If so, when the changes are ready, complete the pre-review checklist and request an API review. Status of requested reviews is tracked in the API Review project. |
0d643a0
to
6ff75d5
Compare
/cc cheftako |
/assign @cheftako @caesarxuchao @Jefftree |
/priority important-soon |
I believe https://github.com/kubernetes/kubernetes/blob/master/cluster/gce/gci/configure-helper.sh#L938 and https://github.com/kubernetes/kubernetes/blob/master/cluster/gce/gci/configure-helper.sh#L956 will need to be updated as well. |
@Jefftree Forget to change gce scripts: cluster/gce/gci/configure-helper.sh. Thanks. |
e7c325c
to
c39a5b6
Compare
@@ -527,7 +527,7 @@ func TestValidateEgressSelectorConfiguration(t *testing.T) { | |||
}, | |||
}, | |||
{ | |||
Name: "master", | |||
Name: "controlplane", |
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.
This is dup test case.
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.
Duplicate error is like
Calling ValidateEgressSelectorConfiguration expected error, got [service.connection[1]: Duplicate value: "controlplane"]--- PASS: TestValidateEgressSelectorConfiguration (0.00s)
c39a5b6
to
6ab02e0
Compare
6ab02e0
to
8e226f9
Compare
8e226f9
to
0db40ff
Compare
Signed-off-by: pacoxu <paco.xu@daocloud.io>
…ontrolplane' Signed-off-by: pacoxu <paco.xu@daocloud.io>
0db40ff
to
f6d9d2a
Compare
/retest |
staging/src/k8s.io/apiserver/pkg/apis/apiserver/v1alpha1/conversion.go
Outdated
Show resolved
Hide resolved
f6d9d2a
to
c0b2c3f
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cheftako, liggitt, pacoxu 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 |
* remove deprecated validEgressSelectorNames 'master' Signed-off-by: pacoxu <paco.xu@daocloud.io> * update gce configure: replace deprecated egress name 'master' with 'controlplane' Signed-off-by: pacoxu <paco.xu@daocloud.io> * add dup error for EgressSelection & fix converting alpha/beta to v1 name
What type of PR is this?
/kind cleanup
/kind deprecation
What this PR does / why we need it:
Which issue(s) this PR fixes:
Follow up of #95235
/cc andrewsykim
Special notes for your reviewer:
#94900 test related is still not done: only part one #100181
Does this PR introduce a user-facing change?