script execution should include snappy path (/snap/bin)
Bug #1635634 reported by
Andreas Hasenack
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Landscape Client |
Fix Committed
|
Undecided
|
Alberto Donato | ||
landscape-client (Ubuntu) |
Fix Released
|
Undecided
|
Andreas Hasenack |
Bug Description
The current PATH value used for landscape executed scripts is set like this:
constants.
It does not include "/snap/bin", needed by snaps like livepatch. We should add /snap/bin to the end of that list.
The snapd package expands PATH by dropping this file:
# dpkg -S apps-bin-path.sh
snapd: /etc/profile.
Which does:
apps-bin-
Related branches
lp:~ack/landscape-client/snap-bin-path
- Bogdana Vereha (community): Approve
- 🤖 Landscape Builder: Approve (test results)
- Geoff Teale (community): Approve
-
Diff: 31 lines (+9/-1)2 files modifiedlandscape/constants.py (+3/-1)
landscape/manager/tests/test_scriptexecution.py (+6/-0)
Changed in landscape-client: | |
assignee: | nobody → Alberto Donato (ack) |
status: | New → In Progress |
Changed in landscape-client: | |
status: | In Progress → Fix Committed |
Changed in landscape-client (Ubuntu): | |
assignee: | nobody → Andreas Hasenack (ahasenack) |
status: | New → In Progress |
To post a comment you must log in.
Let's not re-work how we determine path, just add /snap/bin to the hardcoded list we have now please.