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

Dynamic proxyProtocol.trustedIPs from the created service #1282

Open
2 tasks done
Gitopolis opened this issue Dec 13, 2024 · 1 comment
Open
2 tasks done

Dynamic proxyProtocol.trustedIPs from the created service #1282

Gitopolis opened this issue Dec 13, 2024 · 1 comment
Labels
kind/proposal a proposal that needs to be discussed.

Comments

@Gitopolis
Copy link

Welcome!

  • Yes, I've searched similar issues on GitHub and didn't find any.
  • Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?

When I install traefik with this chart I can't know the list of trusted ips for the proxy protocol, because these ips will be available only after the LoadBalancer service is created. So as a workaround I first install traefik with the empty list, wait for the LoadBalancer to be created, then use dig to get the list of IPv4 and IPv6 addresses of my balancer. After that I can put them into the values file and redeploy the traefik.

The proposal is to automate this process. We can create a job that waits for the LB, and then update the traefik arguments. We can use ArgoCD sync waves to ensure the correct order in which the resources are created.

@mloiseleur mloiseleur added kind/proposal a proposal that needs to be discussed. and removed status/0-needs-triage labels Dec 16, 2024
@mloiseleur
Copy link
Contributor

Hello @Gitopolis,

That seems quite specific. 🤔 I am not sure if and how it could be implemented in this chart without adding a significant amount of complexity.

With this Chart, it's possible to disable the Service, and manage it. It seems possible to create a LoadBalancer Service before installing Traefik, get the IP (with dig or from the Service status field) and use this information as parameter in values of this Chart.

We are interested in this issue, but we’re unsure about the use case and the traction it will receive. We are going to leave the status as kind/proposal to give the community time to let us know if they would like this idea.

We will reevaluate as people respond.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/proposal a proposal that needs to be discussed.
Projects
None yet
Development

No branches or pull requests

2 participants