plans: set sbin-inclusive PATH for test execution (fix tmt 1.75.0 regression)#65
Open
andrewlukoshko wants to merge 2 commits into
Open
plans: set sbin-inclusive PATH for test execution (fix tmt 1.75.0 regression)#65andrewlukoshko wants to merge 2 commits into
andrewlukoshko wants to merge 2 commits into
Conversation
added 2 commits
June 7, 2026 18:48
tmt 1.75.0 stopped leaking the runner's process environment into test commands (teemtee/tmt#4900), so /usr/sbin and /sbin are no longer on PATH by default. The legacy t_functional scripts and several ng tests invoke privileged tools (useradd, ip, service, auditctl, setsebool, tcpdump, mdadm, cracklib-check, bpf-*, ...) by bare name and broke with 'command not found' on tmt >= 1.75.0, while passing on 1.74.0. Set an explicit sbin-inclusive PATH in the legacy and ng plan environments so the suite is independent of tmt's env-forwarding behavior.
Same tmt 1.75.0 regression (teemtee/tmt#4900): HOME is no longer forwarded to test commands. p_go's 'go build' then fails with 'build cache is required ... neither $XDG_CACHE_HOME nor $HOME are defined', leaving /var/tmp/go-test uncreated. Restore HOME=/root (the value tmt forwarded before 1.75.0) in both plan environments.
Member
Author
|
Follow-up: added `HOME: "/root"` to both plan environments. The PATH fix cleared all the
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
The compose-tests suite started failing on the same commit that previously passed:
Both runs are the same commit (
1b49e3e) with the same test scripts, so the regression is environmental.Every failing test fails with
command not found(or a missing artifact downstream of one) on a binary that lives in/usr/sbinor/sbin:p_auditauditctl(auditd never starts)p_bridge-utils,p_network,p_tcpdumpip,ifconfig,tcpdumpp_openssh,p_passwd,p_shadow-utils,p_dovecotuseradd,userdel,usermod,runuserp_initscripts,p_tftp-serverservicep_nfs,p_vsftpdexportfs,setseboolp_cracklibcracklib-checkp_logwatch,p_iptraf,p_mdadmlogwatch,iptraf-ng,mdadm/tests/podmanuseradd,usermod,userdel/tests/libbpf-toolsbpf-*helpersRoot cause
The workflow
pip installstmtunpinned. Between the two runs it moved 1.74.0 → 1.75.0, which shipped teemtee/tmt#4900 — "Do not populate the default command environment with tmt process envvars." Commands run on the guest no longer inherit tmt'sos.environ, so the runner'sPATH(which included/usr/sbin:/sbin) is no longer forwarded. The legacyt_functionalscripts and several ng tests call privileged tools by bare name and depend on sbin being onPATH.Fix
Set an explicit sbin-inclusive
PATHin thelegacyandngplan environments, making the suite independent of tmt's env-forwarding behavior (robust across future tmt upgrades).