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

Pull in hcat bug fix on Consul connect timeouts #133

Merged
merged 2 commits into from
Nov 4, 2020

Conversation

findkim
Copy link
Contributor

@findkim findkim commented Nov 3, 2020

The changes fixes the case where CTS would hang indefinitely when
running on a machine with DNS issues that cause timeouts while
connecting to Consul. Added a log line to provide users context
about attempting Consul connection for easier debugging connection
issues instead of hanging silently.

Context: hashicorp/hcat#23

The changes fixes the case where CTS would hang indefinitely when
running on a machine with DNS issues that cause timeouts while
connecting to Consul. Added a log line to provide users context
about attempting Consul connection for easier debugging connection
issues instead of hanging silently.
@findkim findkim added the bug Something isn't working label Nov 3, 2020
@findkim findkim added this to the Technology Preview 2 milestone Nov 3, 2020
@findkim findkim requested a review from a team November 3, 2020 00:02
@findkim
Copy link
Contributor Author

findkim commented Nov 3, 2020

Looks like i need to update the template helper functions with recent changes to hcat lib.

@findkim
Copy link
Contributor Author

findkim commented Nov 4, 2020

Updated! hcat helper funcs were friendly to include 🎉

Copy link
Member

@lornasong lornasong left a comment

Choose a reason for hiding this comment

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

looks good to me!

@findkim findkim merged commit 9c18b74 into master Nov 4, 2020
@findkim findkim deleted the fix-consul-connection branch November 4, 2020 16:20
sarahethompson pushed a commit that referenced this pull request Nov 6, 2023
Update TFC driver with SuppressNotification notifier for scheduled tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants