Re: So I wanted to update a dependency . .

2016-08-11 Thread Casey Marshall
On Thu, Aug 11, 2016 at 5:44 PM, Nicholas Skaggs < nicholas.ska...@canonical.com> wrote: > This is a simple story of a man and a simple mission. Eliminate the final > 2 dependencies that are in bazaar and launchpad. It makes juju and it's > dependencies live completely in git. A notable goal, and

Re: Some Juju CLI usability thoughts before we close 2.0

2016-08-11 Thread Ian Booth
On 11/08/16 17:46, Ian Booth wrote: > > On 11/08/16 17:03, John Meinel wrote: >> On Thu, Aug 11, 2016 at 9:30 AM, Ian Booth wrote: >> >>> A few things have been irking me with some aspects of Juju's CLI. Here's a >>> few >>> thoughts from a user perspective (well, me

So I wanted to update a dependency . .

2016-08-11 Thread Nicholas Skaggs
This is a simple story of a man and a simple mission. Eliminate the final 2 dependencies that are in bazaar and launchpad. It makes juju and it's dependencies live completely in git. A notable goal, and one that I desired for getting snaps to build with launchpad. I don't feel I need to explain

Re: Uniter test failure from utils update

2016-08-11 Thread Nicholas Skaggs
Thanks for the insights here. I ended up reverting the offending change, https://github.com/juju/utils/pull/203. @bogdanteleaga, my apologies for reverting, but I'm not sure how to fix with your change so I went back to something that worked. It's worth following up and figuring out how to re-land

Re: Uniter test failure from utils update

2016-08-11 Thread William Reade
(or, at least, it's something like that -- the general point is that you shouldn't throw away values you don't know you don't need) On Thu, Aug 11, 2016 at 2:14 PM, William Reade wrote: > worker/uniter/operation/runcommands.go:68... uses `context.ErrReboot` to >

Re: Uniter test failure from utils update

2016-08-11 Thread William Reade
worker/uniter/operation/runcommands.go:68... uses `context.ErrReboot` to indicate "there is a valid response value that should be handled, but we also need to queue a reboot". The change throws away the valid response. On Thu, Aug 11, 2016 at 1:47 AM, Martin Packman <

Re: Some Juju CLI usability thoughts before we close 2.0

2016-08-11 Thread Ian Booth
On 11/08/16 17:03, John Meinel wrote: > On Thu, Aug 11, 2016 at 9:30 AM, Ian Booth wrote: > > > A few things have been irking me with some aspects of Juju's CLI. Here's a > > few > > thoughts from a user perspective (well, me as user, YMMV). > > > > The following pain

Re: Some Juju CLI usability thoughts before we close 2.0

2016-08-11 Thread John Meinel
On Thu, Aug 11, 2016 at 9:30 AM, Ian Booth wrote: > A few things have been irking me with some aspects of Juju's CLI. Here's a > few > thoughts from a user perspective (well, me as user, YMMV). > > The following pain points mainly revolve around commands that operate on