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(db) remove pre-workspace blue-green deployment workaround #8896

Merged
merged 1 commit into from
Jun 14, 2022

Conversation

bungle
Copy link
Member

@bungle bungle commented Jun 3, 2022

Summary

When Kong 2.1.0 was release we needed to add special workaround to enable blue-green deployment with the introduction of workspaces.

We needed to add special select_by_cache_key to Plugins DAO. The check that we added is hurting the Kong performance, as seen reported in #8788. We got further PR from the author with #8789. While the fix is good, I do think we can remove this by now altogether. The only downside of this, is that for people who migrate directly from Kong < 2.1.x to 3.0.0, we don't support the blue-green deployment.

Issues Resolved

Fixes #8788
Closes #8789

@bungle bungle requested a review from a team as a code owner June 3, 2022 08:34
@bungle bungle force-pushed the chore/pre-workspace-workaround-removal branch from e6233c8 to 7c127ef Compare June 3, 2022 09:21
@bungle
Copy link
Member Author

bungle commented Jun 3, 2022

Alternatively, we need to merge #8789.

### Summary

When Kong 2.1.0 was release we needed to add special workaround to enable
blue-green deployment with the introduction of workspaces.

We needed to add special `select_by_cache_key` to `Plugins` DAO. The check
that we added is hurting the Kong performance, as seen reported in #8788.
We got further PR from the author with #8789. While the fix is good,
I do think we can remove this by now altogether. The only downside of this
is that is people migrate directly from Kong < 2.1.x to `3.0.0` we don't
support the blue-green deployment.

### Issues Resolved

Fixes #8788
Closes #8789
@bungle bungle force-pushed the chore/pre-workspace-workaround-removal branch from 7c127ef to 2ca39e1 Compare June 3, 2022 13:16
@bungle bungle merged commit 9ac6cc2 into master Jun 14, 2022
@bungle bungle deleted the chore/pre-workspace-workaround-removal branch June 14, 2022 08:25
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.

Plugins/DAO: incorrect function field reassignment
2 participants