Skip to content
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.

Sign up for GitHub

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

Jump to bottom

aaa: convert k8s NodePort services to ClusterIP services #2488

Closed
5 tasks done
ameukam opened this issue Aug 6, 2021 · 3 comments
Closed
5 tasks done

aaa: convert k8s NodePort services to ClusterIP services #2488

ameukam opened this issue Aug 6, 2021 · 3 comments
Assignees
@ameukam
Labels
area/apps/k8s-io k8s.io redirector, code in apps/k8s-io/ area/apps/perfdash perfdash.k8s.io, code in apps/perdash/ area/apps/sippy sippy.k8s.io, code in apps/sippy/ area/apps/slack-infra slack.k8s.io, slack-infra, code in apps/slack-infra area/apps/triageparty-release release.triage.k8s.io, triage-party, code in apps/triageparty-release area/infra Infrastructure management, infrastructure design, code in infra/ priority/backlog Higher priority than priority/awaiting-more-evidence. sig/k8s-infra Categorizes an issue or PR as relevant to SIG K8s Infra.
Milestone
v1.23

Comments

@ameukam
Copy link
Member

ameukam commented Aug 6, 2021 •
edited
Loading

Followup of #2316.

Convert NodePort services to ClusterIP services using NEGs:
@ameukam ameukam added this to the v1.23 milestone Aug 6, 2021
@ameukam ameukam added area/apps/k8s-io k8s.io redirector, code in apps/k8s-io/ area/apps/perfdash perfdash.k8s.io, code in apps/perdash/ area/apps/sippy sippy.k8s.io, code in apps/sippy/ area/apps/slack-infra slack.k8s.io, slack-infra, code in apps/slack-infra area/apps/triageparty-release release.triage.k8s.io, triage-party, code in apps/triageparty-release area/infra Infrastructure management, infrastructure design, code in infra/ labels Aug 6, 2021
@ameukam ameukam self-assigned this Aug 6, 2021
@spiffxp
Copy link
Member

spiffxp commented Sep 2, 2021

/priority backlog

@k8s-ci-robot k8s-ci-robot added the priority/backlog Higher priority than priority/awaiting-more-evidence. label Sep 2, 2021
ameukam added a commit to ameukam/k8s.io that referenced this issue Sep 15, 2021
@ameukam
apps: Switch jobs to ClusterIP and NEG
2444c35 
Ref: kubernetes#2488.

Convert kubernetes services for world-accessible from NodePort to
ClusterIP.

Signed-off-by: Arnaud Meukam <[email protected]>
@ameukam ameukam mentioned this issue Sep 15, 2021
Merged
ameukam added a commit to ameukam/perf-tests that referenced this issue Sep 27, 2021
@ameukam
perfdash: use NEG for kubernetes service
261b8c7 
Ref: kubernetes/k8s.io#2488.

Signed-off-by: Arnaud Meukam <[email protected]>
ameukam added a commit to ameukam/perf-tests that referenced this issue Sep 27, 2021
@ameukam
perfdash: use NEG for kubernetes service
0233cd8 
Ref: kubernetes/k8s.io#2488.

Switch to ClusterIP for the service.

Signed-off-by: Arnaud Meukam <[email protected]>
ameukam added a commit to ameukam/perf-tests that referenced this issue Sep 27, 2021
@ameukam
perfdash: use NEG for kubernetes service
d936710 
Ref: kubernetes/k8s.io#2488.

Switch to ClusterIP for the service.

Signed-off-by: Arnaud Meukam <[email protected]>
@ameukam ameukam mentioned this issue Sep 27, 2021
Merged
@k8s-ci-robot k8s-ci-robot added sig/k8s-infra Categorizes an issue or PR as relevant to SIG K8s Infra. and removed wg/k8s-infra labels Sep 29, 2021
ameukam added a commit to ameukam/k8s.io that referenced this issue Oct 8, 2021
@ameukam
apps/k8s.io: Change service type
b2d2976 
Ref: kubernetes#2488

The service already use a NEG.
Signed-off-by: Arnaud Meukam <[email protected]>
@ameukam ameukam mentioned this issue Oct 8, 2021
Merged
ameukam added a commit to ameukam/perf-tests that referenced this issue Oct 21, 2021
@ameukam
perfdash: use NEG for kubernetes service
da22ce8 
Ref: kubernetes/k8s.io#2488.

Switch to ClusterIP for the service.

Signed-off-by: Arnaud Meukam <[email protected]>
@ameukam
Copy link
Member Author

ameukam commented Dec 8, 2021

/close

@k8s-ci-robot
Copy link
Contributor

@ameukam: Closing this issue.

In response to this:

/close

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ameukam ameukam

Labels
area/apps/k8s-io k8s.io redirector, code in apps/k8s-io/ area/apps/perfdash perfdash.k8s.io, code in apps/perdash/ area/apps/sippy sippy.k8s.io, code in apps/sippy/ area/apps/slack-infra slack.k8s.io, slack-infra, code in apps/slack-infra area/apps/triageparty-release release.triage.k8s.io, triage-party, code in apps/triageparty-release area/infra Infrastructure management, infrastructure design, code in infra/ priority/backlog Higher priority than priority/awaiting-more-evidence. sig/k8s-infra Categorizes an issue or PR as relevant to SIG K8s Infra.
Projects
None yet
Milestone
v1.23
Development

No branches or pull requests
3 participants
@spiffxp @ameukam @k8s-ci-robot