[02:17] externalreality: what thumper said - if you call the API with the right creds (tag and password), you shouldn't get permission denied, no matter whether the agent's been removed from init. [02:19] babbageclunk, thumper I am calling the endpoint once (no problem), then I call it again after killing the init system service, after that I get permission denied errors. [02:20] I don't understand how you are killing it and calling the endpoint at the same time [02:20] whom are you killing? [02:20] externalreality: you're definitely killing the unit agent and not the machine agent? [02:20] babbageclunk: I am killing the unit agent's service directly [02:21] Then I am trying to update a record based by calling a service with its tag [02:22] I'm not clear what you mean by "calling a service with it's tag". [02:22] What facade/method are you calling? How are you connecting? [02:24] externalreality: (not trying to be pedantic, just want to understand what you're seeing) [02:24] babbageclunk, no worries, gathering logs to make things clear [02:25] babbageclunk, endpoint calls `canAccess(tag)` [02:26] babbageclunk, in the case before I call the service all is fine [02:26] babbageclunk, after killing the service - I get: [02:26] babbageclunk, `juju.worker.dependency "upgrade-series" manifold worker stopped: Access permission denied for string unit-ubuntu-0` [02:27] Indicating the I no longer can call `canAccess(tag)` where the tag is `unit-ubuntu-0` the tag of the service for whom I have killed their systemd/upstart service [02:27] Admittedly, I find it hard to follow the Auth code. Which is why I just thought I'd ask [02:28] I'll do more testing tomorrow to ensure that I am seein what I think I'm seeing before digging through Auth code. [02:28] I'm fairly sure canAccess isn't looking at services. I can help you track through the auth code if you want? [02:29] babbageclunk, I'll take a look in the morning. I am 5:28 minutes past EOD. [02:29] externalreality: Ok, ping me tomorrow if you want help then! (Where are you at the mmoment?) [02:29] babbageclunk, ack [02:31] babbageclunk, I am EST -1 [02:32] Oh right - so pretty late for you then! [02:32] kelvinliu_: ping, just a heads up taht if those changes work for the snap we need to update https://git.launchpad.net/~juju-qa-bot/+git/juju-edge-snap/tree/snap/snapcraft.yaml#n17 and the other git branches there as well. [02:33] kelvinliu_: veebers might be able to get you the full links. I don't have my phone on me to 2fa into LP atm [02:34] rick_h_: I'm on it :-) Just pulled myself to make sure I could [02:35] veebers: gotcha cool. you all rock! :) [02:35] kelvinliu_: so the sep packaging branch uses source-type, as it's source it pointing to a repo. This may trip us up [02:38] rick_h_, yeah, i will need veebers help me understand how everything is setup. thx [03:18] kelvinliu_: remind me the command to cleanup after dep? [03:18] veebers, clean up what? [03:19] kelvinliu_: ah sorry, i.e. when I'm switching between 2.4 and develop i need to remove vendor I think (I'm pretty sure this is covered in the discourse post and I should just RTM) [03:19] ah right, just remove it :-) [03:20] ah make godeps [03:24] kelvinliu_: if you remove vendor from your working dir and try a snap build it'll fail [03:24] veebers, develop branch? [03:25] I'm not 100% sure why yet, but snapcraft will do a go get, then some sort of check/build before it's finished that and does the prepare (or now override-build) [03:25] kelvinliu_: yes [03:25] kelvinliu_: it only worked because we had that dir pre-populated and use a source of ./ [03:26] wallyworld, thumper FYI the win unit test job is running again. It has a build failure, seems to be due the the length of a filename or extension. That rings a bell from previously [03:26] ty [03:26] hmm... [03:26] is it in the vendor directory? [03:27] thumper: http://10.125.0.203:8080/job/RunUnittests-win2012/666/console [03:27] ooohh... 666 [03:27] thumper: hah yeah ^_^ Aye, it looks like it is [03:28] kelvinliu_: I've got a 'going to fail' run with --debug so hopefully will get some more info [03:29] veebers, yes, we need to inject `make dep` after git clone immediately [03:29] wallyworld: this switches the default https://github.com/juju/juju/pull/9061 [03:30] wallyworld: I'll add another later to deal with not starting the pingers [03:30] kelvinliu_: aye, indeed. Just digging in to try figure that out [03:30] veebers, either write dep plugin or ensure dep before plugin [03:31] am I being blind or is there no search for https://docs.snapcraft.io/build-snaps/ [03:34] kelvinliu_: much like the override-build, there might be an override-pull [03:35] FYI https://docs.snapcraft.io/build-snaps/scriptlets looks like it's possible (read the 1st paragraph so far) [03:36] * veebers tries something out [03:51] kelvinliu_, wallyworld: I'm failing to see a way around this. I tried override-pull, but the pull does a go get and tries to do some build (as it chokes on the deps), if I override to not do the pull then we don't have the source there to do anything with. [03:52] veebers: i'm not up on the current core issue [03:52] I also tried changing back to the godeps plugin and doing an override-pull: snapcraftctl pull || true; make dep. But the pull still fails [03:53] veebers, im looking to write dep plugin [03:53] wallyworld: essentially we fail at the pull stage, as the goplugin and godeps plugin does a go get ... + something else that fails so we have no chance to inject the make based dep handling [03:53] kelvinliu_: ack, I think that's the only way forward [03:53] kelvinliu_: I imagine it'll be a big ol' cribbing of the godep one [03:54] hah you could potentially subclass the godep one and change pull(). [03:54] hmm, sad that it's this hard [03:56] I see babbageclunk has a flying spaghetti monster WIP he's working on^_^ [03:56] #dadkjokes [03:56] ha [03:58] thumper: lgtm [04:04] thumper: if you have a moment could you sanity eyeball https://github.com/CanonicalLtd/juju-qa-jenkins/pull/71 please? It's adding a caas image build step after a edge snap build trigger [04:04] then I'll merge that and deploy a handful of jobs that have changed recently [04:11] veebers, im looking to see if we can put the dep.py to ./snap/plugins/ [04:12] kelvinliu_: intersting [04:12] kelvinliu_: I take it you saw: https://docs.snapcraft.io/build-snaps/plugins [04:13] (it outlines what you're trying to do I think) [04:14] veebers, yes, [04:14] sweet ^_^ [05:57] wallyworld: I am waiting for ur approval have pushed commits in both. [05:57] ok, will look [05:58] best to ping me when you need a re-review [06:18] vinodhini: looks like there's still confusion about errors. let me know if you need a HO to clarify [08:14] wallyworld: are u around [08:14] i am [08:15] I reverted whatever u have mentioned in the review [08:15] pushed the commit [08:16] I have made the previous chnages wheever the cause wasnt captured i changed all to fmt.Errorf that was my fault. [08:17] i reverted back all. [08:17] ok, looking [08:28] vinodhini: almost there, main thing is a missing test still === alephnull_ is now known as alephnull [08:30] the reason for wanting the test is that we have shown that auth will fail if there's a mistake in the code in that area [08:30] we need to also guard against future code breakages [08:34] wallyworld [08:34] I was going thru ur comments === salmankhan1 is now known as salmankhan [09:21] Howdy; i messed up. I've killed the host running the juju controller. :( Its not a big problem, its a testing environment. But; what would be the suitable path to start a new controller on the same cloud. [09:23] would destroy-controller (or kill-controller), and the just bootstrap the new one be suitable, or would it potentially have some side effects on the running models? [09:24] My guess, it would not, as the device running the controller is dead, hence the destroy command would have no way to talk to the models..Which also were hosted on that device.. [09:38] https://github.com/juju/layer-index/pull/39 anyone around to give me a merge? [09:39] wallyworld: are u around [09:40] I have left comments for the test u have requested. [09:40] Its already there without TenantName and TenantID [09:42] wallyworld I have added the same comment in PR. Please let me know your input. [09:46] parlos: you can juju unregister the existing controller (to clear it from the local juju cache). if the controller machine also hosted the model worker machines via containers or whatever, then they are all gone anyway and you can start again [09:49] vinodhini: there's still no test [09:50] for the v3auth scope [09:52] wallyworld; thanks. The controller was just controller, the models/applications are fully functioning. But, I will redeploy them as to have them under control. I'll also start doing controller backups.. [09:53] parlos: if you want to redeploy, you should also manually kill the machines running the workloads [09:53] juju unregister is the command you want to clear the controller away from juju's cache [09:54] wallyworld; eventually; but one of the workloads is an openstack deployment, and its in use....( for another couple of weeks). [09:54] ah ok [09:55] if you had a juju backup you could have recreated the controller [09:56] I'm reading on the backup, does this only apply to a specific model? [09:56] thanks [09:57] magicaltrout: done [09:57] parlos: if backs up the entire controller database; all models [10:01] Ok, and if the controller has gone to /dev/null, then you need the downloaded backup file. Correct? This will basically bootstrap a new controller when the information in the backupfile.. [10:43] vinodhini: the test that's there isn't testing what I'm asking for. the code that was changed in the PR was to change how auth.Auth.Scope is set according to what cred values we have. I can't see a test for that [10:43] the tests you are referring to appear to be testing other aspects [10:44] we specifically need to ensure that what is passed to the identity endpoint is what we need [10:45] it's very much a whitebox test. we need it because if the code is wrong, auth will fail [11:48] I just had an issue with SCP (https://github.com/juju/python-libjuju/issues/259) and raised a PR fixing it: https://github.com/juju/python-libjuju/pull/260 [11:49] Thanks beforehand for the review! [12:38] tinwood, cory_fu: what's the reactive way of handling the 'stop' hook [12:38] ? [13:10] jamespage, good question. Probably @hook for the moment, unless @hook has been removed now [13:10] * tinwood goes off to check [13:11] tinwood: It hasn't been removed. [13:11] jamespage: Yeah, I think @hook, I guess [13:11] thanks cory_fu [13:11] ok good [13:11] I was not wrong then - someone asked me [13:27] jamespage: Ideally I'd like to add a framework-driven flag for it, just for consistency, but stop is always going to be a bit of a special case [15:35] rick_h_: with this mutliple credentials, so this issue is that when using lxc to auto detect credentials, localhost breaks, as there are always multiple credentials - i wonder if we just never return locally discovered credentials? [15:35] stickupkid: processing ... [15:36] i.e. we can discover the remote, but you have to add the credential manually? [15:36] so it will show in "add-clouds", but you can't bootstrap to it [15:36] stickupkid: k works for now to move forward [15:37] although you're likely to get a cryptic message about localhost not the right credentials for nuc5 [15:37] and puts the right priority on what's easier vs more akward [15:37] stickupkid: ? [15:38] i think it's both akward [15:38] maybe we should back out autodectection of local lxc configurations [15:38] stickupkid: when would you get that message? [15:38] bootstrap [15:39] stickupkid: but if I bootstrap a remote I added as a cloud with a credential why would I get that error? [15:40] so say you have some lxc remotes in your lxc client, you forget to add a credential for that lxc remote, and then do bootstrap [15:40] it will try and use localhost, because LXD is special provider and always returns localhost creds :| [15:41] as we've conflated local vs remote for LXD we can't differentiate when you want which [15:42] stickupkid: I thought we were looking into stopping that. What came of auto creating the localhost cloud credential automatically? [15:43] hmmm... i'm thinking when we could do that [15:43] during bootstrap? [15:44] stickupkid: HO? [15:44] if so, we would need something like "providers.RegisterCloudCredentials()" <- tbh that would solve a lot of issues :| [15:44] sure please [15:57] https://discourse.jujucharms.com/t/connectivity-issues/163/3 [15:57] if anyones bored [16:00] magicaltrout: ping'd the charmstore folks to see what's up. none of them in here :( [16:01] thanks rick_h_ [16:18] Yeah, charmstore is having a bad day here as well. [16:47] rick_h_: this is looking and feeling good - i'm going to play with it more... [16:48] rick_h_: at least I'm on the right track \o/ [17:32] stickupkid: good to hear [20:24] Morning all o/ [22:07] externalreality_: You're working on something to do with update-series, is that right? [22:07] babbageclunk, the owner of the connection to the api was the machine agent, it was trying to auth as its units. [22:07] Ah, right. [22:08] babbageclunk, instead we now auth as the machine and collects its units apiserver side [22:08] babbageclunk, do you think that sounds more correct... I kind of think it does [22:08] Ok, that sounds like exactly what I'd have suggested, awesome. [22:09] babbageclunk, ack, thank you for the ear sir [22:09] kelvinliu_, thumper, rick_h_: Note: with the snap changes being made for the dep change we'll have to add some extra smarts to the release job so we can release either a 2.4.* or a 2.5.* in the future) [22:10] externalreality_: no worries, it sounds like you sorted it out independently anyway!