haproxy missing install dependency
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
haproxy (Juju Charms Collection) |
Fix Released
|
Undecided
|
Matt Bruzek |
Bug Description
Hi,
I used the haproxy charm on digitalocean and noticed that it's missing a dependency on python-yaml. That package is probably installed in the ubuntu cloud images elsewhere, but not in DO's:
2014-04-04 14:20:19 INFO juju.worker.uniter uniter.go:469 running "install" hook
2014-04-04 14:20:19 INFO worker.uniter.jujuc server.go:103 running hook tool "juju-log" ["Invoking charm-pre-install hooks"]
2014-04-04 14:20:19 DEBUG worker.uniter.jujuc server.go:104 hook context id "haproxy/
2014-04-04 14:20:19 INFO juju-log Invoking charm-pre-install hooks
2014-04-04 14:20:19 INFO worker.uniter.jujuc server.go:103 running hook tool "juju-log" ["Invoking python-based install hook"]
2014-04-04 14:20:19 DEBUG worker.uniter.jujuc server.go:104 hook context id "haproxy/
2014-04-04 14:20:19 INFO juju-log Invoking python-based install hook
2014-04-04 14:20:19 INFO install Traceback (most recent call last):
2014-04-04 14:20:19 INFO install File "hooks/hooks.py", line 11, in <module>
2014-04-04 14:20:19 INFO install import yaml
2014-04-04 14:20:19 INFO install ImportError: No module named yaml
2014-04-04 14:20:19 ERROR juju.worker.uniter uniter.go:475 hook failed: exit status 1
Suggested fix:
--- install.orig 2014-04-04 10:38:05.260191000 -0400
+++ install 2014-04-04 10:38:19.972191000 -0400
@@ -6,6 +6,8 @@
DEBIAN_
}
+apt_get_install python-yaml
+
juju-log 'Invoking charm-pre-install hooks'
[ -d exec.d ] && ( for f in exec.d/
Related branches
- José Antonio Rey (community): Approve
-
Diff: 12 lines (+3/-0)1 file modifiedhooks/install (+3/-0)
tags: | added: landscape |
Changed in haproxy (Juju Charms Collection): | |
status: | New → Confirmed |
assignee: | nobody → Matt Bruzek (mbruzek) |
Changed in haproxy (Juju Charms Collection): | |
status: | Confirmed → Fix Released |
This was fixed in an unrelated bug against the precise series.