/srv/irclogs.ubuntu.com/2017/11/29/#juju-dev.txt

thumperbabbageclunk: got 5-10 minutes to catch up soonish?00:46
babbageclunkthumper: sure - give me 2 mins?00:47
thumperack00:47
babbageclunkwallyworld: ok, take another look at https://github.com/juju/juju/pull/8138 plz?00:50
babbageclunkthumper: go for babbageclunk00:50
thumperbabbageclunk: 1:1?00:50
babbageclunkja00:51
* babbageclunk goes for a run02:02
axwwallyworld: how do I push a new image into the running k8s without updating the registry? I see Make targets, but I guess I need to configure docker to talk to the remote host somehow?02:18
axwah, local-operator-import, not the other one02:19
* axw tries02:19
jamaxw: I don't know if anyone told you, but we're co-opting the tech board today to discuss the interview candidates.02:35
jam(I just got my reminder for t-b and I meant to cancel the meeting yesterday)02:36
axwjam: yeah wallyworld mentioned this morning, thanks tho02:36
axwwallyworld: FYI, with minikube: eval $(minikube docker-env); make operator-image; kubectl set image po/juju-operator-ubuntu juju-operator=juju/caas-jujud-operator:latest02:42
babbageclunkhey wallyworld: are you happy with https://github.com/juju/juju/pull/8138?04:20
wallyworldlooking04:20
wallyworldbabbageclunk: just one small comment fix04:22
babbageclunkwallyworld: cool thanks!04:22
wallyworldaxw: here's a refactpromg branch to extract common hook command logic https://github.com/juju/juju/pull/815004:54
axwwallyworld: ok, will take a look after lunch05:00
wallyworldno rush05:00
axwwallyworld: not sure why it's showing up as outdated immediately, but I left a comment05:38
axwwallyworld: also I got the operator downloading/unpacking teh charm, will send a PR soon05:38
wallyworldaxw: ah, sorry. i pushed a change just now - thought you were still at lunch05:38
wallyworldaxw: there's a drive by fix for a windows error, plus i renamed the common/hooks package05:39
wallyworldyay for thr charm work!05:39
axwwallyworld: yeah I saw the fix, thanks05:39
axwrename...05:39
axwah I didn't see that05:40
axwthat's fine05:40
wallyworldaxw: ok, i'll rename the hooks/testing package to hookstesting and land.05:40
axwwallyworld: thanks05:40
wallyworldaxw: so next i'll be extracting common runner stuff so that the operator *could* run a hook if it needed to05:41
axwwallyworld: sounds good05:41
wallyworldaxw: but that really needs to land first before we mess with the operator itself to get it to run the first hook i think05:42
wallyworldnot sure if you wan tto look a tthat bit05:42
axwwallyworld: I've still got tests to write for this stuff05:42
wallyworldk, sounds good05:42
axwoh and SHA256 validation, nearly forgot05:43
axwwallyworld: it would be nice if we could just mount the charm in as a volume, but I suppose that's tying too closely to k8s05:43
axwalso would require a custom volume driver I think05:44
wallyworldyeah, i sort of went there but had the same reservation05:44
=== frankban|afk is now known as frankban
rogpeppejam: hiya. do you know what happens to panic output from jujud these days? i just did a kill -QUIT of a jujud instance and i don't see any traceback in /var/log/juju/machine-0.log09:06
jamrogpeppe: I've used SIGQUIT not too long ago, I thought it would have ended up in the machine log. So if you're not seeing that, then we probably have introduced an issue.09:08
rogpeppejam: yeah, that's what i'm concerned about09:08
rogpeppejam: i'm using 2.2.609:08
rogpeppejam: well /var/lib/juju/init/jujud-machine-0/exec-start.sh *looks* like it should send stderr to /var/log/juju/machine-0.log09:11
rogpeppejam: hmm, i think i might see the issue.09:18
rogpeppejam: jujud no longer prints its logs to stdout09:18
rogpeppejam: so there will be a race between juju's file descriptor and stderr, both going to the same file.09:19
rogpeppejam: i'll raise an issue09:19
rogpeppejam: https://bugs.launchpad.net/juju/+bug/173512009:25
mupBug #1735120: jujud panic output does not appear in log <juju:New> <https://launchpad.net/bugs/1735120>09:25
axwwallyworld: https://github.com/juju/juju/pull/815109:53
wallyworldrighto09:53
wallyworldaxw: i wonder - the unit currently just gets the charm url and not the url plus sha56. that seems like a bit of an ommission10:17
axwwallyworld: the uniter? it does get it, it's just buried deep in the bowels10:48
wallyworldok10:48
axwworker/uniter/charm/BundlesDir.download10:48
wallyworldi haven't looked at that code in a bit - i recalled thr CharmURL() API10:49
jamwpk: care to review https://github.com/juju/juju/pull/815210:57
rogpeppejam: do you have any opinion on https://bugs.launchpad.net/juju/+bug/1734725 ?11:03
mupBug #1734725: add-model sometimes ignores specified region <juju:Triaged> <https://launchpad.net/bugs/1734725>11:03
rogpeppejam: and do you know of any workaround for it?11:04
jamaxw: are you still around? It seems the new leadership test failed under '--race': bug #173515311:06
mupBug #1735153: state_leader_test LeadershipSuite.TestCheck has a race condition <intermittent-failure> <leadership> <test-failure> <juju:In Progress by jameinel> <https://launchpad.net/bugs/1735153>11:06
jamrogpeppe: I haven't paged it in to have a specific opinion yet, I can try to look11:06
axwjam: looking11:07
jamaxw: there were other failures during --race, one of which is genuine and reproducible and I have a fix for11:07
jamI haven't reproduced the failure yet11:08
jamwpk: lgtm on your Clock change11:09
jamwpk: that's for 'develop', right?11:09
wpkyes11:09
wpkwell, it's just a test change so it could be backported, but for now freeze is freeze, and it's not critical11:10
jamwpk: is it blocking us getting a clean run?11:17
jamin which case, its worth a 2.3 target (as my --race fix is)11:17
wpkjam: it's -really- rare11:18
jamwpk: k, then we can not worry about it.11:27
jamrogpeppe: I'm unable to reproduce bug #1735120, I get a traceback every time with 2.2.611:28
mupBug #1735120: jujud panic output does not appear in log <juju:Incomplete> <https://launchpad.net/bugs/1735120>11:28
rogpeppejam: you see the traceback in machine-0.log ?11:28
jamrogpeppe: yes11:29
jamI did a -QUIT about 4 times, and it showed up every time11:29
rogpeppejam: i'll try again11:29
jamif I -QUIT the exec-start.sh it just gets ignored11:30
jamrogpeppe: so make sure you see the "running jujud" line, which means we really did die and start a new process11:30
jamalso, its certainly possibel that there is a race, but I'm unable to reproduce on LXD or whatever.11:30
rogpeppejam: oops, i left jujud stopped, and now i have to work out how to ssh to the localhost machine 0 again...11:33
jamrogpeppe: lxc list ?11:34
rogpeppejam: ssh -i /home/rog/.local/share/juju/ssh/juju_id_rsa ubuntu@10.0.8.14911:34
jamaxw: so the leadership thing doesn't look like a 'drop everything, must fix', just something I came across while digging into  different issue with getting a CI bless11:39
axwjam: pretty sure I know the issue (bug in test), just verifying now11:39
rogpeppejam: ok, so i got a stack trace this time11:41
rogpeppejam: but i definitely didn't last time11:41
jamI'm out for a bit11:41
rogpeppejam: thanks for your reply to https://bugs.launchpad.net/juju/+bug/173512012:07
mupBug #1735120: jujud panic output does not appear in log <juju:Incomplete> <https://launchpad.net/bugs/1735120>12:07
rogpeppejam: FWIW the problem is made worse because there's no way to find out what credentials are stored on the controller12:08
rogpeppejam: i have a theory as to what's happening with the stack trace. not sure of a good way to fix it though. https://bugs.launchpad.net/juju/+bug/173512012:45
mupBug #1735120: jujud panic output does not appear in log <juju:Incomplete> <https://launchpad.net/bugs/1735120>12:45
=== frankban is now known as frankban|afk
thumperhml: morning19:07
hmlthumper: good morning19:08
thumperhml: I'd like to chat with you about the cloud-init stuff if you have some tome19:08
thumpertime19:08
hmlthumper: sure - give me a few minutes first19:08
hmlthumper: ready19:10
thumperhml: https://hangouts.google.com/hangouts/_/canonical.com/hml-thumper19:11
=== salmankhan1 is now known as salmankhan
thumperwallyworld: when are you on?21:09
hmlthumper: after looking at our cloud-init for a controller machine - i’m rethinking the list we agreed to, new list is postruncmd, preruncmd and users?21:26
wallyworldthumper: yep22:00
thumperhml: I'll go through the doc and comment there, so we have history22:07
thumperwallyworld: hangout?22:07
wallyworldsure22:07
thumper1:122:08
hmlthumper: sounds good22:09
thumperwallyworld: https://github.com/juju/juju/pull/8153 change is +25/-11, so really not big22:37

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