/srv/irclogs.ubuntu.com/2015/02/23/#juju-dev.txt

waiganithumper: http://reviews.vapour.ws/r/955/, thanks00:09
=== kadams54 is now known as kadams54-away
=== kadams54-away is now known as kadams54
=== kadams54 is now known as kadams54-away
katcowallyworld_: fyi i plan on attending the stand up tonight00:42
wallyworld_wot? it's the weekend :-)00:42
katcowallyworld_: WHAT! no one told me! ;p00:43
=== kadams54 is now known as kadams54-away
thumperanastasiamac_: review done on your mega branch01:14
thumperanastasiamac_: I didn't go through all of it as I wanted some response on what I had suggested first01:15
axwkatco: we lost you01:17
katcoaxw: doh01:17
anastasiamac_thumper: it's amazing! thnx :D i wasn't expecting u to have a chance to look but really appreciate u did!01:34
anastasiamac_thumper: sendError() on httpHandler only takes an error message02:17
thumperI know02:18
anastasiamac_thumper: i need both coe and message :D hence an additional sendError method that taeks and error instead of string02:18
thumperanastasiamac_: read my answer again, and if you still have questions, we should talk02:18
thumperbut OTP right now02:18
axwwallyworld_: I forget if I already sent you this link before, this is my WIP for the storage provisioner: https://github.com/axw/juju/tree/worker-storage02:27
wallyworld_thanks, will look after i finishing looking at the storage lifecycle branch02:28
axwwallyworld_: thanks. it doesn't need review, just an FYI02:28
wallyworld_kk02:29
=== kadams54 is now known as kadams54-away
wallyworld_axw: in type VolumeParams struct, should the Attachment field be of type *VolumeAttachmentParams rather than *AttachmentParams?03:46
axwwallyworld_: yeah, I think so03:46
wallyworld_ta, will fix03:46
axwI was thinking not before, but there may be volume-specific attachment params03:46
wallyworld_yeah03:47
wallyworld_axw: the path == "" check in rootfs createfilesystem - you say that the path value in FilesystemAttachmentParams should be set by the caller - that would imply it's up to the storage provisioner to set it. But the storage provisioner would then need to keep track of the ConfigStorageDir param. It is not easier to keep the code as is and let rootfs provider use a default path?03:57
axwwallyworld_: hold up, need to refresh my memory03:57
wallyworld_http://reviews.vapour.ws/r/956/03:57
axwwallyworld_: do you mean location==""?03:58
wallyworld_axw: i've change location to path03:58
axwwallyworld_: hmmm. so, this is a provider-independent thing. I think it's unnecessarily onerous to make it the responsibility of the provider to come up with a unique path to mount at04:00
axwwallyworld_: I think I'04:00
axwd prefer if we just had the storage provider set location to /var/lib/juju/storage/<source>/tag04:01
wallyworld_axw: that's what storageDir is for04:01
wallyworld_it's the only thing it is used for atm04:01
wallyworld_i guess it can be the absoluet tpath04:01
wallyworld_but then we'd not need the attachment params path attr04:02
axwwallyworld_: I don't see why the provisioner having to track ConfigStorageDir is a problem. in fact, it doesn't track it, it specifies it based on the agent's data-dir04:02
axwwallyworld_: yes we do, because the user can specify path04:02
axwwallyworld_: this storage-dir path should only be used if the user didn't specify one04:03
axws/user/charm author/04:03
wallyworld_right, ok. then perhaps the config attr should be called DefaultStorageDir04:03
axwwallyworld_: actually I don't think that config attr should come into it04:04
axwwallyworld_: that directory was meant to be for storing info about volumes and filesystems04:05
axwwallyworld_: not for where the mount points end up04:05
axwwallyworld_: e.g. in the loop provider, we should create the file in ConfigStorageDir, but then mount it at "location"04:05
wallyworld_ok, so we need to ensure then that Path is always set, and the provider needs to choose a sensible location if the user doesn't specify one04:07
wallyworld_provisioner04:07
axwright04:07
wallyworld_np. the way i've had it till now is how we had id in the feature branch04:08
wallyworld_but that was all done for the demo04:08
menn0axw: hiya. in environs/cloudinit/cloudinit.go:AddAptCommands, do you know why required packages aren't being installed when enable-os-refresh-update is false04:39
menn0?04:39
menn0axw: it means that rsyslogd is completely broken when enable-os-refresh-update is turned off.04:39
axwmenn0: I think it's because some of the packages may not be found, because they're in cloud-tools04:40
menn0axw: i feared that there might be some complication like that.04:41
axwmenn0: rsyslogd-gnutls should always be found I think ... we could be more selective there04:41
menn0axw: yeah rsyslog-gnutls is needed in order for logging to work04:41
axwwallyworld_: were there specific tests you found missing in my branch, apart from the existing tests you mentioned?04:46
axwwallyworld_: I've identified a problem with the RemoteStorageInstance test, and I ended up removing that method04:46
axwsince I don't think it should be possible to remove a storage instance directly, only destroy & remove attachments which will do the job04:47
wallyworld_axw: mainly just tests to check that the business logic rules that cause the txns to retry when things are done concurrently04:47
axwwallyworld_: ok04:47
wallyworld_do you know the bits i mean?04:47
wallyworld_around checking for attachment count > 0 (or = 0) etc04:48
wallyworld_can't recall specifics04:48
axwwallyworld_: yup04:48
wallyworld_menn0: if you set refresh to false, you are promising to juju that your image has everything needed04:49
wallyworld_that's why it's true by default04:49
menn0wallyworld_: i'm doing scale tests with just the ubuntu charm so I figured it would be safe to turn off to speed things up04:50
menn0wallyworld_: i'll turn it back on for now04:50
wallyworld_ah, i see04:50
wallyworld_menn0: what series are you using? precise? trusty?04:51
menn0wallyworld_: this is trusty on EC204:51
wallyworld_hmmm, i would have hoped trusty would be ok with false04:51
wallyworld_:-(04:51
menn0wallyworld_: i'm not convinced the logic in juju is quite right. as axw also indicates, there are probably some package that should always get installed even if an apt-get update isn't being done.04:52
wallyworld_yeah, that is true04:53
wallyworld_it's a blunt instrument right at the moment04:53
axwwallyworld_: most of these tests can't be done concurrently, because of lack of support for shared storage. e.g. in theory you could add unit, then concurrently add another unit and remove the first one's attachment, ensuring the instance isn't removed04:56
axwbut we can't do the in theory bit yet04:56
axwI'll do what I can, but I don't think it's much04:56
wallyworld_ok, just wanted to be really sure that if itwere possible to test the bad cases, we could04:56
wallyworld_axw: there's 2 choices with the Path passed to rootfs provider already existing - 1. be really paranoid and assume that another storage is already using that path and return an error (will revisit later when shared storage supported), or 2. see if dir is empty and return an error if not04:57
wallyworld_or 3. - something else?04:58
wallyworld_eg just let it slide, but i don't think we should do that04:58
axwwallyworld_: we should be guaranteeing #1 is not a possibility, I think04:58
axwin which case, #204:59
wallyworld_axw: that's the provisioner's job yeah04:59
wallyworld_hence i was also leaning to#204:59
axwwallyworld_: partially, we also need to prevent multiple storage instances with the same path04:59
axwstorage attachments*04:59
axwi.e. explicitly specified path in the charm metadata05:00
wallyworld_yes, but not inside rootfs05:00
axwyep05:00
axwI mean at a level higher than the provisioner, and also in the provisioner05:00
wallyworld_yeah05:00
wallyworld_i'll add the empty check to rootfs05:00
wallyworld_axw: you going to land the disktag rename pr?05:01
axwwallyworld_: I'll do it later, I'd like to get the state one fixed first05:01
wallyworld_ok05:02
axwwallyworld_: it's just a sed to fix up05:02
wallyworld_my rootfs one depends on it also, but it can wait05:02
axwwallyworld_: it depends on my branch?05:03
axwwallyworld_: can you just land using DiskTag, and I'll fix up later?05:04
wallyworld_axw: i added filesystemtag on top of your volumetag branch in juju/names05:04
axwah, right05:04
axwwallyworld_: I'll land it now then05:04
wallyworld_it's no biggies, i can wait05:04
wallyworld_i'm still fixing the rootfs branch05:04
axwwallyworld_: merged05:24
wallyworld_axw: awesome, ty05:24
=== Guest33721 is now known as benonsoftware
wallyworld_axw: http://reviews.vapour.ws/r/956/ has been updated; once you are free from your other stuff, would be great if you could look. i have to duck out to do school pickup, so bbiab05:58
axwwallyworld_: thanks, just pushing changes to http://reviews.vapour.ws/r/969/ too05:58
wallyworld_ok, will look as soon as i'm back05:58
=== Murali_ is now known as Murali
jamthumper: did you manage to get the rework from executive summary through the rest of the JES CLI document?06:25
thumperjam: yes06:25
* thumper shouldn't be here06:25
=== benonsoftware is now known as Guest48867
=== Guest48867 is now known as benonsoftware
axwwallyworld_: ;) I can rename it to "TestCDSIRSARI" ? ;)06:53
wallyworld_yeah, why not :-)   .... kidding :-)06:53
dimiternhey wallyworld_, thanks for that cloud-image-utils fix for precise07:09
wallyworld_dimitern: hi there, and thanks for landing07:10
wallyworld_dimitern: did you see the little govet issue in provisioner.gp?07:10
dimiternwallyworld_, np07:10
dimiternwallyworld_, mm no?07:10
wallyworld_checking: go vet ...07:10
wallyworld_apiserver/provisioner/provisioner.go:1028: arg addr for printf verb %q of wrong type: *github.com/juju/juju/state.IPAddress07:10
wallyworld_was introduced yesterday i think07:11
dimiternwallyworld_, cheers, I'll have a look07:11
wallyworld_there's about 6 of them07:11
jamwallyworld_: sabdfl just gave a +1 on storage CLI07:11
jamand liked your layout07:11
wallyworld_i hope we can release 1.22 finally :-)07:11
wallyworld_jam: that's awesome news, ty07:11
wallyworld_none of us liked the CLI n the spec07:12
jamhe really liked using the "juju status --help" syntax to understand how it all looks07:12
wallyworld_yeah, that approach was done with the juju metadata stuff for tools and images07:12
jamwallyworld_: can you do the edits to the actual Storage Spec ?07:13
wallyworld_jam: yep, can do, just need to check i have dit permissions07:13
wallyworld_edit07:13
jamI think I gave you edit for the last round07:14
wallyworld_yep, just checked07:14
dimiternwallyworld_, where have you seen these go vet warnings? I've run go vet ./... on trunk now and 99% of what I get is "using unkeyed fields"07:17
anastasiamacjam, wallyworld_: thnx for nice storage CLI approval :D07:18
wallyworld_dimitern: i basically saw it when i pushed my branch, after rebasing. it is a legit error - %q expects a string07:18
jamanastasiamac: wallyworld_: thanks for a nice proposal :)07:18
dimiternwallyworld_, and in that case you've mentioned "%q" for a *state.IPAddress is fine, because it has String() method07:18
anastasiamacjam: wallyworld_and axw proposed:D i just had fun with 3 level commands :)07:19
dimiternso it's not a string but it's a Stringer07:19
wallyworld_dimitern: so, it's the pre push hook that is invoking go vet - perhaps there's a warning level that's different?07:19
dimiternwallyworld_, I'm not sure I have that pre-push hook configured locally - what should I do?07:20
wallyworld_dimitern: the pre-push hook should be in VCS07:20
wallyworld_in the scripts dir07:21
dimiternwallyworld_, ok, but do you enabled it somehow?07:21
dimiternenable*07:21
axwanastasiamac: cool :)07:21
jamdimitern: you have to edit .git/config07:22
wallyworld_yeah, i can't remember exactly - it should be on the contributing page07:22
anastasiamacwallyworld_: u know how i could not get my PR to have RB request?07:22
dimiternjam, wallyworld_ , ok, will do07:22
anastasiamacwallyworld_: and i had to resolve conflicts and re-propose?07:22
wallyworld_which branch?07:22
anastasiamacwallyworld_: well, i have just pushed to the same PR and found that there are conflicts again... The RB is not updated tho :(07:23
anastasiamacwallyworld_: block-cmd-update...07:23
wallyworld_967?07:23
anastasiamacwallyworld_: https://github.com/juju/juju/pull/163807:23
anastasiamacyes07:23
wallyworld_if rb isnot updating, let eric know and he might be able to help fix it07:24
anastasiamacwallyworld_: and btw, u r no longer on PM07:24
wallyworld_i am on free node07:24
wallyworld_i can also see channels on canonical server07:25
anastasiamacwallyworld_: couple of minutes ago, i was told u were not ;D07:25
axwjam: thanks for fixing my blunder in liveSource07:26
wallyworld_anastasiamac: go figure, i have no idea wtf is happening with my stupid network07:26
anastasiamacwallyworld_: u r putting it under a lot of pressure? :D07:27
wallyworld_not that i know of :-)07:28
wallyworld_anastasiamac: so i'd fix your merge conflicts and repush to GH and see if that then allows rb to updates07:33
anastasiamacwallyworld_: thnx :D will try, just one more push... :D07:34
jammenn0: scale testing sometime ?08:15
TheMuemorning08:19
menn0jam: i've been doing exactly that08:27
jammenn0: k, I don't think I'd seen anything back from you, so just figured I'd give you a poke and see how its goin08:27
menn0jam: i08:28
menn0jam: i was going to contact you when I had some numbers. I have numbers for the logging to mongodb case but logging via rsyslog has been giving me problems.08:29
=== ChanServ changed the topic of #juju-dev to: https://juju.ubuntu.com | On-call reviewer: see calendar | Open critical bugs: none
perrito666restore is finally landed09:49
* perrito666 cries09:49
dimiternperrito666, \o/09:50
perrito666now its no longer just my problem :p09:51
axwperrito666: epic :)10:16
anastasiamacdimitern: o/11:18
dimiternanastasiamac, hey there11:19
anastasiamacdimitern: got trunk to resolve my conflicts and am getting test failures in apiserver/provisioner/prepare_test.go11:19
anastasiamacdimitern: how r u?11:19
dimiternanastasiamac, oh, let me have a look?11:20
dimiternanastasiamac, I'm better than last week :) - no blockers11:20
anastasiamacdimitern: ""cannot allocate addresses: dummy.NetworkInterfaces is broken"11:20
anastasiamacline138/28811:20
anastasiamac:D11:20
dimiternanastasiamac, hmm I mean can you paste the failures?11:21
dimiternanastasiamac, to see the context11:21
anastasiamacdimitern: http://pastebin.ubuntu.com/10370778/11:22
anastasiamacdimitern: really appreiate ur help :D11:22
anastasiamacappreciate even11:22
dimiternanastasiamac, right, have you merged the changes to the dummy provider?11:22
anastasiamacdimitern: ?11:23
anastasiamacdimitern: i got the trunk, created my branch and applied my changes11:23
anastasiamacdimitern: running tests, gives me this error :D11:23
anastasiamacdimitern: what m i missing?11:23
anastasiamacdimitern: :)11:23
dimiternanastasiamac, let me have a look..11:23
anastasiamacdimitern: do u want the link to PR?11:24
dimiternanastasiamac, my tests on trunk are passing, so maybe you haven't merged all changes?11:24
dimiternanastasiamac, yes please11:24
anastasiamacdimitern: https://github.com/juju/juju/pull/165211:24
anastasiamacdimitern: RB http://reviews.vapour.ws/r/979/11:24
dimiternanastasiamac, looking11:26
anastasiamacdimitern: tyvm11:26
TheMuedimitern: ping11:27
dimiternanastasiamac, well, istm you haven't pulled everything in there from master - e.g. the apiserver/provisioner changes I made recently (#1646) require some changes in state (#1631) and the dummy provider (#1632)11:27
dimiternanastasiamac, these are probably missing (the last one for sure), hence the error11:28
dimiternTheMue, pong11:28
anastasiamacdimitern: k. thnx :D i'll double check!11:28
dimiternanastasiamac, i suspect you've merged master partially11:30
TheMuedimitern: stumbled over the usage of network.Address in state. once we use it directly, so w/o any tags, the other time we have already a local substitute with a tag giving the Scope field the name networkscope11:30
dimiternTheMue, can you give me examples?11:30
dooferladdimitern: hangout?11:31
dimiterndooferlad, in 5m pls11:31
TheMuedimitern: yep, look at addmachine.go:45 and address.go:18311:32
dooferladdimitern: sure.11:32
TheMuedimitern: similar with hostPort in address.go few line later too11:32
dimiternTheMue, I'll get back to you after my 1:1 with dooferlad, if that's ok11:33
TheMuedimitern: sure, thx11:34
anastasiamacdimitern: contacted on PM11:54
dimiternanastasiamac, sure, in a meeting will get back to you shortly11:57
* TheMue is out for lunch12:16
jamdimitern: have you seen fwereade today?13:09
dimiternjam, no, but I've heard he's having connection issues13:10
jamk13:10
jw4OCR PTAL http://reviews.vapour.ws/r/981/  <-- fixes ./scripts/verify.bash pre-push hook warnings when using newer versions of go14:23
perrito666jw4: I would have expected the formatting function to call strig14:27
perrito666string14:27
jw4perrito666: agreed, but it seems %q isn't handled quite the same as %s14:28
perrito666jw4: wellI would assume it is, since the output of calling that actually works14:29
jw4perrito666: and I think it works, it's just that newer versions of go vet warn about it14:29
perrito666It sounds to me that govet is broken14:29
jw4perrito666: possibly14:29
perrito666ma windows takes a lot to download14:29
perrito666man*14:29
perrito666sinzui: ping me when you have a moment :) I am about to make you the happiest QA person in the world14:31
sinzuiperrito666, really? you can tell me my our maas 1.5, 1.7, and 1.8  hate juju 1.22 and 1.2314:33
perrito666sinzui: no but, I just merged the last bit of restore :)14:34
sinzuiperrito666, NOoooooo......14:34
sinzuiperrito666, the restore tests have been very reliable this year. Why complete a feature to risk a failure?14:35
sinzuiperrito666, sorry about the sarcasm14:35
perrito666sinzui: I have not nuked the old restore14:35
perrito666:p14:35
dimiternjw4, ping14:37
perrito666sinzui: I need to discuss with you how to do this, I could just make the old restore call the new one with parameters that make it do the same, but output might change14:37
dimiternjw4, yeah - go vet seems amiss here14:38
jw4dimitern: pong14:38
sinzuiperrito666, we do need old restore to call new restore because enterprises may have scripted juju 1.18 restore.14:38
jw4dimitern: from previous experience I doubt we'll get go vet fixed if it is a bug in go vet, and anyone using go1.4+ won't be able to get past the pre-push hook14:39
sinzuiperrito666, as for the output change, I don't think there is a contract for the output.14:39
perrito666sinzui: ack, btw, there are a couple of new test cases to add :p sorry14:39
sinzuiperrito666, if CI fails because it scanned output, then we will change ci14:39
jw4dimitern: fwiw I'm about to propose a tweak to verify.bash that allows ignoring go vet with an envar14:39
sinzuiperrito666, old restore only need to work its way. New cases can require the new restore14:40
perrito666sinzui: yes, the new cases are for the new restore, just sending a heads up14:41
perrito666will send you an email with details14:41
dimiternjw4, sure, sgtm, I'll have a look at your PR as well14:42
jw4dimitern: ta14:42
* dimitern needs to start using go1.414:42
perrito666dimitern: I am14:42
sinzuithank you very much perrito666 . the email will be helpful as I am not fully caffeinated.14:42
perrito666sinzui: I am on that process, I assisted a wedding on saturday and my liver seems to be off for repairs14:43
sinzui:)14:43
jw4dimitern: http://reviews.vapour.ws/r/982/ <-- allows environment variable to bypass go vet failing the pre-push hook14:45
jw4perrito666, OCR ^14:45
dimiternjw4, already approved :)14:45
jw4dimitern: thanks!14:46
jw4:)14:46
dimiternjw4, hmmm you know what? looking at your last PR gave me an idea14:47
jw4dimitern: yeah?14:47
dimiternjw4, it seems we're not adding Logf to the list of printfuncs in verify.bash, and it does seem 1.4 go vet reports only them, but not e.g. logger.Tracef one, which have %q as well14:48
jw4dimitern: I think Logf is automatically included and we have to explicitly add the others...14:48
dimiternjw4, ah, no - my mistake, go vet did report logger.Infof("assigned address %q to container %q", addr, container) for example14:49
dimiternjw4, so it's that14:49
dimiternit's not that even14:49
dimiternjw4, LGTM on the other one14:52
jw4tx dimitern14:55
dimiternnp14:56
=== ChanServ changed the topic of #juju-dev to: https://juju.ubuntu.com | On-call reviewer: see calendar | Open critical bugs: 1424669
sinzuidimitern, natefinch can you find someone to look into a gccgo problem in bug 142466915:02
mupBug #1424669: ppc64el fail multiple definitions of juju_juju_api_networker.NewState <ci> <gccgo> <ppc64el> <regression> <test-failure> <juju-core:Triaged> <https://launchpad.net/bugs/1424669>15:02
dimiternsinzui, I was looking at this since saturday15:02
dimiternsinzui, it's another random ppc64 gccgo compiler issue15:02
dimiternsinzui, like the last blocker15:02
dimiternsinzui, I've requested access to a ppc64 machine to all me to run tests and analyze whether building a more recent version of gccgo will resolve this15:03
sinzuidimitern, It is consistent since those the commits, the ppc suite are not unreliable15:03
sinzuidimitern, I will get you access to stilson-09 in a few minutes15:04
dimiternsinzui, yes, but it only happens on ppc64, right?15:04
dimiternsinzui, ah, great!15:04
sinzuidimitern, yes, but the duplicate definition issue was identified to be a gccgo problem. Test need to be written a certain way to ensure the real impl isn't seen by the compiler15:05
dimiternsinzui, I can usually reproduce such gccgo issues easily on my trusty amd64 machine with gccgo, but not this one15:05
dimiternsinzui, hmm is there some document or instructions on this?15:06
sinzuidimitern, I don't know, jam and wallyworld fixed the bugs in the past15:07
sinzuidimitern, I think the issue related to passing interfaces vs passing strings15:08
dimiternsinzui, ok, I'll look into that15:08
aisraelnatefinch: Have you had a chance to look more into bug 1424069?15:16
mupBug #1424069: juju resolve doesn't recognize error state <juju-core:New> <https://launchpad.net/bugs/1424069>15:16
natefinchaisrael: no, sorry, haven't put any more time into it.  I'll spend some time trying to figure it out today.15:18
aisraelnatefinch: much appreciated. It's a bit of a blocker for me, unfortunately15:18
=== whit_ is now known as Guest39130
sinzuinatefinch, dimitern : The QA team are looking for advice regarding bug 1424695. We don't know if this issue is juju or CI. We are happy to change CI to make juju pass15:55
mupBug #1424695: cloud-init cannot download agent from state-server <ci> <maas-provider> <network> <regression> <trusty> <juju-core:Triaged> <https://launchpad.net/bugs/1424695>15:55
dimiternsinzui, I'll look into it15:56
sinzuidimitern, we are working on getting logs, so the bug is reported from my memory15:56
dimiternsinzui, yeah logs will be good to have - I've been monitoring those failing maas jobs on 1.22 for a few days15:57
sinzuidimitern, if John's fix isn't ready for the next run, we will intercept machine-1 before it is destroyed15:58
dimiternsinzui, great, please let me know15:58
alexisbnatefinch, do you think that https://bugs.launchpad.net/juju-core/+bug/1424069 may be a result of current LE work?16:15
mupBug #1424069: juju resolve doesn't recognize error state <juju-core:New> <https://launchpad.net/bugs/1424069>16:15
ericsnowwwitzel3: http://reviews.vapour.ws/r/983/16:16
natefinchalexisb: no idea, I'm working on investigating it now.  At least it's easily reproducible16:26
alexisbnatefinch, ack16:26
wwitzel3ericsnow: will take a look16:29
ericsnowwwitzel3: ta16:29
TheMuedimitern: beside the types to move we also have methods on State responsible those types (would be nice to move them too) and which use the package private network related docs, like ipaddressDoc16:37
dimiternTheMue, that sounds like a good idea, but can it be done?16:44
dimiternTheMue, i.e. state.State can't have methods defined on it in a subpackage16:44
TheMuedimitern: I'm currently checking, many depending places16:45
TheMuedimitern: yes, otherwise the new sub-package has to provide functionality used by those functions16:46
TheMuedimitern: like e.g. State.AddIPAddress()16:46
TheMuedimitern: a wonderful candidate16:46
TheMue:)16:46
dimiternTheMue, yeah :)16:47
TheMuedimitern: thinking of a NetworkState type in the network package, returned by State and containing all needed data16:48
TheMuedimitern: and then move the methods to this type16:48
dimiternTheMue, experiment a bit and I'll see what you've pushed later16:49
TheMuedimitern: so nwst := st.NetworkState() and nwst.AddIPAddress()16:49
TheMuedimitern: will do16:49
=== kadams54 is now known as kadams54-away
dimiternTheMue, can I get an LGTM on this fix for the blocker bug please? http://reviews.vapour.ws/r/986/17:44
dimiternnatefinch, perrito666, ^^17:46
dimiternkatco, ^^17:48
dooferladTheMue / dimitern: could you take a look at this monster diff? https://github.com/juju/juju/pull/1657 Two whole lines!17:48
dimiterndooferlad, looking :)17:49
dimiterndooferlad, reviewed17:51
dooferladdimitern: thanks!17:52
dooferladdimitern: Doh! Good point!17:52
dimiterndooferlad, ;) np17:53
dimiternericsnow, ping17:53
ericsnowdimitern: hey17:53
dimiternericsnow, hey :) I'm glad someone is around - can I get an LGTM on this fix for the blocker bug please? http://reviews.vapour.ws/r/986/17:54
* ericsnow takes a look17:54
dimiternericsnow, thanks!17:54
ericsnowdimitern: done17:57
dimiternericsnow, cheers17:57
mbruzekahoy abentley!  We are seeing an error on the automated testing system.  Can you take a look?  http://juju-ci.vapour.ws:8080/job/charm-bundle-test-azure/46/console18:03
mbruzekabentley: I see a few error messages in there that I don't know how to resolve.18:04
mbruzekERROR cannot assign unit "kubernetes/0" to machine 2: unit placement is not supported with availability-sets-enabled18:04
=== kadams54-away is now known as kadams54
=== kadams54 is now known as kadams54-away
dimiternmbruzek, AFAIK azure does not support --to for deployments18:08
mbruzekdimitern: OK I thought it was just a setting in the azure setup that we could change.18:08
dimiternmbruzek, when you have "availabiliy-sets-enabled: true" set18:08
mbruzekdimitern: could we set that to false?18:09
mbruzekor do we need that?18:09
dimiternmbruzek, I suppose if you haven't heard of it you don't need it :)18:09
dimiternmbruzek, it's an env.yaml setting18:09
dimiternmbruzek, but it's enabled by default for azure18:10
mbruzekdimitern: We want to be able to pass on Azure, and our bundle needs some co-located charms for it to work.18:11
mbruzekdimitern: So are you saying that if we set that to false, Azure will not honor the --to command?18:12
dimiternmbruzek, so then try adding "availability-sets-enabled: false" in your env.yaml for that environment and bootstrap again (it can't be changed after bootstrap)18:12
dimiternmbruzek, I'm saying the opposite :)18:12
mbruzekdimitern: OK.  This is the automated CI system I need abentley or sinzui to help with that.18:13
dimiternmbruzek, if it's false --to is supported18:13
dimiternmbruzek, +118:13
mbruzekdimitern: thanks for your help18:13
dimiternmbruzek, np18:14
dimiternsinzui, one blocker down, one to go - but I'm at eod for some time now, so somebody else perhaps; getting logs from the machine (1,2) will definitely help there18:18
sinzuidimitern, I am almost to it.18:24
dimiternsinzui, great! I hope it works18:24
=== kadams54-away is now known as kadams54
sinzuinatefinch, abentley : we need to discuss bug 142477719:15
mupBug #1424777: local-provider precise failed to upgrade <ci> <local-provider> <precise> <regression> <upgrade-juju> <juju-ci-tools:New> <juju-core:Triaged> <https://launchpad.net/bugs/1424777>19:15
abentleysinzui: sure thing.  The usual hangout?19:16
sinzuiabentley, sure19:16
natefinchsinzui: sure, the release meeting hangout?19:18
=== kadams54 is now known as kadams54-away
thumperrick_h_: around?21:19
rick_h_thumper: yep21:19
rick_h_for a bit, what's up?21:19
=== kadams54-away is now known as kadams54
alexisbwallyworld, I will be back after bed time if you need to talk with me re releases23:14
=== kadams54 is now known as kadams54-away
wallyworldthumper: got 5?23:38
wallyworldsinzui: apt fix merged into 1.2223:44
beisnercurious - is juju bootstrapping an enviro with   default-series:  vivid   defined in the environments.yaml expected to work at this point in the cycle?  i keep getting bootstrap timeouts attempting to do so.23:50
sinzuithank you wallyworld23:50
beisner^ ERROR failed to bootstrap environment: waited for 10m0s without being able to connect: Permission denied (publickey).23:53
beisner^ setting to default-series:  utopic   or  trusty succeeds.23:53
beisnerif it's a no-op atm, please tell me to stop trying for now  ;-)23:54
wallyworldbeisner: i thought it worked. sinzui, you have seen vivid work?23:55
sinzuibeisner, it certainly work in our local provider tests.23:56
beisnersinzui, wallyworld - fyi that's using the openstack provider and the maas provider.23:57
sinzuibeisner, are you using ppc openstack and vivid?23:57
beisnerall amd64 here23:58
beisnerfor this one23:58
sinzuibeisner, from my experience new ubuntu series do work, but you make need to create the images. We have built and published the agents (except for ppc64el)23:59

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