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

chore(env) update router default #911

Closed
wants to merge 2 commits into from
Closed

Conversation

rainest
Copy link
Contributor

@rainest rainest commented Oct 20, 2023

What this PR does / why we need it:

Changes the router default to expressions.

Notes this as a breaking change in unreleased release notes.

Which issue this PR fixes

Part of Kong/kubernetes-ingress-controller#4719

Special notes for your reviewer:

Ditto open questions in Kong/kubernetes-ingress-controller#4934

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • PR is based off the current tip of the main branch.
  • Changes are documented under the "Unreleased" header in CHANGELOG.md
  • New or modified sections of values.yaml are documented in the README.md
  • Commits follow the Kong commit message guidelines

@rainest rainest requested a review from a team as a code owner October 20, 2023 23:31
@rainest rainest force-pushed the chore/router-default branch 2 times, most recently from 68698bc to 3a9ca76 Compare October 23, 2023 08:37
charts/kong/CHANGELOG.md Outdated Show resolved Hide resolved
@rainest rainest marked this pull request as draft October 23, 2023 09:25
@rainest rainest force-pushed the chore/router-default branch from ad18df5 to c2289d7 Compare October 23, 2023 09:29
@rainest
Copy link
Contributor Author

rainest commented Jan 6, 2024

Given Kong/kubernetes-ingress-controller#5018 (comment) we're presumably doing a bit more around this other than just flipping it. Nothing of particular interest here anyway, so just closing this to clean up stale PRs.

@rainest rainest closed this Jan 6, 2024
@rainest rainest deleted the chore/router-default branch January 6, 2024 00:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants