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

Ingress annotations #1903

Merged

Schnitzel merged 3 commits into master from ingress-annotations

May 28, 2020

Contributor

Schnitzel commented May 22, 2020

Checklist

Affected Issues have been mentioned in the Closing issues section
Documentation has been written/updated
PR title is ready for changelog and subsystem label(s) applied

Explain the details for making this change. What existing problem does the pull request solve?

Closing issues

Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such).

Schnitzel added 3 commits

May 21, 2020 22:35


          remove outputToYaml and use a folder with multiple files

93db6e7

outputToYaml unfortunately swallows some newline characters, causing all kind of fun issues


          remmove double tls-acme annotation

cf4d3e6


          fix cat of yaml files

75984ee

Schnitzel added this to the v1.6.0 milestone

Schnitzel added the 2-build-deploy label

Schnitzel merged commit 981ce14 into master

tobybellwood deleted the ingress-annotations branch

July 2, 2020 02:31

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