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

thumperoh poo01:25
thumperwanting to add something to a worker manifold, but it just defines itself as a copy of engine.AgentApiManifoldConfig01:26
* thumper sighs01:26
* thumper unpicks01:26
thumperperhaps after a dog walk01:26
mupBug #1271744 changed: bootstrap on maas with --metadata-source fails <bootstrap> <maas> <maas-provider> <simplestreams> <upload-tools> <juju-core:Triaged by anastasia-macmood> <https://launchpad.net/bugs/1271744>03:16
mupBug #1567763 changed: bootstrapping private openstack, with --metadata-source fails when instance-type constraint is specified <bootstrap> <constraints> <simplestreams> <juju-core:In Progress by anastasia-macmood> <juju-core 1.25:Triaged> <https://launchpad.net/bugs/1567763>03:16
thumperbugger03:20
thumperclean up a test and now it fails03:20
* thumper reverts03:20
menn0axw: the more I think about it, the more I think your approach for those flaky MachineSuite tests is the right thing to do for now03:29
menn0axw: i'll give it a try soon03:29
axwmenn0: cool03:29
thumperfark03:33
menn0thumper: http://reviews.vapour.ws/r/5342/03:54
mupBug #1588115 changed: json: cannot unmarshal string into Go value of type nova.jsonEntity <bootstrap> <juju> <juju-core:Expired> <https://launchpad.net/bugs/1588115>04:22
menn0thumper: http://reviews.vapour.ws/r/5343/05:35
rogpeppeaxw: ping08:26
axwrogpeppe: pong08:26
rogpeppeaxw: yo!08:26
axwhowdy08:26
rogpeppeaxw: is it a bug or a feature that credentials can't be updated or deleted?08:26
axwrogpeppe: bug08:27
rogpeppeaxw: ok, thought so, just checking08:27
axwrogpeppe: working on the foundations of updating them still08:27
rogpeppeaxw: (if you try to update an existing credential, you get a "state changing too quickly" error)08:27
rogpeppeaxw: (not the most informative error in the world :] )08:27
axwrogpeppe: oh, well that's a different type of bug :)   I just meant that we don't react to changes at the moment08:28
axwI guess we're assuming they don't exist at the moment08:28
rogpeppeaxw: yeah, so there's no way for us to change them08:28
rogpeppeaxw: i was wondering if immutability was by design08:29
rogpeppeaxw: but it seems not :)08:29
rogpeppeaxw: i think it should make an update txn if the doc already exists08:29
axwrogpeppe: yep. not set in stone, but at the moment we don't assume immutability. there's no documented workflows for updating creds, but I don't see any problem with updating existing ones08:30
rogpeppeaxw: i think that perhaps it's a mistake for the transaction Run method to *always* return "state changing too quickly" if the transaction fails08:31
rogpeppeaxw: whether that's appropriate surely depends on whether the transaction is designed to work in all cases08:31
axwrogpeppe: it should always be valid, except if the transaction asserts are wrong?08:32
axw(which seems to be the case here)08:32
rogpeppeaxw: well, there might well be operations which are designed to fail in certain cases (for example, a Remove operation might fail because the doc doesn't exist)08:33
rogpeppeaxw: or it might well be legitimate to have an operation that is supposed to fail if the doc already exists (insert vs update)08:34
axwrogpeppe: sure, but you check that at the top of the transaction loop. if it doesn't exist, you return errors.NotExistsf. otherwise you assume it's there, but add an assert; the failure causes the loop to restart08:35
axwyou only get the "state changing too quickly" error if what you're chekcing at the top of the loop doesn't match what's in the assert, or if it really is happening too quickly08:36
axwchanging*08:36
rogpeppeaxw: that implies even more unnecessary round trips08:36
rogpeppeaxw: but i guess that's par for the course08:36
axwrogpeppe: you don't *have* to use the transaction loop08:36
axwrogpeppe: this error only occurs when you do08:36
rogpeppeaxw: you don't have to use the transaction loop? i thought that was the only way to run transactions these days.08:37
axwrogpeppe: state.State.runTransaction doesn't use a loop08:38
axwrogpeppe: state.State.run does08:38
rogpeppeaxw: ah, i didn't know about runTransaction or the distinction between that and run08:40
rogpeppeaxw: not great naming tbh - i think that Run should probably be named something that implies it will loop08:41
axwrogpeppe: agreed on that08:41
rogpeppeaxw: just reported these bugs:08:55
rogpeppeaxw: https://bugs.launchpad.net/juju-core/+bug/160842108:55
mupBug #1608421: UpdateCloudCredentials should be able to change existing credential <juju-core:New> <https://launchpad.net/bugs/1608421>08:55
rogpeppeaxw: https://bugs.launchpad.net/juju-core/+bug/160842208:55
mupBug #1608422: it is not possible to remove cloud credentials <juju-core:New> <https://launchpad.net/bugs/1608422>08:55
axwrogpeppe: thanks08:55
rogpeppeaxw: you're planning to fix these, right? (if not, we can probably do some work on it)08:56
axwrogpeppe: eventually yes, but I don't know when I will get to them08:56
rogpeppeaxw: ok08:56
mupBug #1608421 opened: UpdateCloudCredentials should be able to change existing credential <juju-core:New> <https://launchpad.net/bugs/1608421>09:02
mupBug #1608422 opened: it is not possible to remove cloud credentials <juju-core:New> <https://launchpad.net/bugs/1608422>09:02
mupBug #1608421 changed: UpdateCloudCredentials should be able to change existing credential <juju-core:New> <https://launchpad.net/bugs/1608421>09:05
mupBug #1608422 changed: it is not possible to remove cloud credentials <juju-core:New> <https://launchpad.net/bugs/1608422>09:05
mupBug #1608421 opened: UpdateCloudCredentials should be able to change existing credential <juju-core:New> <https://launchpad.net/bugs/1608421>09:08
mupBug #1608422 opened: it is not possible to remove cloud credentials <juju-core:New> <https://launchpad.net/bugs/1608422>09:08
rogpeppeaxw: is it my imagination, or are there no tests at all for the State.UpdateCloudCredentials method?09:12
axwrogpeppe: indeed, that one slipped through09:19
rogpeppeaxw: i'm doing a coverage test on state now :)09:20
fwereadehuh, irc wasn't open11:25
fwereadeanastasiamac, can we chat about supported architectures?11:25
anastasiamacfwereade: yes we can :)11:31
fwereadeanastasiamac, tell me if I get any of the following statements wrong11:32
anastasiamacfwereade: k11:32
rogpeppeaxw: as a followup to the UpdateCloudCredentials thing, there are a few other methods that could do with tests in state... https://bugs.launchpad.net/juju-core/+bug/160849411:32
mupBug #1608494: state: not all public methods are tested <juju-core:New> <https://launchpad.net/bugs/1608494>11:32
fwereadeanastasiamac, *in general*, bootstrapping with local image metadata will cause the image metadata to be uploaded and put in the database and made accessible to all environ instances created on the controller11:33
anastasiamacfwereade: right (when database is present and we have a connection to it)11:34
fwereadeanastasiamac, there is a window of badness that affects *only* the environ instance that's actually bootstrapping the controller11:34
anastasiamacfwereade: kind of... i have a gut feeling that the "window of badness" has a flow on affect. If it's fixed the flow-on effect *may* be fixed too11:36
fwereadeanastasiamac, that problematic environ instance is running on the same machine as the one that got the image metadata in the first place11:36
fwereadeanastasiamac, (I'm pretty sure that agents won't get env config except via an api conn? or at least via the database)11:36
anastasiamacfwereade: i *think* it maybe that environ instance *has access* to the machine that has image metadata11:37
anastasiamacfwereade: well, metadata has several data soureces - database is one of thethem and we hope is the main one - but ther are also url and file data sources..11:39
fwereadeanastasiamac, I don't understand the follow-on effect11:39
fwereadeanastasiamac, well, sure, but I don't think we need to care what the data sources are -- we just need to care that they *match*11:40
anastasiamacfwereade: so I ahve seen bugs where private clouds (or no intrnet env) can bootstrap using --metadtaa-url (= web served data) not metadata source (= file locations)11:40
anastasiamacfwereade: but had troubles deploying workload11:40
anastasiamacfwereade: right about the match11:40
anastasiamacfwereade: so form what i've seen, the "badness" occurs before stat is open where we select tools and get custom metadata from locally accessible location11:41
anastasiamacfwereade: we detemrine supported architectures from declared data sources (which we have not declared yet) and are ignoring bootstrap custom metadata11:42
fwereadeanastasiamac, so, the local stuff is not a properly configured data source for the environ? shouldn't it be?11:42
anastasiamacfwereade: when we get it, we do not have environ yet11:43
anastasiamacfwereade: we pass it into startinstance as params11:43
fwereadeanastasiamac, all the more reason to create the environ knowing what data source it should use11:43
anastasiamacfwereade: data source is created11:43
anastasiamacfwereade: but data source is just the location to look at11:43
anastasiamacfwereade: what we need here is a collection of actual values11:44
anastasiamacfwereade: since we do not have anywhere to put them et. we can only pass them around11:44
anastasiamacfwereade: i *think* :)11:44
fwereadeanastasiamac, the data source is an abstraction allowing us to get those values, isn't it?11:44
fwereadeanastasiamac, how does an environ know SupportedArchitectures if not by looking at what some datasource provides?11:44
anastasiamacfwereade: yes, and as soon as we get a chance we put them in db :)11:45
anastasiamacfwereade: suppoted architectures does not look i db at this stage anyway.. it's somethign that needs to happen but has not yet11:45
anastasiamacfwereade: putting this custom metadata into db will not help11:45
fwereadeanastasiamac, well, the provider should pretty much never be looking in the db, tbh11:45
fwereadeanastasiamac, I'm not suggesting we should11:46
anastasiamacfwereade: becasue we only obtain it and use it before db is started11:46
fwereadeanastasiamac, right11:46
anastasiamacfwereade: \o/11:46
fwereadeanastasiamac, I'm saying that the environ we use to bootstrap is misconfigured if it's using a different data source to the one specified11:46
fwereadeanastasiamac, (and that a local source needs to be uploaded and put in the db to be accessible later, but it sounds like that all works correctly anyway)11:47
anastasiamacfwereade:db upload works exactly as expected :)11:47
anastasiamacfwereade: peerfectly :)11:48
fwereadeanastasiamac, (and also cautioning you against imagining that an environ ever "should" look in the db, that may be the ultimate source of the data but db concerns belong nowhere near environs)11:48
fwereadeanastasiamac, cool11:48
fwereadeanastasiamac, I'm sorta implicitly dismissing the follow-on concerns then11:48
anastasiamacfwereade: even better for me then :)11:49
fwereadeanastasiamac, I can see an easy way to bootstrap and not be able to deploy, if you use a metadata url that's accessible to the client but not the controller11:49
fwereadeanastasiamac, and that is sorta a Don't Do That Then situation11:49
fwereadeanastasiamac, and it sounds like the upload of local metadata is also fine, so managing to bootstrap should imply we can deploy anything covered by that local metadata11:50
fwereadeanastasiamac, remind me how data sources *are* associated with an environ?11:50
mupBug #1608494 opened: state: not all public methods are tested <juju-core:New> <https://launchpad.net/bugs/1608494>11:50
anastasiamacfwereade: this specific fix does not deal with URL served metadata - this works. Metadat-source is a file location11:50
anastasiamacfwereade: metadata-url vs metadata-source11:51
fwereadeanastasiamac, yeah -- and it really seems like there's *one* environ, that we're trying to bootstrap, which will fail if it doesn't have proper access to the local metadata11:52
fwereadeanastasiamac, am I missing something? or am I pointing at the central problem11:52
anastasiamacfwereade: right. of course with my changes, i managed to bootstrap :)11:52
rick_h_morning11:56
mupBug #1608527 opened: lxd bootstrap now slow, timing out, fails <canonical-is> <juju-core:New> <https://launchpad.net/bugs/1608527>13:33
mupBug #1608527 changed: lxd bootstrap now slow, timing out, fails <canonical-is> <juju-core:New> <https://launchpad.net/bugs/1608527>13:45
mupBug #1608528 opened: machineSuite.TestShortPollIntervalWhenNoStatus timing problem: expect 2, obtained 2 <ci> <intermittent-failure> <regression> <unit-tests> <juju-core:Triaged> <https://launchpad.net/bugs/1608528>13:45
mupBug #1608529 opened: suprious 'd' in log output during lxd bootstrap <juju-core:New> <https://launchpad.net/bugs/1608529>13:45
mupBug #1608533 opened: Race in github.com/juju/juju/apiserver/tools_test <ci> <race-condition> <regression> <unit-tests> <juju-core:Triaged> <https://launchpad.net/bugs/1608533>13:45
=== natefinch-afk is now known as natefinch
perrito666will be back later15:01
natefinchug, taxes on part of my property went up by 54% since last year :/  Luckily, it's the smaller of the two parcels, but it still means over $200 a month extra I'm paying... except that it's doubled by my mortgage company this year to put extra in escrow :/15:58
mupBug #1608597 opened: CalledProcessError for juju-2.0 deploy -m slaveX:default deployment_file <juju-core:New> <https://launchpad.net/bugs/1608597>16:15
mupBug #1608597 changed: CalledProcessError for juju-2.0 deploy -m slaveX:default deployment_file <juju-core:New> <https://launchpad.net/bugs/1608597>16:27
mupBug #1608597 opened: CalledProcessError for juju-2.0 deploy -m slaveX:default deployment_file <juju-core:New> <https://launchpad.net/bugs/1608597>16:33
perrito666natefinch: I understood very little of that16:43
perrito666you can move to argentina, 200 dollars is about 1/3 of my yearly tax16:44
natefinchperrito666: the property that my house is on is in two towns... one of the towns decided that the piece of land I own in their town is worth twice as much this year as was worth last year.16:45
perrito666ah, I see, we have a much more strict definition of town16:45
* rick_h_ goes for lunchables17:02
natefinchrick_h_: you should really find something less processed than lunchables. Those things are terrible.17:03
rick_h_natefinch: just my way of saying 'food for lunch'17:26
rick_h_leftover blue apron meal today17:26
natefinchrick_h_: I know, just giving you a hard time :)17:26
natefinchrick_h_: I had leftover thai food :)17:27
perrito666I had .... too hard to translate, dry tomatoes, some greens, mozzarela and ham on ciabatta with a flavoured cream :p17:31
perrito666lunch caught me off my house, otherwise it would be way less pretentious17:31
natefinchperrito666: sounds tasty17:48
natefinchalso, verdict on the taxes: evidently the town did a review, and realized they'd been taxing us at the residential rate, but that piece of land is in a commercial zone, which is a lot more expensive.  sigh.17:49
alexisbnatefinch, 54% is crazy in one year18:00
alexisbI would have something to say about that18:00
natefinchalexisb: well, the land *is* in a commercial zone, I knew that when we moved in.  So, basically we've been getting a 33% discount for the last 6 years :/18:01
hatchwhen attempting to build Juju from master it is hanging on cmd/jujud has anyone ever run into this before?18:14
natefinchhatch: nope.18:22
natefinchhatch: when's the last time you did a successful build?18:22
hatchnatefinch: on this machine, never18:22
hatchfresh install18:22
natefinchhatch: hm.. suspicious18:22
hatchGo 1.6.318:22
natefinchhatch: are you using the makefile or running go install manually?18:23
hatchI was running `go install -v ./...`18:23
natefinch*nod* did you run godeps first?18:23
hatchyep18:23
natefinchI got nuthin.18:23
hatchmaybe Juju doesn't build with 1.6.3?18:24
natefinchI can try it locally.... I was using 1.6.2, can try 1.6.3 easy enough18:25
hatchI've just deleted everything and it's rebuilding18:25
hatchwill see if it hangs18:26
hatchyup hung again on jujud18:26
hatchI'll try downgrading to 1.6.218:27
hatchnatefinch: fwiw 1.6.2 also hangs18:30
rick_h_natefinch: ping for meeting18:31
natefinchrick_h_: oops, thanks, coming18:32
natefinchhatch: building now... probably something weird with the machine you're building with18:32
hatchnatefinch: yeah, I am just not sure how I go about debugging it as there isn't any output18:33
hatchnatefinch: go build also hangs18:33
hatchas expected I suppose18:34
* rick_h_ goes to get boy from camp biab19:21
rick_h_katco: can you add some step by step QA docs to https://github.com/juju/juju/pull/5906 please and once there natefinch can you please look at and review and validate QA please?20:15
natefinchrick_h_: sure thing20:16
katcorick_h_: pointer to step-by-step instructions are in the review's "testing" section20:16
rick_h_katco: ah, sorry. I got an email on the PR but didn't see anything to the RB20:16
rick_h_natefinch: also please mark https://bugs.launchpad.net/juju-core/+bug/1604474 fix comitted20:28
mupBug #1604474: Juju 2.0-beta12  userdata execution fails on Windows <azure-provider> <ci> <juju2.0> <oil> <oil-2.0> <regression> <windows> <juju-core:In Progress by natefinch> <https://launchpad.net/bugs/1604474>20:28
natefinchrick_h_: done20:28
rick_h_ty20:29
rick_h_katco: while your branch is up for review can you peek at https://canonical.leankit.com/Boards/View/122969419/123586962 please?20:31
rick_h_katco: looks like it just didn't get landed due to conflicts that maybe you could resolve/get it landed?20:31
rick_h_katco: as a quick win for the b1420:31
katcorick_h_: it looks like it doesn't have a ship it20:32
rick_h_katco: oh nvm, reviews say needs lots of love. More than just conflicts.20:32
* rick_h_ follows bug to pr to RB to ... :)20:33
katcohehe20:33
natefinchman, the github CLI helper, hub, really makes testing other people's branches easy.  I can just hub checkout https://github.com/juju/juju/pull/5906  and run the code from a PR.20:47
rick_h_natefinch: <3 if you find a good pattern write it up please. I was using my old tricks from before with custom git aliases20:58
rick_h_natefinch: making it easier/faster to qa-prs is <320:58
rick_h_natefinch: https://github.com/juju/docs/blob/adc36a78430d84f5c6ab05b271eb3b700768ba40/README.md#using-git-aliases is what we used to use21:02
natefinchkatco, rick_h_: ran out of time to test that PR.  I have it all deployed.. but hit a weird situation while removing and re-adding.  Will continue looking at it after the kids are in bed21:04
=== natefinch is now known as natefinch-afk
mupBug #1608677 opened: Remove filesystem cache for charm archives <juju-core:New> <https://launchpad.net/bugs/1608677>21:04
rick_h_natefinch-afk: ty, have a good evening21:04
bdxhey whats up guys?21:26
bdxI'm having some issues with network spaces on aws ... I guess I'll write the list ... thought I would check here first21:27
bdxI have successfully bootstrapped into my aws vpc using `juju bootstrap mycloud aws --credential mycred --config vpc-id=my-vpcid --config force-vpc-id='true' --upload-tools`21:29
bdxand subsequently created a model on my aws controller in the vpc with `juju add-model my-new-model -credential mycred --config vpc-id=my-vpcid --config force-vpc-id='true'`21:30
bdxfollowing which, I add a network space, and then my subnet21:31
bdxhttp://paste.ubuntu.com/21814798/21:32
bdxok, everything looks great so far .... then I go and launch an instance and it all falls apart -> http://paste.ubuntu.com/21815678/21:34
bdxany ideas?21:34
bdxsent to the list21:46
mupBug #1608723 opened: "juju test" command needs updating for juju 2.0 <juju-core:New> <https://launchpad.net/bugs/1608723>22:47
menn0thumper: o/ http://reviews.vapour.ws/r/5346/22:57
menn0this makes that flaky MachineSuite test fail reliably as well as cleaning up a bunch of other stuff22:57
* thumper looks22:58
thumpershipit23:01
rick_h_bdx: can you confirm the rest of the story there? Did the bootstrap instance start on the right vpc you asked it to?23:13
rick_h_bdx: from the list it seems like you asked it to do the right thing, but I'm curious if juju did all the right things with the vpc for the controller, machines for the units deployed, etc.23:13
perrito666thumper: point me to your review23:43
perrito666thumper: the liiiiiiink23:59
thumperhttp://reviews.vapour.ws/r/5345/23:59

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