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.

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

Split internal and external kube-apiserver addresses #16282

Conversation

justinsb
Copy link
Member

This allows us to select the right IP address when we have multiple
load balancers (one for internal, one for external) as we are planning
to have on GCE.

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. area/api labels Jan 26, 2024
@k8s-ci-robot k8s-ci-robot added area/provider/aws Issues or PRs related to aws provider area/provider/digitalocean Issues or PRs related to digitalocean provider area/provider/gcp Issues or PRs related to gcp provider area/provider/hetzner Issues or PRs related to Hetzner provider area/provider/openstack Issues or PRs related to openstack provider labels Jan 26, 2024
pkg/model/hetznermodel/loadbalancer.go Outdated Show resolved Hide resolved
@k8s-ci-robot k8s-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 18, 2024
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from hakman. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@justinsb justinsb force-pushed the split_internal_and_external_apiserver branch from 95d38fc to 24007e2 Compare February 18, 2024 00:34
@k8s-ci-robot k8s-ci-robot added area/provider/azure Issues or PRs related to azure provider and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Feb 18, 2024
@justinsb
Copy link
Member Author

Renamed to KopsControllerInternal, and also expanded the comments for the enum values (in pkg/wellknownservices/wellknownservices.go)

I'm using "Internal" for in-cluster traffic, and "External" for other traffic. InCluster might be clearer than Internal, but I don't know what a better name would be for External...

@justinsb justinsb force-pushed the split_internal_and_external_apiserver branch from 24007e2 to b3fdf46 Compare February 18, 2024 00:41
This allows us to select the right IP address when we have multiple
load balancers (one for internal, one for external) as we are planning
to have on GCE.

Co-authored-by: Ciprian Hacman <[email protected]>
@justinsb justinsb force-pushed the split_internal_and_external_apiserver branch from b3fdf46 to 4ab1bf4 Compare February 18, 2024 14:48
@justinsb
Copy link
Member Author

WDYT @hakman ... I can probably make #16249 work in a different way, but right now that PR depends on this one (and I think that one is comparatively more important!)

@k8s-ci-robot
Copy link
Contributor

PR needs rebase.

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.

@k8s-ci-robot k8s-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 25, 2024
@justinsb
Copy link
Member Author

The approach we took in #16379 replaces the need for this (at least that's the belief!)

/close

@k8s-ci-robot
Copy link
Contributor

@justinsb: Closed this PR.

In response to this:

The approach we took in #16379 replaces the need for this (at least that's the belief!)

/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
Labels
area/api area/provider/aws Issues or PRs related to aws provider area/provider/azure Issues or PRs related to azure provider area/provider/digitalocean Issues or PRs related to digitalocean provider area/provider/gcp Issues or PRs related to gcp provider area/provider/hetzner Issues or PRs related to Hetzner provider area/provider/openstack Issues or PRs related to openstack provider cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants