=== thumper is now known as thumper-dogwalk
wallyworldanastasiamac: why did you tag 1612645 as a blocker?00:52
anastasiamacwallyworld: for 1.25 not master00:53
anastasiamacwallyworld: it's not a critical on master so should not block it00:54
wallyworldwe haven't really considered adding a new region being a blocker before00:54
wallyworldthere's plenty of other regions to choose from in the interim00:55
wallyworldi guess it's 1.25 so it doesn't matter so much00:55
anastasiamacwallyworld: i've added based on comments00:57
anastasiamacwallyworld: and precedence is not a great driver when we strive to improve :D00:58
axwmenn0: which HO?00:59
wallyworldi guess my definition of what a blocker is is different00:59
menn0axw: good question. let's use the standup one00:59
menn0wallyworld: standup hangout?01:00
anastasiamacwallyworld: plz get a chance to review https://github.com/juju/juju/pull/586901:16
thumper-dogwalkmenn0: all done chatting?01:16
=== thumper-dogwalk is now known as thumper
menn0thumper: yep01:17
thumperanything problematic?01:17
menn0thumper: nope all good actually - I'm removing work from the MM completion doc as the way the config stuff was done works well with MM01:18
wallyworldanastasiamac: one thing to add to tech debt sprint - finish complete removal of env storage, including http://reviews.vapour.ws/r/4303/01:40
anastasiamacwallyworld: pleaae bug it up and add `tech-debt` tag. the plan is to tackle that list01:47
axwwallyworld: FYI I've just verified that destroy-controller ends up cleaning juju from a workload machine01:47
axwwith my change01:47
wallyworldaxw: awesome thanks01:47
wallyworldanastasiamac: there may be a bug already, i'll try and find it01:47
anastasiamacwallyworld: \o/ even better01:48
axwwallyworld anastasiamac: if there's any chance that the MAAS provider could be updated to use tags instead of storage at the same time that would be great01:49
wallyworldaxw: i think maas 2.0 can do that but not 1.9 :-(01:49
axwprobably out of scope though01:49
anastasiamacaxw: ^^ would b awesome to bug it up and tag as 'tech-debt' :D altho, I m not sure what our stance is with maas 1.9.. probably criticals only too?..01:50
axwwallyworld: we can keep it for just 1.901:50
wallyworldthat is true. in that case, we should do it01:50
axwanastasiamac: yeah not sure if there's a bug, will have a look01:50
wallyworldnow that we have 2 provider implementations01:50
axwanastasiamac: covered by this I think: https://bugs.launchpad.net/juju-core/+bug/161126701:51
mupBug #1611267: maas:  kill-controller should take down known hosted models <juju-core:Triaged> <https://launchpad.net/bugs/1611267>01:51
anastasiamacaxw: k. i'll a tech-debt card01:51
axwanastasiamac: I've  just tagged it01:52
anastasiamacaxw: u r awesome01:52
anastasiamacaxw: of course, u'd b even more awesomer (!) if u joined the sprint that would fix it :D01:53
axwanastasiamac: is it going to be in Perth? ;p01:53
anastasiamacaxw: :) dunno yet. i need to talk to HR/sprint org admins... unlikely tho :)01:54
anastasiamacaxw: if it was, do i have ur word, u'd join?01:54
axwanastasiamac: tongue is in cheek, nobody wants to go to perth01:54
anastasiamacaxw: i can think of at least couple of ppl who'd b keen and several others are not attached to a location :) so everything is possible...01:55
axwoh true, forgot about andy :p01:56
anastasiamacaxw: :) that's one of these ppl \o/01:56
axwwallyworld: the more I dive into azure auth, I don't think we're going to be able to automatically generate creds. AFAICS, authentication with username/password must be interactive02:34
wallyworldawesome :-(02:34
axwwallyworld: that would be OK for bootstrap, but not for adding creds afterwards02:34
axwbecause you might not have custom azure code in the client02:34
axwwallyworld: I think the best we're going to be able to do for now is a separate plugin/tool02:35
wallyworldwe'll need to identify the gaps and talk to the ms guys maybe02:35
wallyworldstop gap would be good, that can probably happen next week02:35
blahdeblahquick Q: juju 1.x has "api-endpoint" and "api-info" - juju 2.x doesn't.  How should a client program work out the correct API endpoint of the controller under 2.x?02:51
blahdeblahanastasiamac: ^ That's the root cause of the issue we were talking about earlier02:52
anastasiamacblahdeblah: tvm! thumper:wallyworld:axw:menn0: ^^02:53
blahdeblahanastasiamac: My issue with juju-deployer on 2.x, that is.  I don't think this is going to help you prioritise or fix any bugs. :-)02:54
wallyworldjuju list-controllers02:55
axwblahdeblah: "juju show-controller" contains the list of addresses02:55
axwthe first one in the list is the most recently used02:55
anastasiamacblahdeblah: i figured ;)02:56
blahdeblahaxw: where "most recently used" == the one that juju switch will say is selected?02:56
wallyworldjuju switch pertains to controller/model. most recently used refers to the api endpoints for a given controller02:57
wallyworldcontroller cluster02:58
wallyworldin an HA setup, there's more than one endpoint to choose from, they are rotated02:58
blahdeblahWhat about if you've got multiple client processes locally hitting multiple juju envs on different controllers?  Is there a way to reliably determine from the CLI which API endpoint you should hit for env X?02:59
wallyworldjuju show-model will get the controller uuid03:01
wallyworldfrom there you can get the endpoints03:01
wallyworldnot sure if there's a better way, would need to look into it03:01
blahdeblahwallyworld: that will do for now03:04
blahdeblahshow-controller gives the uuid, so they can be compared03:04
wallyworldblahdeblah: it could be with the 2.x rework stuff has got harder in some areas, so would be good to know those pain points03:04
wallyworldraise a bug if you want it tracked etc03:04
blahdeblahwallyworld: yeah - that's the point of what I'm doing today; try to find where the gaps are in getting us ready for 2.x03:05
wallyworldhope you don't find too many :-)03:05
blahdeblahwallyworld: well, the big one is juju-deployer03:05
wallyworldi thought folks had been using that ok with the recent api changes to it03:06
blahdeblahwell, I could be doing it wrong03:06
wallyworldi'm only going on hearsay, i've not used it03:06
blahdeblahI've just pulled down the trunk version of that + python-jujuclient, and it's definitely not working out of the box03:06
wallyworldthere was an email to the list recently i think about it all being updated03:07
wallyworldnot sure now though03:07
* blahdeblah goes to find it03:07
blahdeblahAnyway, context for all of this is that in the conversations anastasiamac and I have been having, it has become evident that we'll need to have a 2.x story if we want our non-critical bugs fixed; so that's what I'm deep-diving into today.03:07
menn0thumper: https://github.com/juju/juju/pull/5992 pls03:47
mupBug #1613150 opened: migrationmaster continually restarts once a model has migrated back to a controller <juju-core:New for menno.smits> <https://launchpad.net/bugs/1613150>04:09
mupBug #1613150 changed: migrationmaster continually restarts once a model has migrated back to a controller <juju-core:New for menno.smits> <https://launchpad.net/bugs/1613150>04:18
mupBug #1613150 opened: migrationmaster continually restarts once a model has migrated back to a controller <juju-core:New for menno.smits> <https://launchpad.net/bugs/1613150>04:24
menn0thumper: I was surprised to find the user access constants in core/description too04:36
blahdeblahDo you folks have any API-level doc for juju2?  I'm waist-deep in the python-jujuclient code, and all I can work out is that its getting refusals from the API server based on the parameters its constructing.  But I have no idea what those should be.04:36
menn0thumper: i think someone is misunderstanding what core/description is for04:36
thumperblahdeblah: which call?04:37
blahdeblahthumper: login04:37
thumperwhich params do you have questions about?04:38
thumperI'm not clear on the macaroon stuff04:38
blahdeblahthumper: All of them?  I suppose I should just read the source and find out where the error is originating and infer back from there, but go is almost as unreadable as lisp to me...04:40
blahdeblahthumper: Here's what's happening: https://pastebin.canonical.com/163111/04:40
blahdeblahLine 8 is the params its sending on the RPC call04:40
blahdeblahI hacked it to remove the password, because the Canonistack controller doesn't seem to want one.04:40
blahdeblahBut it gives the same response with or without a password04:41
thumperblahdeblah: you need to prefix "user-" to the auth-tag04:41
thumperit is expecting a stringified tag,04:41
blahdeblahhang on - I just realised that04:41
thumperand user tags all need "user-"04:41
blahdeblahand I had fixed it, but not pushed the code to my live copy04:42
* blahdeblah tries again04:42
blahdeblahOK - here's the result this time: https://pastebin.canonical.com/163112/04:42
blahdeblahthumper: and if I include 'credentials' (which is None), I get the same result04:44
blahdeblahI really wonder whether this is due to the Canonistack controller running a hacked authentication package. <-- wallyworld, any thoughts?04:47
thumperblahdeblah: it'll be expecting some creds now04:48
thumpernot sure about the macaroon bits...04:48
blahdeblahWhat are "the macaroon bits"?04:48
thumperopaque credentials with magic HMAC chaining04:49
blahdeblahthere's no password in ~/.local/share/juju/accounts.yaml04:49
wallyworldblahdeblah: i think you need to juju login04:49
blahdeblahSo all of this was just another missing step? :-(04:50
blahdeblahwallyworld: where username == launchpad username?  Or launchpad email address?04:50
blahdeblahERROR already logged in04:51
wallyworldhmmm, not sure in that case04:51
wallyworlduros will be online in  a couple of hours04:51
blahdeblahhmmm - pretty sure there was a login step actually, and it worked, because when I log out it removes the accounts.yaml entry for that controller04:53
wallyworldthat seems correct04:54
blahdeblahOK, I might wait for Uros04:56
wallyworldyeah, sorry, he'll know straight away what's wrong04:58
veebersHmm, If I've added a user with permission 'write' and attempt to add a model and get this error, what might I be doing wrong? cmd supercommand.go:458 permission denied (unauthorized access)04:59
veebersI"m confident that it's passing --config authorized-keys="..."05:00
veebersmenn0: Would you have any insight? ^^ user with write perms should be able to add a model right?05:22
menn0veebers: writer perms on the controller model?05:23
veebersmenn0: hmm, I believe so, let me check the test code.05:25
wallyworldveebers: write only pertains to models05:25
wallyworldyou need addmodel permission on controller to add models05:26
wallyworldcontroller permissions are login, addmodel, superuser05:26
wallyworldif that's ot in the release notes we'll need to fix them05:26
veeberswallyworld: ah ok, seems I'm going off old information (I'm resurrecting some old code of mine)05:27
wallyworldaxw: if you had time, here's a small tweak to fix upgrade-juju http://reviews.vapour.ws/r/5433/ (main change was a few lines, but then tests...)05:28
veeberswallyworld: so this makes sense? juju add-user myusername --models testmode,controller --acl 'addmodel' -c controllername05:28
veeberserr, testmode -> testmodel. Or does addmodel only pertain to controller and that will error?05:28
wallyworldveebers: i thin --models have been removed05:32
wallyworldand -acl is replaced by a positional arg05:32
wallyworldcheck the release notes05:32
veeberswallyworld: ack, will do. juju add-user --help lists --models as an arg btw05:34
wallyworldlooks like another case of cli help out of date05:34
wallyworldassuming my memory has not failed me05:35
veeberswallyworld: which is the most up to date release notes? https://jujucharms.com/docs/devel/temp-release-notes or https://docs.google.com/document/d/1ID-r22-UIjl00UY_URXQo_vJNdRPqmSNv7vP8HI_E5U/edit# (or some third option)?05:35
veeberssweet cheers05:36
wallyworldi don't know about the jujucharms ones, not looked at them05:36
axwwallyworld: reviewed05:37
veebersugh, I'm really confused now. Trying to add-user with addmodel gives me: error: invalid model access permission "addmodel"05:40
* veebers re-reads release notes incase he missed something05:40
veeberswallyworld: I can't see mention of any of "login, addmodel, superuser" in the release notes, perhaps like you say they need updating. Or perhaps that part hasn't landed yet (as per error I'm seeing)05:42
wallyworldveebers: it's in master, are you running from source?05:42
=== menn0 is now known as menn0-afk
veeberswallyworld: I am, I pulled and rebuilt this morning. I'll re-pull now to confirm05:43
wallyworldveebers: i tested last week and it seemd to work ok05:44
wallyworldi used grant05:44
wallyworldafter adding a user05:45
veeberswallyworld: hmm right, might be what I'm doing wrong (I'm trying to use add-user). Thanks I'll try that05:45
wallyworldveebers: i juest checked the code - adduser still has the --models but it shouldn't i don't think, i 'll need to check the spec. and you can only specify model permissions, not controller (by design). i think the model permssions are going away too, but i'll need to check to be sure05:49
wallyworldso add-user becomes just that - adds a user05:49
wallyworldthen use grant afterwards05:49
veeberswallyworld: makes sense. When you do check can you confirm re: add-user changes (and using grant) as that'll need a change in the CI tests etc.05:51
wallyworldusing grant will not change - that's the supported use case, give me a few minutes to check for the other bits05:52
wallyworldveebers: https://docs.google.com/document/d/15zwHWctS7WhNIzLNIeG_ly6udIVt_8wRlZTLFqJF-zY05:58
wallyworldaxw: those 2 things fixed, ok to ship?05:59
axwwallyworld: ship it06:00
wallyworldveebers: i added a card to our board to fix the add-user help text etc06:01
veeberswallyworld: sweet thanks06:05
wallyworldveebers: everything is now positional arguments, and if you leave off model, it assumes a controller permission06:05
veeberswallyworld: ack. I'll touchbase with my team to make sure this is known. It's possible it's just me thats out of date06:07
rogpeppe1menn0-afk: thanks for the review comments06:12
=== rogpeppe1 is now known as rogpeppe
rogpeppewallyworld: hiya06:14
rogpeppewallyworld: apparently i need approval from the tech board for this PR to land. any idea what the procedure is for that? http://reviews.vapour.ws/r/542806:14
wallyworldrogpeppe: the PR seems like a reasonable goal to me. pop an email to john, menno, andrew, william just to highlight you want them to check it06:17
rogpeppewallyworld: thanks.06:17
wallyworldrogpeppe: maybe cc juju-dev so everyone can see the discussion?06:21
rogpeppewallyworld: ok06:21
rogpeppewallyworld: but that means i have to write a much longer email... :)06:21
rogpeppewallyworld: and not quite so frivolous :)06:21
wallyworldwhatever works for you then :-) don't want to block06:21
wallyworldaxw: we'll never want to make an api call to get more than just the defaut cloud tag? seems reasonable to return relevant details. maybe a CLI command may want that info to display?06:22
axwwallyworld: maybe, but we don't have a need for it now, so I don't want to guess how it should look any more. as it is, we can get hte default cloud name, and then the details of that cloud06:24
axwwallyworld: really I think we should be shuffling the cloud's Regions so that the default one (i.e. the controller's region) is at the front06:24
axwthen we'd have all we need06:24
rogpeppewallyworld: tbh it's kinda trivial; i think it's probably not worth the noise on juju-dev06:25
axwrogpeppe: probably should've emailed just you and frankban about that one06:25
wallyworldaxw: agree with regions i think. i worry though about designing a chatty diestibuted api. gotta run to school picku, bbiab06:25
rogpeppeaxw: have you actually done DestroyModels or are you waiting for me to do it?06:26
axwrogpeppe: I've done it, merging now06:26
rogpeppeaxw: awesome, thanks!06:26
axwrogpeppe: https://github.com/juju/juju/pull/599106:26
axwrogpeppe: I'm looking at creds now06:27
rogpeppeaxw: if you have a moment or three, it would be great if you could take a look at http://reviews.vapour.ws/r/542806:27
axwrogpeppe: ok06:27
axwrogpeppe: LGTM, please answer katco's issues - QA in particular is the big one06:41
rogpeppeaxw: yeah06:41
rogpeppeaxw: and i know for a fact that QA in one instance will fail - just working on fixing that nowq06:42
rogpeppeaxw: 'cos the command-line tests don't actually test against the real API06:42
rogpeppeaxw: for DestroyModel06:42
rogpeppeaxw: the fix will require your DestroyModels that's being tested06:43
axwrogpeppe: indeed, we should probably have a featuretest06:43
rogpeppeaxw: that's better than just making the command-line test use the actual API?06:44
axwrogpeppe: that's the way we're going, yeah. at least one "happy path" test in featuretest, the rest are unit tests06:44
rogpeppeaxw: ok06:46
rogpeppeaxw: i worry that that's not really enough (is one "happy path" good enough to actually determine that all the variations work?), but fair enough06:47
axwrogpeppe: I think your message before indicates you are, but can you confirm that you're happy with this? http://reviews.vapour.ws/r/5435/diff/#06:52
rogpeppeaxw: looking06:52
rogpeppeaxw: looks fine to me06:54
axwrogpeppe: ta06:55
rogpeppeaxw: glad to see the back of a bulk API call :)06:55
axwshh :)06:55
* rogpeppe still thinks that the bulk API call thing has been the biggest waste of time ever06:55
axwrogpeppe: for the creds issue, I'm thinking of doing this: introducing a credential tag which includes owner, cloud, and cred name. then I'll have the Cloud.Credentials method return the set of credential tags that a user has access to (possibly still with a cloud filter)06:56
axwrogpeppe: then I'll update CreateModel to take a credential tag06:57
rogpeppeaxw: that seems reasonable to me i think07:00
rogpeppeaxw: CreateModel already takes a CloudCredential field07:00
rogpeppeaxw: i'm wondering what the best ordering of owner, cloud and cred name is07:00
rogpeppeaxw: maybe it should be cloud/owner/cred-name07:01
axwrogpeppe: I'm not sure. I had started on owner/cloud/cred, but *shrug*07:02
axwrogpeppe: bbs, school pickup07:02
rogpeppeaxw: k07:02
rogpeppeaxw: are there any existing precedents07:02
blahdeblahSo is there any way to get juju 2.x to tell me where the problem is in my bundle that it's saying is invalid?07:16
=== frankban|afk is now known as frankban
frankbanmorning al07:26
rogpeppeaxw: good point about ControllerTag - i didn't know about that07:31
rogpeppeaxw: i think probably the API should be changed to use that07:32
=== frankban is now known as frankban|afk
=== menn0-afk is now known as menn0
menn0rogpeppe: no worries07:40
menn0wallyworld: I'm seeing uploadSuite.TestMockBuildTools failing consistently on my machine but only under Go 1.7 (and in CI it seems)07:43
menn0wallyworld: looks to be related to your recent changes07:43
wallyworldmenn0: axw mentioned there could be a checksum different between 1.6 and 1.707:44
wallyworldso maybe if the algorithm is different our test values need to be changed07:44
menn0wallyworld: the checksum and length are different07:45
wallyworldawesome, so we have a 1.6 vs 1.7 issue07:45
menn0the tools size is 132 under Go 1.7 and 127 under 1.607:45
rogpeppemenn0: do you know if the controller tag UUID is different from the controller model tag's UUID, by any chance?07:46
menn0wallyworld: which probably makes sense if it's being built with a different Go version right?07:46
wallyworldyeah. can you send me the test failure?07:46
wallyworldrogpeppe: they are supposed to be but not yet IIRC07:46
menn0rogpeppe: AFAIK, they're always the same, but I think there was talk of making them different. wallyworld or axw might remember?07:47
wallyworldwe need to fix that07:47
wallyworldon the todo list07:47
wallyworldtest changes will suck07:47
rogpeppemenn0: ah, ok, that's good to know07:47
rogpeppemenn0: i guess the login API should return a controller tag not a model tag for the controller then07:47
menn0rogpeppe: yes, that seems right07:48
menn0wallyworld: here's the test failure: http://paste.ubuntu.com/23057634/07:48
menn0wallyworld: it also appears to have failed for me during a CI merge run ... although there's no details of the actual failure.07:49
menn0this is why I started looking07:49
menn0that code has zero to do with the change I'm making07:50
wallyworldmenn0: i see that one intermittently even on 1.6, NFI07:50
wallyworldyou can go to environs/sync and no errors07:50
menn0wallyworld: perhaps the compiler output isn't completely predictable?07:50
wallyworldrun tests from a level higher using ./... and intermittent07:50
menn0wallyworld: well it's failing for me consistently under 1.7. not sure if it's the same issue or not though.07:51
wallyworldmenn0: i think that mock build tools test is bogus anywat07:51
wallyworldi have no idea why we are relying on specific compiler behaviour07:51
wallyworldsurely the test can be rewritten07:52
menn0wallyworld: yeah, seems dumb07:52
wallyworldi'll try and look tomorrow07:52
menn0wallyworld: so that test doesn't actually compile anything. it just creates a tarball with a fake jujud in it.07:55
menn0wallyworld: so it's not compiler specific07:55
menn0wallyworld: I bet the tar or gzip implementation has changed in 1.7 though07:56
wallyworldyeah, something has changed07:56
menn0wallyworld: the test shouldn't be comparing the size or hash ... that's bogus07:56
axwrogpeppe: no precedent that's relevant AFAIK07:57
menn0wallyworld: what it /could/ do is extract the contents and make sure it's as expected. that is 100% predictable.07:57
wallyworldthat's what it should have done07:57
rogpeppeaxw: i guess my inclination is to keep user and name together as they're together in other places too07:57
menn0wallyworld: I'll tackle it now if you like.07:57
menn0wallyworld:  easy fix07:58
rogpeppeaxw: so user/name/cloud or cloud/user/name but not user/cloud/name07:58
axwrogpeppe: sure, sounds fine to me07:58
wallyworldmenn0: if you want, ok, thanks, i'm just tidying up some other stuff, sorry07:58
menn0wallyworld: np, I've got it07:58
axwrogpeppe: I think I'll go for cloud/user/name. on the CLI, I'll make it so you can refer to credentials when adding a model with either "name" (short hand for current-user/name), or user/name07:59
axwrogpeppe: (and the code will tack on the cloud name when crafting the tag)07:59
rogpeppeaxw: is that in the API or just on the command line?08:00
mupBug #1613200 opened: TestMockBuildTools fails intermittently under Go 1.6 and always under 1.7 <juju-core:New for menno.smits> <https://launchpad.net/bugs/1613200>08:07
rogpeppewallyworld, axw: speaking of tests failing under different Go versions, apiserver/provisioner tests fail under Go tip08:08
wallyworldyeah, that's been the case for a while :-(08:09
rogpeppewallyworld: *sigh*08:09
* rogpeppe goes off to fix it08:09
wallyworldhasn't been a huge issue since most are still running go 1.608:10
rogpeppewallyworld: i think it's very useful to be able to run tip08:10
wallyworldthose running tip can scratch their own itch :-)08:10
rogpeppewallyworld: because it means we can proactively get fixes into Go before they actually break us later08:10
rogpeppewallyworld: and usually the test failures under tip point to smelly tests rather than bad Go changes08:11
=== frankban|afk is now known as frankban
rogpeppewallyworld: ha, in apiserver/provisioner, one of the failing tests (TestProvisioningInfoWithStorage) passes in Go 1.6 when run as part of the whole test suite but not when run on its own08:24
rogpeppewallyworld: the failure is because of image metadata - do you think there might be a mock failure or something?08:25
rogpeppewallyworld: it looks like it's actually managing to get real image metadata from the network08:26
anastasiamacrogpeppe: it should b mocked out \o/ please open a bug and tag it with `tech-debt`08:29
menn0wallyworld, anastasiamac: fix for TestMockBuildTools - http://reviews.vapour.ws/r/5436/08:30
menn0wallyworld, anastasiamac: no rush. I'm about to EOD. feel free to merge if you're happy with it.08:31
axwwallyworld rogpeppe: https://github.com/juju/names/pull/70 -- PTAL if you have a moment08:43
rogpeppeaxw: reviewed08:52
axwrogpeppe: ta08:52
babbageclunkdimitern: no fwereade still :(08:55
rogpeppewallyworld: yeah, that test is definitely dialing out to
rogpeppeanyone know what mechanism is supposed to stop test code from dialing out to the ubuntu cloud images server?08:57
wallyworldrogpeppe: sorry, was having dinner. john found te same root cause a while back. i think tge issue is that patching the DefaultImaheURL to "" fails08:59
rogpeppewallyworld: ok, thanks08:59
rogpeppewallyworld: DefaultImageURL doesn't seem to exist as a word in the juju sources09:00
wallyworldit's something like that, i can look09:01
rogpeppewallyworld: do you mean DefaultUbuntuBaseURL ?09:01
wallyworldprobs, let me check09:01
wallyworldthat and also DefaultJujuBaseURL09:01
wallyworldrogpeppe: there's also this function PatchOfficialDataSources()09:02
rogpeppewallyworld: well, it looks like withoutControllerSuite doesn't call that function09:06
rogpeppewallyworld: i don't understand why it only fails sometimes though09:06
wallyworldme either, but i haven't looked too deeply into it09:07
frobwarebabbageclunk: ping - can we sync/HO?09:14
babbageclunkfrobware: sure09:14
babbageclunkfrobware: where?09:14
babbageclunkfrobware: just in core?09:15
rogpeppewallyworld: ok, i've uncovered the root of the issue - our "prevent outgoing calls" hack doesn't work under Go 1.7: https://play.golang.org/p/P4L5nQTsBp09:51
rogpeppewallyworld: that program prints "ok" which it shouldn't09:51
wallyworldactually, that rings a bell from when john looked at it09:57
wallyworldobviously no one has looked into how to fix09:57
rogpeppewallyworld: it's quite easy to fix09:59
rogpeppewallyworld: in Go 1.7 and later, DialContext overrides Dial09:59
rogpeppewallyworld: and DialContext is set in DefaultTransport by defailt10:00
rogpeppewallyworld: so we're setting Dial but DialContext is overriding it10:00
wallyworldah, i see, makes sense10:00
rogpeppewallyworld: so we need to set DialContext too10:00
wallyworldmaybe i should try Go 1.7, i guess there's a PPA somewhere10:01
frobwaredimitern: ping - can we sync?10:08
dimiternfrobware: hey, welcome back :) sure - standup HO?10:09
mwhudsonwallyworld: ppa:gophers/archive!10:53
rogpeppewallyworld, axw, dimitern: here's a fix for the broken-in-go1.7 outgoing-connection-checking logic: https://github.com/juju/utils/pull/23310:53
rogpeppei'm thinking that it might be better to make that logic panic when trying to access an external address10:57
rogpepperather than just returning an error10:58
rogpeppemwhudson: fancy a little review that might even be a tad enlightening (i found out something i didn't know) https://github.com/juju/utils/pull/23311:21
rogpeppemgz, katco, mattyw, dooferlad: ^11:22
mgzrogpeppe: sure11:24
rogpeppemgz: ta!11:24
mattywrogpeppe, can't see anything wrong with that, lgtm11:26
rogpeppemattyw: tyvm11:27
mgzrogpeppe: change makes sense to me, my only real complaint is neither the implementions of the install... functions nor their call sites actually say they're about preventing test outbound network access11:31
rogpeppemgz: yeah, i've just added a comment11:32
mgzwith that then go ahead11:32
dimiternrogpeppe: ship it!11:33
rogpeppemgz, dimitern, wallyworld: thanks11:33
wallyworldrogpeppe: np. ensure you add testing notes as per new review policy11:34
wallyworldthanks fir biting the bullet and fixing11:35
rogpeppewallyworld: for such a simple low level fix, i think that requiring a QA from the top level of Juju is overkill tbh11:36
wallyworldrogpeppe: really? changing how http works could break everything. and it is the new policy for core11:36
rogpeppewallyworld: i'll certainly QA when this version of utils gets incorporated as a dependency into juju-core11:36
wallyworldoh right11:37
wallyworldit's utils, sorry11:37
wallyworldyeah, when core deps get updated, test then11:37
wallyworldjumped the gun sorry11:37
rogpeppewallyworld: np11:37
mupBug #1613262 opened: simplestreams: error message can hide real error <juju-core:New> <https://launchpad.net/bugs/1613262>12:05
mupBug #1613262 changed: simplestreams: error message can hide real error <juju-core:New> <https://launchpad.net/bugs/1613262>12:11
mupBug #1613262 opened: simplestreams: error message can hide real error <juju-core:New> <https://launchpad.net/bugs/1613262>12:26
dimiternrick_h_: ping12:26
rick_h_dimitern: pong12:28
dimiternrick_h_: hey, frobware and me would like to have a chat re future work/planning/breakdown - when will be a good time for you? it shouldn't take more than ~20m12:29
rick_h_dimitern: any time, welcome back frobware12:29
rick_h_dimitern: frobware let me know when works for you and will join in the standup room?12:30
dimiternrick_h_, frobware: top of the hour? i.e. in ~30m?12:30
rick_h_dimitern: wfm12:31
dimiternrick_h_: ok, cheers12:31
dimiternfrobware: ^^12:33
frobwaredimitern, rick_h_: yep good for me12:49
rogpeppei've updated http://reviews.vapour.ws/r/5428/ with QA instructions (wasn't quite sure where I should have put them). if someone could QA it for me, that would be great.12:54
frobwarerick_h_: if you time to jump back in before standup that would be useful13:27
rick_h_frobware: sure thing, I'm back sorry13:34
frobwarerick_h_, dimitern: the google part of the net has disappeared for me13:52
rick_h_frobware: google knows all, you were thinking bad thoughts :P13:52
rick_h_voidspace: ping for standup14:01
voidspacerick_h_: omw14:02
natefinchsinzui, mgz: seems like we still have a windows error in CI.  I think I need to do a run with --keep-env, but the scripts don't support that yet.  I tried looking for where I should put support for that, but it's a little twisty. Give me a hint where to look?14:13
mgznatefinch: the failing test seems like just a standard deploy job? so we can add --keep-env there fine?14:17
sinzuinatefinch: all CI scripts support --keep-env14:17
mgznatefinch: do you want us to kick off a run with that?14:18
natefinchmgz, sinzui: oh, sorry, looking at notes it was constraints, but I don't need that anymore14:20
sinzuinatefinch: yeah, contraints remain a gap. Which do you need?14:21
natefinchsinzui: it's ok. it was only as a workaround for the azure thing that andrew already fixed.  So I should be all set now14:25
rick_h_frobware: one other thing for your "welcome back" is that dimitern looked at this lxd spec but would be good if you also gave it a look over this week: https://canonical.leankit.com/Boards/View/122969419/12376188914:25
mupBug #1613300 opened: github.com/juju/juju/environs/sync fails with little info <centos> <ci> <jujuqa> <test-failure> <juju-core:Triaged by alexis-bruemmer> <https://launchpad.net/bugs/1613300>14:41
rick_h_dimitern: what came of: https://bugs.launchpad.net/juju-core/+bug/1612624 last week?14:43
mupBug #1612624: Bootstrap fail on MAAS if ipv6 is disabled <juju-core:Triaged> <https://launchpad.net/bugs/1612624>14:43
dimiternrick_h_: I haven't had time to look into that14:44
rick_h_dimitern: ok14:44
dimiternrick_h_: I could try later today14:45
rick_h_dimitern: all good, I was just reviewing bugs and recalled we had the conversation of simplifying that work14:45
rick_h_dimitern: so wanted to see what came after that discussoin we had.14:45
rick_h_dimitern: hold off and wrap up WIP before we move forward there.14:45
dimiternrick_h_: frobware and I discussed a few ways to detect ipv6 being enabled or not and if not - don't pass --ipv6 to mongod14:46
dimiternrick_h_: but couldn't get as far as a wip fix for it14:47
rick_h_dimitern: all good, ty for the update14:47
mupBug #1613311 opened: show-log and list-models disagree after migration <ci> <intermittent-failure> <jujuqa> <model-migration> <juju-core:Triaged> <https://launchpad.net/bugs/1613311>14:50
frobwarerick_h_: apologies for dropping out - have plumber here and needed somebody to stop the flow...14:51
rick_h_frobware: :) all good14:52
frobwarerick_h_: there's less water now :)14:55
rick_h_frobware: as long as it's less out of the pipes, and more in the pipes, seems like a net win to me14:55
frobwarerick_h_: houses - just like s/w14:55
frobwarerick_h_: I took a look over LXD spec - looks good15:00
natefinchif someone could review https://github.com/juju/jsonschema/pull/1  it would be a big help15:04
frobwarevoidspace: ping - scripts, what were the issues?15:05
voidspacefrobware: the scripts have various dependencies - python libraries and system libraries15:06
voidspacefrobware: and they're not detailed anywhere15:06
voidspacefrobware: so I had to work them out15:06
frobwarevoidspace: that's outrageous! :-D15:06
voidspacefrobware: e.g. xmlstarlet15:06
voidspacefrobware: so run, fail, work out why it failed and what package provides the missing bit, repeat...15:07
voidspacefrobware: :-)15:07
frobwarevoidspace: let me add a prerequisites.sh15:07
voidspacefrobware: cool15:07
voidspacefrobware: I have an empty vm I was just about to re-start the procedure so I could work out what they were again15:07
frobwarevoidspace: did this not work?15:08
frobwaretype -p xmlstarlet > /dev/null || die "please install xmlstarlet(1)"15:08
voidspacefrobware: I didn't see that message15:08
voidspacefrobware: I saw one telling me to *reinstall* xmlstarlet15:08
voidspacefrobware: that was just the first missing dependecy - there were about four I think15:08
frobwarevoidspace: must come from something else15:08
voidspacefrobware: it was just add-node I was using15:09
frobwarevoidspace: that 'type -p' is in add-node15:09
voidspacefrobware: *really* useful15:09
voidspacefrobware: so thanks15:09
voidspacefrobware: it failed at commissioning, but probably because I don't have power types set up15:09
frobwarevoidspace: we need (MUST!) do more of this15:09
voidspacefrobware: but I can do that manually15:09
voidspacefrobware: rick_h_ said move it to github.com/juju so we can maintain it collectively15:10
voidspacefrobware: and I agree15:10
frobwarevoidspace: sometimes I do see commisionning fail, but I believe that's a maas issue15:10
voidspacefrobware: I don't care about that anyway - creating the vm and enlisting is cool enough15:11
mupBug #1613311 opened: show-status and list-models disagree after migration <ci> <intermittent-failure> <jujuqa> <model-migration> <juju-core:Triaged> <https://launchpad.net/bugs/1613311>15:11
frobwarevoidspace: I need to fix a small issue with remove-node, so will look at this now15:12
voidspacefrobware: ok15:12
voidspacefrobware: past 6pm here, dinner time15:12
voidspacefrobware: I'll be on later though15:13
voidspacefrobware: want to finish this actions stuff15:13
frobwarevoidspace: sure15:13
natefinchmgz, sinzui: you guys seen this before, when destroying an azure env? ERROR listing resource groups: azure.ServicePrincipalToken#Refresh: Failure sending request for Service Principal 10f015aa-8519-47cf-bd76-d60b1b06ba7d: StatusCode=0 -- Original Error: http: nil Request.URL15:14
* rick_h_ gets his taco on15:34
dimiternfrobware: I'll need to skip the networking call btw15:48
rogpeppekatco: any chance of a "shipit" on http://reviews.vapour.ws/r/5428/ ?16:04
rogpeppekatco: i added some QA steps, got it approved by tech board, QA'd it myself16:04
rogpeppekatco: and it's in the way of a couple of other branches now16:05
rogpeppe... or anyone else... voidspace? mgz? dooferlad?16:08
mgzrogpeppe: are the open issues actually all addressed/responded to now?16:11
=== frankban is now known as frankban|afk
rogpeppemgz: all except the ones i replied to to say i didn't think they were worth doing, yes16:12
mgzrogpeppe: code changes look good to me, I don't fully get the implications of changing the return signature of ModelTag or adding a new /api alias16:19
mgzpresumably both are safe/sane enough?16:19
rogpeppemgz: thanks16:19
mgzyou have a +1 from me, and a minor comment shipit from the other martin16:21
rogpeppemgz: ta!16:25
natefinchmgz, sinzui: --keep-env doesn't seem to actually keep the environment?  does that just keep the machines around?17:00
sinzuinatefinch: --keep-env means don't call kill-controller at the end of the script. It exits after collecting logs17:01
natefinchsinzui: hmm weird17:01
natefinchsinzui: so the controller should show up if I do JUJU_DATE=./cloud-city juju controllers, right?17:02
natefinch(obv JUJU_DATA)17:02
mgznatefinch: that's the bit I'm not sure is correct right now17:03
mgzI think our poking of code around config/creds might have confused somethig17:03
rogpeppemgz: when a 'bot merge fails now, where do we go to find the test failure output?17:03
natefinchmgz: ok... I don't seem to have a controllers.yaml in cloud-city17:04
rogpeppemgz: (it used to be in the console output but no longer)17:04
mgzbecause last time I wanted to do something similar I couldn't see the controller of a running env outside the script17:04
mgzrogpeppe: you need to look at the artifacts, linked from one level up on the job page17:04
natefinchmgz: to be fair, it did throw an exception during log collections because ./logs didn't exist :/17:04
rogpeppemgz: ah!17:04
mgznatefinch: it probably should not have actually called kill-controller though? that should be apparent from the log17:05
sinzuinatefinch: not exactly the controller is JUJU_DATA=cloud-city/jes-homes/<name-from-commandline>17:05
mgzrogpeppe: generally trusty-out.log for test failures17:05
natefinchsinzui: ahh, that's the problem17:06
mgzrogpeppe: well, that's interesting: juju/juju/environs/sync a test failed, but there's no output for the failure17:06
rogpeppemgz: yes17:06
rogpeppemgz: i was just writing that to you17:06
natefinchmgz, sinzui: works like a charm, thanks17:07
rick_h_mgz: rogpeppe there's a bug on that filed this morning17:07
mgzcan you repo locally by just running that package?17:07
mgzrick_h_: ah17:07
rick_h_mgz: rogpeppe https://bugs.launchpad.net/juju-core/+bug/161330017:07
mupBug #1613300: github.com/juju/juju/environs/sync fails with little info <centos> <ci> <jujuqa> <test-failure> <juju-core:Triaged by wallyworld> <https://launchpad.net/bugs/1613300>17:07
mgzrogpeppe: so, it's master, not new with you17:08
rogpeppemgz: i guess i should just $$merge$$ again17:08
mgzrogpeppe: well, ideally not, but I guess seeing how intermittent it is caan also be interesting17:10
rogpeppemgz: i can't get it to fail on my machine17:10
natefinchug... user admin@local is really confusing when it refers to a remote controller17:11
rogpeppenatefinch: ha ha17:14
rogpeppenatefinch: maybe the command line clients should rewrite admin@local to admin@$controllername17:15
natefinchrogpeppe: that would be helpful I think....  I saw this:17:19
natefinch juju switch controller17:19
natefinchnate-win-debug:admin@local/nate-win-debug -> nate-win-debug:admin@local/controller17:19
natefinchand I thought for a minute I'd switched to a different controller, since it say "admin@local/controller at the end17:20
rogpeppenatefinch: well that's the problem with the name of the controller model17:20
rogpeppenatefinch: i still think that name is an endless source of problems17:21
natefinchrogpeppe: I agree... using the same name for two different things is asking for trouble17:21
natefinchbut the same goes for "local" :)17:21
rogpeppenatefinch: if it had been nate-win-debug:admin@local/admin, it wouldn't have been as much of a problem, i think17:21
rogpeppenatefinch: well, @local names are always relative to the controller they're in17:22
natefinchrogpeppe: local means on my machine17:22
natefinchrogpeppe: we may not be using it that way, but that's what it means to most people17:22
rogpeppenatefinch: and mhilton points out to me that we really can't add arbitrary @domains17:22
rogpeppenatefinch: maybe it should be admin@3ecffebe-c248-41a2-893c-f67df84ec6bf :)17:23
natefinchrogpeppe: I don't even really understand why there's an @ at all.  Aren't all users defined in the namespace of the controller they're on?17:23
rogpeppenatefinch: no17:23
rogpeppenatefinch: external users are not17:23
rogpeppenatefinch: the @ is optional17:26
rogpeppenatefinch: tbh, as per a discussion in juju-dev some time ago, i'd really like to get rid of the whole special-case name "local" completely17:26
natefinchme too... I'd love to drop the @ entirely for users local to the controller17:27
rogpeppenatefinch: then any name without an @ is treated relative to whereever it's used17:27
rick_h_rogpeppe: yes, we've agreed that it needs to go away, just don't think it's been removed through and through yet17:27
rogpeppenatefinch: and that simplifies the names logic too17:27
rogpepperick_h_: really? cool!17:27
rogpepperick_h_: that would simplify a bunch of stuff on lots of places17:28
natefinchrick_h_: that helps a lot... optional input is nice, but if we always still output it, it's even more confusing, because now I've input "nate" and juju replies with "nate@local"17:28
rick_h_rogpeppe: yea, and it's not a case of 'by default users shouldn't need to mentally grok wtf is local' since to them there's no other option.17:28
rick_h_natefinch: right17:28
natefinchwell, awesome :)17:28
natefinch sinzui, mgz: I keep getting "no matching tools" when trying to run the deploy CI test on windows... what am I doing wrong?17:48
natefinchI'm running this command: JUJU_HOME=./cloud-city JUJU_REPOSITORY=./repository ./juju-ci-tools/deploy_job.py parallel-azure-arm $GOPATH/bin/juju ./logs nate-win-debug --series win2012r2 --keep-env17:49
sinzuinatefinch: you might is in a bad position17:49
natefincham I getting screwed by the new auto-upload-tools?17:50
sinzuinatefinch: you need streams to provide two different agents17:50
mgznatefinch: do you need to be testing your exact binary?17:50
sinzuinatefinch: I don't think so in this case since --upload-tools will fail the same ways17:50
mgzif not, you can probably use our streams17:50
mgzotherwise, you're going to need to build some windows tools and make streams17:51
rick_h_natefinch: --upload-tools is still there today?17:51
sinzuinatefinch: If you can use the streams that CI is building then use the current revision add this to your command: --agent-stream=revision-build-425417:52
natefinchsinzui: is there a way to say "just use the latest stream"?17:53
natefinchsinzui: I think I can use streams, if they've been built since andrew made his azure storage fix17:53
sinzuinatefinch: no, there is no way to detect the latest testing stream17:53
sinzuinatefinch: but you can see a stream for *weeks*17:53
sinzuinatefinch: I tend to use the version that matches master at the start of my day: http://reports.vapour.ws/releases17:54
natefinchsinzui: but what I want 99% of the time is whatever is the latest stream, since that may have fixes I need17:54
sinzuinatefinch: right. You can check that page to see which version is available17:54
rogpeppehere's a small PR that removes some API server cruft from the code: https://github.com/juju/juju/pull/599717:54
natefinchmgz: how hard is it to make my own stream?  making my own jujud.exe is trivial, at least17:58
mgznatefinch: it's pretty trivial, I believe abentley has some tools to make it less painful as well.17:58
natefinchmgz: those two statements seem to be at odds with each other ;)17:59
mgzit's easy to do and easy to do wrong :)18:00
natefinchcan I just do sync tools or something?  I just want to tell juju "here's a couple different tools, please use them" :/18:01
natefinchexcept not sync tools, obv18:01
mgzyerp nope.18:02
natefinchwe really seem to work hard to make things difficult for ourselves18:02
mgzjuju metadata generate-tools does get you most of the way though.18:02
sinzuinatefinch: I sent uyou a scipt that makes streams from you local jujuds18:03
sinzuinatefinch: pull lp:juju-ci-tools and look at streams-from-local.bash which you can pass jujus and jujud.exe to18:07
natefinchsinzui, mgz: should I be using series win2012hvr2?  I've been using win2012r218:09
sinzuinatefinch: does azure have that? We use it in our maas, but but in azure because I could find tht image18:12
natefinchsinzui: I have no idea what azure has18:12
natefinchsinzui: I honestly can't even tell what azure has when looking at their list of VMs to create18:14
sinzuinatefinch: no, you can't :(. We installed the azure python sdk to write a script to query that was available18:15
natefinchnot sure what "Windows Server 2012 R2 Datacenter" maps to in our list of series names18:15
natefinchwell I'm glad it's not just me18:15
sinzuinatefinch: win2012r2 did with with juju om july 618:16
sinzuinatefinch: I am still looking for some sample streams that show the available series18:16
natefinchsinzui: that streams-from-local.bash gives me line 68: unexpected EOF while looking for matching `"'18:25
* sinzui looks18:25
natefinchsinzui: WIN_JUJUD=${3"-}18:26
sinzuithank you name18:27
natefinchI presume that's wrong, although it may be a bashism I don't understand18:27
sinzuithank you natefinch18:27
* natefinch gets the feeling he's being addressed by a bot18:27
sinzuinatefinch: bash is not pleasant18:27
sinzuinatefinch: oh, we need to also add the other win series too18:28
sinzuinatefinch: I used this with the maas. I will fix the quote and add the other win series18:28
sinzuinatefinch: pull lp:juju-ci-tools. I added win2012r2 and replaced " with :18:34
natefinchsinzui: thanks18:37
natefinchsinzui:  I've never made my own streams before.  after running the streams-from-local script, do I just bootstrap with --metadata-source, or is there more I need to do?18:43
sinzuinatefinch: --metadata-source worked last time I used (when I wrote the script).18:44
natefinchgah, of course --metadata-source doesn't work with the ci script :/    I'm sorta surprised we don't just pass any unrecognized flags to bootstrap... seems like that would be the right thing to do most of the time18:45
natefinchmgz, sinzui: to add support for a new arg, do I just add another add_argument call to deploy_job_parse_args?18:52
sinzuinatefinch: probably not. Most args ar needed by all scripts. utility.py add_basic_testing_arguments() fif the arge is for a general juju or test case. If the arg is specific to a single test, then the script can get it.18:56
natefinchsinzui: ok, so how do I get the flag to actually be passed to bootstrap?  I added it to add_basic_testing_arguments ... but not exactly sure what to put as the action19:20
sinzuinatefinch: you will need to update BootstrapManager in deploy_stack.py. You probaly need the arg passed down to booted_contex()19:23
sinzuinatefinch: which arg to you need19:24
natefinchsinzui: metadata-source19:26
sinzuiabentley:  we need the new juju bugs reported to discuss them on the release call19:27
* rick_h_ goes to get the boy from summer camp19:27
abentleysinzui: I reported bug #1613300 and bug #1613311.  Is that what you're asking?19:28
mupBug #1613300: github.com/juju/juju/environs/sync fails with little info <centos> <ci> <intermittent-failure> <jujuqa> <test-failure> <juju-core:Triaged by wallyworld> <https://launchpad.net/bugs/1613300>19:29
mupBug #1613311: show-status and list-models disagree after migration <ci> <intermittent-failure> <jujuqa> <model-migration> <juju-core:Triaged> <https://launchpad.net/bugs/1613311>19:29
sinzuinatefinch: I would consider a cheat to to get moving. you could change EnvJujuClient.bootstrap() to append the extra arg. You need to do that to add full support. Just added enough to get yourself unblocked. You cna push your change up for yourself to complete or maybe the qa team. I am sure we will want comprehensive support in the future19:31
natefinchman I miss static typing19:33
natefinchis args a string or a list?19:33
natefinchanswer: neither, it's a tuple :/19:36
katcorick_h_: ping20:18
rick_h_katco: pong20:19
katcorick_h_: got a sec? i have a usability error message q20:20
rick_h_katco: otp atm, will ping when off20:20
katcorick_h_: np, ta20:20
rick_h_katco: meet you in standup room when you're free20:29
katcorick_h_: omw20:30
mupBug #1613429 opened: introspectionSuite.SetUpTest failure <intermittent-failure> <test-failure> <testing> <juju-core:Triaged by thumper> <https://launchpad.net/bugs/1613429>20:33
natefinchsinzui: I can't get metadata-source to work... I just always get no tools available, even though I have win2012hvr2 in my local streams stuff20:37
sinzuinatefinch: is metadata-source broken?20:37
sinzuinatefinch: you can push the streams to people.canonica.com then set image-metadata-url. I did this earlier this year (http://people.canonical.com/~curtis/images)20:39
natefinchsinzui: I'm not sure.  I don't know enough about how it is supposed to work.  Definitely there seems to be a lack of logging around it.  Just saying "no tools available" is kind of useless for debugging.20:40
sinzuinatefinch: yeah. that's below average.20:40
veebersnatefinch, sinzui: is it possible that might be related to the changes wallyworld did for the upload-tools/build-agent stuff?20:41
natefinchI think streams data is a separate path, but it's possible this is an unintended side effect20:43
sinzuiveebers: I don't think so since CI isn't seeing failures yet. Juju is very bad at letting developers test mixed os/arch. the new feature is actively ignoring that we must support windows, centos, and 3 other archs20:43
veeberssinzui: ack. I ask because he is double checking something for me based on his recent changes :-)20:44
sinzuiveebers: I removed --upload-tools when I woke so we know deployments without streams continue to work.20:45
sinzuinatefinch: You hit bug 159148820:46
mupBug #1591488: Unable to bootstrap with --metadata-source <cpe-sa> <juju-core:In Progress by anastasia-macmood> <juju-core 1.25:Won't Fix> <https://launchpad.net/bugs/1591488>20:46
veeberssinzui: huh I should have checked trunk changes more closely :-\ You already fixed this bug before I came across it: https://bugs.launchpad.net/juju-ci-tools/+bug/161319220:50
mupBug #1613192: EnvJujuClient.grant() passes invalid argument <juju-ci-tools:New> <https://launchpad.net/bugs/1613192>20:50
natefinchsinzui: well, I guess I have to do the image-metadata-url thing20:52
sinzuinatefinch: just add image-metadata-url to the env in environments.yaml. It will be added to config.yaml for bootstrap20:54
natefinchsinzui: cool... do you have a link to instructions on how to get stuff up to people.canonical.com?20:55
natefinchgotta run, back later20:59
=== natefinch is now known as natefinch-afk
rick_h_natefinch-afk: you can just use dropbox and get a url for it, or use the s3 file web UX to upload and get a generated url for it.21:00
rick_h_natefinch-afk: don't let people.canonical.com hold you up, unblock21:00
rick_h_natefinch-afk: shoot, upload the file into a pastebin and direct link to the raw url path21:00
anastasiamacwallyworld: thumper: could u joun bug scrub?21:02
wallyworldit's not on my calendar, url?21:03
mupBug #1613444 opened: Remove-user doesn't remove user from list-shares <juju-core:New> <https://launchpad.net/bugs/1613444>21:03
thumpernor mine21:04
anastasiamacwallyworld: thumper: https://hangouts.google.com/hangouts/_/canonical.com/bug-scrub?authuser=021:05
voidspaceaxw: ping21:10
voidspacethumper: ping21:12
thumpervoidspace: hey, in a call right now21:12
thumpervoidspace: is this about the actions review?21:12
voidspacethumper: hey, cool21:12
thumperbecause I didn't finish it, sorry21:12
voidspacethumper: sort of - I'm working on QA steps and failing to do a basic migration21:12
voidspacethumper: I'll email you what I'm trying to do and you can correct my bad syntax21:12
thumperhmm... where's menno?21:13
voidspace$ juju help migrate21:13
voidspaceERROR unknown command or topic for migrate21:13
thumperyou need the feature flag21:13
voidspaceah, I did for the migrate command - but not for the help21:13
voidspacethumper: I'll bother menn021:14
thumpermenn0: hey, can you help voidspace with a migration?21:14
menn0voidspace: yep, what's up?21:14
voidspacemenn0: hang on, I'll create a pastebin21:14
voidspacemenn0: my thoughts for QA steps for action migration was that it would look something like this:21:15
voidspacemenn0: http://pastebin.ubuntu.com/23059562/21:15
voidspacemenn0: however the migration step of this seems to not (visibly) do anything21:16
voidspacemenn0: so I've obviously got something wrong21:16
menn0voidspace: the QA steps look sensible to me21:16
voidspacemenn0: after running the migration there is no git unit in B21:17
voidspacemenn0: nor any visible sign of anything happening21:17
menn0voidspace: it might be worth checking `JUJU_DEV_FEATURE_FLAGS=developer-mode juju dump-model` on the model before and after the migration to see if the action data is being included21:17
voidspacemenn0: ah, ok21:17
menn0voidspace: did the model actually make it across?21:18
voidspacemenn0: however, I'm not yet convinced a migration is actually happening21:18
voidspacemenn0: ah, of course21:18
voidspaceI'm in B:default21:18
voidspacenot looking at the new model21:18
menn0voidspace: I see the problem21:18
menn0voidspace: you're trying to migrate "default" across but there's already a default on the other side21:19
menn0voidspace: the migration will have aborted because of that21:19
voidspacemenn0: ok, will fix this21:19
menn0voidspace: to see try something like: juju debug-log -m A:controller --replay -T | grep migrationmaster21:19
menn0voidspace: if you remove "default" from B and then retry the migration you should have better luck21:20
voidspacemenn0: yep, exactly that21:20
voidspacemenn0: or just add a new model, deploy to that and migrate that21:20
menn0voidspace: that's what I normally would do21:20
menn0voidspace: but given you already have it set up as default you could just delete default first21:20
voidspacemenn0: ok, so I can confirm that actions are not migrated on master - now to try it with my branch :-D21:25
voidspacemenn0: thanks for the help21:25
menn0voidspace: no problems21:25
menn0voidspace: migrations will support renaming of the model during migrations at some point21:26
menn0voidspace: it'll also fail in a more obvious way for these kinds of early checks (i.e. before the `juju migrate` command returns)21:26
voidspacemenn0: that will really help21:26
voidspacemenn0: and after much futzing (required merging master in) I can confirm that actions are migrated by my branch21:50
voidspacemenn0: will add the successful QA steps to RB21:50
voidspaceand bedtime21:52
voidspaceg'night all o/21:52
menn0voidspace: good night21:55
mupBug #1613459 opened: MainSuite.TestFirstRun2xFrom1xNotUbuntu forbidden (on closed network) <ci> <regression> <test-failure> <unit-tests> <juju-core:Triaged> <https://launchpad.net/bugs/1613459>21:57
* rick_h_ goes to make the fam dinner, night all22:02
thumperwallyworld: what was the final thoughts on the blocker?22:02
wallyworldthumper: i'm going to remove it22:02
thumpermenn0: ^^22:03
menn0wallyworld: ty, this didn't seem like blocker material to me either22:04
menn0redir: can you show me how you added the config to the cloud? I just tried that (via a new lxd cloud called "local") and none of the config made it the the bootstrapped host22:17
redirmenn0: so far I've been doing it with aws since i am working on something that requires regions. but lemme make sure it works with lxd too22:18
redirmenn0: so I have this file stored where I won't delete it: http://paste.ubuntu.com/23059827 (you only care about lxdtest)22:20
redirand I add it with juju add-cloud lxdtest ~/Sync/juju/2.0/development-config.yaml22:21
redirmenn0: and I can see that the lxd controller is using squid-deb-proxy22:21
menn0redir: that's essentially what I did but none of those options made it into the bootstrapped models22:22
menn0redir: I must have missed something22:22
menn0redir: I see the problem -- "juju add-cloud" ignored all the extra options22:23
menn0redir: looks like I have to edit them into clouds.yaml directly22:23
redirmenn0: I just imported them22:23
redirif you already have a config then you need --replace maybe22:23
rediralso maybe local is overloaded22:24
redirmenn0: ^22:24
menn0redir: it was definitely a new cloud22:24
menn0redir: I'll try a different name22:24
menn0redir: nope22:25
* redir blinks22:25
menn0redir: add-cloud just ignores the extra bits22:25
menn0I end up with a cloud with just one field "type: lxd"22:25
redirmenn0: juju model-defaults no-proxy22:26
redirno-proxy   -        https://lxd-no-regions22:26
menn0redir: very strange ... so you used "juju add-cloud" as well?22:26
redirwhenever I edit I edit in the non-volatile location and --replace22:27
redirdo you have only one cloud?22:27
redirmenn0: ^22:27
menn0redir: yes... there's one cloud in clouds.yaml22:28
redirmenn0: very strange, indeed22:30
redirmenn0: FWIW, I am at ed24813 with a couple commits on top.22:31
menn0redir: I'm a little behind you then (last updated late yesterday)22:40
menn0redir: I'll fiddle with this a bit more later22:41
menn0redir: thanks for your help22:41
redirsome help:)22:41
menn0redir: I certainly think having this config in the cloud config is the way to go22:41
redirI'll update the wiki next week.22:41
redirwfm in AWS too (sans apt proxy)22:43
* redir looks to see if wallyworld is in meetings22:43
wallyworldnot right now22:43
redirwallyworld: looks ripe for interruption22:43
wallyworldalmost, give me a minute22:44
redirgot a few minutes for some model-config ?s22:44
katcowallyworld: change pushed22:46
katcowallyworld: ty22:46
mupBug #1605756 changed: [ juju2 beta11 ] system show up in juju status as pending but there is no attempt to deploy in maas <oil> <oil-2.0> <juju-core:Invalid> <MAAS:Invalid> <https://launchpad.net/bugs/1605756>22:57
menn0wallyworld: so this test I fixed last night isn't the same thing that's causing the mysterious failures in environs/sync22:58
menn0wallyworld: I was just bitten by it again22:58
menn0wallyworld: it's happening a lot22:58
menn0wallyworld: any ideas?22:58
wallyworldmenn0: otp, give me a sec22:59
mwhudsoner are you all in some different standup to me?23:15
mupBug #1613476 opened: juju cli commands should be logged by juju <oil> <oil-2.0> <juju-core:New> <https://launchpad.net/bugs/1613476>23:33
wallyworldredir: did you want to talk some more?23:47
wallyworldor you all good?23:48
redirwallyworld: yeah was mid thought23:48
redirstill in standup23:48
wallyworldredir: gogle crashed, one sec23:49

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