Re: First customer pain point pull request - default-hook

2014-08-18 Thread Curtis Hovey-Canonical
On Fri, Aug 15, 2014 at 4:36 PM, Nate Finch nate.fi...@canonical.com wrote: There's new hook in town: default-hook. If it exists and a hook gets called that doesn't have a corresponding hook file, default-hook gets called with the name of the original hook as its first argument (arg[1]). I

Re: First customer pain point pull request - default-hook

2014-08-18 Thread Gustavo Niemeyer
Rather than passing it as the first argument, I suggest introducing an environment variable: $JUJU_HOOK_NAME. This would be set irrespective of how the hook is being called, so that the same hook can be used both as a symlink and as a default-hook, unchanged. It also means further spawned

Re: First customer pain point pull request - default-hook

2014-08-18 Thread Kapil Thangavelu
That doc implies a completely different style of authoring ie. rewrite then of most extant (95%) charms using symlinks to a single implementation. There are a minority that do indeed reconsider all current state from juju each hook invocation, in which case this level of optimization is useful,

Re: First customer pain point pull request - default-hook

2014-08-18 Thread Gustavo Niemeyer
I don't think I fully understand the proposal there. To have such a something-changed hook, we ought to have a better mechanism to tell *what* actually changed. In other words, we have a number of hooks that imply a state transition or a specific notification (install, start, config-changed,

Juju stable 1.20.5 is released.

2014-08-18 Thread Curtis Hovey-Canonical
juju-core 1.20.5 A new stable release of Juju, juju-core 1.20.5, is now available. This release replaces 1.20.1. Getting Juju juju-core 1.20.5 is available for utopic and backported to earlier series in the following PPA: https://launchpad.net/~juju/+archive/stable Noteworthy This

Re: Juju stable 1.20.5 is released.

2014-08-18 Thread Aaron Bentley
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Woo-hoo! Looking forward to these stability improvements. Aaron On 14-08-18 04:09 PM, Curtis Hovey-Canonical wrote: juju-core 1.20.5 A new stable release of Juju, juju-core 1.20.5, is now available. This release replaces 1.20.1. Getting

Mentored reviewers - ACTION needed by team leads

2014-08-18 Thread Tim Penhey
Hi folks The mentored reviewers spreadsheet [1] has seven people down as people who would like review mentors. I'd like the team leads now to make sure that the people on their teams have a mentor assigned. The purpose of the mentor is to review the review. The idea here is that the mentored

Fwd: Storage server temporarily offline

2014-08-18 Thread Menno Smits
An update from Github about the current outage. -- Forwarded message -- From: John Greet (GitHub Staff) supp...@github.com Date: 19 August 2014 12:37 Subject: Re: Storage server temporarily offline To: Menno Smits menno.sm...@canonical.com Hi Menno, Sorry about that, we're