/srv/irclogs.ubuntu.com/2016/09/01/#juju-dev.txt

axwwallyworld thumper menn0 anastasiamac: could someone please review http://reviews.vapour.ws/r/5572/, fixes the blocker00:01
axwif it looks good, please land - I need to go help with kids00:01
thumperaxw: ack00:01
redirtwo in a row perrito666!00:01
redir~warp speed~00:01
menn0axw: looking00:01
wallyworldmenn0: axw: sorry, just got off a call, menn0 i can look at your PR in a minute00:11
alexisbanastasiamac, ping00:19
menn0wallyworld: cheers00:23
anastasiamacalexisb: pong :)00:26
anastasiamacalexisb: a-team standup?00:26
alexisbsure00:26
wallyworldmenn0: in the PR, it looks like the "lost" processing has been omitted for machines?00:31
menn0wallyworld: it was never there00:31
menn0wallyworld: there should be no functional change in this PR00:32
wallyworldah, right, looks like there was some dumb no-op code there for mchine status00:33
menn0wallyworld: yep there was some dead code there. I meant to mention it in the PR description, sorry.00:33
wallyworldno worries, i just had a reading comprehension problem00:33
wallyworldmenn0: +1, nice to see that refactoring00:36
axwwallyworld: can you please look at http://reviews.vapour.ws/r/5572/ also01:12
wallyworldok01:12
wallyworldaxw: was the main issue adding back the EnableHTTPSListener call?01:16
axwwallyworld: yes01:16
wallyworldaxw: would be nice to test on a virgin system, to repro the scenario that lead to the bug, but maybe that's a bit difficult01:17
axwwallyworld: I reproduced the issue by wiping out the core.https_address config from my lxd01:17
wallyworldawesome, ok01:17
wallyworldlgym01:17
axwwallyworld: sorry, should have included that in QA01:18
wallyworldno worries01:18
axwwallyworld: the CI machine is out of disk, and I need to head out for a bit. can't land my branch just yet01:39
wallyworlddamn01:39
redirdoh01:43
menn0wallyworld: thanks for the review01:49
wallyworldnp01:49
menn0gosh darn it... github.com/juju/charm/hooks has moved/gone02:24
menn0thumper: so the migration export format only stores the charm URL against the application03:57
menn0thumper: but a particular unit might running a different charm version03:57
menn0thumper: if a charm upgrade is in progress03:58
menn0thumper: axw picked up that the prechecks don't prevent a migration if a charm upgrade is in progress03:58
menn0thumper: we either need to extend the export format to store charm URL per unit04:00
menn0thumper: (which is probably needed when the migration format gets used for backup and restore)04:00
menn0thumper: or we block migrations during charm upgrade04:00
menn0thumper: thoughts?04:00
anastasiamacmenn0: this bug has been assigned to u, but I suspect u were snowed under and haven't had a chance to look at.. bug 1453805... is it correct? m thinking to re-target it to next beta...04:02
mupBug #1453805: Juju takes more than 20 minutes to enable voting <ci> <ensure-availability> <intermittent-failure> <regression> <juju:Triaged by menno.smits> <juju-core 1.23:Fix Released by menno.smits> <juju-core 1.24:Fix Released by menno.smits> <https://launchpad.net/bugs/1453805>04:02
menn0anastasiamac: yeah, snowed under ... retarget please04:02
anastasiamacmenn0: \o/04:03
thumpermenn0: precheck for charms04:08
thumpermenn0: we will have to update the format though I think04:08
menn0thumper: ok, easy enough04:08
anastasiamacthumper: menn0: wallyworld: axw: i thought I heard someone say during standup that they were working on fixing bundle deployment.. has bug 1555808 been addressed as part of that work?04:08
mupBug #1555808: Cannot deploy a dense openstack bundle with native deploy <2.0> <2.0-count> <bundles> <cdo-qa> <ci> <deployer> <juju-release-support> <jujuqa> <maas-provider> <juju:Triaged> <https://launchpad.net/bugs/1555808>04:08
* wallyworld doesn't know04:09
axwlikewise - I missed that04:09
thumperanastasiamac: my fix may have addressed that04:09
* anastasiamac have troubles believing that there is something that wallyworld doesn't know :-P04:09
thumperbut it wasn't the specific fix04:09
anastasiamacthumper: \o/ i'll get ppl to re-test :D tyvm!04:10
anastasiamacveebers: do u know if there is a functional test for this ^^ bug?04:11
wallyworldCI doesn't yet deploy openstack bundles04:11
wallyworldto the best of my knowledge; it's a gap that needs to be filled04:12
wallyworldwe tend to rely on OIL04:12
anastasiamacwallyworld: in my converstaions with Torsten and Nicholas, there have been movements in this area... i'll confrim :D Thank you for answering Chris's question \o/04:12
wallyworldi'd love to know how far away such movements are from something concrete04:13
veebersanastasiamac: sorry missed the ping, seems wallyworld answered for me :-)04:17
anastasiamacwallyworld: axw: i thought bug 1582021 was addressed :D04:26
mupBug #1582021: Juju loses track of current controller when destroying model <2.0> <bitesize> <destroy-model> <juju-release-support> <usability> <juju:Triaged> <https://launchpad.net/bugs/1582021>04:26
wallyworldso did i, it could be a dupe04:26
anastasiamacwallyworld: i love those \o/ i'll mark as fix committed :D04:27
axwwallyworld: your PR is obese, too hard to see what's going on. can you please point out where the OpenParams.ControllerUUID is used?05:26
wallyworldaxw: yeah, sorry, i didn't know how to split it up05:26
wallyworldit's used in the ec2 provider (and hopefully soon openstack) to construct the firewaller component05:27
wallyworldthe controller uuid is needed to manipulate security groups05:27
wallyworldremmeber how in openstack we didn;t have the controller uuid avaiable and had to make that api call?05:28
wallyworldwe can now revert that change05:28
axwwallyworld: why would we need controller-wide security groups? shouldn't they be model specific?05:29
axwwallyworld: which API call?05:29
wallyworldlet me check the exact one05:30
wallyworldaxw: before i do that, here's a call that's made in the ec2 provider in current master: controllerSecurityGroups(controllerUUID)05:32
wallyworldmaybe the intent was for controller model uuid05:33
wallyworldah, no, i think controller uuid is to filter on tag05:34
wallyworldwe tag ec2 security groups with model and controller uuid05:34
axwwallyworld: if we just move the security group creation to the Create method, like we were planning to, we don't need OpenParams.ControllerUUID05:34
axwit already has the controller UUID there anyway...05:35
wallyworldthat me be a very valid point05:35
wallyworldi can look at that and revert the controller uuid in open params05:35
axwthanks05:35
axwwallyworld: but before you do, what was the openstack case?05:36
axwmaybe it's needed there05:36
wallyworldaxw: i think with openstack, we can tag security groups, so need to construct the name as juju-controlleruuid-modeluuid05:36
wallyworldlet me check hte code05:37
axwwallyworld: assuming you mean s/can/can't/ -- yeah I think that's the case. probably same deal then, move to Create05:38
wallyworldaxw: it's to get all the security groups for instances to be stopped; i just checked and ec2 also makes an api call - i think openstack didn't and then we had to so it's become no worse than ec2; so i think we are ok to revert that controller uuid in open params05:40
axwcool, sounds good05:40
wallyworldat least it will make the pr smaller05:40
wallyworldaxw: i even now see the todo in ec2 environ create, sigh, forgot about that before05:51
axwwallyworld: :)  is it even necessary at the moment though? the code that's there already has access to controller UUIDs AFAICS05:51
axwwallyworld: I'd prefer to do that change in another PR if it's not necessary, because there are possible negative side effects05:52
axwwallyworld: Create is currently called synchronously, when adding a model. so if we do anything slow in there, it means add-model is going to be slow05:52
wallyworldaxw: i found out that not all places had the controller uuid. but. all i need to do is create the firewaller struct in create05:52
wallyworldand i can call it later when needed using the current code05:53
axwwallyworld: they're not necessarily called on the same Environ05:53
axwin fact, s/necessarily// :)05:53
anastasiamacaxw: do u know if bug 1607347 is related to some credential change?06:03
mupBug #1607347: Password for juju-gui not showing up after a change <cpe-sa> <juju:Triaged> <https://launchpad.net/bugs/1607347>06:03
wallyworldaxw: damn, a bit more to do - appears ec2 tests don't call Create()06:03
wallyworldwhich is kinda important :-)06:04
wallyworldaxw: huh, looks like only prod code for all the providers calls create06:06
anastasiamacwallyworld: axw: also was bug 1614010 addressed as part of recent works in he area?06:07
anastasiamacthe*06:07
wallyworldanastasiamac: that one is invalid06:08
anastasiamacwallyworld: could u plz update the bug and explain why it's invalid as part of the comment? ;D06:09
wallyworldi may not have all the exact reasoning to hand; would be better to ask rick whose team did the work06:09
wallyworldsomething along the lines of needing a separate juju home for each user06:10
wallyworldi'm not sure i agree with it06:10
wallyworldmick did the work i believe06:10
anastasiamacwallyworld: axw: and what about bug 1607347?06:21
mupBug #1607347: Password for juju-gui not showing up after a change <cpe-sa> <juju:Triaged> <https://launchpad.net/bugs/1607347>06:21
=== petevg_ is now known as petevg
axwanastasiamac: we wipe the password off disk when you call change-user-password06:22
anastasiamacaxw: \o/ could you please adda  comment to the bug?06:23
anastasiamacwallyworld: do u know if ur recent work addressed bug 1617046 as a drive-by?06:25
mupBug #1617046: WARNING cannot read current model: current model for controller <name> not found <oil> <oil-2.0> <juju:Triaged> <https://launchpad.net/bugs/1617046>06:25
wallyworldanastasiamac: possibly, not sure off hand, there's been activity in that area06:29
wallyworldaxw: it appears we only call env.Create() during a new model operation; it is not called at bootstrap. and it needs to be for ec2 at least so that the new firewaller is set up06:34
wallyworldyou agree that i need to add a create call at bootstrap?06:35
axwwallyworld: we do not call Create on the same Environ that other methods are called with. so setting up a firewaller in that method will be pointless. you can/should create *security groups* in that method06:35
axwwallyworld: but again: do we actually need to make that change in this PR? from what I could see, controllerUUID is *already* available where we were creating the security groups06:36
axwi.e. in the StartInstances method06:36
wallyworldaxw: firewaller is misnamed perhaps (i copied off the openstack naming) - all it is is a component to do the securty group thing06:36
wallyworldaxw: i'll chedk the code again - there were places where we did not have controller uuid06:37
axwwallyworld: my point is that Create should not modify the Environ object - or if it does, you should not *expect* that it has done so from any other Environ method06:38
wallyworldaxw: that's fine - other environs are opened and created a sneeded06:38
wallyworldand those other ones have the create called to set up things06:39
wallyworldbut for bootstrap, the bootstrap instance startup was failing because the security group bit had not been initialised because create wasn;t called06:39
axwwallyworld: right, Create is only called for hosted models06:40
wallyworldi'll take another look to confirm that we always have controller uuid where needed06:40
wallyworldaxw: yeah, i think in an earlier incarnation i mixed up controller uuid and controller model uuid and so controller uuid is available where needed; i've reverted all the ec2 provider changes and and open params ones, diff is 1 page of files less06:48
wallyworldexcept i left in a create call, need to remove it06:51
wallyworlddone06:52
mupBug #1616832 changed: manual environment juju-db timeout <manual-provider> <juju:Triaged> <juju-core:Won't Fix> <juju-core 1.25:Incomplete> <https://launchpad.net/bugs/1616832>07:06
mupBug #1618798 changed: endpoint not used in lxd provider <juju:Triaged> <https://launchpad.net/bugs/1618798>07:06
=== frankban|afk is now known as frankban
mupBug # changed: 1492000, 1566414, 1584616, 1596597, 1612645, 1614364, 161463307:33
wallyworldaxw: thnks for review - i was 50/50 on dropping the controller uuid from the login result. i couldn't see anything that used it. but i guess some callers may want to08:22
axwwallyworld: seeing as you don't need to know it before hand, I think it's best to keep it08:22
wallyworldok, will revert08:23
voidspacefwereade: ping - got a minute08:26
voidspacefwereade: ?08:26
fwereadevoidspace, sure08:26
voidspacefwereade: working on migration for cloudimagemetadata08:26
voidspacefwereade: cloud image metadata stuff is in a sub-package of state08:27
voidspacefwereade: the migration internal test checks migrated fields so needs access to cloudimagemetadataDoc08:27
voidspacefwereade: but this isn't exported and the test is in the state package08:27
voidspacefwereade: so the two options I can see are either export the doc or copy the migration test infrastructure into the sub-package08:28
voidspacefwereade: can you think of another option?08:28
voidspaceexporting the doc seems like a bad idea08:28
fwereadevoidspace, thinking/spooling08:30
axwwallyworld: I'm looking to see if we can fix the dummy provider now08:32
axwlooks diable08:32
axwdoable*08:32
wallyworldaxw: econtext switch, which bit?08:32
axwwallyworld: getting rid of references to controller UUID from dummy provider's PrepareConfig08:33
wallyworldah ok. did you want me to land this current work first once i fix the comments?08:33
fwereadevoidspace, I'm thinking that the package boundary should be firm enough that it *should* be sufficient to check the data that's exported -- but, yeah, I'm drawing a bit of a blank on how to do that with certainty that we're not missing fields as we evolve08:33
axwwallyworld: no, I don't want PrepareConfigParams getting worse than it is already08:34
wallyworldok08:34
fwereadevoidspace, is the test machinery at all suitable for extraction and reuse rather than copying?08:35
voidspacefwereade: it's a pretty simple function (and a method)08:35
voidspacefwereade: so could easily be movedd08:36
voidspace*moved08:36
voidspacefwereade: the heavy lifting is a 15 line reflection function getting the exported fields08:36
fwereadevoidspace, I'm gently leaning that way at the moment then08:37
voidspacefwereade: I'll do that08:37
voidspacefwereade: just need to find the right place for it to live08:37
voidspacefwereade: and as it will be a single test in cloud image metadata I'll just inline the assert and move the reflection function somewhere resusable08:38
voidspacefwereade: thanks08:38
fwereadevoidspace, cheers08:41
rock_Hi. How can I set bugs-url and homepage  for my newly developed charm? Please anyone respond to this.09:05
wallyworldmarcoceppi: ^^^^^^ you around to help?09:42
wallyworldaxw: i've pushed some changes, will retest, i can wait till you propose and land yours and then rebase09:43
axwwallyworld: thanks, just gotta pull in your restore changes09:43
wallyworldok, will go feed dog etc09:44
perrito666Morning10:00
axwwallyworld: my WIP is here: https://github.com/juju/juju/pull/6137, seems to be an issue with leaking mongo. I need to go help with the kids, will try and be back later10:03
wallyworldok10:03
anastasiamacbabbageclunk: fwereade: do u have any idea what could be wrong in bug 1616832 or if there is a workaround for juju 1.x?10:06
mupBug #1616832: manual environment juju-db timeout <manual-provider> <juju:Triaged> <juju-core:Won't Fix> <juju-core 1.25:Incomplete> <https://launchpad.net/bugs/1616832>10:06
mupBug # changed: 1425808, 1468752, 1484105, 1496166, 1498642, 1500981, 1504602, 1506498, 1510651, 1510675, 1511103, 1511235, 1512875, 1513659, 1517474, 1517535, 1519403, 1519473, 1522409, 1525868, 1532085, 1534296, 1535678, 1539684, 154122810:42
fwereadeanastasiamac, nothing really springs to mind... at least not without stopping mongo, running it locally without auth, and seeing what's actually in the admin database10:47
fwereadeanastasiamac, it does look like machine-0 has forgotten how to log in, but I don't have a mechanism for how that'd happen10:48
dimiternfrobware, babbageclunk: hey guys, I've updated http://reviews.vapour.ws/r/5559/ and still need +1 to land it11:12
frobwarejam: ^^ based on our current discussion we may want to look at this again11:21
anastasiamacfwereade: \o/ thank you. could u plz comment in the bug with these thoughts and i'll follow up with Menno in the morning ;D11:22
* dimitern steps out for ~45m11:55
axwwallyworld: red herring it seems, I had the same failure on master. running tests again and then will propose12:16
wallyworldok12:17
wallyworldlooked good from what i saw12:17
axwwallyworld: if you're happy with https://github.com/juju/juju/pull/6137, please land and rebase off that12:22
wallyworldok12:22
wallyworldaxw: let me know if you're happy with the latest version of mine; i've alredy fixed the controller uuid thing12:24
axwwallyworld: looking12:26
axwwallyworld: LGTM with that field removed12:48
wallyworldaxw: awesome, ta. yours is almost dome landing hopefully12:48
mupBug #1484105 opened: juju upgrade-charm returns ERROR state changing too quickly; try again soon <bug-squad> <upgrade-charm> <upgrade-juju> <juju-core:Incomplete> <https://launchpad.net/bugs/1484105>12:49
mupBug #1573136 changed: kill-controller is stuck, lots of "lease manager stopped" errors <juju:Fix Released by fwereade> <juju-core:Won't Fix> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1573136>13:07
babbageclunkalexisb, wallyworld - Everyone cool for me to pause trying to reproduce bug 1611159 and work on maas2 instance tagging instead?13:11
mupBug #1611159: model not successfully destroyed, and error on "juju list-models" <oil> <oil-2.0> <juju:Triaged by 2-xtian> <https://launchpad.net/bugs/1611159>13:11
wallyworldbabbageclunk: big +1 from me :-)13:11
wallyworldthat bug is a rabbit hole13:11
wallyworldyou can come back to it13:12
babbageclunkwallyworld: I'm definitely feeling that too.13:12
babbageclunkwallyworld: Ok, I'll start on that unless alexisb says otherwise in the meantime.13:13
wallyworldsgtm13:13
wallyworldwe discussed it briefly13:13
wallyworldand it's good to reset your brain a bit as well13:13
babbageclunkyeah, absolutely - letting your subconscious have a go at it in the background wh13:14
babbageclunkrogue wh13:15
mupBug #1573136 opened: kill-controller is stuck, lots of "lease manager stopped" errors <juju:Fix Released by fwereade> <juju-core:Won't Fix> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1573136>13:16
alexisbbabbageclunk, see priv chat13:17
dimiternjam, are you around?13:17
mupBug # changed: 1573136, 1595617, 1597318, 1598329, 160498813:19
dimiternbabbageclunk: if you can have a look at http://reviews.vapour.ws/r/5559/ again, I'd appreciate a +1 :)13:21
mupBug # opened: 1595617, 1597318, 1598329, 160498813:28
babbageclunkdimitern: yup, lookin now13:35
babbageclunkg13:35
=== natefinch-afk is now known as natefinch
mupBug # changed: 1595617, 1597318, 1598329, 160498813:37
dimiternbabbageclunk: ta!13:46
babbageclunkdimitern: LGTM!13:54
dimiternbabbageclunk: tyvm13:57
katcovoidspace: natefinch: standup time14:01
mgzso, go 1.614:29
mgzI thought I'd consistently get random ordering in maps14:29
mgzis that not actually the case?14:29
mgzin specific,14:30
mgziterate over a map of three objects14:30
mgzI'd expect a test expecting a specific order to fail most of the time14:30
mgzis is not actually that random?14:31
frobwaremgz: make your map content bigger14:31
voidspacekatco: booger!14:31
voidspacekatco: I caught up and didn't notice the time, sorry14:31
frobwaremgz: MOAR14:31
voidspaceI'm going to start setting an alarm14:31
mgzfrobware: so, there's special handling for small maps? or something?14:31
dimiternmgz: yes there is14:32
frobwaremgz: Just the hashing of your object may yield the same result each time14:32
dimiternmgz: I can remember the size, but it was ~10 entries14:32
frobwaremgz: s/object/keys14:32
mgzaha, thanks guys14:32
dimitern(from what I remember from gophercon, maps are internally implemented as linked arrays of fixed size - buckets, + overflow space14:33
frobwarenatefinch: do you want to set aside some time to install MAAS before I EOD in a few hours?14:33
dimiternthat's why using a map lookup vs slice iteration for small number of elements is not worth the overhead14:33
natefinchfrobware: yeah, will you have time in like 15 minutes?14:34
frobwaredimitern: let's do 30 - would that work?14:34
frobwarenatefinch: ^^ (sorry!)14:34
dimiternha! :)14:34
frobwaredimitern: habits...14:34
natefinchfrobware: sounds good14:35
natefinchmgz: randomish - https://play.golang.org/p/lzGvkm9VxC14:35
natefinchmgz: it appears as though small maps are just occasionally randomized14:39
mgzthat is interesting, and pretty odd14:43
natefinchmgz: pretty sure small maps are implemented as slices14:44
rock dimitern: Hi. as you said yesterday I upgraded to juju version 2.0-beta16. And also I deployed juju gui manually $juju deploy cs:juju-gui-134 --series xenial. It was deployed. But from gui I am not able to add relation between mycharm to cinder.ERROR: Relation biarca-openstack:juju-info to cinder:juju-info: cannot add relation   "biarca-openstack:juju-info cinder:juju-info" : principal and subordinate applications' series must match14:44
rockI am getting the same issue. juju status pasted info http://paste.openstack.org/show/565721/14:45
natefinchmgz: they add in fake random access so that when you get to a larger map that isn't just a slice, you won't get screwed by having code that relies on ordered iteration14:45
rockdimitern: So I am thinking it might be a juju-gui bug.14:46
frobwarenatefinch: you'll need a server ISO before we start14:50
natefinchfrobware: I can get that14:51
frobwarenatefinch: xenial if you want MAAS 2.x14:51
natefinchfrobware: I wouldn't get anything else :)14:52
frobwarenatefinch: it depends if you need to repro against MAAS 1.9 though - that's trusty only14:52
natefinchwell that's a PITA14:52
dimiternrock: hey! did you also deploy your biarca charm with specified --series ?14:52
natefinchfrobware:This bug is 2.0, and that's likely to be where most bugs are in the future, I'd imagine14:53
* frobware chuckles... :)14:53
dimiternnatefinch, mgz: here's the gophercon talk that explains how maps are implemented, if you're interested :) https://github.com/gophercon/2016-talks/tree/master/KeithRandall-InsideTheMapImplementation14:54
mgzit's a journey of fun discovery :)14:54
mgzokay, I'm going to go ahead and change this delete implemtation to preserve order14:55
katconatefinch: frobware: andreas has offered up help in reproducing bug 1614635 simply14:56
mupBug #1614635: Deploy sometimes fails behind a proxy <landscape> <juju:Triaged by natefinch> <https://launchpad.net/bugs/1614635>14:56
katconatefinch: frobware: please ping him if you run into trouble getting the environment set up14:57
natefinchlooks like almost exactly 75% of the time, iterating a small map will do so in order14:57
rockdimitern: Yes.14:58
perrito666natefinch: iirc express randomization was added to that14:59
natefinchperrito666: yep.... I just thought it was interesting that it wasn't just always random14:59
rockdimitern: Manually, using juju CLI I deployed juju-gui by specifying series as xenial. It worked. It was showing juju-gui series as Xenial.15:00
perrito666natefinch: well that is the definition of random15:00
natefinchperrito666: well, no... sometimes it decides to be random, and a lot of the time it decides not to be15:01
perrito666its random15:02
natefinch50% of the time I iterate over a map of 5 items, it's exactly 0 1 2 3 4.... that's not random.  If it was random, that order would only come up 1/120 times15:03
natefinchit's choosing to be random some of the time, and a bunch of the time it's choosing just to iterate in order15:04
perrito666natefinch: I recall a random implementation in turbo C that would spit the same number sequence each time15:04
perrito666it is random at random, how meta15:04
natefinchyes... that probably helps keep the average runtime near constant15:04
natefinchnotably, the bigger the map, the less often it goes in order... 3 items it's 75% in order, 5 items, it's 50%  8 items it's 12%15:07
natefinch...those numbers still being hugely higher than pure random would indicate15:07
frobwarenatefinch, katco: do we still want to setup locally?15:08
katcofrobware: natefinch: andreas said the setup is easy; maass in vmware, add 1 node, boom15:09
katcofrobware: natefinch: so if it starts sounding any more complicated than that, i would say ping him15:09
natefinchuh... well, maas in vmware is still ¯\_(ツ)_/¯ to me15:10
frobwarenatefinch: want to start now?15:10
natefinchfrobware: yes :) Thank you :)15:10
frobwarenatefinch: https://hangouts.google.com/hangouts/_/canonical.com/andrew-mcdermot?authuser=015:11
dimiternrock, right and your charm - was it showing xenial as well?15:18
katcovoidspace: sorry for delay, meetings... np about standup. how goes migrations?15:30
voidspacekatco: there's progress - it compiles, so mostly fixing tests, so not far off now. Probably tomorrow though.15:31
katcovoidspace: cool15:32
voidspacekatco: ran into trouble with packages - cloudimagemetadata doesn't live in the state package, so our usual import and test techniques don't work15:32
voidspacekatco: plus I did a grand rename to get rid of the abomination "cloudimagemetadatas" I originally used :-)15:32
katcohaha15:32
katcowhat's it named now?15:32
voidspacekatco: some places I could simply use CloudImageMetadata, even though it returns a slice of Metadata15:33
voidspacekatco: and the places I couldn't I used cloudimagemetadataset15:33
voidspacekatco: which is not ideal (it's not a set) but still better15:33
natefinchfrobware: all installed15:39
frobwarenatefinch: ok let's jump back in to the HO15:40
redirmorning16:00
mgzheya redir16:04
redir:)16:04
macgreagoirg'day redir16:05
frobwarekatco, natefinch: all done. working maas setup. \o/16:19
katcofrobware: ya! thanks a lot frobware16:19
redir\o/16:20
redirbbiab, going for a haircut.16:50
=== frankban is now known as frankban|afk
lazyPowerHey core devs o/  do juju-gui bugs get targeted against http://launchpad.net/juju now that its moved into core?17:06
natefinchlazyPower: no idea17:09
cmarshatch, do you know? ^^17:10
* hatch looks17:10
cmarsi thought we just used github issues, but wasn't sure17:11
hatchlazyPower: if it's a GUI bug it should be reported https://github.com/juju/juju-gui/issues unless it specifically has to do with the juju core cli17:11
lazyPowerwell, i filed it here: https://bugs.launchpad.net/juju/+bug/161938917:11
mupBug #1619389: juju-gui fails to parse exported bundle in beta-16 <juju:New> <https://launchpad.net/bugs/1619389>17:11
lazyPoweri'll xpost to juju-gui repo17:11
cmarslazyPower, thanks17:11
hatchlazyPower: it's a known bug17:11
hatchand it's fixed in gui tip17:12
hatchlazyPower: https://github.com/juju/juju-gui/issues/193717:12
lazyPowerhttps://github.com/juju/juju-gui/issues/196617:13
hatchthanks lazyPower we're working on getting the next GUI out but we have some bigger tasks we're just wrapping up first17:14
lazyPowerno stress, magicaltrout found it so was just making sure we had it on the docket17:15
hatchyup, and been fixed :)17:15
=== bcsaller_ is now known as bcsaller
hatchlazyPower: also to clarify, the GUI project has not been moved into core, only the ability to serve the GUI was added to core.17:21
lazyPoweryou're now part of core, tl;dr17:21
hatchlol definitely not17:22
hatchor maybe I have been and I just don't know it yet!17:22
tvansteenburghcan anyone give me an example of the 'directive' and 'scope' portions of a placement17:46
tvansteenburghlike if i have lxc:7, is that all directive, and the default scope is the model?17:47
tvansteenburghbasically trying to figure out how to convert a placement string into a Placement map17:48
katcotvansteenburgh: try: https://github.com/juju/charm/blob/aece7b0e56c298641968239a7fa0b3466afa6ef5/bundledata.go#L626-L62917:49
tvansteenburghkatco: thanks!17:59
katcotvansteenburgh: np, happy hacking18:00
mgzI had to add a bunch of refactoring to make one of my tests reliable, can I request a re-review in a sec?18:01
mgznatefinch: around still?18:03
natefinchmgz: I exist in this reality.18:03
mgz:D18:03
perrito666ok, apparently its impossible to bootstrap juju from my ISP18:04
perrito666in aws18:04
* perrito666 goes a bit closer18:05
mgznatefinch: okay, I have pushed a new commit on http://reviews.vapour.ws/r/5545/18:08
mgznatefinch: the base fix has been reviewed, but I did some refactoring to make the new testing not subject to map ordering problems18:08
mgznatefinch: so, I'd appreciate an eye on that part18:09
natefinchmgz: looking18:17
natefinchgod I hate significant whitespace18:39
natefinchwhats the opposite of deploy in maas?18:40
natefinch(unrelated to my previous statement)18:40
mgznatefinch: I was wondering18:41
mgznatefinch: `maas ENV machine delete` for 2, I think it's `... node delete` in 118:42
natefinchdelete or release?  I was just looking at the UI... I had deployed a machine to test that my maas is working correctly.  And now I want to undo that....18:43
natefinchmaybe release is the opposite of commission... not that I know what commission means18:43
mgznatefinch: repoke about review19:48
natefinchmgz: oops, sorry, had started looking at it then got distracted by maas stuff19:50
natefinchmgz: reviewde20:20
mgznatefinch: ta!20:22
cmarshow do i connect to juju's mongodb on xenial?21:38
cmarstried: /usr/lib/juju/mongo3.2/bin/mongo --ssl --username admin --password <value of oldpassword> --sslAllowInvalidCertificates localhost:37017 admin21:41
cmarsdoesn't work21:41
thumpermenn0: ^^21:44
thumpercmars: I think menn0 has a script somewhere21:45
menn0I thought the packaging was being fixed to include the mongo3.2 client21:45
* menn0 checks21:45
cmarsmenn0, it's there21:45
cmarsmenn0, i just can't log in with it, maybe my cmdline is wrong?21:45
menn0could be... let me check my script21:46
menn0mongo 127.0.0.1:37017/juju --authenticationDatabase admin --ssl --sslAllowInvalidCertificates --username "admin" --password "$password"21:46
cmarsah21:46
menn0that's what my script uses21:46
cmarsmenn0, ok, that got me in, thanks21:48
cmarsmenn0, can I have a copy of that script?21:48
menn0cmars: I just confirmed that it still works.21:48
cmarsjust needed the --authenticationDatabase flag21:48
menn0cmars: sure... let me revise it first though. It goes and installs the client from the mongodb.org PPA b/c we didn't use to include the client.21:48
menn0that's unnecessary now21:48
menn0cmars: https://gist.github.com/mjs/0d0b89356654de04adf4935860642c5f21:51
menn0cmars: run it from your machine like this: juju-db <controller:model>21:52
menn0where "model" is the controller model name21:52
menn0thinking about it, it should probably just assume the name "controller"21:53
cmarsmenn0, thanks!21:54
cmarsthis is going in my tools repo..21:54
menn0cmars: i've just improved it21:56
menn0cmars: if you now run "juju-db" without args it assumes machine 0 in the "controller" model for the currently selected controller21:56
menn0which is usually what you want21:56
cmarsmenn0, perfect, even better21:57
wallyworldthumper: menn0: you free for a chat? in s.o. h.o.?22:33
mupBug #1617602 changed: juju status <service-name> stuck <status> <juju:Triaged> <juju-core:Won't Fix> <juju-utils:New> <https://launchpad.net/bugs/1617602>22:35
katcothat feeling when you can iterate through a change because it has an actual unit test22:51
wallyworldcmars: hey, fyi, thumper just landed a fix fo all the recent landing issues. you will want to rebase your kpi branch to fix all the things22:53
wallyworldkatco: to be fair, there would have been a test wouldn't there? maybe more like an intergration test, but still a test to run?22:55
katcowallyworld: yes, but it would have taken significantly longer to run22:56
cmarswallyworld, if we've passed CI and there are no conflicts, i think we ought to be able to just land as-is22:57
wallyworldcmars: true. i was just letting you know in case you were doing any more development on that branch before landing22:58
cmarswallyworld, no, it's ready to go i think22:58
cmarsjust waiting for QA blessing22:58
wallyworldcmars: sweet, in that case it *will* land foirst time now :-)22:58
cmarsawesome. i spent my afternoon retrying so many times..22:59
wallyworldcmars: i just noticed you guys had a bunch od landing bot failures yesterday22:59
wallyworldyeah22:59
wallyworldthought'd you like to know it was fixed22:59
cmarsthat's great. i was getting worried :)22:59
wallyworldcmars: yeah, i blame thumper :-)23:00
thumperwallyworld: http://reviews.vapour.ws/r/5580/23:04
menn0wallyworld: sorry, I've been AFK23:09
menn0wallyworld: still want to talk?23:10
perrito666I am late for standup sorry ill be 10 late23:14
thumper10 what?23:15
thumperms?23:15
thumpers23:15
thumperhours?23:15
anastasiamacyears23:15
anastasiamac?23:15
perrito666Ill ley you guess23:15
katcoperrito666: if you're late by 10 years i'll buy you a beer in 523:21
perrito666lets not tell katco that It was minutes so she buys me abeer in 523:27
katcoperrito666: i didn't specify the unit =|23:27
perrito666I feel  cheated :p23:28
katcoperrito666: i have used your own logic against you! mua! muaha ha!23:28
perrito666It was just me typing on the phone, you attribute me too much inteligence23:28

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