-
Notifications
You must be signed in to change notification settings - Fork 474
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
Release consolidated chart as kong 3.0.0-rc1 #976
Conversation
27514ec
to
cdd7b5a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I approve this change with a big caveat: we understand that the 3.0 release must support various named use cases and this will take one of two forms, and we understand that the former is more likely
- differently named charts (kong/ingress
, kong/konnect-dp
, kong/operator
, etc) with "batteries included" using default values.yaml
- same chart but with different values.yaml examples
Therefore
- I've asked that we remove the definitive language that might otherwise be forgotten to update before we reach a conclusive decision about the 3.0 shape
- we'll continue the march towards 3.0 in Next-gen (3.0) chart layout #977
1c8ddc3
to
6d3608b
Compare
Copy pieces of the ingress chart Konnect example into the kong chart Konnect example.
Add migration notes for the ingress to kong 3.x change. Remove separate ingress templates and such.
Co-authored-by: Michał Flendrich <michal.flendrich@konghq.com>
6d3608b
to
f2ddcbc
Compare
58c6e63
to
1edf3f7
Compare
What this PR does / why we need it:
Releases the consolidated chart as kong 3.0.0-rc1.
Removes most existing
ingress
chart content and consolidates someingress
documentation into thekong
files. Leaves a brief migration message and historical underingress
.This will release an RC chart version from the release branch.
main
will remain as-is for now. When we are ready to finalize the release, we will cut an old version release branch from the tip ofmain
before merging this intomain
.Special notes for your reviewer:
See the other PR. The name for the consolidated chart isn't particularly important, but since it is built from the
kong
templates, releasing askong
requires much less trickery behind the scenes. There's also much less docs burden if existingkong
locations continue to exist.