/srv/irclogs.ubuntu.com/2016/03/22/#juju-dev.txt

rogpeppeericsnow: i've pushed $$merge$$ - perhaps you could take it over if there are some silly errors from the bot?00:00
rogpeppeericsnow: indeed00:00
ericsnowrogpeppe: sure00:00
rogpeppeericsnow: but this should actually work reasonably for most use cases00:00
rogpeppeericsnow: thanks00:00
urulamaty, ericsnow00:00
* rogpeppe beds headward00:00
ericsnowrogpeppe: np00:00
rogpeppeg'night all00:00
mupBug #1560262 opened: relation visibility rules different between service/service and service/subordinate relations <canonical-is> <juju-core:New> <PostgreSQL Charm:Triaged> <postgresql (Juju Charms Collection):Triaged> <https://launchpad.net/bugs/1560262>00:32
wallyworldaxw: with the restore issue - reading the code leads me to believe the issue is because we no longer store admin-secret in bootstrap config. we could add it back but wouldn't it be better to store it in the backup metadata?01:02
axwwallyworld: admin-secret is the password for admin@local, should just grab it out of accounts.yaml instead01:02
wallyworldah, yes, good point01:02
wallyworldthat does ite the restore to the user who did the backup01:03
axwwallyworld: (it would be better to pull something out of backup metadata tho)01:03
wallyworldagreed01:03
wallyworldi may fix the easy way for now01:03
axwwallyworld: +101:03
wallyworldi'd rather the backup be stand alone01:03
axwwallyworld: needs an overhaul, so yeah01:03
wallyworldaxw: joy, we also need ca-cert which is fine, but also ca-private-key which we don't currently store with the controller metadata01:22
axwwallyworld: what do we need that for?01:23
axwI thought we would generate new certs01:23
wallyworldi'll have to look deeper - the current code attempts to create an env config and it complains01:24
davecheneythumper: cherylj https://github.com/juju/juju/pull/483601:26
wallyworldaxw: atm we don't prepare or anything - the code just assumes we had a complete config available via bootstrap config, so it needs to be reworked01:26
anastasiamacwallyworld: axw: last change for openstack virt type http://reviews.vapour.ws/r/4276/01:27
axwanastasiamac: looking01:27
anastasiamacaxw: tyvm \o/01:28
axwwallyworld: yeah, it does assume we have a bootstrap config already. it always did, so the only potential issue is to do with info we may have thrown away, like ca-private-key01:28
axwwallyworld: only immediate problem I mean01:28
wallyworldaxw: yeah, not having the same ca-private-key seems problematic i think?01:29
wallyworldmaybe not01:29
axwwallyworld: sorry, I don't know what's newly generated, and what needs to be the same01:30
wallyworldi guess if we are rebootstrapping, there's no existing machine sout there01:30
wallyworldso no need for existing agents to be able to reconnect01:30
wallyworldwhich i think would be the main issue01:31
axwwallyworld: part of the restore process involves SSHing to the agents and fixing them up, IIANM01:31
axwwallyworld: the controller's going to have a new IP, after all01:32
wallyworldright but doesn't the restore assume everything is gone if we use the -b option01:32
wallyworldmaybe not01:32
wallyworldi'll have to read the code01:33
axwanastasiamac: just a couple of comment changes, otherwise LGTM01:51
anastasiamacaxw: awesome \o/ thnx :D01:51
axwanastasiamac: BTW, "code is kinda weird" is not your fault, just how it is / has to be due to openstack limitation01:51
axwwallyworld: you said there was another bug about permission denied? I don't see it on LP against admin-controller-model. did you say you still had to add it?01:55
wallyworldaxw: it's just not targetted to that branch, let me get it01:55
wallyworldhttps://launchpad.net/bugs/146156101:55
mupBug #1461561: juju run fails with "Permission denied (publickey)" on manual provider <intermittent-failure> <manual-provider> <run> <juju-core:Incomplete> <juju-core 1.24:Triaged> <https://launchpad.net/bugs/1461561>01:56
axwta01:56
mupBug #1441302 changed: Vivid unit tests are not reliable enough <test-failure> <vivid> <juju-core:Won't Fix> <https://launchpad.net/bugs/1441302>01:56
axwwallyworld: I suspect it's just a race with the worker to update authorized_keys01:58
wallyworldsounds likely yeah01:58
mupBug #1496032 changed: backups restore won't create bootstrap on GCE <backup-restore> <docteam> <gce-provider> <juju-core:Triaged> <https://launchpad.net/bugs/1496032>02:11
wallyworldaxw: reviewed02:36
axwwallyworld: thanks02:36
axwwallyworld: please see reply about type name03:01
wallyworldok03:01
wallyworldaxw: no reply yet, did you publish?03:02
axwwallyworld: derp. published now03:03
wallyworldaxw: GenericResource I think is ok, but I'm not strong;y opinionated either way03:05
axwwallyworld: in that case I'll leave it. if it becomes a pattern, we should strongly consider changing the framework to allow injecting things directly03:05
wallyworldsgtm03:06
axwwallyworld: do you want me to wait before merging to admin-controller-model?03:06
axwwallyworld: wait for CI to be a bit happier?03:06
wallyworldmay as well merge now, what could possibly go wrong :-)03:06
wallyworldwe should have a run in say 5 hours03:07
wallyworldstill time to fix stuff before tomorrow and better to find issues early at this stage03:07
wallyworldwe could merge the other 2 fixes first i guess03:07
cheryljgah, I see the problem with the joyent provider03:46
cheryljthey're ignoring the tags passed into startinstance03:46
anastasiamaccherylj: :D03:46
davecheney  thumper mwhudson https://github.com/golang/go/issues/1490403:47
davecheneyi'm detouring into this bug this arvo03:48
davecheneycmd/juju/status trips it up 100% of the time03:48
wallyworldaxw: this backup thing is messy - apart from old bootstrap config, the only place we store the admin-secret aka oldpassword is in the agent.conf file, and the same is true for the ca-private-key. So assuming we have that file, which the backup does include in the archive, we can parse it on restore, but i think it would be better to do that stuff server side and include in the json metadata for the backup. but that sucks because03:56
wallyworldnothing so far in apiserver needs to know about agentconfig and i'd rather keep it that way. so it's all a bit yuck03:56
wallyworldaxw: maybe we should just include the private key and admin secret in bootstrap config03:58
axwwallyworld: it would be nice not to, but I guess we could as a stop gap at least03:59
wallyworldaxw: yeah, i don't really want to. so what would we prefer - parsing the conf files on restore? or extracting the info srver side on backup04:00
axwwallyworld: we definitely can't generate new ones at restore time?04:00
wallyworldi'm not sure. the private key i think needs to be paired with the cacert04:00
axwwallyworld: yeah, we would need to update that too.04:01
thumperdavecheney: interesting...04:01
wallyworldaxw: and a new admin secret would not be the end of the world i guess. but we do have all the info in the backup archive04:02
wallyworldso we'd really need to look to use that info if availab;e04:02
axwwallyworld: for a quick solution, I'd go with parsing agent.conf on restore. then we can look at improving.04:06
axwwallyworld: we also need to think about how we're going to persist user auth details to log back into the controller after restore, in lieu of an accounts.yaml that's valid with the creds at time of backup04:07
axwwallyworld: options are to store in the backup metadata, or to require the user to login. but these are things we could look into at the same time as the rest04:08
wallyworldok, was thinking the same thing, also we only need this if we are rebootstrapping04:09
axwwallyworld: I can't make heads or tails of the SSH thing. the system key is added to authorized-keys at bootstrap, and I've confirmed they're in the hosted model config; and that is present in cloud-config when machines are started04:10
axwso there's no race AFAICT04:10
wallyworldfark04:10
* axw disappears to make lunch04:10
wallyworldaxw: i think adding admin-secret to bootstrap config is ok. it's used for the gui login having it accessible like that makes sense to me05:28
axwwallyworld: I don't understand why we need it there. it's already in accounts.yaml.05:29
axwwallyworld: admin-secret == password for admin@local05:29
wallyworldah true05:29
wallyworldthere's not a nice way that avoids yuk to get it server side, so i'll read from there for now05:30
wallyworldparsing the agent conf files client side is also no go05:30
wallyworldso i'll get that ingo srrver side and include with backup metadata05:30
axwwallyworld: why is it no go?05:31
wallyworldall the code to open the archive and extract the files is all server side. there's a lot of it. and there's no guarantee what you'll get in terms of file names - it depends on machine id.05:32
wallyworldi'd have to refactor a lot of code05:32
axwwallyworld: ok05:32
wallyworldrestore works by uploading the archive to state server and server side code does the work05:33
wallyworldso none of that code is client side05:33
axwwallyworld: ah, I think I figured out when the race could happen with authorized-keys. if you specify it explicitly in --config05:34
axwwallyworld: that'll go into hosted model config, whereas admin model's config will have the system's public key added05:35
* axw works on a patch05:35
wallyworldah i see05:35
wallyworldbbiab, school pickup05:35
anastasiamacaxw: wallyworld: tag fixes for joyent http://reviews.vapour.ws/r/4282/06:10
axwanastasiamac: is there some significance to the "tag." prefix?06:23
axwanastasiamac: do we need to add that to the prepopulated tags?06:23
anastasiamacaxw: I am fixing the broken code. what ever was there or whatever it was copied from, i've kept06:23
anastasiamacaxw: but i would have to say that probably not..06:24
axwanastasiamac: I guess if you tested and it worked, there's no significance. alrighty06:24
axwanastasiamac: LGTM, thanks06:26
anastasiamacaxw: thnx. I'll re-google tag names in joyent but m pretty it's an arbitrary prefix :D06:27
mupBug #1560331 opened: juju-br0 fails to be up when no gateway is set on interface <juju-core:New> <https://launchpad.net/bugs/1560331>06:48
jamaxw: are you around for a bit? I'd like to figure out what we need for storage and LXD containers07:12
jamespecially for normal deployments (maas/ec2) but also for LXD provider.07:13
axwjam: sorry, didn't see notification - not on usual machine (desktop's on the fritz)07:22
axwjam: still around for a while07:22
jamaxw: so the LXC code has something about passing through the loop device07:23
jamand I need to know what we actually use, and how we need to do that with LXD07:23
jamAIUI if we want a device, then we talk directly to LXD and ask it to pass a device through (might need to be in the lxd profile, I need to investigate that side)07:23
jamaxw: but I'd like to know what we're using and how I can measure that I've done it correctly.07:24
axwjam: the existing LXC storage code wasn't great. it was simply allowing access to all loop devices to all containers07:25
jamaxw: who was creating the loop device to then pass in?07:25
axwjam: the container does it internally. they have access to the /dev/loopX files07:25
axwjam: (it would be much better if the host passed it in)07:26
jamaxw: but who on the outside would create the fil to then mount?07:27
jamyou'd ask the container broker ?07:27
axwjam: there's several places where we can create storage: dynamically on the machine (e.g. loop); dynamically on the controller (e.g. ebs); at machine provisioning time (e.g. maas)07:28
axwjam: the last one might fit with creating in the container broker07:28
axwjam: but it would be nice if there's some way we could do it dynamically through the LXD API07:29
jamaxw: LXD would let us pass the device through (supposedly), but it wouldn't do the mkfs, etc. stuff07:34
jamI don't believe07:35
axwjam: all we need is a device. juju will create filesystems as necessary07:40
axwjam: does LXD have an API for allocating block devices? I don't see it in the REST API doc, but that may just be outdated07:40
axwwallyworld: https://github.com/juju/juju/pull/484407:41
wallyworldlooking07:41
wallyworldaxw: what if the user has explictly set their auth keys for the hosted model07:44
axwwallyworld: how?07:45
wallyworldah, i was thinking we had a capability for host model config on cli07:45
axwwallyworld: you can only specify the config for both admin and hosted model07:45
wallyworldone day we should add that separate host model config perhaps07:46
wallyworldaxw: lgtm07:46
=== menn0 is now known as menn0-afk
wallyworldaxw: off to soccer, restore tests are a bitch, need to refactor a bit, will finish when i get back and hopefully land tonight08:05
axwwallyworld: okey dokey08:05
wallyworldaxw: i'm so not happy with the solution, but there's little other option right now08:06
wallyworldthat we can do quickly08:06
axwjam: so thinking about it a bit more, if we were to pass disks through to the containers, I think we'd need to do that in two places: an environment-level storage provider for the lxd provider; and a machine-level storage provider for lxd-type containers08:09
axwjam: for hte latter, we'd need to update machine-level storage to manage disks for containers as well as disks for the machine itself08:10
axwjam: AFAICT there's no help from LXD for dynamically allocating volumes from ZFS or BTRS for use in containers (apart from root volume), which would be ideal08:11
axwat worst we could do what we were doing with LXC: open up loop devices to all containers and let them have at them. not ideal tho08:11
axwjam: I'd prefer if we could allocate any volumes the host supports to containers, though08:12
TheMueaaargh, building a shell script generating a shell script but then doing wrong escaping. wonderful, costed me some time.08:26
TheMuemorning btw08:27
mupBug #1560391 opened: apt-mirror is not used in containers with MAAS provider <juju-core:New> <https://launchpad.net/bugs/1560391>09:30
fwereadedon't suppose anyone who did backup/restore work is online?09:48
fwereadebecause apparently I have perpetrated lp:1559712 and I need a bit of context about why this is an error09:51
fwereadeperrito666, ericsnow perhaps? ^09:52
perrito666Well i am having breakfast and chatting from a phone09:54
perrito666But shoot09:54
perrito666Fwereade^09:54
perrito666In short, prepare restore puts juju in a pseudo ro mode for that iirc, it writes info in mongo so I presume that is what is failing hence the transaction aborted10:00
voidspacedooferlad: standup?10:02
perrito666Fwereade gtg need to take my wife to work + not feeling so well10:09
mupBug #1560428 opened: cmd/juju/common depends on gopkg/check.v1 <juju-core:New> <https://launchpad.net/bugs/1560428>10:31
mupBug #1560428 changed: cmd/juju/common depends on gopkg/check.v1 <juju-core:New> <https://launchpad.net/bugs/1560428>10:43
fwereadeperrito666, np, let me know if/when you're available10:46
fwereadeagain :)10:46
mupBug #1560428 opened: cmd/juju/common depends on gopkg/check.v1 <juju-core:New> <https://launchpad.net/bugs/1560428>10:49
dimiternfwereade, davecheney, FYI - updated the pprof wiki page with instructions how to access it from a browser11:41
dimiternawesome tool btw11:41
dimiternhttps://github.com/juju/juju/wiki/pprof-facility11:41
mupBug #1560457 opened: help text for juju bootstrap needs improving <juju-core:New> <https://launchpad.net/bugs/1560457>12:13
wallyworldaxw: not sure if you have a moment to look at http://reviews.vapour.ws/r/4287/12:25
wallyworldaxw: you may need to hold your nose, but we need to re-write backup/restore somewhat for multi-model12:26
rick_h_wallyworld: hold on that though. I want to talk through that and what's in 2.0 vs 2.1 and how we go about that please12:30
wallyworldrick_h_: oh, no fear, we have no much for anything right now :-)12:30
wallyworlds/much/time12:31
rick_h_wallyworld: k12:31
wallyworldrick_h_: what we need to do to get 2.0 out the door given how the cloud credentials stuff now removes the need for storing an entire bootstrap config means that there's a few things we need to clean up for 2.112:32
rick_h_wallyworld: rgr12:33
marcoceppikatco natefinch how does deploying a charm with resources local work?13:17
marcoceppilike, what's the command line work like?13:18
marcoceppis/work/look/13:18
natefinchmarcoceppi: yep13:18
natefinchmarcoceppi: juju help deploy ;)13:18
natefinchmarcoceppi: uju deploy foo --resource bar=/some/file.tgz --resource baz=./docs/cfg.xml13:19
natefinchmarcoceppi: you just specify the resources you want to have uploaded when you deploy the charm.   You can do that for store resources too13:19
marcoceppinatefinch: I DON'T HAVE TIME FOR YOUR REASONABLE SUGGESTIONS13:20
natefinchmarcoceppi: er, store charms that is13:20
natefinchmarcoceppi: lol13:20
marcoceppinatefinch: yeah, but I'm waiting for charmstore update, etc13:20
natefinchmarcoceppi: right, just wanted to clarify that it's not just for local charms13:20
marcoceppinatefinch: yeah, lookikng forward to that, but I've got a charm now with resources13:20
marcoceppinatefinch: I know there's a filename param, etc13:21
marcoceppinatefinch: and I remember that it's mostly ignored13:21
natefinchmarcoceppi: the extension of the file you upload needs to match the extension of the filename in the charm metadata, it'll complain if you don't have them match13:22
marcoceppinatefinch: okay, one file is literally `python-jujusvg` with no ext13:22
marcoceppiwhat happens then?13:22
marcoceppias in, it's just a binary13:23
natefinchmarcoceppi: then we enforce that what you upload doesn't have an extension13:23
marcoceppicool13:23
marcoceppi<313:23
natefinchmarcoceppi: yeah, the filename in the metadata is really just used for the extension for uploads, and its' what we store the file data as on the units... though since we return the full path from resource-get, it shouldn't matter too much to the charm itself13:24
marcoceppinatefinch: perfect, I remember that being shook out in capetown happy to see it implemented as such13:24
natefinchmarcoceppi: yeah, I'm mostly really happy with how the feature turned out.  I think it'll be super useful.13:25
marcoceppinatefinch: a few days ago the instance containing svg.juju.solutions died, so I'm charming that up with resources. I'll let you know how it goes13:26
natefinchmarcoceppi: cool cool.  Please let us know any pain points or other sharp edges13:26
mupBug #1560487 opened: local provider fails to create lxc container from template <juju-core:New> <https://launchpad.net/bugs/1560487>13:43
rogpeppedoes anyone know the difference between jujuclient.Controller.Servers and jujuclient.Controller.APIEndpoints ?13:45
rogpeppeit's a new type, so *presumably* they both have a role to play, but in my limited experimentation they both hold exactly the same thing.13:46
rogpeppefwereade, dimitern, natefinch, axw, wallyworld: ^13:47
natefinchrogpeppe: no idea, I haven't used that package13:48
rogpeppenatefinch: np13:48
wallyworldrogpeppe: they mean the same as when used with the jenv stuff - servers are the host names, api endpoints the ip addresses13:48
rogpeppewallyworld: what's the difference? both seem to contain host names and port numbers13:48
rogpeppewallyworld: ah, you mean one has resolved IP addresss + port and the other has the equivalent hostnames ?13:49
wallyworldrogpeppe: as i understand it, if the hostnames are known they are used13:49
wallyworldyes13:49
rogpeppewallyworld: that is *really* confusing!13:49
rogpeppewallyworld: just the naming really13:49
wallyworldrogpeppe: that stuff goes back a looong time13:49
natefinchwe're really bad at naming13:49
rogpeppewallyworld: it's a new type!13:49
wallyworldwhich we ported13:49
wallyworldcopied even13:49
rogpeppewallyworld: hrmph13:49
wallyworldwe didn't want to change the semantics13:50
wallyworldas people would have groked the meaning b now13:50
rogpeppewallyworld: the semantics perhaps didn't need to change, but at least it could be documented that a) they both hold host:port pairs and b) one is resolved and the other isn't13:50
natefinchwallyworld: ...or not :)13:50
wallyworldrogpeppe: it was probaly around your time on juju when this was first done :-)13:50
rogpeppewallyworld: i still can't work out which is resolved and which isn't13:51
wallyworldit's as documented now as it evr was13:51
natefinchwallyworld: you're just digging your hole deeper13:51
wallyworldwhy?13:51
wallyworldwe can't change the world in a day13:51
natefinchwallyworld: if it's been the same for forever and it's been this well documented for forever, that just means it's bad and should be fixed.13:51
rogpeppewallyworld: just about everything else has changed about that stuff13:51
wallyworldyes it should, let me get 25 hours in a day and i'll do it13:52
rogpeppewallyworld: so i'd've thought that aspect could probably change too13:52
wallyworldnot everything has changed, that bit hasn't :-)13:52
rogpeppehow is anyone meant to guess what the intended relationship between these two things?13:53
rogpeppe// Servers contains the addresses of hosts that form the Juju controller cluster.13:53
rogpeppe// APIEndpoints is the collection of API endpoints running in this controller.13:53
rogpeppethey sound very different from one another.13:53
wallyworldguess so, that doc was as we founf it13:53
wallyworlda lot of this stuff has been ported, not reimplemented13:53
wallyworldyou don't need to understand everything to the nth degree if you are porting it13:54
rogpeppewallyworld: it's actually got worse since the original, which was this: http://paste.ubuntu.com/15472129/13:54
rogpeppewallyworld: that at least explained the situation13:55
wallyworldwe should copy that doc back across13:55
rogpeppewallyworld: +113:55
rogpeppewallyworld: BTW both the field names *and* the comments have changed from the original13:57
wallyworldyes, can't recall why now, may have been to comply with the spec we were given actually13:58
katcomorning all14:01
katcowallyworld: evening o/14:01
wallyworldhi14:01
katcowallyworld: doing ok?14:01
wallyworldno, life sucks14:01
cheryljsomeone's not living the dream today :/14:02
katcowallyworld: understandable atm... but the sun will rise on a better day :)14:02
* wallyworld is grumpy14:02
katcowallyworld: anything i can do to help?14:02
wallyworldwave a magic wand and solve all the merge conflicts and test failures14:02
rogpeppewallyworld: sorry if i excacerbated the situation :)14:02
wallyworldrogpeppe: np, i have copied the text across already14:02
katcowallyworld: i can work on merges today if you'd like14:02
rogpeppewallyworld: thanks14:02
rogpeppewallyworld: changing the names would be good too. Something like: APIEndpoints and ResolvedAPIEndpoints ?14:03
wallyworldkatco: thanks, bu there's a lot of ingrained knowledge needed to resolve the conlficts14:03
dooferladfrobware: is there no maas call today?14:03
wallyworldrogpeppe: i'll see what we can do14:03
katcowallyworld: ok. well, lmk14:03
wallyworldwill do14:04
wallyworldkatco: you could review http://reviews.vapour.ws/r/4287/ - it's a temporary fix to get restore working, still a long way to go to fix it all properly14:04
katcowallyworld: tal14:04
rogpeppewallyworld: thanks, that's appreciated14:05
cheryljrogpeppe: is your 070-use-charmstore-v5-api branch something we need to get into the beta this week?14:06
rogpeppecherylj: no, sorry, it was a temporary hack to get a PR in a mutually dependent repo landed. I can delete it now.14:07
cheryljrogpeppe: ah, ok, thanks!14:07
rogpeppecherylj: deleted14:08
cheryljthanks!14:08
rogpeppecherylj: and BTW if you ever need to land something in a repository which both depends on an is depended on by juju-core, that's the way to do it and still have godeps work.14:09
rogpeppes/on an is/on and is/14:09
cheryljgood to know.  Thanks :)14:09
ericsnowrogpeppe: FYI, that merge went through on the first try :)14:09
rogpeppeericsnow: yeah, i saw in the morning and was happy14:09
ericsnow:)14:09
rogpeppeericsnow: thanks for keeping an eye on it14:10
ericsnowrogpeppe: np14:10
katcorogpeppe: hey thanks for working so hard to land that patch :) work continues i guess?14:11
fwereadekatco, if you know backup: ISTM that creating a RestoreInfoSetter should not create a document; or, if it should, that it ought to handle concurrent creations of the same type14:11
fwereadekatco, is there some synchronisation mechanism somewhere in restore that I've missed?14:11
katcofwereade: eh? is this regarding wallyworld's patch?14:11
rogpeppekatco: i'm leaving as is for the time being, as i have to make progress in other areas.14:12
fwereadekatco, not at all I'm afraid14:12
rogpeppekatco: it should be sufficient for most purposes. there may be some bugs :)14:12
katcofwereade: ah. i don't know hardly anything about backup/restore i'm afraid :( but ericsnow and natefinch should14:12
fwereadekatco, np, thanks14:12
katcofwereade: i'm reviewing a patch now, so maybe i'll become an expert in the next bit of time ;)14:13
wallyworldi don't recall doing anything with RestoreInfSetter in my patch?14:13
rogpeppekatco: hope that's ok. did you have any other particularly pressing things?14:13
fwereadeericsnow, natefinch: ^ ? (RestoreInfoSetter)14:13
ericsnowrogpeppe, katco: there are a few things left (like sending the channel through the AddCharm API endpoint)14:13
fwereadewallyworld, nothing to do with you14:13
natefinchperrito666: is Mr. Restore14:13
rogpeppeericsnow: file a bug14:13
ericsnowfwereade: perrito666 did all the work on restore, though I did review much of the work14:13
fwereadewallyworld, my problem entirely :)14:13
katcorogpeppe: the only horse i have in that race is natefinch being able to determine what channel a charm was deployed from14:13
wallyworld\o/14:13
ericsnowrogpeppe: k14:13
* katco head explodes from the different threads of convo14:14
fwereadeperrito666, last I heard, was not feeling well14:14
natefinchgah.. people talk about sending channels around in go and I get all confused14:14
ericsnowfwereade: :(14:14
fwereadeericsnow, I will do my best and ping you for a review14:14
ericsnowfwereade: sounds good; thanks!14:14
ericsnownatefinch: ha14:15
katcoericsnow: ty14:15
rogpeppekatco: yeah, currently it'll probably always use stable.14:15
perrito666fwereade: I am back fully restored14:16
perrito666fwereade: how can I help you?14:16
ericsnowrogpeppe, katco: we still need to sort out the long-lived macaroon situation, but I don't think it's as urgent14:16
tvansteenburghwhere can i find a list of all the valid Request types for each api facade?14:18
ericsnowtvansteenburgh: I don't believe that is cataloged anywhere, meaning you have to read through the facades code and apiserver/params/*.go14:19
natefinchtvansteenburgh: yeah, unfortunately, what ericsnow said.  We really should document all that stuff.14:20
ericsnowtvansteenburgh: godoc might help make it more manageable though14:20
bogdanteleagado we have something like this for stderr? https://github.com/juju/cmd/blob/master/output.go#L15714:20
tvansteenburghericsnow, natefinch: ok, well if i know the name of the facade, where in the source should i go to find the Request types for it?14:21
ericsnowtvansteenburgh: all the API data types are lumped together in several files in apiserver/params.go14:21
ericsnowtvansteenburgh: so you have to see what types are in the API methods and then find them in those params files14:22
fwereadeperrito666, so the heart of it is that creating a RestoreInfoSetter will sometimes fail14:22
perrito666it being the bug you mentioned early today?14:23
fwereadeperrito666, which is presumably happening now because of timing changes in my branch14:23
fwereadeperrito666, yeah14:23
fwereadeperrito666, and I am worried that just fixing the current symptom is going to leave us unstable all the same14:23
perrito666fwereade: see priv msg14:24
dimiternwallyworld, ping14:28
wallyworldyo14:28
dimiternhey, I've found a storage issue with bootstrapping on maas now14:28
dimiternhttp://paste.ubuntu.com/15472360/14:28
dimiternlooks like related to removing provider storage ?14:29
tvansteenburghericsnow: i don't see a apiserver/params.go?14:29
ericsnowtvansteenburgh: apiserver/params/params.go14:30
ericsnow(in core)14:30
wallyworlddimitern: maas still has provider storage14:31
wallyworldit can't use tags the same way as the other providers, so we still need ito14:31
wallyworldare you referring to this line? DEBUG juju.provider.common state.go:36 putting "provider-state" to bootstrap storage *maas.maasStorage14:32
dimiternwallyworld, nope - it fails like joyent - see the paste above - just after it connected once client logins were unblocked14:32
perrito666dimitern: didnt remove storage in maas14:33
perrito666oh wallyworld said that already14:33
wallyworlddimitern: so why do you think it's a storage issue?14:33
dimiternoh :/ I was hoping it was something known.. I'll keep digging14:33
wallyworlddimitern: cherylj had a tgeory14:34
wallyworldtheory14:34
wallyworldcan't recall exactly what now14:34
dimiternwallyworld, joyent precise jobs failed very similarly and intermittently and it was due to missing tags14:34
* cherylj reads backscroll14:34
perrito666dimitern: joyent iirc, was a porblem with tags not being populated as fast as they where requested14:34
cheryljperrito666 - the problem with joyent was that they were not setting tags passed in as StartInstance args14:35
perrito666ah, I got it wrong then :)14:35
cheryljperrito666: no worries, I didn't discover that until adding in the logic to retry the api connection in bootstrap if the error was "not bootstrapped"14:36
cheryljI saw that even after several minutes, we never got tagged instances14:36
cheryljso, I had to look elsewhere :)14:36
dimiternperrito666, cherylj, wallyworld, false alarm, I found out why it fails - not related to storage14:36
cheryljdimitern: ah, good :)14:37
mupBug #1560511 opened: The AddCharmWithAuthorization API endpoint needs to respect channels. <juju-core:New> <https://launchpad.net/bugs/1560511>14:40
mupBug #1560520 opened: Charm channels must be used on the controller. <juju-core:New> <https://launchpad.net/bugs/1560520>14:52
mupBug #1560520 changed: Charm channels must be used on the controller. <juju-core:New> <https://launchpad.net/bugs/1560520>15:01
mupBug # opened: 1560520, 1560525, 1560527, 156053115:10
mupBug #1560525 changed: Juju 2.0-beta3 stabilization  <blocker> <juju-core:Triaged> <https://launchpad.net/bugs/1560525>15:13
mupBug #1560527 changed: juju get should be able to take a key argument <juju-core:New> <https://launchpad.net/bugs/1560527>15:14
mupBug #1560531 changed: Charm store macaroons must be used on the controller. <juju-core:New> <https://launchpad.net/bugs/1560531>15:14
mupBug #1560525 opened: Juju 2.0-beta3 stabilization  <blocker> <juju-core:Triaged> <https://launchpad.net/bugs/1560525>15:23
mupBug #1560527 opened: juju get should be able to take a key argument <juju-core:New> <https://launchpad.net/bugs/1560527>15:23
mupBug #1560531 opened: Charm store macaroons must be used on the controller. <juju-core:New> <https://launchpad.net/bugs/1560531>15:23
redirmorning15:41
perrito666morning redir15:43
redirfeeling better perrito666 ?15:50
perrito666redir: much, thank you15:50
redirgood good15:51
perrito666redir: did you get your mail and other stuff?15:51
dimiterndooferlad, ping15:53
dooferladdimitern: pong15:53
dimiterndooferlad, hey, I wasn't sure you're around today15:55
dimiterndooferlad, meet babbageclunk - Christian15:55
dooferladdimitern: I am, but was at the hospital this morning for a scan (baby #2)15:56
dooferladhello babbageclunk!15:56
dimiternI see, ok15:56
* dimitern needs to go out, but will be back later15:57
TheMuedooferlad: baby #2?16:01
dooferladTheMue: yep :-)16:01
TheMuedooferlad: great news, grats16:02
dooferladTheMue: due early August.16:02
dooferladTheMue: Thanks!16:02
TheMuedooferlad: so enough time left for the preparation. ours maybe both will have left home by end of year, depends on universities16:04
dooferladTheMue: ah, happy and sad at the same time.16:04
TheMuedooferlad: exactly, but it makes proud to see how they grow up.16:05
redirperrito666: not yet16:05
TheMuedooferlad: hehe, and we already think about downsizing the house, when we don't need so much room anymore16:05
TheMuedooferlad: means selling the big one and buy a new, smaller, and more modern one16:06
dooferladTheMue: heh, for me the only larger thing we may buy is a car!16:07
TheMuedooferlad: here wel already changed to a smaller one (but with a little more luxary). the large station wagon soon hasn't been needed anymore16:09
cheryljfrankban: ping?16:13
frankbancherylj: on call, will ping you asap16:13
cheryljk, thanks!16:14
ericsnowredir: did you get onto ReviewBoard yet?16:16
voidspacealexisb: ping16:18
redirericsnow: I don't beleive so.16:20
redirianb said he'd work on it.16:21
redirgoing to listen to the people and culture orientation thing in a minute16:21
redirThen I'll ping alexisb about the paperwork stuff.16:22
ericsnowredir: go to http://reviews.vapour.ws/ and click on the github button16:22
ericsnowredir: see https://github.com/juju/juju/blob/master/CONTRIBUTING.md#code-review16:22
redirericsnow: to be sure, would I receive and email re: reviewboard?16:22
redirericsnow: will do16:22
ericsnowredir: also see https://github.com/juju/juju/blob/master/doc/contributions/reviewboard.md16:23
redirericsnow: great thanks16:24
ericsnowredir: np16:25
cheryljericsnow: for bug 1560531 and bug 1560520, are these blockers for the next beta?16:31
mupBug #1560531: Charm store macaroons must be used on the controller. <juju-core:New> <https://launchpad.net/bugs/1560531>16:32
mupBug #1560520: Charm channels must be used on the controller. <juju-core:New> <https://launchpad.net/bugs/1560520>16:32
ericsnowcherylj: they prevent the correct behavior when folks use channels, so I'd say so16:32
ericsnowcherylj: rick_h_ could probably say more; and I know it's on urulama's radar16:33
ericsnowcherylj: oh, and regarding the macaroons one, I'm not sure but I'd be inclined to call it a blocker too16:34
rick_h_ericsnow: didn't we say for the beta we'd get the initial deploy in and come back with follow ups to fix the bugs in that behavior after deploy?16:34
ericsnowcherylj: it will keep private charms from doing the right thing or using resources16:34
ericsnowrick_h_: the initial deploy stuff isn't finished relative to channels16:35
ericsnowrick_h_: basically just the --channel flag is added16:35
rick_h_ericsnow: ok, and it won't deploy the right one when used?16:36
ericsnowrick_h_: right; it will always use stable16:36
ericsnowcurrently16:36
rick_h_ericsnow: oh...then boooo16:37
ericsnowcherylj: and by "next beta" you mean the one after the one we're wrapping up this week, right?16:37
redir:)16:37
cheryljericsnow: we're still trying to wrap up the one for this week16:37
ericsnowcherylj: those bugs just need to be in the final release, however that works out16:38
cheryljokay, thanks16:38
ericsnowcherylj: I do not anticipate they will be resolved this week16:38
cheryljericsnow: cool, thanks16:38
ericsnowcherylj: np16:38
katcocherylj: best to talk to urulama for status/eta16:46
urulamaotp16:47
voidspacealexisb: meet babbageclunk :-)16:48
alexisbbabbageclunk, heya man, welcome!16:48
babbageclunkalexisb: hi!16:49
babbageclunkThanks!16:49
frankbancherylj: ping16:50
urulamakatco, cherylj, ericsnow: yes, that's not a full implementation, but it's also not so problematic as you've described. once you have a fully qualified url from the channel pointer, the logic to resolve ACL should allow you to deploy it, not just stable one16:51
urulamaso, i'd like to have a QA performed when this actually brakes16:51
urulamait's a bug as in not fully implemented, but it might not be critical16:51
alexisbfrankban, I a m pestering cherylj so her responses will be slow16:51
frankbanalexisb: np16:52
ericsnowurulama: thanks for clarifying16:52
urulamaso, the plan is to test it and in case it's critical, we'll work in it tomorrow16:53
ericsnowurulama: how can the store determine the channel from the charm URL?16:53
urulamatoday is eod for eu16:53
ericsnow(resolved URL)16:53
urulamaericsnow: so, the case when this will brake is when stable channel has more rights than development channel.16:54
urulamaericsnow: so, you'd get a resolved url, then the CS logic check if you can deploy it from stable first, then from development, then unpublished16:55
urulamaericsnow: if you have access to stable but not development, then you can deploy it16:55
ericsnowurulama: in the case the user specified explicitly the channel they want16:55
katcourulama: that doesn't seem quite right... what if the charm was deployed from development to begin with?16:55
urulamaericsnow: but the revision is the same16:55
ericsnowurulama: right16:55
urulamai said it's not filly implemented16:56
urulamait's just not as critical to be a blocker16:56
katcourulama: ah ok, that's the edge-case16:56
ericsnowurulama: k16:56
urulamawell, unless someone else call it a blocker :)16:56
katcourulama: i don't think we should implement the fallback algo. you described while we wait for full implementation16:56
katcourulama: it's just a guess at a path that may or may not be correct16:57
rick_h_right, it's a feature in a beta. It can not work properly in the beta16:57
rick_h_katco: urulama ^16:57
urulamakatco: that's already in the charmstore ... once the channel is passed to addcharm, everything will be ok16:57
ericsnowkatco: for download all we need is the resolved URL, which is the same for the different channels16:57
urulamaericsnow: yes16:57
katcoericsnow: how does that account for resources?16:58
ericsnowkatco: so the edge case is that the requested channel doesn't actually have that revision, but that is checked when we resolved the URL16:58
ericsnowurulama: I think I get it now16:58
ericsnowkatco: for resources we still need the channel16:58
katcorick_h_: understood... i'm just trying to connect urulama with cherylj/you so the right calls can be made. again, my team does not have a horse in this race.16:59
rick_h_katco: rgr16:59
katcorick_h_: in fact, i would encourage direct communication b/t those parties :)17:00
katcoericsnow: we are just adding confusion. let's remove ourselves from the conversation going forward17:01
urulamacherylj: seems we need to talk ... gimme an hour of my so called life, will be back later :)17:02
cheryljpfft...  "life"17:02
cheryljisn't this job your life?!17:03
urulamait is :)17:04
urulamacherylj: so, in 1h ... is that ok for you?17:05
cheryljurulama: yeah, I'll be here :)17:05
cheryljjust ping me17:05
urulamaok, i do want to verify this, tbh, fell more at ease with proper QA then a bunch of guesses :)17:06
katcoericsnow: have you looked at natefinch 's http://reviews.vapour.ws/r/4269 ?17:07
ericsnowkatco: I reviewed it yesterday but haven't looked since17:09
ericsnowkatco: will take a look17:09
katcoericsnow: it seems like there's a lot of resources functionality that should live in the component? but unsure17:09
=== urulama is now known as urulama|afk
cheryljdamn, I was just about to ping rogpeppe about breaking master!!17:25
cheryljanyone know why he would've removed github.com/gabriel-samfira/sys from dependencies.tsv?17:26
cheryljericsnow: ^^  I see you reviewed the PR.  Any thoughts?17:27
ericsnowcherylj: I expect it was accidental17:27
cheryljericsnow: okay, thanks.  Just wanted to sanity check before I add it back in17:28
ericsnowcherylj: np17:28
* natefinch is back17:38
katconatefinch: wb17:38
katconatefinch: i was mentioning to ericsnow: for your patch, it looked like there was some resources functionality in there that maybe belonged in the component? but wasn't sure17:38
katcoericsnow: how hard would it be to keep http://reviews.vapour.ws/r/4272 going? i don't think we should land it before 2.0 because i don't want to be a huge source of conflicts for incoming branches17:40
natefinchkatco: hmm... good question.17:41
ericsnowkatco: what do you mean "keep it going"?17:42
katcoericsnow: keep the patch updated with changes as they come in17:42
ericsnowkatco: as to conflicts, it would only be a problem for new HTTP endpoints, not new facades17:43
ericsnowkatco: I doubt there will be much churn in that bit of code17:43
katcoericsnow: are you aware of any new endpoints coming in with the 2.0 feature branches?17:43
ericsnowkatco: no, though I could image model migrations potentially having something17:44
katcoericsnow: alright... i suppose we can land this code if we can also assist in any merge conflicts17:44
katconatefinch: btw what are you working on now?17:45
ericsnowkatco: k17:45
redirericsnow: yes17:47
natefinchkatco: I'd spent a little time trying to clean up my patch from last night, so there wasn't so much duplication and unnecessary levels of abstraction... but that's not super pressing right now.  I was going to ask what I should do next.17:47
redirwhoops, was scrolled. nm ericsnow17:47
ericsnowredir: :)17:47
katconatefinch: for now focus on reviews to unblock ericsnow17:49
natefinchkatco: happy to17:49
katconatefinch: i'm peeking at the backlog, but open to any pets you or ericsnow feel need addressing17:49
ericsnowkatco: we do need to sort out the long-lived macaroon issue in the relative "soon" timeframe17:50
katcoericsnow: meaning before the apr. 8th deadline?17:51
natefinchkatco, ericsnow: do we clean up old resources when we remove a service?17:51
katconatefinch: i think ericsnow landed that patch17:51
ericsnownatefinch: yep17:51
katconatefinch: we took care of all the critical bugs17:51
natefinchcool... I thought I remembered that, but just wanted to make sure17:52
katconatefinch: so now we're onto uh... "bothersome"? bugs? and charmstore implementation17:52
katconatefinch: but we should take the rest of the iteration to just do bugs and stay agile for the release17:52
ericsnowkatco: the macaroon issue will keep private charms from working with resources17:52
natefinchkatco" I think there's some gap between the channel work that roger did and what I did, I'd have to look to know for sure.17:52
ericsnownatefinch: FYI, I'm reviewing your patch right now17:53
katconatefinch: there is, and will that will remain until they have a full implementation up. nothing to do for now.17:53
katcoericsnow: natefinch: so sounds like the long-lived macaroon should come next. i'll start drafting that email17:53
natefinchericsnow: yeah, I'm interested to hear your thoughts... looking at it with fresh eyes this morning, I think there's a bunch of cleanup I could do to avoid duplicate abstraction17:54
natefinchkatco: cool17:54
ericsnownatefinch: yeah, I'm jotting down my thoughts right now :)17:54
natefinchericsnow: I think we can collapse Client and BaseClient and just have my clientWrapper interface to abstract away the ugly details of csclient.Client and the charmstore repo dance.17:55
natefinchericsnow: but I may be missing some externalities that you were taking into consideration17:56
mupBug #1560593 opened: debian lintian reports mis-spellings <packaging> <juju-core:Triaged> <https://launchpad.net/bugs/1560593>17:56
mupBug #1560595 opened: help text for juju show-cloud needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1560595>17:56
ericsnownatefinch: baseClient *is* that wrapper17:56
ericsnownatefinch: baseClient and Client represent different things17:57
natefinchericsnow: except with my code, there's a lot more logic inside baseClient than basically anywhere else in the file17:57
natefinchericsnow: maybe that was just me putting the logic in the wrong layer, though17:57
ericsnownatefinch: it's partly a consequence of not having that logic living under the charmrepo repo17:58
ericsnownatefinch: and of dealing with the way channels are handled in csclient.Client17:58
natefinchericsnow: right.. so we need our own wrapper to make using csclient/charmrepo less awful17:59
ericsnownatefinch: and that's what baseClient is for18:00
natefinchericsnow: but that's really the point of the entire package, so why do we need Client?18:00
ericsnownatefinch: let me finish my review and we can discuss it some more18:02
natefinchericsnow: sure :)  Sorry :)18:02
redirwhat's JFDI?18:06
cheryljredir: it's a keyword that lets you try to merge changes into a branch, even if it's blocked by a bug18:06
redirjust do it. got it, tx18:06
cheryljredir: you can see if branches are blocked here:  http://juju.fail/index.html18:07
redirmarked, thanks18:11
cheryljhey rogpeppe, was there a reason https://github.com/juju/juju/pull/4807/ removed github.com/gabriel-samfira/sys from dependencies.tsv?  or was it an accident?18:12
rogpeppecherylj: oh bother18:12
rogpeppecherylj: i forgot to do GOOS=windows godeps.18:12
cheryljso accident, then :)18:13
rogpeppecherylj: yes18:13
rogpeppecherylj: oops18:13
cheryljrogpeppe: http://reviews.vapour.ws/r/4299/18:13
rogpeppecherylj: reviewed18:14
rogpeppecherylj: sorry for the inconvenience18:14
cheryljrogpeppe: it happens :)  You were up rather late getting these things in18:14
rogpeppecherylj: no excuse though :)18:15
natefinchrogpeppe: we should import _ "github.com/gabriel-samfira/sys/unix" somewhere just to avoid that18:15
rogpeppenatefinch: godeps should be better about tagged imports.18:16
natefinchrogpeppe: if only we had a good relation with its author ;)18:17
cheryljheh18:17
ericsnownatefinch: my review is up18:17
rogpeppenatefinch: how would that help?18:17
ericsnownatefinch: as to merging baseClient and Client, baseClient is a wrapper around csclient.Client18:18
natefinchericsnow: but isn't that what Client is, too?18:18
ericsnownatefinch: and Client is a wrapper around BaseClient, thus keeping a tighter control on what functionality we depend on in Juju18:18
mgzrogpeppe: you already looked at the build issues with your charmv5 api branch right?18:19
ericsnownatefinch: Client also provides some Juju-specific functionality18:19
rogpeppemgz: yeah, cherylj pointed out that I'd mucked up the deps18:19
natefinchericsnow: if it lives under github.com/juju/juju, it's all juju-specific18:19
ericsnownatefinch: whereas baseClient adapts csclient.Client (and charmrepo.CharmStore) to a more sensible API18:19
rogpeppemgz: i'm hoping that's the only issue18:19
arosalesaside from 'kill-contoller' any folks have any hints on how to reclaim and juju 2.0 environment?18:20
* arosales stuck in this loop http://paste.ubuntu.com/15473933/18:20
cheryljarosales: you gotta manually delete the info from the cache.yaml18:20
cheryljI've been doing that all week :/18:20
arosalescherylj: ah, ok thanks18:21
cheryljwe should totally fix that for beta318:21
mgzrogpeppe: oh, so you've not looked at reports.vapour.ws/releases/3799 directly?18:21
rogpeppemgz: nope18:21
mgzrogpeppe: not sure the other build problems all come from windows dep18:21
rogpeppemgz: I'm eod18:21
cheryljmgz: that was a temporary branch that rogpeppe already deleted18:21
ericsnownatefinch: there are many things in the core repo that aren't Juju-specific, but no one has taken the time to pull them out18:21
cheryljI pinged him about it this morning :)18:21
mgzcherylj: okay, all I want to know is if I need to report bugs, sounds like not?18:22
cheryljmgz: no, not for that branch18:22
cheryljmgz: and for the windows build failure on master, I've already submitted a fix for dependencies.tsv18:22
mgzexcellent18:22
rogpeppemgz: sorry, we needed a temporary branch to make godeps work in another repo with a circular dependency with juju-core18:22
rogpeppemgz: see mattyw for details18:22
mgzah, but didn't actually want it tested?18:23
ericsnownatefinch: ideally baseClient would go away and we'd use csclient.Client (or some external surrogate) directly18:23
* rogpeppe thinks that "base" is an insidious qualifier for any name.18:23
mattywmgz, yeah, we have some horrific circular dependency thing going on :(18:24
arosalescherylj: safe to just rm ~/.local/share/juju/models/cache.yaml and start anew?18:24
ericsnowrogpeppe: it's a base class <wink>18:24
cheryljarosales: as long as you don't have any other controllers you need to talk to18:24
natefinchrogpeppe: +118:24
arosalescherylj: nope18:24
natefinchericsnow: blech18:25
natefinchericsnow: (re: base classes)18:25
cheryljarosales: you may also need to clean up ~/.local/share/juju/controllers.yaml18:25
ericsnownatefinch: lol18:25
arosalescherylj: ok, thanks for the the help.18:25
cheryljnp, I'll see if we can get that bug fixed on beta318:25
cheryljtoo many people are hitting it18:25
mgzmattyw: you can probably push the rev to the repo without actually creating a branch18:26
mgzthen reference the rev directly18:27
mgzthis is all rather corner-casey though18:27
mgzyou could also ask someone in CI to blacklist a branch you don't want tested18:27
natefinchkatco, ericsnow: so about the problem with GetResource not having all the data... can we just have the charmstore return the rest of the metadata?  The only things it's missing is Origin, Type, and Description, which seem like a trivial amount of data to return along with the bytes themselves.18:29
ericsnownatefinch: the charm store client is supposed to be returning that info (see https://docs.google.com/document/d/1T_7XQ-pmE4gFiD2SSaZnQUqlcLkA7dk0c0PB_ebEkrI)18:30
natefinchericsnow: ok, we need to adjust GetResource on csclient.Client, then18:36
natefinchericsnow: are we doing a multipart body, or dumping the extra data into headers?  Right now there's a header for revision and hash, we'd need additional headers for type, path, description, and origin.18:40
natefinchericsnow: (I presume the latter, since we haven't dealt with multipart bodies anywhere else yet)18:41
katconatefinch: i would do headers as the metadata is metadata about what's in the body18:41
ericsnownatefinch: origin is strictly inferred and the fingerprint should be in a header18:41
ericsnownatefinch: the type, path, and description come from the charm metadata, which may or may not help depending on our use of charmrepo.CharmStore18:42
natefinchericsnow: current use does not make that helpful.  It seems like it makes the API friendlier if we just return all the metadata along with the bytes, given that the payload is tiny compared to the bytes themselves.18:43
ericsnownatefinch: the correct way would be to use a multipart body, but it's a pain18:44
natefinchericsnow: agreed on both points18:44
ericsnownatefinch: at least take a look to see how much work it would be to do a multipart body18:45
natefinchericsnow: there is a library for it... I'll have to check how much of it exists in 1.2, though18:46
ericsnownatefinch: well, we did multi-part for backups with 1.2, so...18:47
=== alexisb is now known as alexisb-afk
mupBug #1560618 opened: Cloud types are unknown <docteam> <juju-core:New> <https://launchpad.net/bugs/1560618>18:56
mupBug #1560624 opened: cmd supercommand.go:448 failed to bootstrap model: no matching tools available <cdo-qa> <juju-core:New> <https://launchpad.net/bugs/1560624>19:08
* thumper goes to find the cat, another trip to the vet for us, yay19:10
natefinchericsnow: so, multipart seems fine, I'll get that adjusted, and then that'll fix the metadata problem for that PR19:31
ericsnownatefinch: sweet19:31
TheMuethumper: oh, hope not too bad.19:34
* TheMue loves cats19:34
thumperTheMue: she has diabetes, and we are getting the insulin levels checked19:39
natefinchericsnow: I'm just going to assume the first part is the metadata and the second is the bytes... that seem reasonable?19:40
ericsnownatefinch: yep19:40
natefinchericsnow: so much cleaner and nicer this way.19:41
TheMuethumper: oh, possible for cats too? our last one sadly had troubles with his kidneys. leaving us after being a family member for more than 14 years.19:41
ericsnownatefinch: yep :)19:41
thumperTheMue: yeah, was a surprise to us too19:41
thumperonly found out a month ago19:41
thumperhence the regular visits now19:41
TheMuethumper: so it's good they found it and you can take care for it19:42
thumperyeah19:42
natefinchmy wife was a vet tech for a while, she says kidney problems and diabetes are super common for cats19:42
TheMuenatefinch: about kidney problems we then learned too. so we're now more careful and try to detect troubles earlier19:45
TheMuenatefinch: but our new cat, we've got it since some weeks now, already trains us. she owns us already. :D19:45
natefinchTheMue: :)  We're cat people, too.  We had one cat, and decided he needed a friend, so we went to the shelter to get one more cat... and came home with two more :D19:47
TheMuenatefinch: *rofl* but you're not only cat people. when I see the pics of your zoo.19:48
natefinchTheMue: the goats and chickens are actually not much harder to take care of than the cats... we do have to trim the goats' hooves every 6 weeks, and clean out the chicken coop once a month or so... but otherwise it's just like cats, make sure they have food and water, and they're fine19:50
TheMuenatefinch: I love it when I see your kids, ok, the girls, take care for them. and the junior surely soon will too19:51
natefinchrogpeppe: how important is it to print out the data in this error message? https://github.com/juju/charmrepo/blob/v2-unstable/csclient/csclient.go#L669   seems like we could use json.Decoder to parse directly off the reader, rather than copying into a buffer first19:52
natefinchTheMue: yeah, it's great having the animals for the kids, and they're fun for us too19:53
TheMuenatefinch: can imagine. I've grown up on countryside with many animals around me. always have been great (ok, sometime much work *g*)19:56
rogpeppenatefinch: i've found those error messages extremely useful in the past20:06
rogpeppenatefinch: particularly when the problem comes from a proxy not the service itself20:07
mupBug #1560665 opened: help text for juju status needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1560665>20:08
mupBug #1560667 opened: help text for juju list-clouds needs improving <helpdocs> <juju-core:New> <https://launchpad.net/bugs/1560667>20:08
natefinchrogpeppe: fair enough... was trying to avoid reading the full data into memory is all20:10
=== alexisb-afk is now known as alexisb
alexisbwallyworld, thumper, fearless juju managers, when you have a moment I would like to steal you guys20:39
thumperalexisb: when and for what?20:40
TheMuehehe, stealing20:40
alexisbreview priorities as we get down to the wire20:40
alexisbthumper, I am flexible20:40
alexisbI can put something on the calendar for this afternoon if that is easier20:41
katcoalexisb: fearless juju managers are now only wallyworld & thumper, yeah?20:41
alexisbkatco, yep20:41
alexisbfeel free to crash but I only need them, lucky souls20:41
katco:)20:41
* TheMue continues to extend his tomb-alike loop to allow also hierarchically monitored and restarted goroutines20:43
alexisbthumper, I am going to send an invite, let me know if the time doesnt work for you20:44
alexisbcherylj, ping21:02
natefinchericsnow: standup?21:02
cheryljalexisb: wrapping up a meeting now21:03
natefinchwallyworld: you up?21:03
ericsnownatefinch: coming21:03
wallyworldnatefinch: have a meeting clash :-(21:03
=== menn0-afk is now known as menn
=== menn is now known as menn0
=== urulama|afk is now known as urulama
thumperalexisb: I can tell when I'm not wanted ;-|21:38
alexisb:)21:38
thumperhttp://reviews.vapour.ws/r/4300/21:39
thumpermenn0: ^^ trade for yours21:39
menn0thumper: looking21:39
menn0thumper: done21:41
urulamakatco, ericsnow: just verified and was able to deploy a development charm from charm store22:08
urulamawith --channel development22:08
urulamawallyworld: ping you around22:09
wallyworldurulama: i am, just finishing a meeting22:10
urulamanp, i'll wait22:10
wallyworldwon't be long22:11
urulamawallyworld: forgot we have sync in a few hours ... in between, take a look at these https://pastebin.canonical.com/152562/ https://pastebin.canonical.com/152560/ ... last one is multiseries charm22:25
=== urulama is now known as urulama|afk
wallyworldurulama: looking in 30 seconds22:25
cheryljwell this is exciting:  "go1: internal compiler error: in fold_binary_loc, at fold-const.c:10124"22:26
mgzcherylj: yeah, I didn't get as far as filing a bug for that but filled in the issue22:29
cheryljI suspect the answer is going to be "use go 1.6"22:29
cheryljmwhudson: how goes 1.6 in trusty?22:29
mwhudsoncherylj: slangasek promised he'd look at it today22:30
cheryljcool22:30
mwhudsoncherylj: is that on ppc64el, or some other platform?22:30
cheryljmwhudson: yes, ppc64el22:31
cheryljso, ppc64 seems to go okay, just ppc64el falls over22:31
mwhudsonppc64?22:33
cheryljmgz:  but the build-binary-trusty-ppc64el succeeded?22:35
* cherylj is confused22:35
mgzcherylj: it may only be a tests compile issue22:37
mgzbuilding binaries doesn't compile *_test.go22:37
cheryljah22:37
cheryljok22:37
=== redir_ is now known as redir
axwwallyworld: sorry I was away from computer last night. have started reviewing, will finish after standup23:14
mgzcherylj: so, it's not test only, for some reason the way dpkg-buildpackage does the build just doesn't hit the issue23:30
mgzprobably because it does itsybitsy steps and then links everything at the end23:30
mgzrather than doing go build .../...23:30
mgzbut I can repo the failure building just that package in charmstore so nfc23:31
mupBug #1560732 opened: Azure endpoint ACLs disappear after machine-0 restart <juju-core:New> <https://launchpad.net/bugs/1560732>23:45
mupBug #1560732 changed: Azure endpoint ACLs disappear after machine-0 restart <juju-core:New> <https://launchpad.net/bugs/1560732>23:51
mgzwelll this works as a fix...23:53
mupBug #1560732 opened: Azure endpoint ACLs disappear after machine-0 restart <juju-core:New> <https://launchpad.net/bugs/1560732>23:54

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