ubuntu-repository-cache: hard-coded cron path to juju-run is wrong for juju v1.24
Bug #1455649 reported by
Robert C Jennings
on 2015-05-15
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| | ubuntu-repository-cache (Juju Charms Collection) |
Critical
|
Dan Watkins | ||
Bug Description
Seen on juju-1.24-beta1.2, juju-run has moved from /usr/local/bin to /usr/bin.
The charm is using an absolute path to juju-run when creating the cronjob on the leader which will be incorrect for juju 1.24.
The current contents of /etc/cron.
/usr/local/
Need to change this to call juju-run without the path and just depend on $PATH.
Related branches
lp:~daniel-thewatkins/charms/trusty/ubuntu-repository-cache/fix_cron_path
- José Antonio Rey: Approve on 2015-06-04
- Robert C Jennings (community): Approve on 2015-06-04
- Adam Israel: Approve on 2015-06-04
-
Diff: 8 lines (+2/-1)1 file modifiedtemplates/cron/ubuntu-repository-cache_rsync.cron (+2/-1)
Robert C Jennings (rcj)
on 2015-05-15
| Changed in ubuntu-repository-cache (Juju Charms Collection): | |
| assignee: | nobody → Robert C Jennings (rcj) |
| status: | New → In Progress |
Robert C Jennings (rcj)
on 2015-05-29
| Changed in ubuntu-repository-cache (Juju Charms Collection): | |
| importance: | Undecided → Critical |
| Marco Ceppi (marcoceppi) wrote : | #1 |
Patricia Gaughen (gaughen)
on 2015-05-29
| Changed in ubuntu-repository-cache (Juju Charms Collection): | |
| assignee: | Robert C Jennings (rcj) → Dan Watkins (daniel-thewatkins) |
| Robert C Jennings (rcj) wrote : | #2 |
Thanks Marco, that's the plan to remove the hard-coded path in the cron template.
José Antonio Rey (jose)
on 2015-06-04
| Changed in ubuntu-repository-cache (Juju Charms Collection): | |
| status: | In Progress → Fix Released |
To post a comment you must log in.
In generaly you should never hard code paths for juju tools as they will (and have in the past and now) change. Best to setup proper path envs in cron instead