snapd remove /usr/local/bin from the PATH for all systemd unit (bionic SRU regression)
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
| initramfs-tools (Ubuntu) |
Undecided
|
Unassigned | ||
| Bionic |
Undecided
|
Unassigned | ||
| snapd (Ubuntu) |
High
|
Unassigned | ||
| Bionic |
Critical
|
Unassigned | ||
| systemd (Ubuntu) |
Undecided
|
Unassigned | ||
| Bionic |
Undecided
|
Unassigned |
Bug Description
Big regression in 2.37.1+18.04 compare to version 2.34.2
all these paths /usr/local/sbin & /usr/local/bin are not anymore in the path of all systemd process .
So we can not start a daemon that use /usr/local/bin
reinstalling package 2.34.2 fix the problem
in 2.34.2 :
~# strings /proc/$(pidof /lib/systemd/
PATH=/usr/
in 2.37.1+18.04 :
~# strings /proc/$(pidof /lib/systemd/
PATH=/sbin:
tags: | added: regression-update |
Oliver Grawert (ogra) wrote : | #1 |
Changed in snapd (Ubuntu): | |
importance: | Undecided → High |
summary: |
- snapd remove /usr/local/bin from the PATH for all systemd unit + snapd remove /usr/local/bin from the PATH for all systemd unit (bionic + SRU regression) |
E. MAS (erwan-mas) wrote : | #2 |
This is not a duplicate of 1771858 . This a critical regression from update for fixing 1771858 .
Since we have the bug we can not deploy anymoore kubernetes charm fro canonical .
E. MAS (erwan-mas) wrote : | #3 |
This is related to 1771858 , i agree .
Alexander Turek (turekaj) wrote : | #4 |
agree, cannot deploy Kubernetes charm from canonical, or add Kubernetes-worker units to existing deployment
Launchpad Janitor (janitor) wrote : | #5 |
Status changed to 'Confirmed' because the bug affects multiple users.
Changed in snapd (Ubuntu): | |
status: | New → Confirmed |
Dimitri John Ledkov (xnox) wrote : | #6 |
Hi,
I cannot reproduce the issue, but it might be relevant how the system is booted and/or was upgraded.
Can you do the following, please?
sudo systemd-run --unit=check-env /usr/bin/env
sudo journalctl --no-pager -u check-env
cat /etc/systemd/
cat /proc/cmdline
journalctl -b -k | grep -e initrd -e command
ls /usr/lib/
And paste the output here?
Michael Vogt (mvo) wrote : | #7 |
Here is what I see in a clean VM:
ubuntu@
Listing... Done
snapd/bionic-
N: There is 1 additional version. Please use the '-a' switch to see it
ubuntu@
Running as unit: run-u7.service
Press ^] three times within 1s to disconnect TTY.
/usr/local/
Finished with result: success
Main processes terminated with: code=exited/
Service runtime: 13ms
After just upgrading snapd (latest version) and rebooting:
ubuntu@
Listing... Done
snapd/bionic-
ubuntu@
Running as unit: run-u7.service
Press ^] three times within 1s to disconnect TTY.
/sbin:/
Finished with result: success
Main processes terminated with: code=exited/
Service runtime: 15ms
So this appears to be real. We added the systemd environment generator (/usr/lib/
Michael Vogt (mvo) wrote : | #8 |
As for the questions of xnox:
"""
ubuntu@
Running as unit: check-env.service
ubuntu@
-- Logs begin at Mon 2018-07-23 18:30:47 CEST, end at Sun 2019-02-03 13:25:21 CET. --
Feb 03 13:25:21 autopkgtest systemd[1]: Started /usr/bin/env.
Feb 03 13:25:21 autopkgtest env[543]: LANG=en_US.UTF-8
Feb 03 13:25:21 autopkgtest env[543]: PATH=/sbin:
Feb 03 13:25:21 autopkgtest env[543]: INVOCATION_
Feb 03 13:25:21 autopkgtest env[543]: JOURNAL_
ubuntu@
# This file is part of systemd.
#
# systemd is free software; you can redistribute it and/or modify it
# under the terms of the GNU Lesser General Public License as published by
# the Free Software Foundation; either version 2.1 of the License, or
# (at your option) any later version.
#
# Entries in this file show the compile time defaults.
# You can change settings by editing this file.
# Defaults can be restored by simply deleting this file.
#
# See systemd-
[Manager]
#LogLevel=info
#LogTarget=
#LogColor=yes
#LogLocation=no
#DumpCore=yes
#ShowStatus=yes
#CrashChangeVT=no
#CrashShell=no
#CrashReboot=no
#CtrlAltDelBurs
#CPUAffinity=1 2
#JoinController
#RuntimeWatchdo
#ShutdownWatchd
#CapabilityBoun
#SystemCallArch
#TimerSlackNSec=
#DefaultTimerAc
#DefaultStandar
#DefaultStandar
#DefaultTimeout
#DefaultTimeout
#DefaultRestart
#DefaultStartLi
#DefaultStartLi
#DefaultEnviron
#DefaultCPUAcco
#DefaultIOAccou
#DefaultIPAccou
#DefaultBlockIO
#DefaultMemoryA
#DefaultTasksAc
#DefaultTasksMax=
#DefaultLimitCPU=
#DefaultLimitFSIZE=
#DefaultLimitDATA=
#DefaultLimitSTACK=
#DefaultLimitCORE=
#DefaultLimitRSS=
#DefaultLimitNO
#DefaultLimitAS=
#DefaultLimitNPROC=
#DefaultLimitME
#DefaultLimitLOCKS=
#DefaultLimitSI
#DefaultLimitMS
#DefaultLimitNICE=
#DefaultLimitRT
#DefaultLimitRT
#IPAddressAllow=
#IPAddressDeny=
ubuntu@
BOOT_IMAGE=
ubuntu@
Feb 03 13:23:52 autopkgtest kernel: Kernel command line: BOOT_IMAGE=
Feb 03 13:23:52 autopkgtest kernel: Freeing initrd memory: 50848K
ubuntu@
snapd-env-generator
"""
Changed in snapd (Ubuntu Bionic): | |
importance: | Undecided → Critical |
Hello E., or anyone else affected,
Accepted snapd into bionic-proposed. The package will build now and be available at https:/
Please help us by testing this new package. See https:/
If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-
Further information regarding the verification process can be found at https:/
N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.
Changed in snapd (Ubuntu Bionic): | |
status: | New → Fix Committed |
tags: | added: verification-needed verification-needed-bionic |
Will Cooke (willcooke) wrote : | #10 |
Performing the same tests as mvo. Before upgrading...
$ apt list snapd
Listing... Done
snapd/bionic-
N: There are 2 additional versions. Please use the '-a' switch to see them.
$ sudo systemd-run --pty --wait '/bin/echo' '$PATH'
Running as unit: run-u920.service
Press ^] three times within 1s to disconnect TTY.
/sbin:/
Finished with result: success
Main processes terminated with: code=exited/
Service runtime: 10ms
Will Cooke (willcooke) wrote : | #11 |
After upgrade:
$ apt list snapd
Listing... Done
snapd/bionic-
N: There is 1 additional version. Please use the '-a' switch to see it
$ sudo systemd-run --pty --wait '/bin/echo' '$PATH'
[sudo] password for will:
Running as unit: run-u121.service
Press ^] three times within 1s to disconnect TTY.
/sbin:/
Finished with result: success
Main processes terminated with: code=exited/
Service runtime: 8ms
Will Cooke (willcooke) wrote : | #12 |
Ignore the previous comment, I still had the wrong version installed. Let's try that again...
$ apt list snapd
Listing... Done
snapd/bionic-
N: There are 2 additional versions. Please use the '-a' switch to see them.
$ sudo systemd-run --pty --wait '/bin/echo' '$PATH'
[sudo] password for will:
Running as unit: run-u120.service
Press ^] three times within 1s to disconnect TTY.
/usr/local/
Finished with result: success
Main processes terminated with: code=exited/
Service runtime: 13ms
tags: |
added: verification-done-bionic removed: verification-needed-bionic |
tags: |
added: verification-done removed: verification-needed |
Will Cooke (willcooke) wrote : | #13 |
The path is correct again. Marking as verification-done.
Launchpad Janitor (janitor) wrote : | #15 |
Changed in snapd (Ubuntu Bionic): | |
status: | Fix Committed → Fix Released |
The verification of the Stable Release Update for snapd has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.
E. MAS (erwan-mas) wrote : | #16 |
The release of 2.37.1.1 fixed my issue .
Dimitri John Ledkov (xnox) wrote : | #17 |
Also initramfs-tools sets PATH without /usr/local, which may leak into init environment. Fixing this as well.
Launchpad Janitor (janitor) wrote : | #18 |
This bug was fixed in the package initramfs-tools - 0.131ubuntu17
---------------
initramfs-tools (0.131ubuntu17) disco; urgency=medium
* Fix PATH to include /usr/local. LP: #1814355
-- Dimitri John Ledkov <email address hidden> Tue, 12 Feb 2019 13:16:06 +0000
Changed in initramfs-tools (Ubuntu): | |
status: | New → Fix Released |
likely fallout of the fix from 1771858