| wallyworld | babbageclunk: can haz small review at some stage when you are free? no rush https://github.com/juju/juju/pull/8739 | 01:52 |
|---|---|---|
| babbageclunk | wallyworld: yup, looking now | 01:53 |
| babbageclunk | wallyworld: approved | 02:04 |
| wallyworld | yyay ty | 02:05 |
| _thumper_ | proxies suck | 03:27 |
| _thumper_ | just sayin | 03:27 |
| === _thumper_ is now known as thumper | ||
| manadart | jam externalreality: In the HO. | 07:06 |
| manadart | Anyone able to take a look at https://github.com/juju/juju/pull/8736 ? It is small. | 10:43 |
| stickupkid | manadart: looks good to me. | 10:46 |
| manadart | stickupkid: Ta. | 10:47 |
| srihas | hi guys, the space "aci" which is passed as 'juju deploy neutron-openvswitch --bind "data=aci" ' should have DHCP enabled? | 11:51 |
| srihas | how can I remove a subordinate service that has "hook failed: install" status in juju 2.3 | 12:17 |
| srihas | ?? | 12:17 |
| manadart | Another PR for review: https://github.com/juju/juju/pull/8740/files - mostly mechanical. | 12:45 |
| stickupkid | manadart: i added a quick comment to the PR... | 13:26 |
| srihas | thank god, --no-retry saved me | 13:28 |
| manadart | stickupkid: Ta. | 13:40 |
| === zeus is now known as Guest47400 | ||
| === Guest47400 is now known as zeus | ||
| === zeus is now known as Guest58315 | ||
| === Guest58315 is now known as zeus | ||
| thumper | bugger... | 21:15 |
| thumper | simon's gometalinter patch causes all the CI unit tests to fail | 21:15 |
| veebers | thumper: ah, because gometalinter is not installed :-\ | 21:19 |
| 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:20 |
| 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:21 |
| 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:27 |
| veebers | hindsight is 20/20 :-) | 21:31 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!