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

🐛 interactive installer hangs if terminal type is unsupported #8205

Closed
Tracked by #8010
nberlee opened this issue Jan 25, 2024 · 0 comments · Fixed by #8221
Closed
Tracked by #8010

🐛 interactive installer hangs if terminal type is unsupported #8205

nberlee opened this issue Jan 25, 2024 · 0 comments · Fixed by #8221
Assignees

Comments

@nberlee
Copy link
Contributor

nberlee commented Jan 25, 2024

Bug Report

When TERM is set to an unsupported value, talosctl apply-config --mode interactive will hang.

Description

My Terminal sets TERM=alacritty, when running talosctl dashboard it cleanly errors out with terminal type unsupported, which reminds me to set TERM to linux. However, when I use talosct apply-config --mode interactive with an unsupported terminal it will just hang.

I would prefer to be reminded that its actually my fault that is not working, just like dashboard, instead of thinking I have a connection problem.

Logs

Environment

  • Talos version:1.6.3
@nberlee nberlee changed the title interactive installer hangs if terminal type is unsupproted interactive installer hangs if terminal type is unsupported Jan 25, 2024
@smira smira changed the title interactive installer hangs if terminal type is unsupported 🐛 interactive installer hangs if terminal type is unsupported Jan 26, 2024
@smira smira self-assigned this Jan 31, 2024
smira added a commit to smira/talos that referenced this issue Jan 31, 2024
In the previous implementation, even though `installer.err` was set, it
was never checked 🤦.

The run loop was stolen from the dashboard code.

Fixes siderolabs#8205

Signed-off-by: Andrey Smirnov <andrey.smirnov@siderolabs.com>
smira added a commit to smira/talos that referenced this issue Feb 1, 2024
In the previous implementation, even though `installer.err` was set, it
was never checked 🤦.

The run loop was stolen from the dashboard code.

Fixes siderolabs#8205

Signed-off-by: Andrey Smirnov <andrey.smirnov@siderolabs.com>
(cherry picked from commit 593afee)
smira added a commit to smira/talos that referenced this issue Feb 1, 2024
In the previous implementation, even though `installer.err` was set, it
was never checked 🤦.

The run loop was stolen from the dashboard code.

Fixes siderolabs#8205

Signed-off-by: Andrey Smirnov <andrey.smirnov@siderolabs.com>
(cherry picked from commit 593afee)
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 7, 2024
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 a pull request may close this issue.

2 participants