anastasiamacwallyworld: btw, PTAL https://github.com/juju/juju/pull/10586 :)05:20
anastasiamacwallyworld: i've requested u as reviewer on github when I proposed but obviously it's way too subtle :)05:21
wallyworlddidn't see it05:21
anastasiamacall good05:21
wallyworldmost gh email is filtered as it's too noisy05:21
wallyworldanastasiamac: see comment/question about the txn logic05:34
anastasiamacwallyworld: k05:34
anastasiamacwallyworld: i'd rather keep docxists assert but will add buildttxn loop and the test05:35
wallyworldin that case, also add assert for $ne Dead05:36
anastasiamacwallyworld: k05:36
wallyworldto match initial query05:36
anastasiamacwallyworld: funny but this code was almost verbatum from upgrade permissions... i wonder then if there'd b a problem there too..05:36
wallyworldupgrade steps you are talking about" upgrades don't need to deal with concurrent access to db05:37
anastasiamacaltho mayb just rmoving assert is fine... u r right, we don't care if we cannot clear cred on a gone model..05:55
wallyworldi think in this case it may be ok to not care so much06:46
anastasiamacwallyworld: meh.. alreay pushed tansactionality and buildtxn ... want to look?06:48
manadartCan I get a tick on a forward-merge? https://github.com/juju/juju/pull/1059007:13
manadartjam: Ta.07:14
wallyworldanastasiamac: sure, looking now07:30
wallyworldanastasiamac: still lgtm07:31
anastasiamacwallyworld: \o/07:33
anastasiamacah manadart if only that merge was a couple of hours late :)07:33
anastasiamaci'll just cherry-pcik my change into develop tomorrow :D07:34
manadartanastasiamac: If the merge fails, I'll add yours :)07:34
anastasiamacmanadart: mine just went for landing now...07:36
anastasiamacinto 2.6 that is..07:39
anastasiamacwallyworld: and the cherry-pick for develop PTAL https://github.com/juju/juju/pull/1059108:51
anastasiamacmanadart: mayb u could a +1? srsly just cherry picking :)08:52
manadartanastasiamac: Tick!08:55
anastasiamacmanadart: \o/08:55
stickupkidachilleasa, any chance of a CR, this is the changes I talked about last week https://github.com/juju/bundlechanges/pull/5710:31
stickupkidachilleasa, fixing the tests took a lot longer than i hoped :\10:31
achilleasastickupkid: sure thing. looking in a few min10:32
stickupkidachilleasa, it's a shame GUIArgs has to live on, as it's not a very efficient way to transport information... i.e. you need empty values for all options, even if they're empty10:33
achilleasastickupkid: that's only being used for the GUI bits right? We could kindly ask them to switch to the map version instead ;-)10:36
stickupkidachilleasa, yes, i agree10:36
achilleasastickupkid: approved. Yeah, I remember updating the tests to include the empty channel in the GUIArgs output was a pain too11:29
parlosanybody experienced "twd4f3: Unable to allocate static IP due to address exhaustion", and have a fix (not sure what network is exhausted, and from the maas there are plenty of IPs in all networks)...13:31
parlos(ignore twd4f3 its just the MAAS id)13:32
hmlmanadart:  can or should 1 subnet be in 2 spaces?14:25
manadarthml: No.14:25
hmlmanadart:  that’s what i thought.  juju may be broken there.  stumbled on it while updating RelationUnitSuite.TestNetworksForRelationWithSpaces14:26
hmlfor space-314:27
hmlhoping was a copy paste error in the test setup14:27
hmllook like we never check14:30
hmlfound the check… up in the apiserver code.  but that doesn’t prevent state tests from doing interesting things.14:49
=== salmankhan1 is now known as salmankhan
stickupkidrick_h, i've updated to pylibjuju PR, I want to now add more unit tests around the type classes, as it should be much easier to do so. Can you look to see if you've got any concerns15:00
stickupkidrick_h, https://github.com/juju/python-libjuju/pull/35015:00
rick_hstickupkid:  rgr, otp but will look in a few15:00
rick_hstickupkid:  feedback in, looks much sweeter!15:12
stickupkidrick_h, the nice thing is, we can use mocks to test the run methods now!15:13
rick_hstickupkid:  exactly15:13
stickupkidrick_h, yarp, just adding some tests15:13
rick_hstickupkid:  sorry to create the work but I think it's on a nicer path for sure15:13
stickupkidrick_h, i'd rather have it right, 100% agree15:13
yan0sHi all!19:27
yan0shas anyone used openstack as a cloud provider in Juju?19:28
yan0sI'm getting this error: ERROR juju.cmd.juju.commands bootstrap.go:697 failed to bootstrap model: no image metadata found19:31
yan0salso: skipping index "http://my_ip:8080/simplestreams/images/streams/v1/index.json" because of missing information: index file has no data for cloud {Louros https://mykeystoneurl:5000/v3} not found19:32
yan0ssame steps have been working about a year ago..19:36
hmlyan0s:  check the index.json to see if the exact url is given:  https://mykeystoneurl:5000/v319:36
hmlyan0s:  i’ve seen where it’s https://mykeystoneurl:5000/v3 vs https://xx.xx.xx.xx:5000/v3 where technically they are the same, but it fails.19:37
hmlit’s a straight string match, no lookup19:37
yan0shml: it is the same in index.json20:07
hmlyan0s: which version of juju?20:09
hmlyan0s:  any possibility of a link to or pastebin of the the index.json file?  and the full bootstrap —debug output please?20:09
yan0sactually here is my simplestream uploaded
hmlyan0s: there was a mistype in creating the simplestream files… “https:///keysto”  3 slashes instead of 2.20:17
yan0soh god..20:18
yan0sI'm so sorry for wasting your time20:19
hmlyan0s:  no problem.  another set of eyes can be a good thing20:19
timClicksyan0s: great to hear that this was an easy fix :)20:20
timClicksyan0s: do let us know if you have any further problems, we will be delighted to help20:23
magicaltrouthttps://discourse.jujucharms.com/t/square-brackets-in-env-vars/2024 if anyone knows what goes on under the hood and what I have to escape to get this working, that would be very nice indeed! :)23:18
thumpersome more intermittent test failure fixes https://github.com/juju/juju/pull/1059423:43
thumperwallyworld: you may have commentary for magicaltrout on that post23:44
thumpermagicaltrout: I'm guessing that it is being parsed as a yaml array23:45
* thumper thinks23:45
thumpermagicaltrout: now I read it a bit more, it does seem very strange23:46
thumpermagicaltrout: you can use yaml type forcing23:46
thumperI'd expect23:46
thumpermagicaltrout: also, what is dc4?23:46
thumpermagicaltrout: you also don't mention where you are putting it in juju23:47
thumperwallyworld: I think I know what is going on though23:47
thumperwallyworld: I bet if you take a string like '["hello"]' and parse it as yaml, then write it out as yaml you get ["hello"]23:48
wallyworldthumper: sorry, was otp23:48
thumperwhich the second parser deeper is treating as a list23:48

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