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.

Sign up for GitHub

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

Jump to bottom

Update gocql to resolve #12879

Closed
wants to merge 1 commit into from
Closed

Update gocql to resolve #12879

wants to merge 1 commit into from

Conversation

qk4l
Copy link
Contributor

@qk4l qk4l commented Oct 20, 2021

Resolve #12878

@vercel vercel bot temporarily deployed to Preview – vault-storybook October 20, 2021 09:24 Inactive
@vercel vercel bot temporarily deployed to Preview – vault October 20, 2021 09:24 Inactive
@qk4l qk4l changed the title Update gocql to resolve #12878 Update gocql to resolve Oct 20, 2021
@hsimon-hashicorp
Copy link
Contributor

Thank you for this submission! We will get it reviewed soon. :)

@player1-github player1-github mentioned this pull request Jan 18, 2022
Open
HridoyRoy
HridoyRoy reviewed Jan 21, 2022
View reviewed changes
Copy link
Contributor

@HridoyRoy HridoyRoy left a comment

Choose a reason for hiding this comment

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

Can we resolve the go.mod conflicts and re-run the go get? I'm happy to approve the PR after that.

@vercel vercel bot temporarily deployed to Preview – vault-storybook February 3, 2022 17:01 Inactive
@qk4l qk4l requested a review from a team February 4, 2022 07:41
@vercel vercel bot temporarily deployed to Preview – vault-storybook February 4, 2022 07:41 Inactive
@vercel vercel bot temporarily deployed to Preview – vault-storybook February 4, 2022 08:00 Inactive
@qk4l
Copy link
Contributor Author

qk4l commented Feb 4, 2022

Hi,

@HridoyRoy I've updated PR to resolve merge conflict.

@hghaf099
Copy link
Contributor

hghaf099 commented Feb 4, 2022

@qk4l Would you please add a change log indicating the update on those go packages?

@conradkleinespel
Copy link
Contributor

@HridoyRoy @hghaf099 I have re-created this Pull Request here, with the changelog entry added: #14973

Let me know if this helps or what else I can do to help get this fix merged.

@conradkleinespel conradkleinespel mentioned this pull request Apr 8, 2022
Merged
@swenson
Copy link
Contributor

swenson commented Apr 18, 2022

Closing in favor of #14973, which was just merged.

Thanks everyone!

@swenson swenson closed this Apr 18, 2022
This was referenced Apr 18, 2022
Merged
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@HridoyRoy HridoyRoy HridoyRoy left review comments

@acahn acahn Awaiting requested review from acahn

@austingebauer austingebauer Awaiting requested review from austingebauer

@taoism4504 taoism4504 Awaiting requested review from taoism4504

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Cassandra backend: error=gocql: no hosts available in the pool
6 participants
@qk4l @hsimon-hashicorp @hghaf099 @conradkleinespel @swenson @HridoyRoy