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

Switch to kubernetes cluster dns for incluster url everywhere #876

Merged
merged 3 commits into from
Apr 13, 2022
Merged

Switch to kubernetes cluster dns for incluster url everywhere #876

merged 3 commits into from
Apr 13, 2022

Commits on Apr 13, 2022

  1. Switch to kubernetes dns for incluster url everywhere 

    This is the recommended, and only documented method on https://kubernetes.io/docs/tasks/run-application/access-api-from-pod/
The legacy method has issues with ipv6 and it's time to retire it.

We trialled the new method for 6months via #587 without any reports.

Closes #874

Signed-off-by: clux <sszynrae@gmail.com>
    @clux
    clux committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    9916723 View commit details
    Browse the repository at this point in the history

  2. remove code for legacy methods, was never actually made public 

    Signed-off-by: clux <sszynrae@gmail.com>
    @clux
    clux committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    258fd52 View commit details
    Browse the repository at this point in the history

  3. simplify kube_dns fn with less unwraps 

    Signed-off-by: clux <sszynrae@gmail.com>
    @clux
    clux committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    e529292 View commit details
    Browse the repository at this point in the history