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

feat: make expressions router default #939

Closed
wants to merge 5 commits into from

Conversation

czeslavo
Copy link
Contributor

@czeslavo czeslavo commented Nov 9, 2023

What this PR does / why we need it:

Sets the expression router as default in kong/kong. Adds validations that ensure that:

  • expressions router flavor is not used with KIC < 2.10 as it's not supported
  • when expressions router flavor is used with KIC < 3.0, the feature gate must be set.

to both kong/kong and kong/ingress charts.

Validations will output instructions on how to fix the issue if it is encountered.

It also changes the way we refer to kong/kong in kong/ingress's Chart.yaml: instead of using the repository URL that forces us to bump charts in two steps, use file:// to make it possible in one step.

Which issue this PR fixes

Should solve #1188.

Special notes for your reviewer:

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

@czeslavo czeslavo force-pushed the ingress-router-flavor-expressions branch from 488dfd3 to 1f0dc28 Compare November 9, 2023 16:07
@czeslavo czeslavo self-assigned this Nov 9, 2023
@czeslavo czeslavo added the enhancement New feature or request label Nov 9, 2023
@czeslavo czeslavo force-pushed the ingress-router-flavor-expressions branch 4 times, most recently from 5f529f9 to 4b2e7f2 Compare November 9, 2023 17:08
@czeslavo czeslavo force-pushed the ingress-router-flavor-expressions branch from 4b2e7f2 to 830bccb Compare November 9, 2023 17:19
@czeslavo czeslavo enabled auto-merge (squash) November 9, 2023 17:22
@@ -44,3 +44,4 @@ gateway:
env:
role: traditional
database: "off"
router_flavor: "expressions"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we add a template to check image of KIC?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please elaborate on what you mean by this? There are two validation rules added in validations.tpl file: one for KIC < 2.10 and one for KIC < 3.0.

@czeslavo czeslavo requested review from randmonkey and a team November 13, 2023 08:19
@czeslavo czeslavo changed the title feat: make expressions router default in kong/ingress feat: make expressions router default Nov 13, 2023
@czeslavo czeslavo force-pushed the ingress-router-flavor-expressions branch from 1becd23 to d61f920 Compare November 13, 2023 10:11
@czeslavo czeslavo force-pushed the ingress-router-flavor-expressions branch from d61f920 to d777988 Compare November 13, 2023 10:13
@czeslavo
Copy link
Contributor Author

Per @mheap, I'm labeling this with HOLD/DO NOT MERGE until Konnect is compatible with expressions.

@pmalek
Copy link
Member

pmalek commented Nov 20, 2023

Could potentially be put in kong v3 milestone

@czeslavo czeslavo closed this Mar 27, 2024
auto-merge was automatically disabled March 27, 2024 10:10

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request HOLD/DO NOT MERGE
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants