/srv/irclogs.ubuntu.com/2016/10/26/#juju-dev.txt

menn0wallyworld: review done00:32
wallyworldthanks menno, will look soon00:32
axwwallyworld: I was planning to do a small fix for cinder in goose today, but won't be able to if we can't use the version changes in juju01:02
axwwallyworld: you think it's too close to be doing that?01:03
natefinchrick_h_: thanks for all the feedback01:21
thumperwallyworld: ffs01:39
thumperwallyworld: I'm trying to get controller details for a non-responsive controller01:39
thumperbut show-controller makes an api call01:39
thumperso doesn't return01:39
natefinch<sad trombone>01:49
natefinch..aww, I miss Dave01:49
menn0axw: prometheus review done. good stuff.01:55
axwmenn0: thanks!01:56
axwmenn0: develop seems buggered, I got a test failure to do with SCP commands which I went nowhere near01:56
menn0axw: yep, that's failing each run in CI too01:57
axwI'll dig a bit later01:57
menn0axw: I worked in that area a few months ago so I can try and take a look too.01:57
menn0axw: i'm bringing up the issue of develop being permanently cursed in the tech board meeting01:58
axwmenn0: dimiter made some changes last week to do with connecting to multiple addresses, most likely introduced then01:58
menn0axw: ah right. that seems likely.01:59
axwmenn0: ok, sounds good. it's been a week since it merged :/01:59
menn0axw: actually, I'll take a look now unless you really want it02:00
axwmenn0: nope, knock yourself out02:00
* thumper sighs02:02
thumpermenn0: got things to panic02:02
thumpernot sure if this is moving in the right direction02:02
menn0thumper: this is the cloud credentials handling?02:03
thumperyeah02:03
thumperoh ffs02:04
* thumper wants to slap whoever did the core/description part of cloud-credentials02:05
thumpermenn0: the cloud credentials are never set on the outgoing model02:05
thumpernow that I'm passing them in02:05
thumperit panics with "" not valid02:05
wallyworldthumper: show-controller does make an api call as it needs that info. list-controller doesn't. juju controllers --format yaml is what you want02:30
thumperthanks02:30
natefinchthat is uh... not very intuitive02:35
axwwallyworld: just want to check I understand your email to heather. are we not updating dependencies.tsv now, because we're too close to 2.0.1?04:11
axwwallyworld: I was hoping to get https://github.com/go-goose/goose/pull/26 in and updated in juju, but can't do it without the api version changes04:12
wallyworldaxw: i was refering to pull 25 which is the base work to support multi version endpoints. i didn't see the point in landing that just before 2.0.1 if there was no functional change04:13
wallyworldbut if you have a 26 which actually fixes something....04:13
axwwallyworld: yes, this will fix https://bugs.launchpad.net/juju/+bug/163664804:13
mupBug #1636648: cinder fails with badRequest... "Invalid input for field/attribute device" <openstack-provider> <storage> <juju:In Progress by axwalk> <https://launchpad.net/bugs/1636648>04:13
wallyworldthen that changes my thnking04:13
wallyworlddo you see where i was coming from?04:13
axwwallyworld: yes04:14
axwwallyworld: but I thought this was meant to go through OIL before we cut 2.0.104:14
axwso not sure on the timing04:14
wallyworldyeah, we will need to ensure that04:14
axwok04:14
wallyworldlet's land yours and we can track progress through OIL04:14
axwwallyworld: ok, sounds good. if you have a momen to review that, please do - it's pretty trivial04:15
* axw lunches04:15
wallyworldlooking now in fact :-)04:15
axwwallyworld: thanks04:46
wallyworldnp04:46
wallyworldaxw: oops (one liner) https://github.com/juju/juju/pull/650005:04
axwwallyworld: LGTM05:06
wallyworldta05:06
axwwallyworld: OK for me to land https://github.com/juju/juju/pull/6428 now then?05:07
axwthen I'll base my update on top05:07
wallyworldyep05:07
wallyworldif you are happy with it05:08
axwlooked fine, I'll take another look over05:08
axwwallyworld: actually hrm, looks like we're never assigning s.Openstack, so we'll leak HTTP servers05:11
axwwallyworld: I'll cherry-pick and fix that, and include in my PR05:11
wallyworldsgtm, ta05:11
axwwallyworld: bugger, seems there's an issue with keystone metadata source not working on heather's branch05:47
wallyworldoh, for the simple streams data05:48
wallyworldhmmm05:48
wallyworldyo think the issue is on the goose side?05:48
wallyworldor in that juju pr05:49
axwwallyworld: dunno, looking into it now05:49
wallyworldok05:49
axwwallyworld: it looks like the object-store check is not quite sufficient for product-streams05:55
axwwallyworld: what do you think about disabling the API version check if the user passes in "" for the version?05:55
axw(and just use whatever's in the service URL)05:56
axwservice catalogue05:56
wallyworldi *think* that's ok testing notwithstanding05:56
axwwallyworld: https://github.com/go-goose/goose/pull/27 please06:06
axwrunning a test in Juju now, so far so good06:06
axwit found the image metadata06:07
wallyworldaxw: sorry, quassel has stopped beeping at me, missed ping06:26
wallyworldaxw: lgtm with a suggestion. glad it was a simple fix06:29
axwwallyworld: last one I promise: https://github.com/juju/juju/pull/650106:44
wallyworldno worries06:44
axwwallyworld: first commit is heather's, unchanged06:45
wallyworldok06:45
wallyworldaxw: lgtm, suggested a test for the no device case06:47
menn0wallyworld or axw: here's the fix for the issue that's causing lots of failures in develop: https://github.com/juju/juju/pull/650207:09
axwmenn0: looking07:10
axwand thank you07:10
menn0just writing up QA steps07:10
wallyworldyay07:10
axwmenn0: LGTM, thanks07:15
menn0axw: cheers07:15
menn0wallyworld, axw: there's still other tests that are failing intermittently and need attention, but this gives us a better chance of a bless07:16
wallyworldyes, ty menno07:16
* wallyworld off to soccer, bbiab07:16
=== frankban|afk is now known as frankban
dimiternmacgreagoir: hey there07:52
macgreagoirdimitern: \o07:52
dimiternmacgreagoir: rick_h_ suggested we should pair on the ipv6 setup / stuff07:52
dimitern:)07:52
dimiternmacgreagoir: unless you are busy otherwise atm, ofc :)07:53
macgreagoirdimitern: Can I get back you about that? I'm just trying to finish part of it now.07:53
Mmikehi, lads. Where do I find wishlist for juju2 on launchpad?10:58
MmikeAnd ladies, hello!10:58
mgzhm, you mean as in features?11:01
deanmanIs ftp used as a protocol in any operation of juju ?11:07
mgznope11:07
deanmanI have enabled TRACE logging to see why controller running inside LXD fails to download image to deploy a charm on a seperate LXD. I get the following http://paste.ubuntu.com/23382970/. Information there doesn't help. What could i further look into? pcap?11:09
=== Guest59736 is now known as ahasenack
=== ahasenack is now known as Guest33774
mgzdeanman: do you actually have routing to streams.canonical.com from that machine?11:22
deanmanThe key differentiator is proxy, if not using proxy my setup works just fine, if using proxy the guest Xenial machine is able to download image and boot LXD container but not when deploying a charm.11:26
deanmanmgz: You mean whether this URL is resolvable?11:27
mgzyeah, in this case with the same proxy settings I guess11:27
mgzyou probably do need some lower level inspection to see whether the correct requests are being sent out11:28
deanmanmgz: I can manually curl that address and see content returned, i guess juju would have nagged anyway if it couldn't reach that URL, could it be that somehow proxy messes with content returned?11:28
mgzit's possible, but that log does look like juju nagging (at info level only)11:31
deanmanI can deduce from https://github.com/juju/juju/blob/juju-2.0.0/tools/lxdclient/client_image.go that it should have failed earlier if it was a network problem.11:38
=== petevg is now known as petevg_afk
=== Guest33774 is now known as ahasenack
=== ahasenack is now known as Guest66764
perrito666hey I suddenly have to run to the bank, bbl12:56
mgzvoidspace: do you have a mo to help me work out if the bug I have is in juju or maas?13:00
voidspacemgz: yes13:15
voidspacemgz: hangout or PM?13:15
mgzvoidspace: hangout?13:25
voidspacemgz: cool - I have 5 mins before 1:1 with rick_h_ though.13:25
mgzlets be fast13:25
voidspacekk13:25
voidspacemgz: see you in core13:25
voidspacemgz: right, 15 minutes until hangout - I'm digging out the maas docs13:46
* mgz hopes back in core13:47
mgzhops13:47
mgznot hopes13:47
voidspacemgz: hopes is better13:47
natefinchodd, somehow my calendar got set to tuesday as the first day of the week13:52
natefinchoh, no, it was on 7 days.. wacky13:53
rick_h_katco: voidspace dimitern dooferlad ping for standup14:00
dimiternomw14:02
dimiternmgz: it will be useful to dump the NIC params here e.g.: 2016-10-26 11:31:50 INFO Changed existing interface: node-71f20964-71e9-11e5-8007-525400c43ce5 eth014:02
dimitern 14:02
mgzdimitern: yeah, that could do with some more details14:03
dimiternmgz: I'm not sure maas can satisfy *both* a placement with --to and bootstrap spaces constraints14:13
dimiternmgz: if it happens some times but not others, the issue is probably hidden behind the log I pasted above14:14
mgzI expect --to overrides14:15
mgzanyway, the --to on bootstrap is fine14:15
mgzit's the bundle deploy which just uses constraints where things get interesting14:15
* rick_h_ heads home from coffee shop14:58
katcorick_h_: are you in the 2.0 retro?15:04
katcoballoons: hey i can't chat in bluejeans, but please don't hold back!16:00
balloonskatco, I think the feedback is most useful in the context of how to make it work at Canonical. I am +1 for doing it -- have been for some time16:05
katcodooferlad: +1 to all of that16:09
dooferladkatco: thanks16:09
katcoballoons: yes, i agree: it's paramount to confine the discussion to how it should work at canonical. that's a very agile concept anyhow :)16:17
=== frankban is now known as frankban|afk
=== Guest66764 is now known as ahasenack
=== ahasenack is now known as Guest21645
alexisbrick_h_, ping18:30
rick_h_alexisb: pong18:35
alexisbheya rick_h_redir has a request18:35
katcorick_h_: re. your request: what priority is this? should i stop working on everything else?18:41
rick_h_katco: yes please to unblock release18:41
katcorick_h_: ok, starting on this18:41
perrito666k, new isp about to connect me expect me to be unresponsive, apologies18:43
katcorick_h_: so wait... we want develop -> 2.0? we're skipping staging?18:49
rick_h_katco: yes, staging is a subset of develop at the moment. We need the updates in develop.18:52
katcorick_h_: can you hangout rq?18:52
rick_h_katco: sure thing, meet you in ?core18:53
thumperrogpeppe, katco: morning, I've been thinking more about the retry thing, and I'm starting to get your viewpoint around the error handling. As my wife knows, I can be very stubborn but hopefully teachable19:20
rogpeppethumper: hiya19:21
katcothumper: lol "strongly held beliefs loosely held" (thanks fwereade) is not a fault in my book19:21
thumperI just wanted to say so "in person"19:21
rogpeppethumper: thanks for continuing to think it through :)19:21
thumperrather than just email19:22
rogpeppethumper: that's appreciated19:22
thumperthere is a wonderful saying19:22
katcothumper: rogpeppe: and thank you to both of you for putting so much effort into getting this right19:22
thumperyou shouldn't hold on to a wrong idea just because you spent a long time making it19:22
katcothumper: yeah i immediately stole that one. not sure where he got it from19:22
* katco has a moment of silence to miss fwereade's contribution here19:23
rogpeppethumper: i like that - that is a good motto. apposite on many occasions i think.19:23
* thumper nods19:23
rogpeppethumper: shall we just go with gopkg.in/retry.v1 then? ISTM that it's something that deserves a larger scope than juju/19:24
rogpeppethumper: (i removed the dependency on utils/clock, so it should be usable by external projects that want to keep their dependencies tight)19:25
thumperI'd like to bring things together first, then think about where19:26
thumperI also think that we shouldn't depend on utils/clock19:26
thumperwe have always said that github.com/juju libraries aren't just for juju19:27
rogpeppethumper: i agree that those libraries aren't just for juju, but i think the name does imply some juju relatedness to people.19:28
thumperyeah...19:28
thumperthere is some squatter on the canonical name19:28
rogpeppethumper: and some parts are definitely juju-centric - the way that an import of juju/utils mucks with gobal variables in net/http means I'd never recommend that to anyone external, for example.19:29
thumperwhat in juju/utils mucks?19:29
rogpeppethumper: FWIW i think that snappy is already using the above import.19:29
rogpeppethumper: http.go:2019:30
thumperugh19:30
thumperthat seems like a wrong place19:31
rogpeppethumper: yes19:31
thumperlibraries that modify globabls of other packages is very antisocial19:31
rogpeppethumper: at least it doesn't make all HTTP connections drop after a single request like it did for years19:31
thumperpfft19:31
rogpeppethumper: https://bugs.launchpad.net/juju/+bug/149160819:32
mupBug #1491608: importing juju/utils should not side-effect http.DefaultTransport <tech-debt> <juju:Triaged> <juju-core:Won't Fix> <https://launchpad.net/bugs/1491608>19:32
mupBug #1636960 opened: Manual provider that spawns LXD containers <juju-core:New> <https://launchpad.net/bugs/1636960>19:32
natefinchrogpeppe: wow, that's bad19:39
katcoballoons: ping, can you sanity check? https://github.com/juju/juju/pull/650419:52
natefinchIt is surprisingly hard to get url.parse to actually reject things20:13
katcoso i'm on a new machine and having trouble building gh.com/lxc/lxd20:26
katcoi can't remember if i ever had to set anything up... it looks from the README.md that a go get github.com/lxc/lxd isn't expected to work?20:27
katcothere's additional libs required? can anyone confirm?20:27
natefinchkatco: from the makefile:20:29
natefinch# Must a few times due to go get race20:29
natefinch-go get -t -v -d ./...20:29
natefinch-go get -t -v -d ./...20:29
natefinch-go get -t -v -d ./...20:29
katconatefinch: i'm getting errors on includes of .h files20:30
katconatefinch: i highly suspect my setup, but i wanted some confirmation20:30
katconatefinch: you're able to just do a go get?20:30
natefinchkatco: I think I've built it before... possibly did some work to get there, I don't remember.  I presume you followed the steps in the readme?20:31
katconatefinch: nope, trying to avoid it20:31
natefinchha20:32
katconatefinch: i don't want bleeding-edge lxd on my system, and the ppa doesn't seem to separate out that from the build-deps?20:32
natefinchI dunno.... tych0?20:33
tych0i htink the ppa bundles everything, and the archive version uses the archive's deps20:35
tych0what's the problem, exactly?20:35
katcotych0: is it possible to use the ppa for building but not for installing a binary?20:35
tych0katco: sure, just build the binary but don't copy it to /usr/bin20:36
tych0or did i misunderstand?20:36
katcotych0: if i add the ppa, and do apt install lxc, won't it install the binary from the ppa?20:36
katcotych0: or upgrade for that matter20:36
tych0the `lxc` binary comes from lxd-client20:36
katcotych0: ah ok different ppa? ppa:ubuntu-lxc/lxd-git-master is safe for building?20:36
tych0katco: i don't even think you need the PPA to build20:37
tych0i think the archive versions of golang should be okay for every release20:37
katcotych0: you are correct, it was indeed the way i had things set up. thanks for confirmation! and ty natefinch20:46
tych0katco: sure, np20:46
katcoballoons: ping? ut?20:47
balloonskatco, yes20:47
balloonskatco, sure let me take a quick look20:47
katcoballoons: ta20:48
balloonskatco, the buildbot is going to ignore this -- personally I think if we're happy we can just merge it20:48
balloonsit's been tested once, and we'll give it a full run once it's merged20:48
balloons(it ignores pr's not going into develop)20:49
katcoballoons: ah ok20:49
katcoballoons: so i'll just merge and we can start a test run?20:49
balloonsyea20:49
balloonslooks like you have the proper commit20:50
katcoballoons: cool, merged.20:50
wallyworldkatco: sent you an email about 2.0.121:43
katcowallyworld: ok tal21:43
wallyworldthanks in advance :-)21:43
katcowallyworld: hth, shouldn't take long (WITH THE POWER OF EMACS!)21:43
wallyworldlol21:43
wallyworldkatco: emacs, making development great again21:44
katcowallyworld: lol21:44
* wallyworld prefers vi :-)21:45
katcowallyworld: no one is perfect21:45
wallyworldfunny, funny21:45
katcowallyworld: i thought you liked ides?21:45
wallyworldyeah i do, but when i need to do quick editing, vi is it21:45
katcowallyworld: for me it's emacs -nw -q21:45
wallyworldof course it is :-)21:46
katcowhich i have aliased to qmacs for "quick emacs" ;p21:46
katcowallyworld: so you can't cherry-pick merge-commits, so i'll have to try and CP all the commits for that merge21:52
wallyworldkatco: oh joy. i had a thought - just after that PR was another (test only fixes) to solve bad intermittent juju scp test fixes. might be worth taking that one so we get a good/better CI run21:53
katcowallyworld: was that the PR from menn0?21:53
wallyworldyeah21:54
wallyworldnext commited PR after the goose one21:54
katcowallyworld: the next commit i see is bebd627 which is from dooferlad21:55
katcowallyworld: i don't see anything from menn021:55
menn0katco: https://github.com/juju/juju/pull/650221:56
katcomenn0: ta... weird wonder why my local log is displaying something different21:56
katcomenn0: dang it, last fetch didn't go through21:57
katcomenn0: i see it now, ty21:57
menn0katco: cool. sorry was on a call.22:02
katcowallyworld: i lied earlier, i can cherry merge commits22:03
wallyworldawesome, i'm sure i have done it before so i was a bit confused but in a meeting so didn't really think too hard22:04
wallyworldkatco: so think you are cherry pick 2 commits right?22:04
katcowallyworld: cherry-pick -m 1 <commit>22:05
wallyworldyep22:05
wallyworldthat's what i use22:05
wallyworldso you are taking the goose commit and menno's22:05
katcowallyworld: correcy22:07
katcowallyworld: correct22:08
wallyworldtyvm22:08
katcowallyworld: menn0: sanity check please: https://github.com/juju/juju/pull/650522:16
wallyworldlooking22:16
menn0katco: looking22:16
alexisbanastasiamac, https://hangouts.google.com/hangouts/_/canonical.com/alexis-bruemmer22:17
wallyworldkatco: looks good ty22:17
alexisbwhen you ar eready22:17
katcomenn0: hit merge conflicts on yours22:17
katcomenn0: i think i did the correct thing22:17
menn0katco: LGTM22:18
katcoballoons: are you up on all this?22:18
katcowallyworld: menn0: tyvm22:18
wallyworldyes he is :-)22:18
wallyworldi raised it inthe release call22:19
wallyworldso it's my fault you needed to do this :-D22:19
katcowallyworld: ok :) will need a new test run balloons22:19
katcojust merged22:19
wallyworldwill will let the current run finish22:19
wallyworldand then look to the new one22:19
katcowallyworld: it was a hooge deal. terrible. and some commits i assume were good. but i make the best commits, everyone says so.22:20
wallyworldindeed :-)22:20
katcothat was my attempt at trump-speak if that wasn't evident =|22:20
perrito666axw: already up? I have a vsphere thing to ping pong with you22:43
axwperrito666: I'm here now23:08
perrito666axw: are you on bluejeans?23:11
mupBug #1636960 changed: Manual provider that spawns LXD containers <juju:Triaged> <https://launchpad.net/bugs/1636960>23:12
axwperrito666: I am now23:12
alexisbthumper, menn0, wallyworld ping23:16
mupBug #1636960 opened: Manual provider that spawns LXD containers <juju:Triaged> <https://launchpad.net/bugs/1636960>23:21
mupBug #1636960 changed: Manual provider that spawns LXD containers <juju:Triaged> <https://launchpad.net/bugs/1636960>23:30
wallyworldaxw: ping after your breakfast etc and we can talk to thumper about cred migrations?23:44
thumperI'm looking to head up the road to a cafe for a bit23:44
thumperwhich isn't good for calls or irc23:45
wallyworldok, can do it later23:45
thumperor now?23:45
thumperI really don't think there is a huge amount to talk about23:45
wallyworldi think axw needs to do school drop off etc23:45
axwthumper: now's fine for me. I just resent an email from june if you want to read that first23:45
axwnah, michelle's on leave so she'll take charlotte to school23:45
thumperwhat menn0 and I talked about yesterday was pack up the creds with the model, and error out if there are some that are on the controller already but differ23:46
wallyworldshould we join the standup hangout again?23:47
thumperok23:47

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