[00:00] <blahdeblah> menn0: I see you are the lucky winner of https://bugs.launchpad.net/juju/+bug/1660087; it may still be the same underlying cause as https://bugs.launchpad.net/bugs/1635311, so that might be useful background, depending on what you find in the profiling data.
[00:00] <mup> Bug #1660087: Controller and all models unresponsive <canonical-is> <juju:Triaged by menno.smits> <juju 2.2:Triaged> <https://launchpad.net/bugs/1660087>
[00:00] <mup> Bug #1635311: juju2 eating CPU, units in error <canonical-is> <landscape> <juju:In Progress by axwalk> <https://launchpad.net/bugs/1635311>
[00:05] <menn0> blahdeblah: ok, thanks for joining the dots
[00:05] <blahdeblah> menn0: No guarantees ;-)
[00:35] <alexisb> wallyworld, ping
[00:36] <wallyworld> hey
[00:36] <alexisb> heya wallyworld did you want to meet in our 1x1 HO?
[00:36] <wallyworld> alexisb: give me 5?
[00:36] <wallyworld> or 3
[00:44]  * alexisb taps her fingers impatiently on her desk
[00:44] <alexisb> and stares at wallyworld
[00:44] <wallyworld> alexisb: just looking for hangout, it's gone :-)
[00:45] <alexisb> https://hangouts.google.com/hangouts/_/canonical.com/alexis-ian
[00:45] <alexisb> you just have to look into the past
[00:52] <mup> Bug #1660522 changed: Un-removeable "life: dead", "agent-state: pending"  agents  <canonical-bootstack> <juju-core:Fix Released> <https://launchpad.net/bugs/1660522>
[00:55] <axw> menn0: back
[01:09] <babbageclunk> axw (or anyone else) - is there any way to have the ec2test server return errors for operations?
[01:11] <babbageclunk> axw: Or do I need to introduce indirection in the provider and patch things out with mocked ones? (that seems to be what the other tests do)
[01:13]  * redir looks into the future and sees a beer
[01:13]  * redir eods
[01:13] <redir> maybe
[01:13] <redir> but should find a bug to look at tomorrow
[01:14] <axw> babbageclunk: I don't think so, but don't know for sure. what error do you want to return?
[01:16] <babbageclunk> axw: I just want to test a few error paths in my code.
[01:17] <babbageclunk> axw: Actually I'm not sure there's much need - the error handling is fairly trivial in this case.
[01:17] <axw> for full coverage you probably need a layer of indirection. for more specific validation errors (for example), it would make sense to update ec2test
[01:19] <babbageclunk> axw: I'm not looking for any specific validation errors or anything, so I think this is probably fine.
[01:21] <menn0> axw: was having lunch. i'm back now too.
[01:21] <menn0> axw: quick hangout?
[01:23] <anastasiamac> wallyworld: for bug 1602192, you just want me to say that it'll b better communicated thru status msg and will b addressed in 2.2?
[01:23] <mup> Bug #1602192: when starting many LXD containers, they start failing to boot with "Too many open files" <lxd> <juju:Triaged> <lxd (Ubuntu):Confirmed> <https://launchpad.net/bugs/1602192>
[01:24] <wallyworld> let me look, one sec
[01:24] <axw> menn0: sorry missed your msg, yep
[01:24] <axw> menn0: see you in standup HO?
[01:25] <menn0> axw: yep, see you there
[01:26] <wallyworld> anastasiamac: yeah. i think we also discussed that in clouds, there was no guarantee that the images could support those particular kernal parameters so best not to try and guess and better surface the error
[01:26] <anastasiamac> wallyworld: ack
[01:27] <wallyworld> that was the advice from nic and adam anyway
[01:27] <wallyworld> juju often gets stuff wrong when it tries to guess, so we are trying to be more explicit
[01:53] <redir> I often get stuff wrong when I try to guess
[01:54] <babbageclunk> axw: can you take a look at this if you get a chance? https://github.com/juju/juju/pull/6900
[01:54] <axw> babbageclunk: okey dokey
[01:57] <babbageclunk> axw: thanks!
[01:59] <axw> babbageclunk: done
[02:01] <babbageclunk> axw yay! Yeah, much easier than the Azure one (although a lot of that was me getting to understand the SDK),
[02:34] <veebers> babbageclunk: query is the model migration fix in for aws re: the bug where destroying the origin controller borks things?
[02:41] <babbageclunk> veebers: no - the AWS specific bit is done, but I still need to do the part that calls that from the migration.
[02:41] <veebers> babbageclunk: ack thanks, I'll leave that out from my test script for today then :-)
[02:42] <babbageclunk> veebers: sorry - I'll let you know when it's all done.
[02:42] <veebers> babbageclunk: nw, was just checking before I try it out in my manual testing. Also interested in GCE
[02:43] <babbageclunk> veebers - gce's in the same state.
[02:44] <babbageclunk> Hmm, there's no reason I can't do the migration bit now, if it would help?
[02:44] <veebers> ack. it's not blocking me really at the moment, I'm just doing some manual testing for migrations in the public clouds
[03:34] <menn0> axw: how far away are you with that lxd credentials fix?
[03:36] <axw> menn0: not too far, just updating tests
[03:36] <menn0> axw: cool. I ask b/c I have a fix for the charm URL migration issue but it would be easier to QA if your fix was in place.
[03:36] <menn0> axw: I can work around it regardless
[03:42] <axw> menn0: sorry was otp
[03:42] <axw> menn0: I can push my branch if you want to use it incomplete
[03:42] <menn0> axw: nah it's all good
[03:59] <menn0> axw, babbageclunk or wallyworld: https://github.com/juju/juju/pull/6901
[04:00] <wallyworld> looking
[04:07] <wallyworld> menn0: i think i missing something - the sequence assigns rev numbers 1,2,3,4,5 etc for each base charm url right?
[04:07] <menn0> wallyworld: yes
[04:08] <wallyworld> so if the source model has had charm rev 4 deleted, and migrates rev-1 rev-2 rev-3 rev-5 rev-6, won't that stuff up?
[04:09] <menn0> wallyworld: no b/c the revision is set to max(charm URL rev, next seq value)
[04:09] <menn0> wallyworld: for each charm upload
[04:09] <wallyworld> ah, right ok
[04:09] <menn0> wallyworld: the rev in the uploaded charm is used if possible, unless the sequence is beyond that value already
[04:10] <wallyworld> ok, thanks. lgtm
[04:10] <menn0> wallyworld: thanks for looking
[04:10] <wallyworld> np. was a pretty easy fix in the end
[04:16] <menn0> wallyworld: a bit fiddly but not too hard
[04:17] <menn0> thanks to babbageclunk too
[04:17] <babbageclunk> wallyworld: it looks like there's no method in goose corresponding to the update volume metadata API call: https://developer.openstack.org/api-ref/block-storage/v3/#update-a-volume-s-metadata
[04:17] <wallyworld> wouldn't surprise me
[04:17] <wallyworld> i don't know for sure off hand though
[04:17] <babbageclunk> wallyworld: Ok, cool - just checking I wasn't missing something. I'll add it then.
[04:17] <wallyworld> babbageclunk: if you do change goose, there's a drive by that needs doing
[04:18] <wallyworld> i'll find it
[04:18] <babbageclunk> wallyworld: It's not "upgrade it to the new version of the openstack API" is it?
[04:18] <wallyworld> it's a message that is logged as INFO that should be debug
[04:19] <wallyworld> that could be the message, can't recall exactly
[04:19] <babbageclunk> wallyworld: Oh yeah, that I can handle.
[04:19] <babbageclunk> wallyworld: No I meant a big thing being jokingly described as a driveby. But changing a log level I'll do.
[04:20] <wallyworld> just as well :-)
[04:20] <wallyworld> actually, it's worse than info, it's warning
[04:20] <wallyworld> line 220 of client/apiversion.go
[04:28]  * menn0 is done for now. back again later.
[04:38] <babbageclunk> wallyworld: ok, I'll change that too.
[04:38] <wallyworld> ta
[04:41] <babbageclunk> wallyworld: Do you mean 229? https://github.com/go-goose/goose/blob/v1/client/apiversion.go#L229
[04:41] <wallyworld> babbageclunk: yis (that's kiwi for yes)
[04:42] <babbageclunk> thenks
[04:53] <Lalit>  Hello Experts 	I working on to develop charms . I have two questions  	1) Is there any way to fire command from one juju unit to another ? 	2) Is there a way to copy file from one unit to another ?
[05:07] <axw> Lalit: juju itself does not provide any way to do those things. charms can coordinate with each other to do it using relations, though
[05:07] <axw> wallyworld: can you please review https://github.com/juju/juju/pull/6903
[05:07] <wallyworld> sure
[05:11] <Lalit> Hello Andrew thank for replay . if possible could you please let me know any charm doing this So that i can refer that charm
[05:50] <axw> Lalit: I'm sorry, I don't know of any. I suggest trying in #juju or on the mailing list
[06:28] <axw> wallyworld: you're working on https://bugs.launchpad.net/juju/+bug/1643430?
[06:28] <mup> Bug #1643430: Unassigned units in an error state cannot be removed <juju:Triaged> <juju 2.1:In Progress by wallyworld> <https://launchpad.net/bugs/1643430>
[06:49] <wallyworld> axw: yeah, in between things, just fixing a state changing too quickly error. i also need to think about the fact that we can't assert A && B across collections
[09:23] <jam> for those around that can review: https://github.com/juju/juju/pull/6905
[09:23] <jam> frobware: perrito666: ^^
[09:59] <perrito666> Jam it's 7am here once I finish waking up I'll take a look :)
[11:12] <prankgm> Hi Folks,
[11:12] <prankgm> Have a question. Is there a way to call config-changed hook in charm after relation hooks?
[12:12] <pranav_> Hey folks. Have a query
[12:12] <pranav_> need to fire config-changed hook after all relations are made. Any way to do this?
[15:49] <pranav_> Hey. Can anyone here help me with a query on hooks?
[16:00] <perrito666> pranav_: sorry almost no one is here you had a query regarding config hook right?
[16:00] <perrito666> pranav_: most of these will most likely get a more useful answer in #juju
[16:01] <pranav_> Oh ok thanks. Will try there
[17:03] <redir> morning juju dev
[19:14] <redir> new kernel brb
[19:14] <redir> or bbiab
[20:33] <babbageclunk> perrito666, redir: anyone familiar with the openstack API?
[20:34] <perrito666> Babbageclunk slightly
[20:35] <babbageclunk> perrito666: I'm adding a SetMetadata method to cinder volumes in goose.
[20:36] <babbageclunk> perrito666: My reading of the docs says I want PUT - https://developer.openstack.org/api-ref/block-storage/v3/?expanded=create-metadata-for-volume-detail,update-a-volume-s-metadata-detail#update-a-volume-s-metadata
[20:36] <babbageclunk> perrito666: Only confused because the corresponding method for servers is POST: https://developer.openstack.org/api-ref/compute/?expanded=create-or-update-metadata-item-detail,update-metadata-items-detail,create-or-replace-metadata-items-detail#update-metadata-items
[20:37] <babbageclunk> perrito666: There's also a POST version of the volumes request, but I think PUT is what I want since it explicitly allows partial updates to the metadata (so I don't need to get all of the tags to update one).
[20:38] <perrito666> Definitely not that into it :)
[20:38] <babbageclunk> perrito666: :)
[20:39] <babbageclunk> perrito666: I'll do PUT for now anyway.
[20:41] <redir> babbageclunk: vry minimally
[20:42] <redir> babbageclunk: PUT sounds correct to me
[20:42] <babbageclunk> redir: sweet, thanks
[21:38] <babbageclunk> perrito666: do you know how I can run the cinder live tests for goose?
[21:38] <babbageclunk> perrito666: Can I use creds from cloud-city?
[21:49] <babbageclunk> perrito666: hang on - following along in the nova tests it looks like maybe I don't need to.
[21:51]  * redir lunches
[21:54] <perrito666> babbageclunk: yes you can
[21:56] <babbageclunk> perrito666: I couldn't work out where to get all of the cred bits: I can find URL, user, secret (I assume this can be password), region and tenant. What should domain be?
[21:58] <perrito666> babbageclunk: let me peek at cloud city
[21:58] <perrito666> babbageclunk: you need the credentials for something other than bootstraping?
[21:59] <perrito666> otherwise just JUJU_DATA=/path/to/cloud-city
[22:00] <babbageclunk> perrito666: no, trying to run the goose live tests - that seems to need 6 env vars set.
[22:01] <babbageclunk> perrito666: Ah, looking at the credentials struct, domain is tagged as optional.
[22:02] <babbageclunk> perrito666: Ok, I'll try this - thanks!
[22:02] <perrito666> babbageclunk: source ec2rc
[22:02] <perrito666> :p
[22:02] <perrito666> ah sorry openstack
[22:03] <perrito666> babbageclunk: I used to have hp cloud for tests in openstack
[22:03] <perrito666> now I have no idea what we use, last time someone gave me one
[22:04] <babbageclunk> perrito666: ok, if this doesn't work I'll ping some of the qa folk, thanks
[22:30] <babbageclunk> wallyworld: ping?
[22:30] <wallyworld> otp
[22:34] <wallyworld> babbageclunk: i can ping you after release standup
[22:34] <babbageclunk> wallyworld: ok, just trying to work out how to run goose live tests - if you don't know then no worries.
[22:35] <wallyworld> babbageclunk: unit tests?
[22:35] <wallyworld> ah live tests
[22:35] <babbageclunk> wallyworld: yup
[22:35] <wallyworld> i can help a bit, soon
[22:35] <babbageclunk> wallyworld: ok, awse - thanks
[23:05] <wallyworld> babbageclunk: i have to duckout real quick, but basically if you have your .novarc and source it, the env vars should be set up so you can go test --live
[23:05] <wallyworld> from memory
[23:05] <wallyworld> babbageclunk: but realistically, nowadays, a bootstrap / live functional test is probably best
[23:06] <babbageclunk> wallyworld: what's a .novarc? (I know nothing about openstack)
[23:07] <wallyworld> ah, oh. novarc contains your credentials
[23:07] <wallyworld> and tenant etc
[23:08] <babbageclunk> wallyworld: ok, I'll do some testing once I've done the migration master part
[23:08] <wallyworld> you have not been givwn those for canonistack?
[23:08] <wallyworld> ok
[23:08] <babbageclunk> wallyworld: I've got cloud-city, so I can use those I think
[23:09] <wallyworld> yes you can
[23:09] <wallyworld> there should be a novarc file there
[23:09] <wallyworld> save it as ~/.novarc
[23:10] <wallyworld> then you juju autoload-credentials
[23:11] <wallyworld> babbageclunk: i have to pop out for a bit, back soon
[23:56] <alexisb> anastasiamac, ping
[23:56] <anastasiamac> hi alexisb \o/
[23:57] <alexisb> heya anastasiamac, sorry I missed standups today
[23:57] <anastasiamac> u haven't yet. it's now
[23:57] <alexisb> was curious where we are with the beta5 release
[23:57] <alexisb> ah ok
[23:57] <alexisb> I will join
[23:57] <anastasiamac> it's ready to go. last PR just landed