[00:00] thumper: nagging nag, did you talk to smoser about power64 ? [00:00] wallyworld_: it's left over from the original backup script [00:00] wallyworld_: I'm in the process of sorting out all those hard-coded paths [00:01] ok [00:01] we must have been lucky this hasn't failed then [00:01] in an HA environment [00:06] wallyworld_: for your perusal: http://reviews.vapour.ws/r/519/ [00:06] rightio [00:07] wallyworld_: right [00:11] davecheney: yes, been escelated [00:11] for some spelling of that [00:16] wallyworld_: seen the Azure bug? [00:17] axw: yeah, otp, but i was going to ask you to pick it up. i trusted kapil to have tested it but obviously not :-( [00:17] will do [00:17] ty [00:24] davecheney: I noticed that the juju/utils package tests panic with gccgo [00:24] davecheney: but no idea why [00:24] I couldn't grok the panic [00:25] subpackages are fine, just the main one [00:28] wallyworld_: do you know which juju projects have the autolander? [00:31] ericsnow: it's pretty awful that everything is hardcoded to be able machine-0 but if that's going to get resolved soon then I guess that's ok [00:31] thumper: thanks [00:31] s/able/about/ [00:31] thumper: paste.ubuntu ? [00:32] * thumper looks for it again [00:32] menn0: agreed [00:33] davecheney: http://paste.ubuntu.com/9351357/ [00:34] ericsnow: do you know which juju projects are auto-landed? [00:35] thumper: you talking about the bot or for reviewboard? [00:35] ericsnow: bot [00:36] ericsnow: although knowing about reviewboard would also help [00:36] thumper: RB is currently just core and utils [00:36] ericsnow: it seemed that juju/cmd didn't have the rb hookup [00:36] thumper: sorry, my internets shat themselves [00:36] ericsnow: ah... [00:36] you were saying ? [00:36] davecheney: http://paste.ubuntu.com/9351357/ [00:37] ta [00:37] I think I'll just use the github magic merge button for juju/utils [00:37] thumper: first thougth is you have the wrong gccgo === kadams54 is now known as kadams54-away [00:37] davecheney: I've not changed it... [00:37] prety much everyone has the wrong gccgo [00:37] thumper: for the bot I think it's just core [00:37] thumper: do this [00:37] go test -c $PKG [00:37] gdb --args ./$PKG.test [00:37] r [00:38] davecheney: can I use '.' for $PKG? [00:38] yes [00:38] so if your in juju/cmd [00:38] the bin will be called [00:38] ./cmd.test [00:38] davecheney: but also need to specify the compiler right? [00:38] yes [00:38] wallyworld_, menn0: fix landed for 1398448 [00:39] -c basically says "don't throw away the test binary afterwads, and give it a predictable name" [00:39] ericsnow: awesome [00:39] davecheney: all passed with gdb [00:39] ericsnow: it'll take a while to filter down to the relevant CI jobs [00:40] menn0: for that "unable to get DB names" issue, looks like the mgo session dropped or something (hence EOF) [00:40] thumper: so here is a fun thing [00:40] gccgo development has moved on since 4.9.2 [00:41] menn0: I'm working on making it at least a little more robust [00:41] what are out changes of getting gccgo 5.0 backported to trusty ? [00:41] s/out changes/our chances/ [00:41] nfi, but we can ask [00:42] basically we're going to have to stick with the trunk of gccgo if we want to have any support from uupstream [00:43] ericsnow: sounds good [00:44] thumper: this is on a branch ? [00:44] i'll try to repro [00:44] davecheney: master [00:44] ok, that makes it easy [00:44] github.com/juju/cmd ? === kadams54-away is now known as kadams54 [00:46] davecheney: no, utils === kadams54 is now known as kadams54-away [00:47] * davecheney smacks forhead [00:48] http://paste.ubuntu.com/9351435/ [00:48] thumper: ummm [00:48] what happened here [00:48] oh, wait [00:48] sorry, local problem [00:51] ericsnow: ty, sorry just got out of meeting [00:51] wallyworld_: no worried [00:52] thumper: no, many are supposed to, you mean on github or lp? [00:52] i think the tarmac bot has died [00:52] wallyworld_: I just don't know which ones [00:52] wallyworld_: my general approach is to try $$merge$$ and if nothing happens for a few minutes, do it manually [00:53] we need to follow up there, it's fallen into a hole [00:53] thumper: ok, repro pretty easy [00:53] wallyworld_: github lander not lp [00:54] % env GOMAXPROCS=42 ./utils.test [00:54] Segmentation fault (core dumped) [00:54] wheeee [00:55] davecheney: is the fix as obvious? === kadams54-away is now known as kadams54 [00:56] now I should be able to get it to shit itself under gdb [00:57] urgh, looks like a gc bug [00:57] or a crash in libunwind [00:59] http://paste.ubuntu.com/9351492/ [01:00] interesting [01:01] thumper: what release are you running ? [01:01] trusty [01:01] same [01:02] * thumper afk for a bit [01:07] davecheney: oh, i've seen that one [01:08] davecheney: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64001 [01:10] mwhudson: do you want me to work on a repro ? [01:10] it's related to the http -> tls -> asn code path [01:10] davecheney: a small-ish repro would be great, yes [01:10] davecheney: yeah, but the crash seems dementedly unrelated to that [01:10] you're right about the stack split [01:11] that paste above is pretty clear where it's happening [01:11] yeah, but not why [01:11] somethign it makeing libunwind shit itself [01:11] well no [01:11] ? [01:11] at least not in my case [01:12] it's in the __morestack splitting code [01:12] and $rsp is bogus [01:12] sometimes unaligned, sometimes unmapped [01:12] i guess if you're really lucky it's valid but pointing at some random part of the heap so you get random corruption [01:12] http://paste.ubuntu.com/9351539/ [01:13] mwhudson can you do this [01:13] i saw at least three distinct failure modes fwiw [01:13] env GOGC=1 gdb --args go get github.com/lxc/lxd [01:13] that was one of them [01:13] yeah, i've seen some others [01:13] i think the https one is the easiest to make a repro [01:15] it worked once... [01:15] but then [01:15] Program received signal SIGBUS, Bus error. [01:15] __morestack () at ../../../src/libgcc/config/i386/morestack.S:529 [01:15] oh [01:15] interesting [01:15] (gdb) p $rsp [01:15] $1 = (void *) 0xffffedf45360 [01:16] blergh [01:16] mwhudson: ok, i'll try to make a stand alone repro that blows up [01:16] hopefully ian can figure out what is failing [01:16] 'cos it's way above my ken [01:17] yeah [01:17] i spent an hour or so poking at it when i was in austin and got ~nowhere [01:19] katco: standup? [01:19] mwhudson: do you know if 4.9.2 is available in a trusty-backports ? [01:19] wallyworld_: shoot sorry brt [01:19] davecheney: i do not know [01:19] poop === kadams54 is now known as kadams54-away [01:27] mwhudson: http://paste.ubuntu.com/9351595/ [01:27] worlds smallest repro [01:27] mwhudson: will you be my mule and put this stuff on the gcc bugzilla ? [01:34] menn0: updated https://github.com/juju/cmd/pull/10/files [01:34] davecheney: that is pretty small [01:36] * thumper rushes to get the cmd branch in before anastasiamac [01:42] davecheney: ! [01:42] wallyworld_: sorry I was wrong, D1 fails for me too. I put in a rubbish name and saw "D1", but that's hard-coded in D1 [01:43] err hard coded in Juju [01:44] davecheney: sure, will en-bugzilla [01:47] axw: no worries, glad it's reproduceable [01:50] thumper: looking [01:50] mwhudson: so that one liner can blow up simply [01:50] i'm trying to disect it down to the bit that confuses the gc [01:50] oh [01:51] i know what it could be [01:51] asn1 probably looks like a random field of pointers to the gc [01:54] oh [01:54] it fails with GOGC=off though? [01:55] should do [01:55] repro is super fiddly [01:56] oh, interestingly [01:56] that aspolodes as well [01:56] which means you were right about the stack split [02:00] thumper: u r making me cry but it's k - whoever has the last laugh and all.... :-P [02:00] thumper: done. there's a few doc issues but otherwise LGTM. [02:01] thumper: note that some of the comments are directly against the last commit which Github makes a little tricky to get to. [02:06] menn0: ta [02:07] ah... didn't fix the docstrings on those alias methods [02:07] bugger [02:09] anastasiamac: i like your attitude to code review [02:09] don't get mad, get even [02:10] davecheney: i get made at drivers not coders ;-) [02:10] davecheney: plus thumper is scary [02:10] davecheney: wouldn't want to get mad at him... [02:11] pfft [02:11] yeah, he's fluffy [02:11] like a kitten [02:11] fluffy now with the facial fuzz [02:16] wallyworld_: https://code.launchpad.net/~axwalk/gwacl/rolesizes-fix-dg-names/+merge/243478 please [02:16] looking [02:20] axw: where are the Aliases used? [02:20] wallyworld_: nowhere yet, they'll be used in Juju [02:20] wallyworld_: I'll extend environs/instances/InstanceType to have an "Aliases []string" field [02:21] axw: ok, do we have the ExtraLarge etc ones the right way around? [02:21] wallyworld_: yes [02:21] rightio [02:21] wallyworld_: from Azure: [02:21] 2014-12-02 11:37:29 ERROR juju.cmd supercommand.go:323 failed to bootstrap environment: cannot start bootstrap instance: POST request failed: BadRequest - Value 'D1' specified for parameter 'RoleSize' is invalid. Allowed values are 'ExtraSmall,Small,Medium,Large,ExtraLarge,A5,A6,A7,A8,A9,Basic_A0,Basic_A1,Basic_A2,Basic_A3,Basic_A4,Standard_D1,Standard_D2,Standard_D3,Standard_D4,Standard_D11,Standard_D12,Standard_D13,Standard_D14'. (http code [02:21] 400: Bad Request) [02:21] ok, and the same for the G ones I guess [02:22] those names look like a dog's breakfast [02:23] waigani_: very complex review ... not! http://reviews.vapour.ws/r/575/ === kadams54 is now known as kadams54-away === kadams54-away is now known as kadams54 [02:24] wallyworld_: is the landing bot unblocked? [02:24] should be [02:24] for master? [02:24] i landed a fix last night [02:25] jw4: FAIL: action_test.go:144: actionSuite.TestFindActionTagsByPrefix [02:25] jw4: intermittent? [02:29] davecheney: __morestack is called a _lot_ [02:29] it turns out [02:33] WTH!!! [02:34] that action test fails for me all the time... [02:36] wallyworld_: I always forget. the bot does gwacl? [02:36] axw: i merged by hand, i think tarmac is dead [02:36] ok [02:38] * thumper throws his hands up and leaves the office [02:40] how the fuck did this test land? [02:41] menn0: state/action.go:278 [02:42] menn0: and apiserver/action/action_test.go:144 [02:42] mwhudson: yeah, which is odd [02:42] 'cos there is no escape analysis in gccgo [02:42] so there should be _less_ stack pressure [02:43] * thumper heads to the 'uper duper market [02:44] davecheney: so i was wrong in my initial bug comment, __generic_morestack is returning junk [02:44] davecheney: also, it really doesn't fail with my random gccgo tip build [02:56] wallyworld_: D1 fails to provision still, but it's a different error now (Compute.OverconstrainedAllocationRequest) [02:57] this may take a little while... [02:57] balls [02:58] sure that's not just a transient azure snaffu [02:58] wallyworld_: I get the same error on West US and Southeast Asia, for both D1 and D2 multiple times [02:58] trying through the management console now [02:59] hmmm === kadams54 is now known as kadams54-away [03:35] menn0: ping [03:40] jw4: ping [03:41] wallyworld_: can I get you to test something for me please? [03:41] sure [03:41] wallyworld_: on master, run the tests in apiserver/actions plz? [03:41] I get a fail that I can't see how it landed [03:43] it is me [03:43] grr [03:43] fuckity fuck fuck [03:43] I'm bringing in jw4's utils change... [03:44] ok, just gotta shelve, sec [03:44] OK: 8 passed [03:44] PASS [03:44] ok github.com/juju/juju/apiserver/action 5.622s [03:44] thumper: ^^^^ [03:45] ta [03:45] let me check that y master is up to date, i think it ia [03:45] it is [03:45] I know what it is [03:45] kk [03:55] thumper: sorry, I was out giving programming lessons [03:55] menn0: to? [03:56] thumper: still need me to do something? [03:56] thumper: a friend's son (he's 11 but very keen) [03:56] * thumper nods [03:56] doing an hour a week with him [03:56] menn0: got a minute to hangout? [03:56] thumper: yep [03:56] menn0: standup hangout [03:57] thumper: there now [04:25] 4421 lines in one file :-( [04:26] painful... === kadams54 is now known as kadams54-away [05:33] jam1: ping [05:41] davecheney: hey [05:42] bradm: hey [05:42] davecheney: this might be easier than going back and forth via RT :) [05:43] bradm: yeah [05:43] davecheney: so I checked the squid config on batuan, you did not have github on it. [05:43] davecheney: but you do now. :) [05:43] excelelnt [05:43] thanks [05:43] i'll try now [05:43] davecheney: give it a go, all those things you listed should be working. [05:43] bradm: what do I do abour getting mercurial on rubgy ? [05:43] davecheney: is the packaged version ok? [05:44] for juju uses all three of git, bzr and hg [05:44] yup [05:44] davecheney: do you need it in a chroot? or just in the base OS? [05:44] base os [05:44] don't care about that chroot stuff [05:44] in this case i'm just a luser [05:44] you put your hg in your bzr in your git? then you should cvs it, and then rcs that. [05:44] you'd never lose anything then. [05:45] thanks for the tip [05:45] did i mention we also use more than one code review system [05:45] * davecheney stabs self in face [05:46] davecheney: mercurial package is installed. [05:47] danka [05:48] davecheney: how's that look? need anything else to get you going? [05:50] squid.internal gives some public ip [05:50] can I check the proxy settings with you ? [05:50] sure [05:51] what are you using? [05:51] http_proxy=http://squid.internal:8123 [05:51] https_proxy=$http_proxy [05:51] export http_proxy https_proxy [05:52] try http_proxy=http://batuan.canonical.com:3128 [05:52] and the https_proxy bit too [05:53] squid.internal is mostly used by UK hosts, I only mentioned it because I wasn't sure what you had [05:53] bradm: thanks [05:53] all working now [05:53] davecheney: perfect! let us know if you have any further issues [05:54] bradm: dfc@rugby:~$ gcc [05:54] The program 'gcc' is currently not installed. To run 'gcc' please ask your administrator to install the package 'gcc' [05:55] sorry, i didn't even think to check this [05:55] could I get build-essential and gdb pls [05:55] davecheney: running [05:57] davecheney: done. [05:57] thanks [06:05] * axw pulls hair out [06:05] wallyworld_: now I'm finding that some locations don't have some role sizes. gotta filter that out too... [06:05] le sigh [06:10] ffs [06:48] fwereade_: if you did have time, here's the work to generate the server cert on each state server. i've removed the cert from state entirely. http://reviews.vapour.ws/r/552/ [07:39] wallyworld_: https://code.launchpad.net/~axwalk/gwacl/listlocations/+merge/243495 - another small one, please [07:41] looking [08:27] morning [09:00] jam1: a few seconds, missed to install the plugin [09:01] jam1: dimitern: so, in the hangout [09:03] TheMue, what hangout? [09:04] dimitern: I thought you would be part of the 1:1:1 ;) [09:04] TheMue, ah, I though yours was yesterday [09:05] thought [09:05] dimitern: jam1 moved it due to his holiday [09:05] TheMue, I don't have the link - can you add me to the guests? [09:05] dimitern: yep [09:06] dimitern: just invited you [09:07] TheMue,yeah, thanks - hmm.. it came to my phone though [09:07] dimitern: *lol* that's the magic of the google cloud [09:14] dimitern: when you get a chance old boy [09:14] dimitern: http://reviews.vapour.ws/r/564/ [09:17] voidspace, sure, will have a look shortly [09:33] dimitern: "PickNewAddress" (or whatever we call it). Subnet method or State method? [09:34] dimitern: I think Subnet method. State has a big enough API already. [09:34] coffee - brb [10:02] dimitern: TheMue: standup ? [10:02] jam1, omw [10:02] jam1: coming, just finished 1:1 [10:09] voidspace: ping [10:12] good morning [10:13] jam1: after your standup, can you ping me back? i need to ask about bug 1397376 [10:13] Bug #1397376: maas provider: 1.21b3 removes ip from api-endpoints [10:14] wallyworld_: we'll want to make sure dimitern is in that conversation, as there seems to be some strong disagreement about whether things should be talking DNS names or IP addresses. [10:14] sure [10:14] customers want ip addresses i think [10:15] and there are claims only returning one address is a regression [10:15] since it used to return multiple [10:15] wallyworld_: IIRC from the maas discussion, the IP address can change while the DNS name would stay consistent [10:15] and the api is called api-endpoints after all [10:15] so the MaaS guys asked us to talk in terms of DNS names [10:15] oh, i see [10:16] rogpeppe: hey. would there be any reason for charm.Meta to have bson tags, if we duplicate the structure in state and put bson tags there? [10:16] wallyworld_: api-endpoints did return only 1, then it started returning 2, then we reverted back to 1 [10:16] davecheney: did you mean http://golang.org/pkg/os/#Create ? in your email about chmod? [10:16] wallyworld_, jam1, I can bring this up at today's maas cross team call [10:17] axw: yeah, i think we're storing it directly in mongo in the charm store [10:17] ok [10:17] dimitern: that would be good. i fear (perhaps unnecessarily) that other providers would work better with ip addresses [10:18] eg openstack autopilot [10:18] see comment #8 [10:18] so it seems we want one behaviour for maas, and another for openstack [10:19] but if the provider gives us the correct info for machine addresses, it all should just work [10:19] wallyworld_, afaik maas did not return dns names before due to a bug - it was supposed to [10:19] and we need the prefered address a 0 [10:19] as [10:19] as per comment #9 [10:20] dimitern: so it seems then that with a mass that works correctly, the bug becomes a matter of ensuring that we ensure that the preferred ip address is in Addresses[0] so that's te one printed [10:22] wallyworld_, or we can just try to resolve dns names before saving [10:22] for maas [10:22] dimitern: wallyworld_: so I believe the bug *here* is that MaaS guys are saying "use the DNS name" but the Autopilot guys are saying "but I don't want to have to add MaaS as my DNS source" [10:22] It makes some sense fo inside the MaaS cloud as everything is run by MaaS, but for client machines [10:22] they are fairly likely to just know the MaaS endpoint, and not be configured to talk to "foo.maas" [10:23] client machines would want ip address i think [10:23] wallyworld_: AIUI, api-info is intended to supersede api-endpoints [10:24] as it can give information on stuff like CA Cert [10:24] that could well be true, i'm not across the detail on this bit of the system, hence asking you guys :-) [10:24] but [10:24] we do need to consider backwards compatibility, no? [10:26] dimitern: jam1: so can i leave this bug in your guys' capable hands? :-) [10:26] wallyworld_, sure thing :) [10:27] ty :-) \o/ [10:28] dimitern: andrew is working the azure bug - it got complicated, so he has to add in extra apis to query location as not all locations support all the role sizes :-( [10:28] wallyworld_, sweet! I underestimated that one badly [10:28] dimitern: I'm just makign coffee, but I'll be at the next meeting. [10:29] dimitern: me too. i thought that the gwacl changes made by kapil were all good to go, bad assumption :-) [10:29] as it turned out [10:29] azure is complicated [10:30] yeah, and often broken as well [10:30] yep, through no fault of ours it seems many times [10:30] fwereade_, jamespage, gnuoy, networking call? [10:31] dimitern, yes - sorry - still in stockholm [10:34] hi axw, you still around? [10:36] bac: heya, yes I am [10:37] axw: good, i know it is late for you. i saw a problem with azure yesterday that i wanted to tell you about. [10:37] bac: is it this one? https://bugs.launchpad.net/juju-core/1.21/+bug/1398406 [10:37] Bug #1398406: Azure provider attempts to deploy with unsupported "D1" RoleSize [10:37] axw: no. [10:37] okey dokey [10:38] rogpeppe: pong [10:38] rogpeppe: sorry, only just seen your ping [10:38] voidspace: np [10:38] axw: it involved having two environments up with the same credentials but different storage for the state servers. i used destroy-environment on one and it took down both. [10:38] voidspace: how much do you know about USSO? [10:38] eep [10:38] rogpeppe: well, I've never heard of the acronym [10:38] axw: i have the remnants on azure but have not been able to create a minimal reproduction of the problem [10:39] rogpeppe: so not a good start... [10:39] voidspace: just saw your name on the identityprovider source code and thought you might be able to help us... [10:39] axw: i was using 1.20.12-utopic-amd64. [10:39] bac: I suspect azure is not using the env UUID to separate things... I will see if I can figure it out. did you raise a bug already? [10:39] rogpeppe: heh, I worked a lot on identityprovider - but not actually on the identity protocols in the end [10:39] axw: i did not since i could not reproduce [10:39] rogpeppe: and I don't recall ever hearing USSO, so it might have been added after I left [10:39] fair enough [10:39] voidspace: ubuntu single sign on [10:40] bac: thanks, I'll take a look at the code to see if I can think up a repro [10:40] axw: the problem was quite costly as it destroyed our CI environment [10:40] :( [10:40] rogpeppe: ah... [10:40] axw: if you'd like to look at the jenv files or azure storage i've kept them [10:40] rogpeppe: oh, it depends on the question then [10:40] rogpeppe: it's been a while though... [10:40] bac: that would be good to have [10:40] axw: and i'm happy to file a bug [10:40] voidspace: perhaps you could join us in a hangout for a few moments? [10:41] rogpeppe: sure [10:41] axw: do you have access to chinstrap? i'd like to not sanitize the jenv files so i don't want to attach them to the bug report [10:41] bac: yes I do [10:41] axw: cool, i'll put it there and reference it. [10:42] axw: thanks and good night [10:42] bac: thanks [10:43] (and sorry about your CI env :() [10:58] PTAL, I tweaked the isLimitedRoleSize code a bit [10:58] err [10:58] wallyworld_: ^^ [10:58] looking [11:01] axw: just a typo, might be good for one more live test before landing, just to be sure with the tweaks [11:02] wallyworld_: thanks. sure [11:03] * axw dinners first [11:03] np, ty [11:12] voidspace, whew.. no meetings for a while, so back to your review === ashipika1 is now known as ashipika [11:16] dimitern: cool, thanks === ashipika1 is now known as ashipika [11:34] voidspace, reviewed, please ping me if something is unclear === ashipika1 is now known as ashipika [12:05] dimitern: sure, thanks [12:22] Is there a create version tat supports permissions? [12:43] what exactly will _posix tag match? [12:43] build* tag [12:56] fwereade_: if you did get a chance or the inclination to look at http://reviews.vapour.ws/r/552/ that would be great, but you're busy so i'll bother someone else tomorrow if needed [12:57] wallyworld_, thanks, I'll try [12:57] sure, no hassle if not [13:28] wallyworld_: the Azure fixes still need backporting to 1.21, I'll do that tomorrow [13:28] axw: sure, np [13:28] thanks for fixing [13:30] axw: i filed bug 1398820, not sure if you saw it [13:30] Bug #1398820: destroying Azure environment took down other environment === kadams54 is now known as kadams54-away === kadams54-away is now known as kadams54 [14:35] wallyworld_ ping [14:36] sinzui: thanks for the insight on that bug. i'm trying to reproduce with azure and azure-1 now. [14:36] mbruzek1: it's 00.35am where wallyworld is :) [14:37] but sadly he is here [14:37] ping retracted go to sleep Ian [14:37] mbruzek1: how can i help? [14:37] wallyworld_: OMG!! [14:37] like you can talk :-) [14:37] bac: abentley and I are trying to remember the issue we found about a year ago. [14:37] same time for you [14:37] wallyworld_: I had 89 comments on my tiny PR... how can I sleep? [14:38] all/mostly trivial [14:38] wallyworld_: true. in fact, I was hoping u could review tomorrow in hopes of landing before ur holiday :) [14:38] sinzui, bac: https://bugs.launchpad.net/juju-core/+bug/1257481 [14:38] Bug #1257481: juju destroy-environment destroys other environments [14:39] anastasiamac: sure [14:40] wallyworld_: thnx ;-) m EOD-ing now [14:40] as well you should [14:40] thanks abentley. [14:40] wallyworld_: u2 [14:40] sinzui, landing is blocked on 1398448 - but it looks like a fix has landed for it? [14:42] * sinzui checks test [14:44] mattyw, ha ha, that bug is indeed fix released, but is replaced by another bug 1398837 [14:44] Bug #1398837: cannot extract configuration from backup file: "var/lib/juju/agents/machine-0/agent.conf [14:45] sinzui, so still blocked - but for a different reason? [14:45] mattyw, yes, sorry. [14:48] sinzui, no problem - is someone working on that bug? [14:49] mattyw, not yet, it was reported when abentley was checking on the other blocking bug. I am sending out an email about the many hot bugs targeted to 3 milestones [14:50] sinzui, ok - I'm not vounteering as I'm going to be out for a few days - but just wanted to make sure they weren't being ignored [14:50] sinzui, so I'm not being helpful - but I'm being supportive ;) [14:50] understood Makyo [14:50] understood mattyw [14:53] ooh ooh, can I be unhelpful but supportive, too? ;) [14:56] natefinch, I've taken that job, you'll have to be helpful but unsupportive [14:57] or you can be unhelpful and unsupportive [14:57] I can't believe you fools! here's the fix. [14:57] natefinch: /ignore sinzui ? [14:58] I didn't even know /ignore was a thing... [14:58] * natefinch is an IRC n00b [14:58] odd, you seem to have the age to have been young when IRC was a thing [14:58] * perrito666 hides [14:59] thou dost injure me, perrito666 [15:00] I used IRC back in the late 90's, and figured everyone had moved on since then. But, you know, linux people seem to pine for 1999 for some reason. [15:00] natefinch: you do realize that if I did not spent so much time watching tv and movies that joke would have been completely lost on non english native speakers [15:08] wallyworld_, jam1, the dns issue with api-endpoints for maas is not maas-related, it stems most likely from the changes in address selection logic (i.e. prefer hostnames to public/cloud-local IPs - the latter being most common with maas) [15:17] sinzui: ping [15:17] hi katco [15:17] sinzui: good morning :) [15:17] sinzui: i'd like to provide andreas with some binaries for tip of 1.21 [15:18] sinzui: what's the best-practice for doing so? i want to make sure he's testing what will eventually be b4 === kadams54 is now known as kadams54-away [15:19] katco, http://juju-ci.vapour.ws:8080/job/publish-revision/1254/ lists the binaries we built and tested [15:19] katco, , but there are no streams for these [15:19] sinzui: that's probably _perfect_ [15:20] sinzui: i haven't wrapped my head around simple streams yet, but i suspect he could use these binaries to update his existing testing environment? [15:21] perrito666: FYI, that blocker is due to the hard-coded "machine-0" in the old restore :P [15:21] katco, --upload-tools is the only option. [15:22] sinzui: hm. he can probably build a local environment if pressed. [15:22] katco, our testing streams are volatile, they are master at the moment. there were 1.21-beta4 for about 3 hours when those packges were made [15:23] katco, yes, the local env will be fine he just needs the state-server and two services [15:23] sinzui: i'm pretty confident in that use-case. but i want to make sure it works for his more complex example [15:23] ericsnow: well, i told you that there was no guaranteeof compatibility with thenew backup [15:24] perrito666: I'm expecting that the new restore will break in the same way (under HA) [15:24] sinzui: i am able to reproduce bug 1398820 and have added more information. [15:24] Bug #1398820: destroying Azure environment took down other environment [15:24] ericsnow: most likely, but it is waaaay easier to fix [15:25] perrito666: easier to test at least :) [15:26] sinzui: due to documented loss of user data (the blunder cost us 10 engineer hours and denied 12 engineers the use of our landing automation for five hours) i'd suggest it be marked critical. a work-around after-the-fact is not very useful. [15:26] ericsnow: inded [15:26] thank you bac [15:31] dimitern: one of your suggestions [15:31] dimitern: adding the following to the comment for the State.IPAddress method "with the given value." [15:31] dimitern: you don't think it's entirely obvious that one returned will represent the value you pass in? [15:32] voidspace, yeah, I really meant to say "missing full-stop" [15:32] dimitern: heh, you got me on that one. [15:32] dimitern: cool [15:33] voidspace, in general doc comments should be proper sentences when possible I think [15:33] dimitern: ok, I've never been quite sure. [15:33] dimitern: I'll stick to that from now on. Easy enough. [15:33] voidspace, cheers [15:39] dimitern, ping [15:40] alexisb, hey [15:40] alexisb, just replying to your mail btw [15:40] hey there dimitern [15:40] nws, no rush on that [15:41] I saw some of your irc chatter earlier; are you working this bug: [15:41] https://bugs.launchpad.net/juju-core/+bug/1397376 [15:41] Bug #1397376: maas provider: 1.21b3 removes ip from api-endpoints [15:41] ?? [15:42] alexisb, I did investigate the issue - it seems like a juju-specific regression (if you can call it that, since it was never documented nor claimed anywhere api-endpoints should return IPs instead of hostnames) [15:43] dimitern, given it is currently marked as critical and blocking the 1,21 release we need to decide on a path forward and get it resolved [15:44] alexisb, there are several possible solutions [15:45] alexisb, and I prefer to have the same one across all providers, i.e. always return a usable IP as first endpoint (if we only have a hostname try resolving it first) [15:45] alexisb, I'll add a comment to the bug [15:45] dimitern, thank you [15:46] alexisb, and propose a fix + backports [15:46] make sure to make it clear if you are looking for a response from the bug commiter, etc [15:47] sure === jog_ is now known as jog [15:49] dimitern: instead of enforcing that InterfaceId or MachineId can only be set once [15:50] dimitern: how about we only do it once? [15:50] dimitern: if we do that then the check is unnecessary. If we *need* to do it more than once then the code is actually a hindrance. [15:50] dimitern: so at *best* the code is useless... IMO [15:51] * fwereade_ swears at the jujud tests and kicks things for a bit [15:51] voidspace, how can you guarantee it's only once [15:51] dimitern: by only doing it in one place [15:51] dimitern: by understanding the code [15:51] dimitern: if we attempt to do it more than once and it fails juju will have problems due to a code path that stops [15:51] dimitern: so we need to do that *anyway* [15:52] dimitern: I can add an assert, it's easy enough [15:52] dimitern: I'm just not convinced it's very useful, and may actually be the opposite of useful (at which point we just take it out again) [15:52] fwereade_, I see you are having a good evening ;) [15:52] alexisb, a delight, as always :) [15:54] :) === kadams54-away is now known as kadams54 [15:58] voidspace, ok, let's think about it for a bit [16:00] voidspace, the reason we have similar "hard states", e.g. once a machine is provisioned it can't be "unprovisioned", is because these steps happen at different times and more importantly in different workers [16:01] dimitern: right [16:01] dimitern: in our case an IP address will be requested *for* a machine [16:01] dimitern: so nothing else will attempt to use it [16:01] dimitern: and once allocated the MachineId will be set [16:02] dimitern: so there's no use case for changing it, but nor is it possible that something will try [16:02] voidspace, yeah, it seems reasonable to bind setting state to allocated with setting the machine id [16:03] voidspace, how about having a AllocateTo(machineId, interfaceId) method ? [16:04] dimitern: instead of individual setters - ok. [16:04] dimitern: to add the asserts I need to get rid of omitempty it would seem [16:04] I *would* need to get rid of omitempty [16:04] voidspace, it asserts the state is unknown and machine / interface ids are empty before setting state to allocated + mid +iid [16:05] voidspace, yeah - omitempty should go on both ids [16:05] dimitern: so you still want the asserts... [16:05] to protect us against something that can never happen... :-p [16:06] I should sprinkle anti-polar-bear dust around the code as well just in case [16:06] voidspace, imagine multiple workers trying AllocateTo() in parallel [16:06] dimitern: but they'll be given different ip addresses [16:06] voidspace, true [16:07] technically there's a race between fetching the full set of existing addresses and generating a new one [16:07] voidspace, ok, at least let's have an error when it's already allocated [16:07] ok :-) [16:08] voidspace, indeed, that's why we need to assert the txn-revno of ipaddressesC haven't changed since we fetched them :) - good point [16:08] a fair compromise and that will protect against the race [16:08] we should do that in the code that generates the ip address [16:08] voidspace, but that's relevant to the picking algorithm [16:08] voidspace, yep [16:09] voidspace, dimitern: not sure if this is how fwereade_ intended this to be used, but the new leasing stuff will provide a sort of "environment mutex" [16:10] where you could say, "i'm mucking in the ip addresses, give me that lease" [16:10] voidspace, so AllocateTo() returns nil on success or - say ErrAlreadyAllocated with message "IP "1.2.3.4" is already allocated to machine "1"" [16:10] and then when others would like to know about ip addresses they could grab that lease [16:10] katco, oh GIL ? :) [16:10] or we can call it JIL instead :D [16:10] not familiar with that term [16:10] katco, good to know [16:10] katco, I suspect that will not be a very happy pairing of components [16:11] katco, that's python's global interpreter lock [16:11] ahh ok === tvan-afk is now known as tvansteenburgh [16:11] fwereade_: thanks for chiming in :) i retract my suggestion! :p [16:11] katco, no worries :) [16:12] dimitern, voidspace: I'm not really up to date with what you're dicussing, btw [16:13] dimitern, voidspace: but txn-revno is a very big hammer to be using in general [16:13] voidspace, and the other method can be SetUnavailable(value) - returning some (doesn't have to be typed) error when the address is already unavailable [16:14] fwereade_, we're discussing how to do the address allocation algorithm for containers [16:15] fwereade_, creating a new placeholder doc in the ipaddresses collection with state "unknown" (similarly to local charm uploads), then allocating it (or failing) and changing the state to "allocated" or "unavailable" depending on the result [16:16] natefinch: stand up? [16:16] dimitern: using txn-revno would also prevent *another* address being allocated [16:16] perrito666: momentarily [16:17] voidspace, correct [16:17] voidspace, dimitern: I'm probably being dense, but txn-revno on what doc exactly? [16:17] fwereade_: ipaddresses - a new one [16:17] fwereade_, voidspace, on the new ipaddresses collection Michael is doing now [16:18] voidspace, so the documents in that collection will be one-to-one with machines? [16:18] fwereade_, it will contain all machine addresses juju knows about [16:18] dimitern, not just one doc with all addresses, surely? [16:18] fwereade_, yes - machines and network interfaces [16:19] dimitern, that will be a hell of a bottleneck [16:19] fwereade_, wow, that haven't occurred to me *lol* [16:19] dimitern, even without txn-revno, all writes will be serialized [16:20] fwereade_, right, so txn-revno is no good then [16:21] fwereade_, we can't just fetch all addresses, pick a random in the same range, not yet existing, then create it asserting the collection hasn't changed [16:22] fwereade_: forgive my ignorance, but doesn't this really sound like an environmental mutex would be helpful? what am i missing? [16:22] katco, how is that mutex supposed to work? [16:23] dimitern, voidspace: I am most likely misunderstanding the problem -- but I'm not following who needs to allocate these addresses? [16:23] dimitern, voidspace: surely in the general case we are asking the provider for new addresses, aren't we? [16:23] fwereade_: no [16:23] dimitern: well, the lease service is a fifo stack; 1 exists per state machine, and only 1 person can own a lease for a namespace at a time [16:24] fwereade_: for "reasons" [16:24] fwereade_: let me remember specifically [16:24] fwereade_, no, we're asking for a specific address [16:24] fwereade_: with ec2 we can request they give us a new address - but they don't tell us what it is [16:24] fwereade_: so we then have to look on the network interface and find "the new address" [16:25] fwereade_: and if we do that in parallel (adding several machines to an interface) we have race conditions working out which one is for which machine [16:25] fwereade_, yeah - ec2 is quite unhelpful and most providers allow you to request a specific address to allocate [16:25] fwereade_: but both ec2 and maas (and openstack but we only initially care about ec2 and maas) [16:25] voidspace, dimitern: oh, what fun [16:25] fwereade_: will allow us to pick an address and request that specific one [16:25] fwereade_: so we're generating and storing allocated addresses [16:26] fwereade_, and keeping track which machine uses them, or marking them as unavailable (i.e. something else uses it, we'll try another but remember this) [16:26] voidspace, dimitern: ok, so, to try to restate in my own words [16:27] voidspace, dimitern: an example would be: you have machine N, with a bunch of containers N/lxc/A,B,C [16:28] voidspace, dimitern: something running in the agent for machine N observes that there are 3 new addresses available, and needs to assign them to the appropriate containers? [16:28] voidspace, dimitern: or have I completely misunderstood the context? [16:29] * fwereade_ suspects he completely has [16:30] fwereade_: it's more that we notice there are three new containers, we need to create three new addresses [16:30] fwereade_: more likely we will try to create one address three times [16:30] fwereade_, yeah - that thing will be the container provisioner most likely, but it won't notice the new addresses, it will request them and allocate them to each container, so that the address updater (after this happens) can see the machine has 4 IPs (as seen via the cloud api) but only 1 IP is for the machine (already allocated) the rest are for the containers [16:30] fwereade_: if we arean't picking the addresses ourself, but we have a worker that requests an address and then has to work out what the new one is [16:31] fwereade_: if that happens three time in parallel - when the worker checks the network interface and sees three new addresses, how does it know which is the "new one" [16:31] dimitern, that cannot be the container provisioner, can it? [16:31] dimitern, we don't want to give it the cloud credentials necessary to ask for the addresses [16:32] fwereade_, it won't talk to the cloud directly, obviously the api will be used [16:32] :) [16:32] dimitern, ok cool :) [16:33] dimitern, not 100% sure it's the container provisioner then? but, ehh, that's an irrelevant detail at this point I guess [16:33] voidspace, fwereade_, so effectively: the apiserver will be creating ipaddresses docs, calling the cloud to allocate them and assigning them to their machineid [16:33] voidspace, fwereade_, that's a detail yes, we'll get there [16:37] dimitern, so, what we have at the moment is that the machiner calls SetMachineAddresses with all the addresses it knows about -- and presumably we'll keep on doing roughly the same thing? just doing so more often, and not in the machiner? [16:38] bac: building amulet now [16:38] will be released in about 20 mins [16:38] thanks marcoceppi [16:39] fwereade_, IIRC SetMachineAddresses discovers what's on the machine, not via the cloud api [16:39] dimitern, yes [16:39] fwereade_, so it will be the same [16:41] fwereade_, the address updater is problematic and has to be fixed carefully to separate container addresses from host addresses when refreshing the instance info from the cloud [16:41] dimitern, and we also get a stream of SetAddresses~s coming from the instanceupdater as well [16:42] dimitern, how *can* we separate them? what's the difference between a provider address that was allocated for a container, and one that was allocated for the host machine? [16:43] fwereade_, yeah - which in turn also calls SetAddresses at the end via the apiserver [16:43] sinzui: we have confirmation that 1397995 is fixed in v1.21. discussions are ongoing about expected behavior, but that's just a spec thing, not a bug. [16:43] fwereade_, ha! there's the beauty of it :) [16:43] katco, \o/ [16:43] sinzui: tyvm for your efforts and assistance. you did a great job! :) [16:44] fwereade_, because we pre-allocate the address we know what it is and for which machine id (or container) it's allocated before the container starts [16:44] dimitern, ok, so we (could) have a provider id for each address that comes in via the instanceupdater? [16:45] fwereade_, I think the least painful way to resolve all these is to change state.Machine.SetAddresses() internally to do the separation considering what's in ipaddressesC [16:45] fwereade_, by provider id you mean instance id in this case? [16:46] dimitern, no? I think I mean a provider id for the address [16:46] fwereade_, there's no specific id - the IP itself is the key - at least in maas, openstack and ec2 [16:46] dimitern, instance id is not a problem, is it? we know which *host* machine already because it's 1:1 with instances [16:47] dimitern, I am suspicious that there is an implicit assumption that those addresses won't change [16:48] dimitern, which does not gel with what I understand ec2 in particular is likely to do to you if you suspend your instances for a bit [16:51] fwereade_, it depends on how the instance was started (i.e. the termination behavior) - we don't do suspend/resume ourselves [16:52] dimitern, ok, I am still questioning the assumption that addresses won't change [16:53] dimitern, eg when an AZ falls over for a bit I would like it if we were able to absorb that when everything was running again [16:59] evilnickveitch: ping [17:00] voidspace, hey [17:00] evilnickveitch: hey [17:00] evilnickveitch: you just sent me an email about juju docs [17:00] evilnickveitch: I suspect it was intended for someone else... [17:01] voidspace, oops - sorry! I am blaming gmail autocomplete [17:01] evilnickveitch: :-) [17:03] fwereade_, the address won't change *at will* [17:04] fwereade_, it will happen due to some specific user action - i.e. stopping an instance and restarting it perhaps - juju should be able to detect this and know how to handle it [17:06] fwereade_, an address that was allocated (reserved, in-use, whatever) cannot be reused by any other node on the same subnet (except for some weird clustering/balancing/etc. on L2) [17:07] fwereade_, a machine can get a new (allocated) address and become unreachable under the old one, but the old one won't just appear on some machine (as long as the machine is up) [17:08] dimitern, no arguments there... the problem STM to be that i-abc123 can reasonably return 2 non-overlapping sets of IPs in two consecutive requests to the provider [17:08] dimitern, and if we don't have some notion of what the underlying provider id of a given IP is, we can't cleanly map the first set onto the second [17:09] dimitern, (that said, I believe that the underlying provider ids do not necessarily exist -- I'm not saying you should find them, but I am fixating on the nasty edge cases that I think I can see) [17:10] dimitern, in general it's not outside the bounds of probability that we might see, for a single machine [17:10] fwereade_, I understand the problem and appreciate you bringing it up - I'll make sure we do have tests for such cases [17:11] dimitern, SA: a; SMA: b; SA: c, d; SMA: e,f [17:11] dimitern, cool [17:11] dimitern, have I just been massively derailing you then? :( [17:12] fwereade_, shouldn't we prefer addresses discovered on the machine rather than the ones coming via the cloud api? [17:12] dimitern, it still sort of feels like the problem you're talking about is "how do we reconcile these streams of addresses from two different sources" [17:12] dimitern, hmm [17:12] dimitern, I *think* the cloud api is more likely to give us usable addresses, isn't it? [17:13] dimitern, like how in address-get we want to return the advertise address rather than the bind address? [17:13] fwereade_, no, it's very useful as I did have much time to think in detail about how to solve the 2 address sources problem [17:13] dimitern, I assert that what the cloud tells us is more likely to correspond to the addresses which will cause the traffic to be delivered to the right place [17:14] fwereade_, well, if the cloud *thinks* node A uses IP1, but eth0 on A says IP2, the latter will be usable (all other things equal - i.e. no restrictive firewalling/routing for IP1 vs IP2) [17:14] fwereade_, yeah [17:15] dimitern, my concern is exactly that all other things will not be equal [17:16] fwereade_, at some point we'll need a way to actually *verify* a given node's IP is usable (inbound and out) [17:16] dimitern, eg when you expose something to the public internet, you need to advertise the public address that will get the packets to the right place, which is not necessarily the one you're binding to at all [17:16] dimitern, that's a nice-to-have where it's practical [17:16] fwereade_, until then I guess trusting the cloud makes more sense [17:17] dimitern, but I'm not sure it's something we can depend on in all cases [17:17] dimitern, in particular [17:17] dimitern, these thorny sorts of questions [17:17] dimitern, are why I'm keen that we continue to record the stuff that gets set via SA/SMA unchanged [17:18] fwereade_, the "just-in-" case :) [17:18] dimitern, and we recalculate our best guess at reality, taking *both* of those things into account, in response to a change in either [17:19] dimitern: when you have a minute (shouldn't take any longer) can you check the implementation of AllocateTo [17:19] dimitern: http://reviews.vapour.ws/r/564/ [17:19] dimitern: all issues resolved [17:19] fwereade_, right, but the ips known to be allocated should be used when needed, while the SA/SMA ones only when they change [17:19] voidspace, sure, will look shortly [17:20] fwereade_, (the IPs in ipaddressesC I mean) [17:20] dimitern, true -- the providers that *do* tell us what IPs they've allocated should probably be trusted over and above the SA/SMA ones [17:22] fwereade_, what if the cloud says node A's IP is now b (used to be a)? Should we also ensure SMA reports b as well? (I mean reconfigure the interface on the machine) [17:23] dimitern, I have literally no idea, I'm afraid :( [17:24] dimitern, I suspect "it depends" but I don't really know what on [17:24] fwereade_, and also how often/when this is possible [17:38] voidspace, reviewed [17:38] voidspace, I'm ok with moving subnet method tests from state_test.go into a new subnets_test.go as a follow-up [18:01] dimitern, you joining the networking call? [18:06] dimitern: ok [18:07] dimitern: thanks, still a few things to fix [18:08] dimitern: do we *need* DeepEquals for comparing structs? [18:08] dimitern: I wasn't sure [18:09] equals will work for simple structs that don't have, for example, maps or slices in them [18:10] natefinch: and if it doesn't work it should complain, right? [18:10] natefinch: as it worked I didn't feel the need to use DeepEquals [18:10] yep [18:10] cool [18:11] natefinch: so no reason to prefer jc.DeepEquals over gc.Equals if it's not required? [18:15] voidspace: there's probably no reason not to use DeepEquals, the difference in speed is going to be negligible..... trying to think if there might be a reason why Equals would be preferred... [18:18] natefinch: heh, your answer is the opposite of what I asked :-) [18:23] voidspace: my point is, jc.DeepEquals should always work, so there may be no reason to ever use gc.Equals unless you're doing a very large number of comparisons. If one way always works, and one way usually works... why not just always use the way that always works? [18:24] natefinch: because I already have a way that works and I need a reason to swap... [18:25] natefinch: I understand what you're saying though [18:26] voidspace: if it works, it works, don't muck with it ;) [18:26] my thoughts exactly :-p [18:26] I'll start using jc.DeepEquals just to not have to ever have this conversation again... [18:27] haha [18:27] voidspace, it's good to use DeepEquals for structs, otherwise you might end up comparing .String() or .GoString() values [18:27] dimitern: ah, so there is a reason [18:27] dimitern: when would that happen? [18:32] voidspace, it depends on how the Equals checker is implemented I guess [18:32] voidspace, can't really say without experimenting a bit [18:32] ah [18:34] voidspace, I tend to use DeepEquals (the jc version, not the gc one as it prints more comprehensive error messages with deeply nested structs / long maps, etc.) [18:34] voidspace, ..for anything than a simple type [18:34] dimitern: better failure messages is a good reason to prefer it [18:35] voidspace, but maybe it's just me being paranoid :) [18:35] dimitern: when adding a new test file is there anything else I need to do? [18:35] dimitern: it doesn't look like my tests are being run [18:35] as in, I inject a failure and nothing fails [18:35] voidspace, yeah :) [18:35] voidspace, register the suite [18:36] gc.Suite(...) [18:36] or something elsse? [18:36] var _ = gc.Suite(&machineSuite{}) [18:36] that's there === kadams54 is now known as kadams54-away [18:37] voidspace, and no tests run still? [18:37] doesn't look like it [18:37] package is state_test, file is state/subnet_test.go === kadams54-away is now known as kadams54 [18:38] hmmm... maybe it should be subnets_test.go [18:38] dimitern: does each test file need to match a source file? [18:39] voidspace, yeah, but that shouldn't matter [18:39] dimitern: you're on late [18:39] voidspace, btw - see this http://paste.ubuntu.com/9356541/ [18:39] dimitern: fair enough :-) [18:40] hmmm... but the ipaddresses_test is definitely being run [18:40] voidspace, even though I added a .GoString() method on network.HostPort to shorten the output considerably (from network.HostPort{Value: 1.2.3.4, Port:42, ...} to 1.2.3.4:42 [18:40] voidspace, hmm.. let me see if something else was needed [18:41] voidspace, I'm on since 7am :) [18:41] really need to stop soon [18:42] voidspace, can you paste your complete subnets_test.go ? [18:42] dimitern: http://pastebin.ubuntu.com/9356566/ [18:43] dimitern: there's a deliberate error in assertSubnet inside TestAddSubnet (first test) [18:43] that should fail [18:44] voidspace, you don't need to embed ConnSuite, do you?\ [18:44] dimitern: StateSuite does, which is what I copied for this and IPAddressSuite [18:44] if we can get rid of it then cool [18:44] voidspace, hmm.. no sorry - you need it [18:44] we need s.State [18:45] voidspace, yeah [18:45] I've renamed to subnets_test.go [18:45] that didn't help [18:45] voidspace, why do you have that s.policy.GetConstraintsValidator in SetUpTest? [18:46] voidspace, HA! found it [18:46] voidspace, a single character is needed :) [18:47] go on [18:47] voidspace, var _ = gc.Suite(&SubnetSuite{}) - instead of var _ = gc.Suite(SubnetSuite{}) - all methods have pointer receivers [18:47] gah [18:47] of course [18:47] it registers an empty one [18:47] thanks [18:47] no worries [18:47] i'll be off then :) [18:47] sorry [18:48] dimitern: g'night [18:48] voidspace, g'night [18:58] I'm off too [18:58] g'night all [19:50] natefinch: could you take a look at http://reviews.vapour.ws/r/577/ [19:50] natefinch: (and http://reviews.vapour.ws/r/573/ too) :) [19:50] natefinch: that first one fixes the blocker [19:50] ericsnow: cool, looking [19:57] jw4: ping [19:57] jw4: I actually have a branch that fixes one of the actions bits https://github.com/juju/juju/pull/1261/files [19:57] jw4: may clash with your dependency update too [20:01] jw4: interesting that both our solutions were identical - completely identical :-) [20:12] ericsnow: a couple small concerns, but nothing too hard to fix. [20:12] natefinch: k === kadams54 is now known as kadams54-away [20:35] natefinch: I've addressed your comments [20:38] ericsnow: looking [20:40] natefinch: keep in mind that cmd/plugins/juju-restore/restore.go is going away, so it probably isn't worth nitpicking there :) For that file I just want to be sure I have the logic right since there isn't any simple way to test it [20:41] ericsnow: I did a review too, nothing to add [20:41] ericsnow: I'd wait for natefinch's LGTM before landing though [20:41] thumper: thanks :) [20:41] thumper: will do [20:41] ericsnow: I just gave it a shipit :) [20:41] \o/ [20:42] natefinch: thanks [20:42] * thumper awaits an unblocked laner [20:42] lander [20:43] thumper: while you're waiting, do you have some time to talk about provider configuration? I'm porting fwereade_'s skeleton provider from 10-months-ago juju to today-juju, and some of the internals bug me. He thought you might have some insight. [20:43] sure [20:43] natefinch: make a hangout [20:45] thumper: https://plus.google.com/hangouts/_/canonical.com/moonstone?authuser=1 [21:00] davecheney: I was just reading through your functional options talk (very interesting) [21:00] davecheney: do you think there are any spots that could stand to benefit from functional options in juju? [21:02] davecheney: would it be a good approach for pulling in the different paths that need to be backed up (I think mattyw was hinting at this a while back) [21:02] ericsnow: sure [21:02] i think there are a few places we could use it [21:09] thumper: thanks! [21:11] ok, time for breakfast [21:11] then, REVIEWING! [21:21] ericsnow: did you see that your branch failed early? [21:21] thumper: yeah, fixing it now [21:21] kk [21:21] * thumper wants to land stuff [21:52] anastasiamac: I'm reviewing your mega branch [21:58] anastasiamac: done [22:02] anyone know how to force a CI test to run (functional-ha-backup-restore, specifically)? [22:02] thumper: my fix landed but I hesitate to mark the bug as committed until the CI test (functional-ha-backup-restore) runs [22:03] ericsnow: I think they happen automatically [22:03] fix committed won't clear CI til it's marked as fix released [22:03] ericsnow: and marking the bug fix committed doesn't release the bot [22:03] thumper: right [22:03] thumper, jw4: thanks for clarifying [22:42] interesting, windows does not run update as an atomic process [22:43] * perrito666 has an install running in a machine next to him that installed 170 packages, failed on the 171 and just rolled back the whole thing... this all thing took 4 hs [22:44] thumper: https://bugs.launchpad.net/juju-core/+bugs?field.searchtext=&orderby=-importance&field.status%3Alist=NEW&field.status%3Alist=CONFIRMED&field.status%3Alist=TRIAGED&field.status%3Alist=INPROGRESS&field.status%3Alist=FIXCOMMITTED&field.status%3Alist=INCOMPLETE_WITH_RESPONSE&field.status%3Alist=INCOMPLETE_WITHOUT_RESPONSE&field.importance%3Alist=CRITICAL&field.tag=ci+regression+&field.tags_combinator=ALL [22:44] thumper: when that list is empty CI will be unblocked [22:44] * thumper sighs [22:44] * thumper waits [22:44] thumper: I personally use https://api.launchpad.net/devel/juju-core?ws.op=searchTasks&status%3Alist=Triaged&status%3Alist=In+Progress&status%3Alist=Fix+Committed&importance%3Alist=Critical&tags%3Alist=regression&tags%3Alist=ci&tags_combinator=All [22:44] impatiently [22:45] handy link [22:45] * jw4 has had code to land for a week now [23:16] davecheney: do you feel answered by my email? I was not sure if you answered the review with an email or added a review and then deleted it [23:17] natefinch: fwereade_ ericsnow you should all have a mail with the shared draft for b&r specs === kadams54-away is now known as kadams54 === kadams54 is now known as kadams54-away [23:31] thumper: thnx for the review!!! [23:31] davecheney: i think you're ocr? you able to take a look at http://reviews.vapour.ws/r/552/ for me? === kadams54-away is now known as kadams54 [23:54] wallyworld_: looking [23:55] perrito666: could you summarise the state of play for me [23:56] ty [23:56] i'm unclear what or if there is a problem [23:58] wallyworld_: on a scale of 1 to "don't be picky dave", is the importance of this PR ? [23:58] davecheney: i would like to land but if there are issues, record them [23:58] understood [23:59] wallyworld_: URGH params.StateServingInfo [23:59] that fucking type [23:59] such a mess