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

Update Enterprise manifest examples and docs #909

Merged
merged 4 commits into from
Oct 23, 2020
Merged

Conversation

rainest
Copy link
Contributor

@rainest rainest commented Oct 14, 2020

What this PR does / why we need it:

  • Updates Enterprise manifests to use kong-enterprise-edition at the current latest patch version.
  • Updates registry secret instructions to use kong-enterprise-edition.

I missed the KIC repo version/registry in the course of updating various pieces of documentation to reflect the switch from kong-enterprise-k8s to kong-enterprise-edition, which broke instructions elsewhere that were updated when attempting to deploy using the example Enterprise manifests here.

That may have been intentional when 2.1 Enterprise was still in beta. It no longer is, so everything should now reflect the current 2.1 kong-enterprise-edition guidance for new installs. I think this was the last of the holdouts 🤞

Special notes for your reviewer:
@lena-larionova not actually adding this to the changelog here since there's no good release to put it in. Doc fixes in this repo are a bit odd, since changelog items are tied to code releases, and doc fixes usually go on top of the current tip of main regardless of whether they accompany a release. 1.0.1 or 1.1.0 (depending on whether or not we do a patch release) notes should include it.

Travis Raines added 2 commits October 14, 2020 11:11
Change Enterprise manifests from kong-enterprise-k8s to
kong-enterprise-edition.

Update tag to current latest 2.1 patch.
lena-larionova
lena-larionova previously approved these changes Oct 15, 2020
Copy link
Contributor

@lena-larionova lena-larionova left a comment

Choose a reason for hiding this comment

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

Changelog comment makes sense to me. I think the changes here should be sufficient to stop most of the confusion anyway.

@rainest rainest merged commit 8c94846 into main Oct 23, 2020
@rainest rainest deleted the fix/enterprise-version branch October 23, 2020 18:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants