Re: Null provider and failing early

2013-08-26 Thread Andrew Wilkins
On Tue, Aug 27, 2013 at 8:44 AM, Tim Penhey tim.pen...@canonical.comwrote: Perhaps we should have a sanity-check type callback into the provider with the constraints at the time we want to add a machine. This would give the null provider the early fail mechanism, and could also allow other

Re: [ANN] juju-core 1.13.3 has been released

2013-09-03 Thread Andrew Wilkins
Hi Kapil and Antonio, Dave and I decided not to include manual provisioning in the release notes on account of it needing some documentation. There are some things that require a bit of explanation: the target machine needs to be able to route to the state server; currently only Ubuntu is

Re: Clean Code: Intro, Clean Code and Meaningful Names (up to the end of chapter 2)

2013-09-17 Thread Andrew Wilkins
On Tue, Sep 17, 2013 at 2:02 PM, Jeroen Vermeulen jeroen.vermeu...@canonical.com wrote: On 17/09/13 12:42, Andrew Wilkins wrote: I don't think many people will disagree that deduplication of code is a bad idea in general. However, as with database denormalisation, there are exceptions

Re: Automatic tools-upload/sync during bootstrap

2013-10-14 Thread Andrew Wilkins
On Fri, Oct 11, 2013 at 2:19 PM, Ian Booth ian.bo...@canonical.com wrote: So I'd like to see what people think of this: - Keep juju bootstrap --upload-tools, but make that the only time we attempt to upload during juju bootstrap. There will not be any automatic upload if tools aren't

API work

2013-11-01 Thread Andrew Wilkins
I've created a bug to do the CLI-API work. If there's something existing, or a more appropriate place to do this, let me know and this can disappear. https://bugs.launchpad.net/juju-core/+bug/1246983 I'm going to look at doing destroy-machine, which looks pretty trivial. Cheers, Andrew --

Re: State interfaces

2013-11-03 Thread Andrew Wilkins
On Fri, Nov 1, 2013 at 8:31 PM, roger peppe rogpe...@gmail.com wrote: On 31 October 2013 18:42, Andrew Wilkins andrew.wilk...@canonical.com wrote: type State interface { AllMachines() ([]Machine, error) } // another name would be needed, as this clashes with state.Entity; just

Re: Datasource Fallbacks (simplestreams)

2013-11-04 Thread Andrew Wilkins
=:- On Oct 10, 2013 11:16 AM, Andrew Wilkins andrew.wilk...@canonical.com wrote: On Fri, Oct 4, 2013 at 9:24 PM, Curtis Hovey-Canonical cur...@canonical.com wrote: Do we place a specific warning in the log. If I bootstrap with --show-log, do I see a message that clearly states the image

Re: API work

2013-11-05 Thread Andrew Wilkins
I thought I'd summarise where things are at now, in case anyone else starts looking for work to do. Ignoring the DONE's, we have the following remaining (see below for commentary): api-endpoints: TODO [axwalk] debug-hooks: INPROGRESS [axwalk] debug-log: INPROGRESS deploy: TODO

Re: API work

