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(kuma-cp) updated go control plane #2082

Merged

Conversation

sudeeptoroy
Copy link
Contributor

@sudeeptoroy sudeeptoroy commented Jun 2, 2021

Signed-off-by: Sudeepto Roy sudeepto.roy@gmail.com

updated go-control-plane

Fix #2115

Signed-off-by: Sudeepto Roy <sudeepto.roy@gmail.com>
@sudeeptoroy sudeeptoroy requested a review from a team as a code owner June 2, 2021 10:57
jpeach
jpeach previously requested changes Jun 3, 2021
Copy link
Contributor

@jpeach jpeach left a comment

Choose a reason for hiding this comment

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

Please update pkg/test/xds/client/go.mod as well.

@sudeeptoroy
Copy link
Contributor Author

Hi @jpeach, continuing with the conversation on slack. Do you still suggest to update pkg/test/xds/client/go.mod

@jpeach
Copy link
Contributor

jpeach commented Jun 8, 2021

Hi @jpeach, continuing with the conversation on slack. Do you still suggest to update pkg/test/xds/client/go.mod

Since this is a temporary hack and @jakubdyszkiewicz approve, I'm fine with it. I think that updating this properly requires fixing #2087.

@jpeach jpeach dismissed their stale review June 8, 2021 22:40

out of band consensus

@jakubdyszkiewicz jakubdyszkiewicz merged commit 95c66c1 into kumahq:master Jun 9, 2021
@sudeeptoroy sudeeptoroy deleted the feat/upgrade-go-ctrl-plane branch July 26, 2021 14:46
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.

go-control-plane upgrade
3 participants