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

Generic handling of server region errors #4087

Merged
merged 2 commits into from
Mar 30, 2018
Merged

Generic handling of server region errors #4087

merged 2 commits into from
Mar 30, 2018

Conversation

dadgar
Copy link
Contributor

@dadgar dadgar commented Mar 30, 2018

Does not early exit of errors and instead collects all and displays a warning message:

nomad server members
Name            Address         Port  Status  Leader  Protocol  Build      Datacenter  Region
server2.foo     192.168.18.171  5658  alive   false   2         0.8.0-dev  dc1         foo
server1.global  192.168.18.171  4648  alive   true    2         0.8.0-dev  dc1         global

Error determining leaders: 1 error(s) occurred:

* Region "foo": Unexpected response code: 500 (rpc error: failed to get conn: dial tcp 192.168.18.171:5657: connect: connection refused)

@dadgar dadgar force-pushed the b-server-members branch from 6fdcd3b to 9f99b9d Compare March 30, 2018 17:57
@dadgar dadgar merged commit d681330 into master Mar 30, 2018
@dadgar dadgar deleted the b-server-members branch March 30, 2018 19:35
@github-actions
Copy link

github-actions bot commented Mar 7, 2023

I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 7, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants