[01:52] <wallyworld> babbageclunk: can haz small review at some stage when you are free? no rush https://github.com/juju/juju/pull/8739
[01:53] <babbageclunk> wallyworld: yup, looking now
[02:04] <babbageclunk> wallyworld: approved
[02:05] <wallyworld> yyay ty
[03:27] <_thumper_> proxies suck
[03:27] <_thumper_> just sayin
=== _thumper_ is now known as thumper
[07:06] <manadart> jam externalreality: In the HO.
[10:43] <manadart> Anyone able to take a look at https://github.com/juju/juju/pull/8736 ? It is small.
[10:46] <stickupkid> manadart: looks good to me.
[10:47] <manadart> stickupkid: Ta.
[11:51] <srihas> hi guys, the space "aci" which is passed as 'juju deploy neutron-openvswitch --bind "data=aci" ' should have DHCP enabled?
[12:17] <srihas> how can I remove a subordinate service that has "hook failed: install" status in juju 2.3
[12:17] <srihas> ??
[12:45] <manadart> Another PR for review: https://github.com/juju/juju/pull/8740/files - mostly mechanical.
[13:26] <stickupkid> manadart: i added a quick comment to the PR...
[13:28] <srihas> thank god, --no-retry saved me
[13:40] <manadart> stickupkid: Ta.
=== zeus is now known as Guest47400
=== Guest47400 is now known as zeus
=== zeus is now known as Guest58315
=== Guest58315 is now known as zeus
[21:15] <thumper> bugger...
[21:15] <thumper> simon's gometalinter patch causes all the CI unit tests to fail
[21:19] <veebers> thumper: ah, because gometalinter is not installed :-\
[21:20] <thumper> veebers: yeah, I emailed the crew asking if it should be added to the makefile or just the jenkins jobs
[21:20] <veebers> thumper: we can either update the jobs to install it as part of the process, or add it to the make file
[21:20] <veebers> hah
[21:20] <rick_h_> No, the linter was put in make check with what's already being run
[21:20] <thumper> rick_h_: where?
[21:20] <rick_h_> I know simon checked on that and already updated the makefile for make check to add the linting
[21:20]  * rick_h_ is looking
[21:20] <thumper> rick_h_: clearly it isn't working
[21:20] <veebers> rick_h_: installing the linter, not running it
[21:21] <rick_h_> ah, installing it...good question.
[21:21] <veebers> running the linter is there (that's what's causing the error). It needs to be installed
[21:21] <rick_h_> I guess I assumed it would be part of updating deps but ... guess not
[21:27] <thumper> how are users expected to install gometalinter?
[21:27] <thumper> it would have been good to get an email to the crew about the addition if it was going to cause problems
[21:27] <thumper> or even the juju email list
[21:31] <veebers> hindsight is 20/20 :-)