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

tctl silently exits when the server is down #28129

Closed
Tener opened this issue Jun 22, 2023 · 0 comments · Fixed by #29281
Closed

tctl silently exits when the server is down #28129

Tener opened this issue Jun 22, 2023 · 0 comments · Fixed by #29281
Labels
bug good-starter-issue Good starter issue to start contributing to Teleport tctl tctl - Teleport admin tool

Comments

@Tener
Copy link
Contributor

Tener commented Jun 22, 2023 •
edited by webvictim

Expected behavior:

If the server is down, and tctl exits due to that fact, it should show an appropriate error message.

Current behavior:

No message is shown unless --debug is provided.

Bug details:

Logs:

> tctl get all
> tctl get all --debug
2023-06-22T11:15:40+02:00 DEBU             Debug logging has been enabled. common/tctl.go:239
2023-06-22T11:15:40+02:00 DEBU             reading config file: /etc/teleport.yaml config/configuration.go:214
2023-06-22T11:15:40+02:00 ERRO             "Cannot connect to the auth server. Is the auth server running on \"boson.tener.io:3025\"? failed direct dial to auth server: dial tcp 192.168.7.186:3025: connect: connection refused\n\tdial tcp 192.168.7.186:3025: connect: connection refused" common/tctl.go:202

cc @stevenGravy
@Tener Tener added bug good-starter-issue Good starter issue to start contributing to Teleport labels Jun 22, 2023
@webvictim webvictim changed the title tctl silently exists when the server is down tctl silently exits when the server is down Jun 22, 2023
@zmb3 zmb3 added the tctl tctl - Teleport admin tool label Jun 26, 2023
@zmb3 zmb3 mentioned this issue Jul 18, 2023
Merged
zmb3 added a commit that referenced this issue Jul 19, 2023
@zmb3
tctl: fix error reporting when server is down
8b35203 
A regresion in #25780 changed text that used to be reported to
standard output to go to the logger instead. Since the logger
output is only visible when the debug flag is set, this caused
a fatal error to go unreported to the user.

Fixes #28129
github-merge-queue bot pushed a commit that referenced this issue Jul 19, 2023
@zmb3
tctl: fix error reporting when server is down (#29281)
bdcb7f4 
A regresion in #25780 changed text that used to be reported to
standard output to go to the logger instead. Since the logger
output is only visible when the debug flag is set, this caused
a fatal error to go unreported to the user.

Fixes #28129
github-actions bot pushed a commit that referenced this issue Jul 19, 2023
@zmb3
tctl: fix error reporting when server is down
76ad093 
A regresion in #25780 changed text that used to be reported to
standard output to go to the logger instead. Since the logger
output is only visible when the debug flag is set, this caused
a fatal error to go unreported to the user.

Fixes #28129
zmb3 added a commit that referenced this issue Jul 19, 2023
@zmb3
tctl: fix error reporting when server is down (#29281)
49bd377 
A regresion in #25780 changed text that used to be reported to
standard output to go to the logger instead. Since the logger
output is only visible when the debug flag is set, this caused
a fatal error to go unreported to the user.

Fixes #28129
@zmb3 zmb3 mentioned this issue Jul 19, 2023
Merged
github-merge-queue bot pushed a commit that referenced this issue Jul 19, 2023
@zmb3
tctl: fix error reporting when server is down (#29281) (#29324)
c9b322d 
A regresion in #25780 changed text that used to be reported to
standard output to go to the logger instead. Since the logger
output is only visible when the debug flag is set, this caused
a fatal error to go unreported to the user.

Fixes #28129
github-actions bot pushed a commit that referenced this issue Jul 19, 2023
@zmb3
tctl: fix error reporting when server is down (#29281)
571402f 
A regresion in #25780 changed text that used to be reported to
standard output to go to the logger instead. Since the logger
output is only visible when the debug flag is set, this caused
a fatal error to go unreported to the user.

Fixes #28129
github-merge-queue bot pushed a commit that referenced this issue Jul 20, 2023
@zmb3
tctl: fix error reporting when server is down (#29322)
8086784 
A regresion in #25780 changed text that used to be reported to
standard output to go to the logger instead. Since the logger
output is only visible when the debug flag is set, this caused
a fatal error to go unreported to the user.

Fixes #28129
github-merge-queue bot pushed a commit that referenced this issue Jul 20, 2023
@zmb3
tctl: fix error reporting when server is down (#29281) (#29329)
2049651 
A regresion in #25780 changed text that used to be reported to
standard output to go to the logger instead. Since the logger
output is only visible when the debug flag is set, this caused
a fatal error to go unreported to the user.

Fixes #28129
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug good-starter-issue Good starter issue to start contributing to Teleport tctl tctl - Teleport admin tool
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

tctl: fix error reporting when server is down gravitational/teleport
2 participants
@Tener @zmb3