[pve-devel] [PATCH installer 0/4] fix #4872: properly timeout `traceroute` command in coutry detection

Christoph Heiss c.heiss at proxmox.com
Fri Feb 9 11:55:57 CET 2024


For all the details, see patch #4.

TL;DR: SIGALRM does not interrupt line reading using <>, causing the
installer to hang on country detection. Fix it by using
Proxmox::Sys::Command::run_command(), which properly interacts with
SIGALRM.

Patch #1 is a rather mundane fix for some niche cases, #2 and #3 are
simply preparatory and to not alter existing behaviour.

Christoph Heiss (4):
  low-level: initialize UI backend for 'dump-env' subcommand too
  sys: command: allow terminating the process early from log subroutine
  sys: command: add option to not print process output to stdout
  fix #4872: run env: use run_command() for country detection

 Proxmox/Install/RunEnv.pm   | 23 ++++++++++++-----------
 Proxmox/Sys/Command.pm      | 13 ++++++++++---
 proxmox-low-level-installer |  1 +
 3 files changed, 23 insertions(+), 14 deletions(-)

--
2.43.0





More information about the pve-devel mailing list