=== CyberJacob is now known as CyberJacob|Away
=== zz_zz_mwhudson is now known as mwhudson
=== mwhudson is now known as zz_mwhudson
=== zz_mwhudson is now known as mwhudson
=== CyberJacob|Away is now known as CyberJacob
[09:53] <marcoceppi> jamespage: it seems we actually have a juju unset to revert to default values
[09:54] <marcoceppi> so I need to propose to the list about allowing None as a valid default type for configuration for charm proofing, but I think it's acceptable now given that unset exists
[09:56] <jamespage> marcoceppi, excellent
[09:56] <jamespage> I noticed that the default "" stuff already got into the rabbitmq-server charm
[09:56] <jamespage> I'm going to revert that
=== mwhudson is now known as zz_mwhudson
=== bcsaller_ is now known as bcsaller
[19:36] <JoshStrobl> Hey guys, I've been trying to debug my install hook for a while now, changed the set -e to set -x to do line-by-line debugging and aside from the usual running commands not as root (I'm under the impression the charm install hook will run as root to download packages), can't really figure out the issue (juju status states the service agent-state-info: 'hook failed: "install"'. Someone mind taking a look at the script?
[19:37] <JoshStrobl> http://pastebin.com/MHV91wKQ
[19:37] <JoshStrobl> charm proof only provides a copyright and maintainer warning, no errors.
=== zz_mwhudson is now known as mwhudson
[20:17] <marcoceppi> JoshStrobl: change the install hook to `set -eux` then copy the output of the charm log
[20:17] <JoshStrobl> will do
[20:19] <marcoceppi> JoshStrobl: also, line 70 is kind of silly to have, since you call wget around line 15
[20:19] <marcoceppi> also, apt-get install is safe to invoke multiple times
[20:20] <marcoceppi> so just having apt-get install <pks> will exit 0 if the packages are installed or not (unless there was an error during installation)
[20:20] <JoshStrobl> ah yea the wget on line 15 is silly :D
[20:20] <JoshStrobl> marcoceppi: so it won't do an unnecessary re-install?
[20:20] <marcoceppi> JoshStrobl: sorrect
[20:20] <marcoceppi> correct*
[20:20] <JoshStrobl> alrighty
[20:20] <JoshStrobl> I'll apply some changes and try again then :)
[20:21] <marcoceppi> JoshStrobl: http://paste.ubuntu.com/6905382/
[20:21] <JoshStrobl> ah :D
[20:21] <JoshStrobl> been a while since I've used Ubuntu myself, to be honest
[20:21] <JoshStrobl> Sorta sticking with archlinux nowadays until 14.04.
[20:22] <marcoceppi> :D
[20:22] <JoshStrobl> Thanks again marcoceppi :)
[20:22] <JoshStrobl> I'll let you know how things go!
[20:22] <marcoceppi> np! cutting down the complexity might reveal where the install hook is breaking better
[20:24] <JoshStrobl> Well, eliminating unnecessarily complexity is always a good thing anyways, makes easier to maintain. Just sorta figured apt-get might do unnecessary package re-installs, hence why I implemented those checks. But w00t, glad it doesn't.
[20:25] <JoshStrobl> Oh and if you're wondering why I'm using echo rather than juju-log, it's because set -e kept failing on juju-log "anksdnfkas" calls, despite me having juju and juju-core installed (running on a local Juju server)
=== mwhudson is now known as zz_mwhudson
[20:39] <JoshStrobl> where are charm logs typically stored?
[20:46] <marcoceppi> JoshStrobl: /var/log/juju/unit-*.log on the machine
[20:47] <marcoceppi> JoshStrobl: if you're using local provider, you can find them in ~/.juju/local/log/unit-*
[20:47] <JoshStrobl> marcoceppi: ah ok, was about to say there wasn't anything in /var/log/juju :D
[20:48] <marcoceppi> JoshStrobl: there is if you do `juju ssh <unit>` then cd to /var/log/juju :)
[20:49] <JoshStrobl> I think I've found the issue
=== zz_mwhudson is now known as mwhudson
=== timrc-afk is now known as timrc
=== CyberJacob is now known as CyberJacob|Away
=== CyberJacob|Away is now known as CyberJacob