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(clustering): remove legacy hybrid protocol #9740

Merged
merged 2 commits into from
Nov 16, 2022

Conversation

flrgh
Copy link
Contributor

@flrgh flrgh commented Nov 10, 2022

This PR removes the legacy hybrid protocol.

Some code was moved out of kong.clustering.control_plane into kong.clustering.utils, but aside from that, I have done as little refactoring as possible in order to keep the PR simple.

@chronolaw
Copy link
Contributor

I am very glad to see these changes, it do make our hybrid code more clean.

The only question is when it will be approved?

@flrgh flrgh requested a review from hbagdi November 10, 2022 22:40
@flrgh flrgh marked this pull request as ready for review November 15, 2022 18:42
@flrgh
Copy link
Contributor Author

flrgh commented Nov 15, 2022

Reviewers: I'm aware this PR lacks a changelog entry and updates to the upgrade doc.

I have an internal ticket assigned to myself for those items and will open a follow-up PR for them.

@flrgh flrgh added this to the 3.1 milestone Nov 15, 2022
@flrgh flrgh requested a review from fffonion November 15, 2022 21:09
Copy link
Contributor

@chronolaw chronolaw left a comment

Choose a reason for hiding this comment

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

It's great.

@flrgh flrgh force-pushed the refactor/remove-legacy-hybrid-protocol branch from 00ff3df to 644ca5d Compare November 16, 2022 21:24
@flrgh flrgh force-pushed the refactor/remove-legacy-hybrid-protocol branch from 644ca5d to 6a2101e Compare November 16, 2022 21:32
Copy link
Member

@kikito kikito left a comment

Choose a reason for hiding this comment

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

approved with a minor comment

kong/clustering/init.lua Show resolved Hide resolved
@flrgh flrgh merged commit 5d9de28 into master Nov 16, 2022
@flrgh flrgh deleted the refactor/remove-legacy-hybrid-protocol branch November 16, 2022 22:52
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.

3 participants