2013-11-05 Thread Andrew Wilkins
On Tue, Nov 5, 2013 at 6:22 PM, William Reade william.re...@canonical.comwrote: On Tue, Nov 5, 2013 at 10:25 AM, Andrew Wilkins andrew.wilk...@canonical.com wrote: As I've been looking for things to bite off, here's some comments from me (excluding things others are already looking

Re: High Availability command line interface - future plans.

2013-11-06 Thread Andrew Wilkins
On Thu, Nov 7, 2013 at 9:23 AM, Ian Booth ian.bo...@canonical.com wrote: So, I haven't been involved directly in a lot of the discussion, but my 2c is: +1 to juju ensure-ha Users don't give a f*ck about how Juju achieves HA, they just want to know their data will survive a node outage.

Re: High Availability command line interface - future plans.

2013-11-08 Thread Andrew Wilkins
On Fri, Nov 8, 2013 at 4:47 PM, Mark Canonical Ramm-Christensen mark.ramm-christen...@canonical.com wrote: I have a few high level thoughts on all of this, but the key thing I want to say is that we need to get a meeting setup next week for the solution to get hammered out. First,

Re: Preparations for juju-core 1.17.0

2013-11-12 Thread Andrew Wilkins
I'd really like to get this one in, which I *hope* to have done by end of next week at the latest (preferably sooner): #1238948 manual provider client cache prevents reuse of env by name And I have a MP waiting review for this, which I'd like to get in, but depends on people reviewing it (hint

Re: Preparations for juju-core 1.17.0

2013-11-14 Thread Andrew Wilkins
On Thu, Nov 14, 2013 at 5:35 PM, William Reade william.re...@canonical.comwrote: On Wed, Nov 13, 2013 at 2:46 AM, Andrew Wilkins andrew.wilk...@canonical.com wrote: I'd really like to get this one in, which I *hope* to have done by end of next week at the latest (preferably sooner

Re: Automatic tools-upload/sync during bootstrap

2013-11-18 Thread Andrew Wilkins
. On Mon, Nov 18, 2013 at 4:01 AM, John Arbash Meinel j...@arbash-meinel.com wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 2013-11-18 10:47, Andrew Wilkins wrote: All, I'm in the process of landing the following MP which removes the --source flag from juju bootstrap

Synchronous Bootstrap

2013-12-02 Thread Andrew Wilkins
Ahoy, Just wanted to inform everyone that synchronous bootstrap has landed on trunk. If you've got any scripts that bootstrap an environment and expect it to chug away in the background, then some script changes will be necessary. If you hit Ctrl+C, juju will catch the SIGINT and cancel the

Re: SSH keys

2013-12-04 Thread Andrew Wilkins
On Thu, Dec 5, 2013 at 1:15 PM, John Arbash Meinel j...@arbash-meinel.comwrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 2013-12-05 8:15, Andrew Wilkins wrote: So, synchronous bootstrap broke CI. The reason for this is that we're now using SSH as part of the process; I can see

Re: Can we get an update on the individual juju CLI/API tasks?

2013-12-05 Thread Andrew Wilkins
For destroy-environment: I first need to land https://codereview.appspot.com/28880043/ and then get back to https://codereview.appspot.com/22870045/ ETA: not sure, but not this week, and probably not next week either. I hope to have the first CL accepted soon, but the second one needs reworking.

Re: Releasing stable, dev, and others

2013-12-09 Thread Andrew Wilkins
On Tue, Dec 10, 2013 at 5:16 AM, Curtis Hovey-Canonical cur...@canonical.com wrote: Gentlemen. Juju-core 1.16.5 has one bug preventing us from doing a release. CI has blessed lp:juju-core/1.16 r1998. Do we want to defer the destroy-machine --force bug?

Re: An interesting scenario

2014-01-23 Thread Andrew Wilkins
On Wed, Jan 22, 2014 at 12:06 AM, Jonathan Aquilina jaquil...@eagleeyet.net wrote: I have an interesting scenario which either is already catered for or should be catered for. I am currently using a local provider for testing and contribution purposes. I bootstrapped an environment and

Re: Local provider - isolating sudo usage

2014-01-23 Thread Andrew Wilkins
: chowning the ~/.juju/ssh dir and keys. Suggestions welcome. Cheers, Andrew On Fri, Jan 24, 2014 at 4:34 AM, roger peppe roger.pe...@canonical.comwrote: Thank you! Let's get rid of every chown in the code base. On 22 January 2014 23:12, Andrew Wilkins andrew.wilk...@canonical.com wrote: Hi

Re: lxc bug blocks 1.17.1 release.

2014-01-24 Thread Andrew Wilkins
This looks a lot like a Trusty-specific issue to me, and nothing to do with sudo. When did CI move to Trusty? Trusty no longer has the /etc/lxc/auto directory, and instead the LXC container config file has an attribute that says the container should be auto-started with the machine. Jesse has an

Re: Local provider - isolating sudo usage

2014-01-24 Thread Andrew Wilkins
On Sat, Jan 25, 2014 at 1:24 AM, roger peppe roger.pe...@canonical.comwrote: On 24 January 2014 10:59, Andrew Wilkins andrew.wilk...@canonical.com wrote: On Fri, Jan 24, 2014 at 11:38 PM, roger peppe roger.pe...@canonical.com wrote: On 24 January 2014 01:14, Andrew Wilkins andrew.wilk

Re: lxc bug blocks 1.17.1 release.

2014-01-25 Thread Andrew Wilkins
On Sat, Jan 25, 2014 at 11:17 PM, Curtis Hovey-Canonical cur...@canonical.com wrote: Hi Andrew. On Fri, Jan 24, 2014 at 8:40 PM, Andrew Wilkins andrew.wilk...@canonical.com wrote: This looks a lot like a Trusty-specific issue to me, and nothing to do with sudo. When did CI move

Re: Juju Docs - SSH key requirements

2014-01-31 Thread Andrew Wilkins
less work that HTML, please just mandate one and be done with it. I would say Markdown, because GitHub Pages. On Fri, Jan 31, 2014 at 6:58 AM, Andrew Wilkins andrew.wilk...@canonical.com wrote: Hi Nick, https://juju.ubuntu.com/docs/getting-started.html On the Intro/Getting Started page

Re: Juju Docs - SSH key requirements

2014-02-04 Thread Andrew Wilkins
PM, Andrew Wilkins andrew.wilk...@canonical.com wrote: Hi Nick, https://juju.ubuntu.com/docs/getting-started.html On the Intro/Getting Started page for Juju, we say that you *need* to generate an SSH key pair. This is no longer true in 1.17.x: Juju will generate one for you. Juju

Targeted unit testing pre-push

2014-02-10 Thread Andrew Wilkins
Hi folks, When I'm about to propose or land something, I'll typically run make check, or just go test ./... to ensure I haven't broken some far-away package. This can be quite slow, unnecessarily so if the packages changed don't feature in the dependencies of some of our longer running tests. I

Re: Two new bugs blocking CI

2014-02-19 Thread Andrew Wilkins
On Thu, Feb 20, 2014 at 9:14 AM, Kapil Thangavelu kapil.thangav...@canonical.com wrote: i ran into this as well, there's a bug in manual provider on trunk which effectively needs --upload-tools to bootstrap correctly (filed previously as https://launchpad.net/bugs/1280678) -k On Wed,

Re: rsyslog configuration worker

2014-02-23 Thread Andrew Wilkins
one and overwriting). It doesn't necessarily have to be a worker, just something that runs once at machine agent startup: wait for environ config, write rsyslog config, restart rsyslog. John =:- On Mon, Feb 24, 2014 at 9:02 AM, Andrew Wilkins andrew.wilk...@canonical.com wrote: I'm been

Re: rsyslog configuration worker

2014-02-23 Thread Andrew Wilkins
On Mon, Feb 24, 2014 at 3:46 PM, Ian Booth ian.bo...@canonical.com wrote: It doesn't necessarily have to be a worker, just something that runs once at machine agent startup: wait for environ config, write rsyslog config, restart rsyslog. That's actually how the upgrade happens now -

Re: manual bootstrap in 1.18

2014-03-03 Thread Andrew Wilkins
On Tue, Mar 4, 2014 at 12:54 AM, William Reade william.re...@canonical.comwrote: Hi all We've been talking about what we need to do to finalise 1.18, and the issue with manual bootstrap addresses in environments.yaml [0] has been causing us some problems; in particular, we think it was a

Re: juju-core tests broken by gwacl

2014-03-14 Thread Andrew Wilkins
I have updated gwacl in preparation for some upcoming changes, but I haven't landed them or updated dependencies.tsv yet. Please revert to whatever's in dependencies.tsv. Cheers, Andrew On Sat, Mar 15, 2014 at 3:43 AM, Nate Finch nate.fi...@canonical.comwrote: I know some people were working

Re: I WANT YOU, to give me your feedback (Juju docs!)

2014-03-21 Thread Andrew Wilkins
+1 to Markdown. Nice integration with GitHub (edit, preview, etc.), pretty much a de facto standard on there. I'd also like to see us using GitHub Pages for unstable/unreleased versions of Juju. On Sat, Mar 22, 2014 at 1:47 AM, Marco Ceppi marco.ce...@canonical.comwrote: Hi everyone, I'm

Re: destroy-environment and /var/log/juju*

2014-03-23 Thread Andrew Wilkins
Thanks John. Removing the log dir on destruction is easy to change. I left it like that in case people were using it to debug things after destruction, but it's probably not needed. In hindsight, provider/local probably would need to be hacked up to call sudo strace juju instead of sudo juju.

Re: Azure Availability Set support on trunk

2014-04-07 Thread Andrew Wilkins
On Thu, Apr 3, 2014 at 8:28 PM, Andrew Wilkins andrew.wilk...@canonical.com wrote: Hi all, I've just landed support for Azure's Availability Sets on trunk today. Here's a little demo. I've written a donkey charm that exposes /bin/hostname via xinetd on port 1234, and deployed 2 units

Re: local provider destroy-environment sometimes fails with a weird error

2014-04-13 Thread Andrew Wilkins
On Mon, Apr 14, 2014 at 7:52 AM, Tim Penhey tim.pen...@canonical.comwrote: Hi folks, I noticed this at the end of last week, but shelved the thoughts until this weed as other things were taking priority. $ juju destroy-environment local -y [sudo] password for tim: ERROR readdirent: no

Re: juju HA lands on tip

2014-04-17 Thread Andrew Wilkins
of the others should take over. ... There is one major caveat: upgrading an existing environment to HA is not supported currently (Andrew Wilkins has a branch in review that should fix this). Thanks for the heads up Roger. We'll definitely take a peek at this. From a GUI perspective

Re: juju HA lands on tip

2014-04-21 Thread Andrew Wilkins
! There is one major caveat: upgrading an existing environment to HA is not supported currently (Andrew Wilkins has a branch in review that should fix this). Upgrade support landed on the weekend. If would be great it some intrepid users would test this out. (I have tested it, of course

Re: transcloud environment woes.

2014-04-24 Thread Andrew Wilkins
On Fri, Apr 25, 2014 at 5:55 AM, Curtis Hovey-Canonical cur...@canonical.com wrote: Esteemed nephologists, My hopes to setup a jenkins slave to test on ppc64el on a remote network using juju itself are vexed. Juju CI is currently in ec2. While I could update many tests to use ssh to run

Re: Juju devel 1.19.1 is released

2014-04-25 Thread Andrew Wilkins
On Sat, Apr 26, 2014 at 4:16 AM, Curtis Hovey-Canonical cur...@canonical.com wrote: juju-core 1.19.1 A new development release of Juju, juju-core 1.19.1, is now available. Getting Juju juju-core 1.19.1 is available for trusty and backported to earlier series in the following PPA:

test-failure bugs

2014-05-15 Thread Andrew Wilkins
Hi all, This past couple of weeks Ian and I have been working on fixing tests so we're not constantly poking the bot. Several of the test-failure bugs I've looked at over the last couple of days have already been fixed, but it took a while to figure that out. It would be a great help if, when

Re: [Proposal] Requiring Go 1.2 across the board

2014-05-16 Thread Andrew Wilkins
On Fri, May 16, 2014 at 3:08 PM, David Cheney david.che...@canonical.comwrote: Hello, This is a proposal that we raise the minimum Go spec from Go 1.1 to Go 1.2. Motivation: * Now that T is out, we have to support Juju on two series (Precise and Trusty[1]) with two compilers, gc and

Re: interface identity (was Re: Proposed new dependency: github.com/juju/errors (and github.com/juju/errgo))

2014-05-27 Thread Andrew Wilkins
On Wed, May 28, 2014 at 8:47 AM, Tim Penhey tim.pen...@canonical.comwrote: On 28/05/14 12:43, Nate Finch wrote: This sounds like one of those if you have to ask this question, you're doing something wrong. Can you give an example of where we need this? Sure... let's say we have a stack

Re: interface identity (was Re: Proposed new dependency: github.com/juju/errors (and github.com/juju/errgo))

2014-05-27 Thread Andrew Wilkins
On Wed, May 28, 2014 at 9:52 AM, Tim Penhey tim.pen...@canonical.comwrote: On 28/05/14 13:48, Andrew Wilkins wrote: On Wed, May 28, 2014 at 8:47 AM, Tim Penhey tim.pen...@canonical.com mailto:tim.pen...@canonical.com wrote: On 28/05/14 12:43, Nate Finch wrote: This sounds like

Re: interface identity (was Re: Proposed new dependency: github.com/juju/errors (and github.com/juju/errgo))

2014-05-27 Thread Andrew Wilkins
On Wed, May 28, 2014 at 10:39 AM, John Meinel j...@arbash-meinel.comwrote: The address of the real value is the same. Are you referring to the backing array? That is not what is being compared, so that's not a useful property. John =:- On May 28, 2014 6:04 AM, Andrew Wilkins andrew.wilk

Re: Not logging jenv data

2014-05-28 Thread Andrew Wilkins
On Thu, May 29, 2014 at 4:25 AM, Nate Finch nate.fi...@canonical.comwrote: Today I learned CI isn't running with --debug because they don't want to expose sensitive data in their jenv... which gets logged when you run with --debug. However, it also means that we don't get all our really

Re: reverting (fixing?) r2814

2014-06-01 Thread Andrew Wilkins
I haven't looked at the revision in question, so lack context, but just wanted to point out that sort.Stable is Go 1.2+ On Sun, Jun 1, 2014 at 6:05 PM, John Meinel j...@arbash-meinel.com wrote: So I missed this when it was up for review, but I think the approach it takes is actually

Re: not rebasing after PR?

2014-06-05 Thread Andrew Wilkins
On Fri, Jun 6, 2014 at 1:22 AM, Casey Marshall casey.marsh...@canonical.com wrote: On 06/05/2014 10:22 AM, Nate Finch wrote: This mashes all your pre-PR commits into one, so hides some commit spam that way, but then keeps the post-PR commits, to preserve comments. It sounds like we can

Re: naming repos on github

2014-06-05 Thread Andrew Wilkins
On Thu, Jun 5, 2014 at 9:28 PM, roger peppe rogpe...@gmail.com wrote: Unfortunately that means that the import identifier isn't intuitive, but perhaps that's ok. It's not so different from the go-thing convention a lot of people use, where the package name is actually just thing. If we were

Re: Reviewing in progress work on Github

2014-06-05 Thread Andrew Wilkins
On Thu, Jun 5, 2014 at 9:26 PM, roger peppe rogpe...@gmail.com wrote: On 5 June 2014 14:22, Andrew Wilkins andrew.wilk...@canonical.com wrote: On Thu, Jun 5, 2014 at 9:02 PM, roger peppe rogpe...@gmail.com wrote: On 5 June 2014 05:07, Menno Smits menno.sm...@canonical.com wrote: One

Re: Replica sets re-enabled for local provider

2014-06-08 Thread Andrew Wilkins
, Jun 6, 2014 at 12:56 PM, Curtis Hovey-Canonical cur...@canonical.com wrote: On Fri, Jun 6, 2014 at 4:50 AM, Andrew Wilkins andrew.wilk...@canonical.com wrote: I've gotten a bit closer to figuring out what the deal is. Even on the CI machine, local provider works with HA enabled, but only

Re: Replica sets re-enabled for local provider

2014-06-09 Thread Andrew Wilkins
On Mon, Jun 9, 2014 at 10:04 AM, Andrew Wilkins andrew.wilk...@canonical.com wrote: On Sat, Jun 7, 2014 at 1:32 AM, Nate Finch nate.fi...@canonical.com wrote: That is awesome. I presume that means we can re-re-re-enable HA on local provider? At least Michael has all the code to do so now

Re: Landing bot merge commit messages need the PR title

2014-06-09 Thread Andrew Wilkins
On Mon, Jun 9, 2014 at 1:39 PM, Menno Smits menno.sm...@canonical.com wrote: I've noticed that the commit messages for the merges performed by the Juju bot don't include the PR title. This can lead to some slightly odd commit logs. For example, have a look at PR #13. The merge commit

Re: Availability zone support

2014-06-14 Thread Andrew Wilkins
, Kapil Thangavelu kapil.thangav...@canonical.com wrote: On Fri, Jun 13, 2014 at 12:23 AM, Andrew Wilkins andrew.wilk...@canonical.com wrote: Hi all, I'm pleased to say that availability zone support for AWS and OpenStack has now landed on trunk, and will be available in 1.19.4. I'll

Re: Availability zone support

2014-06-15 Thread Andrew Wilkins
On Sat, Jun 14, 2014 at 4:38 PM, John Meinel j...@arbash-meinel.com wrote: ... 2) If you have 2 services, you likely would rather them on the same AZ rather than spread across different AZ because of the different cost of network bandwidth. How do we manage multiple services? Do we just

Re: Request to update config-LXC doc

2014-06-15 Thread Andrew Wilkins
On Sat, Jun 14, 2014 at 8:51 AM, Joshua Strobl joshua.str...@outlook.com wrote: I noticed that there is documentation on juju debug-log ( https://juju.ubuntu.com/docs/config-LXC.html) regarding a debug-log bug that prevents it from working as expected. However the bug's status is Fix Released

Re: Relation addresses

2014-06-17 Thread Andrew Wilkins
On Tue, Jun 17, 2014 at 3:42 PM, William Reade william.re...@canonical.com wrote: On Tue, Jun 17, 2014 at 6:39 AM, Andrew Wilkins andrew.wilk...@canonical.com wrote: Hi all, I've started looking into fixing https://bugs.launchpad.net/juju-core/+bug/1215579. The gist is, we currently set

Re: Relation addresses

2014-06-17 Thread Andrew Wilkins
On Tue, Jun 17, 2014 at 11:35 PM, Kapil Thangavelu kapil.thangav...@canonical.com wrote: On Tue, Jun 17, 2014 at 9:29 AM, John Meinel j...@arbash-meinel.com wrote: ... In a nutshell: - There will be a new hook, relation-address-changed, and a new tool called address-get. This

Re: Is upgrade-juju provider/cloud dependant?

2014-06-17 Thread Andrew Wilkins
On Wed, Jun 18, 2014 at 12:27 PM, Curtis Hovey-Canonical cur...@canonical.com wrote: CI tests deploy and upgrade in every CPC because I *think* these two scenario test the provider and the streams that were placed in the clouds. The upgrade test verifies stable juju understands the new

Re: Relation addresses

2014-06-18 Thread Andrew Wilkins
On Tue, Jun 17, 2014 at 9:29 PM, John Meinel j...@arbash-meinel.com wrote: ... In a nutshell: - There will be a new hook, relation-address-changed, and a new tool called address-get. This seems less than ideal, we already have standards ways of getting this data and being notified of

Re: Relation addresses

2014-06-18 Thread Andrew Wilkins
On Wed, Jun 18, 2014 at 2:56 PM, Andrew Wilkins andrew.wilk...@canonical.com wrote: On Tue, Jun 17, 2014 at 9:29 PM, John Meinel j...@arbash-meinel.com wrote: ... In a nutshell: - There will be a new hook, relation-address-changed, and a new tool called address-get. This seems less

Re: Relation addresses

2014-06-18 Thread Andrew Wilkins
On Thu, Jun 19, 2014 at 7:59 AM, Kapil Thangavelu kapil.thangav...@canonical.com wrote: On Wed, Jun 18, 2014 at 5:21 PM, William Reade william.re...@canonical.com wrote: On Wed, Jun 18, 2014 at 7:05 PM, Kapil Thangavelu kapil.thangav...@canonical.com wrote: addresses are just keys

Re: Relation addresses

2014-06-18 Thread Andrew Wilkins
On Thu, Jun 19, 2014 at 5:21 AM, William Reade william.re...@canonical.com wrote: On Wed, Jun 18, 2014 at 7:05 PM, Kapil Thangavelu kapil.thangav...@canonical.com wrote: addresses are just keys in a unit relation data bag. relation-get is the cli tool to retrieve either self or related

Re: Relation addresses

2014-06-19 Thread Andrew Wilkins
On Fri, Jun 20, 2014 at 6:44 AM, William Reade william.re...@canonical.com wrote: On Thu, Jun 19, 2014 at 5:14 AM, Andrew Wilkins andrew.wilk...@canonical.com wrote: If that's the case, why do we not just require charms to implement an address-changed hook to update the relation setting

Re: Proposal: making apt-get upgrade optional

2014-07-01 Thread Andrew Wilkins
it to the user. When you create an instance in ec2 or Azure or whatever, it doesn't come fully up-to-date. You get the released image, and then you can update it if you want to. John =:- On Mon, Jun 30, 2014 at 5:05 PM, Andrew Wilkins andrew.wilk...@canonical.com wrote: Hi folks, I've

Re: Proposal: making apt-get upgrade optional

2014-07-01 Thread Andrew Wilkins
upgrade. We add repos (cloud-tools, ppa), so we need to update for juju's dependencies anyway. I don't think my proposal will affect charms. [0] https://bugs.launchpad.net/juju-core/+bug/1336353 -thanks, Antonio On Tue, Jul 1, 2014 at 4:43 AM, Andrew Wilkins andrew.wilk...@canonical.com wrote

Re: Proposal: making apt-get upgrade optional

2014-07-07 Thread Andrew Wilkins
On Sat, Jul 5, 2014 at 4:05 AM, Antonio Rosales antonio.rosa...@canonical.com wrote: On Thu, Jul 3, 2014 at 7:41 PM, Andrew Wilkins andrew.wilk...@canonical.com wrote: On Fri, Jul 4, 2014 at 5:23 AM, Tim Penhey tim.pen...@canonical.com wrote: I do just want to make the point that we

Re: Friday Fun

2014-07-11 Thread Andrew Wilkins
On Fri, Jul 11, 2014 at 1:19 AM, Jesse Meek jesse.m...@canonical.com wrote: I polished off a script this afternoon to check that imports are group correctly. It deals with comments, if the groups are in the wrong order, if there are too many groups and several other edge cases. I *think* I

Re: Specific configuration on network-bridge interface

2014-07-22 Thread Andrew Wilkins
On Wed, Jul 23, 2014 at 7:29 AM, Jorge Niedbalski jorge.niedbal...@canonical.com wrote: Hello, I need some pointers regarding to provide a Fix for bug LP: https://bugs.launchpad.net/juju-core/+bug/1341524 , as you can see a user is experiencing problems to boot up a new environment cleanly

Re: Master has been broken for 4 weeks

2014-07-23 Thread Andrew Wilkins
On Wed, Jul 23, 2014 at 11:34 PM, Curtis Hovey-Canonical cur...@canonical.com wrote: This is the current list of regressions that block a release C:/Juju/lib/juju/nonce.txt does not exist, bootstrap failed in win https://bugs.launchpad.net/juju-core/+bug/1342725 reported 2014-07-16,

Re: issuing a jujud restart

2014-08-10 Thread Andrew Wilkins
On Mon, Aug 11, 2014 at 5:41 AM, Menno Smits menno.sm...@canonical.com wrote: How this happens is slightly complex but the short answer is that if any of jujud's workers exit with a fatalError (as defined in cmd/jujud/agent.go), then jujud will terminate (and then be restarted by upstart).

Re: Juju trunk now is compatible with Mongo 2.6

2014-08-14 Thread Andrew Wilkins
On Fri, Aug 15, 2014 at 11:58 AM, Ian Booth ian.bo...@canonical.com wrote: There is no mongo 2.6 build; it's all the one code base. juju-core works out-of-the-box on both mongo 2.4 and 2.6. The landing tests are run on top of mongo 2.4 as usual. If any of the tests fail, then the bot will be

Re: Juju trunk now is compatible with Mongo 2.6

2014-08-14 Thread Andrew Wilkins
(if someone tests and finds a bug)... On Fri, Aug 15, 2014 at 12:17 PM, Andrew Wilkins andrew.wilk...@canonical.com wrote: On Fri, Aug 15, 2014 at 11:58 AM, Ian Booth ian.bo...@canonical.com wrote: There is no mongo 2.6 build; it's all the one code base. juju-core works out-of-the-box

Re: changing the simplestreams filename format

2014-08-21 Thread Andrew Wilkins
On Fri, Aug 22, 2014 at 4:37 AM, Nate Finch nate.fi...@canonical.com wrote: Simplestreams generates files with names like this: http://maas.ubuntu.com/images/ephemeral-v2/daily/streams/v1/com.ubuntu.maas:daily:v2:download.sjson Unfortunately, the colons in the filename are illegal on

Re: changing the simplestreams filename format

2014-08-22 Thread Andrew Wilkins
*: *The filename, directory name, or volume label** syntax is incorrect.*) OK. Still, we control those filenames: they don't *have* to be the same as what simplestreams specifies, though it may be a PITA to try to work around it. Regards, Gabriel On 22.08.2014 04:54, Andrew Wilkins wrote: On Fri

Re: Triggering Server Restarts

2014-08-22 Thread Andrew Wilkins
On Fri, Aug 22, 2014 at 11:57 AM, John Meinel j...@arbash-meinel.com wrote: Horacio had an interesting question on IRC and I wanted to bring it up to more people to get input. Specifically, juju restore has some properties that are pretty similar to juju upgrade-juju in that we really want

Re: Triggering Server Restarts

2014-08-25 Thread Andrew Wilkins
On Tue, Aug 26, 2014 at 8:06 AM, William Reade william.re...@canonical.com wrote: On Fri, Aug 22, 2014 at 1:10 PM, Andrew Wilkins andrew.wilk...@canonical.com wrote: On Fri, Aug 22, 2014 at 11:57 AM, John Meinel j...@arbash-meinel.com wrote: I think the problem is that to trigger

Re: juju local bootstrap from tip

2014-08-31 Thread Andrew Wilkins
Works fine on my trusty laptop, but I'm also getting a new error when I try bootstrapping on precise: 2014-09-01 04:51:27 INFO juju.utils.apt apt.go:132 Running: [apt-get --option=Dpkg::Options::=--force-confold --option=Dpkg::options::=--force-unsafe-io --assume-yes --quiet install

Re: juju local bootstrap from tip

2014-08-31 Thread Andrew Wilkins
-release for local? I'm not apt-savvy enough to know what the right thing to do here is. John =:- On Mon, Sep 1, 2014 at 9:16 AM, Andrew Wilkins andrew.wilk...@canonical.com wrote: On Mon, Sep 1, 2014 at 12:53 PM, Andrew Wilkins andrew.wilk...@canonical.com wrote: Works fine on my trusty

Re: juju local bootstrap from tip

2014-09-01 Thread Andrew Wilkins
Reading package lists... E: The value 'precise-updates/cloud-tools' is invalid for APT::Default-Release as such a release is not available in the sources (if I apt-add-repository cloud-archive:tools, it's happy) John =:- On Mon, Sep 1, 2014 at 9:57 AM, Andrew Wilkins andrew.wilk

Getting the API server URL in workers

2014-09-03 Thread Andrew Wilkins
I'm working on moving all tools downloads to download from the API server. There will are a few APIs that return tools: - Upgrader.Tools - Client.FindTools - Provisioner.FindTools These APIs will need to return URLs pointing at the API server. I'm intending to change the facade constructors

Re: Getting the API server URL in workers

2014-09-03 Thread Andrew Wilkins
would be useless for older clients upgrading. Anyway, I'll have another dig at it. Cheers, Andrew Cheers William John =:- On Sep 3, 2014 6:03 PM, Andrew Wilkins andrew.wilk...@canonical.com wrote: I'm working on moving all tools downloads to download from the API server

Re: Copyright information in headers

2014-09-03 Thread Andrew Wilkins
On Thu, Sep 4, 2014 at 5:50 AM, Ian Booth ian.bo...@canonical.com wrote: Hi folks The question recently came up in reviews as to whether we should be updating the date in the copyright statement in the file header when we make a change to the code in that file. I sought clarification from

Re: my first charm

2014-09-03 Thread Andrew Wilkins
On Thu, Sep 4, 2014 at 3:46 AM, Eric Snow eric.s...@canonical.com wrote: Hi all, Here's a write-up on my experience writing a charm for the first time. In summary, it wasn't nearly as bad as I expected it to be. The docs were super useful (both the getting-started side of things and later

Re: Bug 1365828 - mongo log spam

2014-09-09 Thread Andrew Wilkins
On Tue, Sep 9, 2014 at 4:14 PM, John Meinel j...@arbash-meinel.com wrote: I don't think these are from authentication between mongo instances. Agreed, looks like the machine-0 jujud connecting over and over again. Perhaps the agent is unhappy and is bouncing. This looks more like a Mongo

Re: Important Blocking Bug

2014-09-09 Thread Andrew Wilkins
Yep, most likely related to my changes. I'm taking a look too. Thanks Menno. On Wed, Sep 10, 2014 at 6:11 AM, Menno Smits menno.sm...@canonical.com wrote: I've looked into this a bit and the problem is due to one of the machines being unable to download tools for the upgrade. This could be

Re: Please don't use bash when there are syscalls available

2014-09-09 Thread Andrew Wilkins
On Wed, Sep 10, 2014 at 4:45 AM, Nate Finch nate.fi...@canonical.com wrote: A user just complained that he can't bootstrap because Juju is parsing stderr text from flock, and his server isn't in English, so the error message isn't matching.

Re: Please don't use bash when there are syscalls available

2014-09-10 Thread Andrew Wilkins
Juju. I have filed https://bugs.launchpad.net/juju-core/+bug/1367695, assigned to 1.20.8. On Tue, Sep 9, 2014 at 6:56 PM, Andrew Wilkins andrew.wilk...@canonical.com wrote: On Wed, Sep 10, 2014 at 4:45 AM, Nate Finch nate.fi...@canonical.com wrote: A user just complained that he can't

Testing api clients

2014-09-10 Thread Andrew Wilkins
Hi folks, I'd like to bring a small, recent addition to everyone's attention: https://github.com/juju/juju/blob/master/api/base/testing/patch.go PatchFacadeCaller can be used (and is used now in several places) to avoid calling through to a real API server in api tests. Ideally we'd just

Re: Unit Tests Integration Tests

2014-09-11 Thread Andrew Wilkins
On Thu, Sep 11, 2014 at 11:29 PM, Matthew Williams matthew.willi...@canonical.com wrote: Hi Folks, There seems to be a general push in the direction of having more mocking in unit tests. Obviously this is generally a good thing but there is still value in having integration tests that test

Re: Agent API Versioning and Upgrades

2014-09-17 Thread Andrew Wilkins
On Wed, Sep 17, 2014 at 5:15 PM, Dimiter Naydenov dimiter.nayde...@canonical.com wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Hi all, TL;DR When introducing a new agent API facade version used by a worker, which requires an upgrade step (schema changes, migration), I propose to

Re: Landing job uses pre-push script

2014-10-03 Thread Andrew Wilkins
On Thu, Oct 2, 2014 at 1:35 PM, Martin Packman martin.pack...@canonical.com wrote: The landing job for juju now uses the pre-push script from the juju branch before running tests. This means it's running go vet over the source, and I've fixed a couple of errors on master. Great, thanks

Re: reviewboard-github integration

2014-10-18 Thread Andrew Wilkins
On Sat, Oct 18, 2014 at 1:38 PM, Eric Snow eric.s...@canonical.com wrote: With the switch to Reviewboard we introduced extra steps to our workflow (mostly involving rbt). This in turn made things more difficult for new/existing contributors. I've been able to take some time in the last

Re: Azure US West having problems

2014-10-20 Thread Andrew Wilkins
I just changed the Azure provider (locally) to allocate a virtual network using a Location instead of Affinity Group, and it got most of the way through bootstrap. It ends up failing with 2014-10-21 01:43:39 ERROR juju.cmd supercommand.go:323 Get

Reviewboard clutter

2014-10-29 Thread Andrew Wilkins
All, Please close your completed reviews off in Reviewboard. You should only need to do it once for the existing reviews, assuming the integration does its job. Thanks, Andrew -- Juju-dev mailing list Juju-dev@lists.ubuntu.com Modify settings or unsubscribe at:

Re: how to update dependencies.tsv

2014-10-30 Thread Andrew Wilkins
On Thu, Oct 30, 2014 at 10:34 PM, Eric Snow eric.s...@canonical.com wrote: On Thu, Oct 30, 2014 at 4:40 AM, roger peppe roger.pe...@canonical.com wrote: It's a pity then that if you go get a package, origin is the repo you've branched from. There is always one of those, but you don't

Re: Someone fixed the security issue with --debug?

2014-11-09 Thread Andrew Wilkins
On Sat, Nov 8, 2014 at 3:16 AM, Curtis Hovey-Canonical cur...@canonical.com wrote: I am comparing the use of streams during the bootstrap of 1.20 and 1.21. I noticed that 1.21 no longer dumps the content of the cloud-init script, which has user credentials and machine keys, implicitly fixing

Re: Proposal: feature flag implementation for Juju

2014-11-25 Thread Andrew Wilkins
On Wed, Nov 26, 2014 at 6:47 AM, Tim Penhey tim.pen...@canonical.com wrote: Hi all, There are often times when we want to hook up features for testing that we don't want exposed to the general user community. In the past we have hooked things up in master, then when the release branch is

Re: Dear reviewers,

2014-12-02 Thread Andrew Wilkins
On Wed, Dec 3, 2014 at 11:43 AM, Ian Booth ian.bo...@canonical.com wrote: On 03/12/14 13:34, Tim Penhey wrote: Hello there reviewers, I have a number of concerns around reviews that I need to say. Firstly, as an on call reviewer, you only need to look at the reviews that have not yet

Re: Feedback on a base fake type in the testing repo

2015-02-11 Thread Andrew Wilkins
On Thu, Feb 12, 2015 at 2:53 AM, Eric Snow eric.s...@canonical.com wrote: tl;dr Using fakes for testing works well so I wrote a base fake type. [1] While working on the GCE provider, Wayne and I started taking a different approach to unit testing than the usual 1. expose an external

Re: New charms client

2015-01-25 Thread Andrew Wilkins
On Fri, Jan 23, 2015 at 11:32 PM, Kapil Thangavelu kapil.thangav...@canonical.com wrote: I'm having some problems actually using this api, is it enabled? or does it need a feature flag? return self.rpc._rpc({ Type: Charms, Request: List, Params:

  1   2   3   >