/srv/irclogs.ubuntu.com/2014/11/27/#juju-dev.txt

jw4Unit tests enhancement by monkey patching utils.NewUUID : PTAL http://reviews.vapour.ws/r/537/00:04
axwjogging with god, that's deep01:02
axwwallyworld_: get me to look at what?01:02
wallyworld_axw: thumper fixed it, the previous fix for tools symlinks broke upgrades from 1.2001:03
axwoh :/ I'll take a look to see what I did wrong...01:03
wallyworld_https://github.com/juju/juju/pull/122601:03
axwthanks01:03
wallyworld_axw: see comment on https://bugs.launchpad.net/bugs/139679201:04
mupBug #1396792: jujuc symlink creation broken on upgrade to 1.21.beta4 <regression> <tools> <juju-core:Triaged by waigani> <juju-core 1.21:Triaged> <https://launchpad.net/bugs/1396792>01:04
axwah, I see.01:05
=== menn0_ is now known as menn0
=== _thumper_ is now known as thumper
=== kadams54 is now known as kadams54-away
thumperwallyworld_: FWIW upgrading from 1.21 -> 1.22 has the tools with their full paths, so the current code works...02:17
wallyworld_ok, ta02:19
wallyworld_it's just 1.20 then02:19
thumperwallyworld_: just 1.2102:20
thumperwhich has landed02:20
thumperit was the 1.20 -> 1.21 upgrade that was buggered02:20
wallyworld_yeah, that's what i meant02:20
thumperso we should be good now02:21
wallyworld_great, ty02:21
thumperalthough I think I'd like some tests added to ensure that the symlinks are absolute rather than relative02:21
axwwallyworld_ anastasiamac_: FYI ericsnow is doing some work to do with exposing AZs to units. this will be relevant for zone constraints02:44
wallyworld_ok, ta02:44
axwas his work will require that we store the zone in state02:44
wallyworld_axw: is there a spec for his work do you know?02:45
axwwallyworld_: I think it's https://docs.google.com/a/canonical.com/document/d/1yVlzgKqfhKccUbm3WcZBq7WnzglBygknscr_I_u4bUg/edit#02:46
axwnot sure if there's anything else02:46
wallyworld_i'll read, thanks02:46
=== kadams54 is now known as kadams54-away
=== kadams54-away is now known as kadams54
menn0review for fix of bug 1396796 please: http://reviews.vapour.ws/r/538/03:08
mupBug #1396796: local provider all-machines.log has only machine-0 <local-provider> <regression> <juju-core:In Progress by menno.smits> <juju-core 1.21:In Progress by menno.smits> <https://launchpad.net/bugs/1396796>03:08
menn0thumper: ^^^03:08
=== kadams54 is now known as kadams54-away
* thumper looks03:10
=== kadams54-away is now known as kadams54
=== kadams54 is now known as kadams54-away
=== kadams54-away is now known as kadams54
=== kadams54 is now known as kadams54-away
menn0wallyworld_: I this today while investigating other issues: bug 139686204:04
mupBug #1396862: Panic when deploying to the bootstrap node <juju-core:New> <https://launchpad.net/bugs/1396862>04:04
wallyworld_oh joy04:05
menn0yeah, and I can't make it happen again04:05
wallyworld_Fix Committed :-)04:06
menn0wallyworld_: so that's how you guys get so many bugs fixed....04:07
wallyworld_:-D04:07
=== TheRealMue is now known as TheMue
wallyworld_axw: this wget certificate stuff is giving me the shits. i've tried several ways to get it to work. i even tried extracting the certificate from the running web server (in case I was using the wrong pem on disk) as per these instructions http://lnotestoself.blogspot.com.au/2011/02/wget-and-ssl-certificates.html04:10
wallyworld_so far i've had no luck without the --no-check-certificate04:11
axwwallyworld_: I'll have a play04:12
axwright after I have some lunch04:12
wallyworld_axw: ok. i had to generate the pem file as we weren't writing out the one for the state server from what i could see04:13
wallyworld_i could have cut and pasted from env-get also04:13
wallyworld_but writing it out on state server startup work, generating the same data as I got from using firefox to get the cert04:14
* wallyworld_ bbiab04:16
* thumper EODs04:30
menn0anyone seen "inconsistent definition for func" errors?04:31
menn0this happen during a merge attempt in Jenkins: http://paste.ubuntu.com/9261637/04:32
menn0i can't see how it relates to the branch04:32
axwwallyworld_: this works for me: juju get-env ca-cert > /tmp/ca-cert.pem && wget --ca-certificate=/tmp/ca-cert.pem https://localhost:17070/tools/1.22-alpha1.1-utopic-amd6404:37
axwmenn0: weird. looks like a tooling issue04:39
wallyworld_axw: that seems to work for me also, yet the cert extracted using firefox is different. no idea why05:11
wallyworld_axw: and confusingly, it's not the same cert that's in StateServingInfo05:19
axwwallyworld_: StateServingInfo has the server's certificate, not the CA certificate05:19
axwthe CA certificate signs the server cert; given the CA cert you can prove that the server is trusted by the CA05:20
wallyworld_that makes sense, i know little about ssl sadly05:20
wallyworld_axw: so to make it easy for wget i could do what rsyslog worker does and write out the cert when the state server starts up, or will that be a security issue?05:21
wallyworld_i guess that ingo is already in env-get so no05:22
axwwallyworld_: you shouldn't need to do that05:22
axwCA cert is in the env config, you should just write it out as a temporary file when you write the wget wrapper05:22
wallyworld_i didn't want to have to invoke juju get-env each time05:23
wallyworld_but i guess i coud05:23
axwwallyworld_: hold on, let me refresh my memory about your branch05:23
wallyworld_right now i generate the wget wrapper each time05:24
wallyworld_but i could do it once somewhere05:24
axwI think this is happening infrequently enough that that is fine05:24
wallyworld_ok05:26
axwwallyworld_: if you have the ImageURLGetter return a CA certificate as well, it can use that. the API client already has the CA certificate in memory05:26
wallyworld_yeah, i'll look into how to make it as nice as possible05:27
wallyworld_first gotta read the news - Phillip Hughes died :-(05:27
axwah :/05:28
=== kadams54 is now known as kadams54-away
TheMuemorning08:44
wallyworld_jamespage: you ok for juju status meeting in 10 minutes? hopefully you got the invite08:51
wallyworld_dimitern: can you take a look over bug 1395908? might be related to bug 1345433 which was fixed08:52
mupBug #1395908: LXC containers in pending state but no error message <lxc> <oil> <juju-core:Triaged> <https://launchpad.net/bugs/1395908>08:52
mupBug #1345433: cloud-init network error when using MAAS/juju <cloud-init> <juju> <maas> <network> <cloud-init:Confirmed for smoser> <juju-core:Fix Released by dimitern> <https://launchpad.net/bugs/1345433>08:52
wallyworld_lxc containers that came up got no ip address08:53
dimiternwallyworld_, sure, will have a look08:53
wallyworld_hence they stayed in pending08:53
wallyworld_ty08:53
dimiternwallyworld_, is it reproducible ?08:53
wallyworld_dimitern: not sure, happened obviously for that maas deployment, not sure if they tried again08:54
wallyworld_i would have said it was fixed, so might be something new08:54
dimiternwallyworld_, hmm.. I really like those bug reports :)08:54
wallyworld_yeah08:54
wallyworld_fwereade_: not sure if jamespage is around for juju status meeting08:57
fwereade_wallyworld_, it's 9am in the uk I think? worth waiting around a little08:58
wallyworld_rightio08:58
wallyworld_fwereade_: i won't log in to hangout until he pings back08:59
axwfwereade_: (how/where) does juju prevent upgrading a charm when metadata changes incompatibly between revs?09:02
fwereade_axw, inside state, we check for...09:02
fwereade_axw, active relations that have changed or disappeared09:02
fwereade_axw, change in subordinacy09:02
fwereade_axw, not sure about anything else09:03
axwfwereade_: cool, that's where I want to look. thanks09:03
fwereade_axw, Service.SetCharm(URL?) I think09:03
axwyep09:04
=== liam_ is now known as Guest33558
jamespagewallyworld_, just coming - was travelling09:12
wallyworld_sure, np09:12
wallyworld_gnuoy: are you around for a juju status meeting?09:14
gnuoywallyworld_, sorry, yes. Bad nights sleep last night and I'm still catching up09:15
wallyworld_gnuoy: np at all, we're running alte, just joined the hangout ourselves09:15
mattywmorning all09:32
* fwereade_ going for a walk and a think09:51
voidspacedimitern: I guess most of the Americans are off work today...10:01
dimiternvoidspace, thanksgiving? right10:03
dimiternjam, are you around?10:04
fwereade_dimitern, jam thanksgiving I think10:20
dimiternfwereade_, I suspected that, but then again - do they celebrate it in dubai ? :)10:24
fwereade_dimitern, probably not, but he does with his family (and I think he's marked as on holiday)10:24
dimiternfwereade_, right, I've missed that10:25
voidspacedimitern: ah, panic because I did "if err != nil" where I meant "if err == nil"...10:25
voidspacedimitern: :-)10:25
voidspacedimitern: so I was returning a nil subnet10:25
dimiternvoidspace, :) ah, there it is10:26
voidspacererunning the test...10:26
voidspacedimitern: and now the test I marked with "XXX this should fail"10:26
voidspacedimitern: now it fails10:26
dimiternvoidspace, \o/10:27
voidspacedimitern: so you were correct - the unique index *was* working, just failing silently10:27
dimiternvoidspace, good; sorry I didn't mention this earlier10:27
dimiternvoidspace, it took me a day to discover it initially :)10:28
voidspacedimitern: well remembered10:28
voidspacedimitern: sounds like a fun day10:28
dimiternvoidspace, to put it mildly, yeah :)10:28
voidspacedimitern: the bad news is that the non-unique provider id triggering the failure is ""10:31
voidspacedimitern: so I have to fix that...10:32
voidspacedimitern: trying sparse10:32
voidspacedimitern: currently indexes doesn't support sparse - I'll have to add it10:34
dimiternvoidspace, good to know unique indexes do not work with null values out of the box10:34
dimiternvoidspace, to mgo?10:34
voidspacedimitern: no, just our code10:34
voidspacedimitern: I have to add a new column to all the index definitions10:34
dimiternvoidspace, sparse: false ?10:35
voidspacedimitern: yep, effectively10:36
voidspacedimitern: I have to add a new sparse entry in the index struct - make it false for all the existing ones and true for the new one (subnetsC)10:36
voidspacedone it, just running the test now10:36
voidspacefield not column10:37
dimiternvoidspace, sgtm10:37
voidspacedimitern: works10:38
voidspacedimitern: I'll need to run the full test suite to make sure I didn't break anything else10:38
voidspaceI shouldn't have done though - I've just made what was the default before (Sparse: false) explicit10:38
voidspaceI'll run the whole test suite while I make coffee10:39
dimiternvoidspace, cheers10:40
=== mgz is now known as mgz_
wallyworld_jam: in an ec2 environment, i'm trying to connect to the state server using https. but the certificate common name set to "*" causes it to fail11:40
wallyworld_ERROR: certificate common name ''*'' doesn''t match requested host name ''10.73.184.8''11:40
wallyworld_why do we use "*", and is there a way around it do you know?11:40
mgzfwereade_: not that you don't have enough to deal with, but is it possible your last branch (uniter-extract-operations) borked upgrades?11:43
mgzfwereade_: http://reports.vapour.ws/releases/212011:44
fwereade_mgz, well, I'm certainly not going to say it's impossible, and I admit I didn't try an upgrade with the final version of the code11:47
fwereade_mgz, will investigate11:47
mgzfwereade_: thanks, given all the jobs failed I'm pretty certain upgrade is unhappy, but haven't tracked down the whys yet11:48
fwereade_mgz, I see this in the logs: machine-0: 2014-11-26 23:34:18 ERROR juju.worker.upgrader upgrader.go:157 failed to fetch tools from "http://juju-dist.s3.amazonaws.com/testing/tools/releases/juju-1.22-alpha1-precise-amd64.tgz": cannot unpack tools: tarball sha256 mismatch, expected 1d45ea821ab50e719df68d7a887449ff5f63f7d3347b665f2e522a99703a2577, got 00c0b630d88383bf0866a5a9846b1b945cac1936b88242a163d2d84a6eed83aa11:49
mgzright, that's what I'm looking at11:50
fwereade_mgz, I don't believe I touched any related code11:51
mgzthat feels like it should be a non-code issue... but it's only getting generated on the fly in the testing, so am not sure what's changed11:51
mgzit must be a fiddling in the simplestreams output in a change somewhere11:51
fwereade_mgz, if I'd broken it I'm 99% sure it would be the uniter failing to come up and complaining about an invalid state file11:51
fwereade_mgz, (which should not happen, and did not happen at least once, fwiw, but if you see it you should certainly blame me)11:52
mgzI'll file a bug with no specific blame and we can go from there, I'll also requeue the run to be sure11:52
fwereade_mgz, cheers11:52
mgzfwereade_: thanks11:52
mgzhttps://bugs.launchpad.net/juju-core/+bug/139698112:07
mupBug #1396981: Upgrade fails with tools sha mismatch <ci> <regression> <upgrade-juju> <juju-core:Triaged> <https://launchpad.net/bugs/1396981>12:07
dimiternmgz, rogpeppe1, if any of you is around, can you have a look at this MP for goamz please? https://code.launchpad.net/~dimitern/goamz/update-aws-api-version-to-latest/+merge/24305715:30
mgzdimitern: sure15:30
dimiternmgz, ta!15:31
rogpeppe1dimitern: i'm +1 in principle but i'd like assurance that that API version works across all the likely endpoints that people are using goamz for15:32
dimiternrogpeppe1, how do you suggest to guarantee that? :)15:32
rogpeppe1dimitern: as a start, i know that aws has different behaviour in different regions, so it would be good to test that that version is available in all aws regions15:32
rogpeppe1dimitern: i've no idea, but i believe that amazon isn't the only provider that people use goamz with15:33
dimiternrogpeppe1, the api version is the same for all regions15:33
rogpeppe1dimitern: ok15:33
rogpeppe1dimitern: i'd like to get it ok'd with gustavo before landing it.15:33
dimiternrogpeppe1, true, but also there are a lot of forks of goamz for various reasons - not keeping up with the api version changes for now15:34
dimiterns/for now/for one/15:34
dimiternrogpeppe1, sure, I intend to15:34
rogpeppe1dimitern: busy now, but i'll take a proper look at some point15:34
dimiternrogpeppe1, np, when you can, cheers15:35
mgzdimitern: have you tested this on a non-default-vpc account? I guess we're fine in that case the way amazon do their api versioning, but would be nce to check :)15:46
mgzactually changing juju to use this goamz ver has some risk, but the change in this codebase would be fine... the complication being that there are also github versions15:47
voidspacedimitern: ready for re-review: http://reviews.vapour.ws/r/534/15:53
dimiternvoidspace, looking15:54
dimiternmgz, I have no access to such an account15:55
dimiternmgz, but you're right - it should work just as well15:55
dimiternmgz, I did test it on a vpc-only region (us-east-1) though15:55
dimiternoh ffs! my full-stop key stopped working15:57
voidspacedimitern: restrict yourself to one sentence at a time and you'll be find15:57
voidspace*fine15:58
voidspacedimitern: and avoid attribute access / method calls15:58
dimiternvoidspace, :D and don't bother trying to type and struct fields15:58
dimiternyeah :)15:59
voidspace...15:59
voidspacesorry, just showing off15:59
mgzdimitern: reviewed16:00
dimiternmgz, thank you16:01
dimiternvoidspace, reviewed16:08
voidspacedimitern: thanks16:10
voidspacedimitern: CheckValid was jam's name16:10
voidspacedimitern: I'm happy with Validate - don't mind16:10
dimiternvoidspace, yeah, I'll leave the decision whether to rename it to you, I don't really mind16:11
voidspacedimitern: thanks!16:12
dimiternvoidspace, I do mind about adding isAliveDoc assert in EnsureDead though :)16:12
voidspacedimitern: fixing the other two, will make a decision on the third16:12
voidspacedimitern: done, just running tests...16:12
dimiternvoidspace, cheers16:12
voidspaceand then I'll merge the bugger16:12
dimiternvoidspace, as you were testing with goamz recently, would you like to have a look at my branch at some point? https://code.launchpad.net/~dimitern/goamz/update-aws-api-version-to-latest/+merge/24305716:13
voidspacedimitern: sure16:14
dimiternvoidspace, ta!16:14
voidspacedimitern: 1025 lines!!16:14
dimiternvoidspace, yeah, but most of them are simple renames16:15
voidspaceah right16:15
voidspace:-)16:15
voidspaceEOD18:20
voidspaceg'night18:20
hatchanyone here able to tell me where on the launchpad website I can find lp:charms/juju-gui ?18:44
hatchso confused by it haha18:44
hatchoop found it :)18:46
hatchask for help and you find it :D always happens haha18:46
=== kadams54 is now known as kadams54-away
=== kadams54 is now known as kadams54-away
waiganimenn0: I caught the tail end of what sounded like an interesting bug you hit?20:22
menn0waigani: the one that happened during a merge attempt?20:22
menn0http://paste.ubuntu.com/9261637/20:23
menn0it only happened once and the next attempt succeeded20:23
menn0and as mwhudson also found, Google has nothing on it20:23
waiganihmph, interesting20:24
=== benonsoftware is now known as \b
=== \b is now known as benonsoftware
waiganimenn0: adds env-uuids to statusDoc: http://reviews.vapour.ws/r/543/20:56
menn0waigani: sorry... I need to get this CI blocker sorted. I might be a little while.20:59
waiganimenn0: np20:59
=== mup_ is now known as mup
=== kadams54 is now known as kadams54-away
menn0This is the fix for the CI blocker: http://reviews.vapour.ws/r/544/23:17
menn0review please23:17
menn0thumper or wallyworld_: can you have a look at review 544 pls?23:31
wallyworld_sure23:31
menn0wallyworld_: thanks23:31
wallyworld_menn0: sorry, just got off phone, looking now23:40
menn0wallyworld_: np23:50
menn0wallyworld_: good catch re the error message. that needs to change with this change in behaviour23:53
wallyworld_yeah, great, i thought so23:53

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