Skip to content

[self-hosted] Add officially support service type ClusterIP for proxy #10725

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

Closed
6 tasks done
Tracked by #9929
corneliusludmann opened this issue Jun 17, 2022 · 8 comments
Closed
6 tasks done
Tracked by #9929
Assignees
Labels
component: install Terraform installation scripts, helm charts, installer images team: delivery Issue belongs to the self-hosted team type: feature request New feature or request

Comments

@corneliusludmann
Copy link
Contributor

corneliusludmann commented Jun 17, 2022

@lucasvaltl
Copy link
Contributor

Replaces #9981

@szab100
Copy link
Contributor

szab100 commented Jun 20, 2022

@corneliusludmann @lucasvaltl When implementing this, can we move adding these two hard-coded annotations to inside the IF(type == LB) {..} stmt above or remove the hard-coded entries completely and just document in which cases should these be added to ProxyConfig.ServiceAnnotations?

Generally, these should only be used when 'proxy' is a load-balancer. When it's NodePort or ClusterIP, the 'external-dns' annotation on 'proxy' can interfere with other ingress / load-balancer resources that can also be using this annotation with same host values. This creates a race condition in the external dns system, causing the LB to be unreachable.

@mrsimonemms mrsimonemms self-assigned this Jun 28, 2022
@mrsimonemms
Copy link
Contributor

@corneliusludmann should this also remove the ClusterIP from the experimental section?

@mrsimonemms
Copy link
Contributor

In "blocked" until #11011 is merged when July release is published

@mrzarquon
Copy link
Contributor

@mrsimonemms if this setting moves from experimental to main, we should still honor the setting if it's in experimental and flag it with a deprecation warning. otherwise people who had clusterip working in 2022.6 will have it stop working in 2022.7 if they don't update the config (and since it's not easy to download / update the config in the replicated UI, there's no way for them to know)

@mrsimonemms
Copy link
Contributor

mrsimonemms commented Jul 26, 2022

Yes, we still honour the experimental setting and will do until December release #11124

As far as I know, there's no way of flagging a deprecation in KOTS dashboard

@mrzarquon
Copy link
Contributor

gitpod-installer parses config, outputs to a assessment.log, troubleshoot.sh analyzer checks for "has_deprecated_experimental_feature" and reports it?

@mrsimonemms mrsimonemms moved this from ⛔ Blocked to ⚒In Progress in 🚚 Security, Infrastructure, and Delivery Team (SID) Aug 1, 2022
@mrsimonemms
Copy link
Contributor

gitpod-installer parses config, outputs to a assessment.log, troubleshoot.sh analyzer checks for "has_deprecated_experimental_feature" and reports it?

Do you want to raise this as a separate ticket?

Repository owner moved this from ⚒In Progress to ✨Done in 🚚 Security, Infrastructure, and Delivery Team (SID) Aug 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: install Terraform installation scripts, helm charts, installer images team: delivery Issue belongs to the self-hosted team type: feature request New feature or request
Projects
No open projects
Development

No branches or pull requests

5 participants