anastasiamacproposed http://reviews.vapour.ws/r/2469/00:15
davechen1yping http://reviews.vapour.ws/r/2470/01:10
sinzuiwaigani: davechen1y: can you review http://reviews.vapour.ws/r/2471/ to prepare to the 1.25-alpha1 release? This is a merge of net-cli into master.01:37
davechen1ycan someone review http://reviews.vapour.ws/r/2470/01:40
anastasiamacdavechen1y: looking01:40
sinzuithank you davechen1y01:40
anastasiamacdavechen1y: done ;)01:48
davechen1yanastasiamac: thanks02:05
anastasiamacdavechen1y: replied :)02:06
=== mup_ is now known as mup
mupBug #1421260 changed: juju 1.21.1 bootstrap timeout <bootstrap> <oil> <oil-bug-1372407> <juju-core:Expired> <https://launchpad.net/bugs/1421260>04:22
mupBug #1468365 changed: internal compiler error: fault <ci> <intermittent-failure> <juju-core:Expired> <juju-core 1.24:Expired> <https://launchpad.net/bugs/1468365>04:22
mupBug #1487727 changed: TestAddresserWorkerStopsWhenAddressDeallocationNotSupported fails on precise and windows <blocker> <ci> <intermittent-failure> <precise>06:56
mup<regression> <unit-tests> <windows> <juju-core:Fix Released> <juju-core net-cli:Fix Released by dimitern> <https://launchpad.net/bugs/1487727>06:56
=== urulama__ is now known as urulama
dimiternwho can review this http://reviews.vapour.ws/r/2475/ fix for bug 1455627, which is causing a lot of issues in CI ?08:39
mupBug #1455627: TestAgentConnectionDelaysShutdownWithPing fails <ci> <intermittent-failure> <lxc> <test-failure> <unit-tests> <windows> <juju-core:In Progress by dimitern> <https://launchpad.net/bugs/1455627>08:39
dimiternTheMue, voidspace ^^ ?08:40
voidspacedimitern: can do, need to make a phone call first08:42
dimiternvoidspace, sure08:45
voidspacedimitern: interesting technique08:55
voidspacedimitern: looks ok to me08:56
TheMuedimitern: doing it08:56
dimiternvoidspace, thanks :)08:56
voidspacedimitern: so devices work is on master, not 1.2408:59
voidspacedimitern: so it needs adding to the release notes08:59
voidspacedimitern: I'll add something08:59
dimiternvoidspace, awesome, ta!09:00
dimiternvoidspace, TheMue, standup?09:01
voidspacedimitern: omw09:01
voidspacedimitern: I added release notes for devices, maybe too long09:43
dimiternvoidspace, tyvm! it's ok - it's still a draft, so we'll polish it for the release09:45
frobwarefwereade, I sent you a HO invite; feel free to move it to what works for you...10:39
fwereadefrobware, thanks, that should be fine, see you shortly :)10:40
mattywwallyworld, you in bed yet?11:33
mattywfwereade, I'd love a review of http://reviews.vapour.ws/r/2459/ when you have a moment11:33
mattywfwereade, I feel like I really need some advice about what I should be doing11:33
wallyworldmattyw: hey11:35
mattywwallyworld, so my last pr actually broke loads of stuff, would you mind taking another look http://reviews.vapour.ws/r/2466/diff/#11:36
mattywwallyworld, I do miss you, but I'm glad we weren't in the same room for the batting collapse11:37
wallyworldmattyw: yeah, i was sus about it, hence i added a card to audit what we have in that area even if the tests can pass. we still need to do that even once this lands11:37
wallyworldmattyw: hard to gloat when the ashes are lost11:38
wallyworldworst ashes series ever11:38
wallyworldno test went for 5 days, each was one sided11:38
* rick_h_ is trying to figure out if you all are talking code or cricket :P11:39
wallyworldrick_h_: cricket :-D11:40
wallyworldshould have used Test11:40
wallyworldand Ashes11:40
mattywwallyworld, reckon that branch should land?11:41
mattywrick_h_, we'll come up with some sport banter in chicago if you like11:42
wallyworldmattyw: if the tests pass, sure. we can iterate. i added a card to make sure we go back and audit everything11:42
mattywwallyworld, was the card on the uniter sprint board?11:42
wallyworldmattyw: yeah11:42
wallyworldmattyw: https://canonical.leankit.com/Boards/View/116558647/11683770911:43
wallyworldi put a few words in the description11:43
mattywwallyworld, awesome, I'll take a look at that later I think11:45
mattywwallyworld, you really should be in bed though right?11:45
wallyworldmattyw: yeah, meant to be on a swap day. but i have a meeting in an hour11:46
wallyworldtimezones are wonderful11:46
mattywwallyworld, fwereade ClearResolvedFlag() doesn't seem to be being called anywhere except in tests. And it doesn't exist on the operation.Callbacks interface anymore12:50
mattyw^^ any thoughts?12:50
mattywashipika, ^^12:50
wallyworldmattyw: it's gone12:51
wallyworldnot needed anymore12:51
mattywwallyworld, ok - so we can remove any reference to it12:51
mattywwallyworld, why is it not needed anymore?12:52
wallyworldmattyw: yes, but any tests will need to be checked to ensure some sort of equivalent behaviour doesn't need to be tested, but mostly i expect they could be deleted. needs ti be case by case12:52
wallyworldmattyw: eg the RemoteState snapshot may need to be udated12:53
wallyworlddepends on the test etc12:53
mattywwallyworld, ack12:54
TheMuedimitern: voidspace: frobware: maas hangout13:03
mattywwallyworld, regarding what we just discussed http://reviews.vapour.ws/r/2476/13:21
voidspaceTheMue: oh, dammit13:22
voidspaceTheMue: is it still on?13:22
TheMuevoidspace: yep13:22
voidspaceok, omw13:23
wallyworldmattyw: lgtm. but i still have a feeling we need to add different tests that test the new resolved model. theres several similar such things we need tests for too. so we'll need to check all that once we think we've gt everything passing13:37
mattywwallyworld, ack - I'm still trying to get my head around this part of the code, shall I add a card for that?13:38
wallyworldmattyw: yeah, it will be a mega card - we can break it up later - but good to have as a reminder13:38
wallyworldmattyw: we can have a hangout to discuss this work maybe in a day or 213:39
wallyworldonce we get the stuff we know about passing13:39
mattywwallyworld, I'll add something the calendar feel free to move it around as you see fit13:40
wallyworldmattyw: sgtm, i'm too tired to do much else right now13:40
mattywwallyworld, ack np13:40
rogpeppeultra-trivial review anyone? (one line dependency change. https://github.com/juju/juju/pull/309714:15
rogpeppeor http://reviews.vapour.ws/r/2477/14:15
dimiternrogpeppe, ship it!14:18
rogpeppedimitern: ta!14:18
mupBug #1488523 opened: Azure provider attempts to reuse dying vnet <azure-provider> <bootstrap> <ci> <destroy-environment> <reliability> <repeatability> <juju-core:Triaged> <juju-core 1.24:Triaged> <https://launchpad.net/bugs/1488523>14:33
mupBug #1488523 changed: Azure provider attempts to reuse dying vnet <azure-provider> <bootstrap> <ci> <destroy-environment> <reliability> <repeatability> <juju-core:Triaged> <juju-core 1.24:Triaged> <https://launchpad.net/bugs/1488523>14:39
mupBug #1488523 opened: Azure provider attempts to reuse dying vnet <azure-provider> <bootstrap> <ci> <destroy-environment> <reliability> <repeatability> <juju-core:Triaged> <juju-core 1.24:Triaged> <https://launchpad.net/bugs/1488523>14:42
=== natefinch is now known as natefinch-afk
mupBug #1488554 opened: wily + 1.24.5: Failed to execute operation: Unit name /etc/systemd/system/juju-clean-shutdown.service is not valid <amulet> <openstack> <uosci> <juju-core:Triaged> <https://launchpad.net/bugs/1488554>15:42
mupBug #1488554 changed: wily + 1.24.5: Failed to execute operation: Unit name /etc/systemd/system/juju-clean-shutdown.service is not valid <amulet> <openstack> <uosci> <juju-core:Triaged> <https://launchpad.net/bugs/1488554>15:57
mupBug #1488554 opened: wily + 1.24.5: Failed to execute operation: Unit name /etc/systemd/system/juju-clean-shutdown.service is not valid <amulet> <openstack> <uosci> <juju-core:Triaged> <https://launchpad.net/bugs/1488554>16:00
mupBug #1488554 changed: wily + 1.24.5: Failed to execute operation: Unit name /etc/systemd/system/juju-clean-shutdown.service is not valid <amulet> <openstack> <uosci> <juju-core:Triaged> <https://launchpad.net/bugs/1488554>16:09
mupBug #1488573 opened: Juju treats transient vnet failures as permanent <azure-provider> <bootstrap> <ci> <deploy> <intermittent-failure> <reliability> <repeatability> <juju-core:Triaged> <https://launchpad.net/bugs/1488573>16:21
mupBug #1488576 opened: TestAddresserWorkerStopsWhenAddressDeallocationNotSupported fails on pp64el <ci> <intermittent-failure> <ppc64el> <regression> <unit-tests> <juju-core:Triaged> <https://launchpad.net/bugs/1488576>16:51
mupBug #1488581 opened: TestFindToolsExactInStorage fails for some archs <blocker> <ci> <i386> <ppc64el> <regression> <storage> <unit-tests> <juju-core:Triaged> <https://launchpad.net/bugs/1488581>16:51
perrito666anybody knows the whereabouts of the OCR?17:34
alexisbnatefinch-afk, ping ^^17:35
alexisbwwitzel3, ping (for something unrelated)17:35
=== natefinch-afk is now known as natefinch
natefinch'allo... sorry, have been neglecting my duties17:37
perrito666aghh afk17:37
perrito666why people wont use the actual away feature on irc17:38
natefinchperrito666: irc has features?17:38
perrito666and why on earth my client does not realize that natefinch-afk is natefinch17:38
perrito666you are using it wrong </steve jobs>17:38
natefinchsinzui: what version of Go are we going to be building with for 1.26?18:04
sinzuinatefinch: its not decided. We remain on 1.2 until we confirm 1.5 works every where and Ubuntu can put it every where18:05
* natefinch prays for 1.518:05
natefinchericsnow: are there corresponding changes to juju-process-docker to support the changes in your patch for plugins to packages?20:42
ericsnownatefinch: yep20:42
ericsnownatefinch: http://reviews.vapour.ws/r/2426/20:42
=== menn0_ is now known as menn0
davecheneycherylj: http://reviews.vapour.ws/r/2482/21:47
davecheneyship it21:47
wallyworldcherylj: you are awesome for fixing that bug - was going to be somthing i did today and i looked to see you'd already done it :-)21:48
cheryljthanks davecheney, wallyworld21:56
cheryljwallyworld: regarding your review comment, I think we do need it.  My reasoning behind it is in the bug https://bugs.launchpad.net/juju-core/+bug/1488581/comments/221:57
mupBug #1488581: TestFindToolsExactInStorage fails for some archs <blocker> <ci> <i386> <ppc64el> <regression> <storage> <unit-tests> <juju-core:In Progress by cherylj> <https://launchpad.net/bugs/1488581>21:57
wallyworldcherylj: fair enough. perhaps then use arch.AMD64 instead of "amd64" to be consistent with the other lines in the test?21:58
cheryljwallyworld: sure, I'll do that21:59
wallyworldaxw_: anastasiamac: perrito666: running late, will ping when ready, soon23:14
anastasiamacwallyworld: k :D23:14
wallyworldaxw_: anastasiamac: perrito666: there now23:32
davecheneycherylj: thanks for your fix23:58
davecheneydon't worry about adding those extra patch values23:59
davecheneyi'll take care of them with my change set23:59
davecheneythe goal is to remove version.Current.Arch23:59
davecheneyso they won't compile23:59
davecheneyand i'll find them and fix them23:59

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!