| thumper | https://github.com/juju/juju/pull/11076 for anyone wanting a simple PR to review | 01:59 |
|---|---|---|
| kelvinliu | thumper: lgtm ty | 02:01 |
| thumper | kelvinliu: thanks | 02:02 |
| kelvinliu | np | 02:08 |
| wallyworld | babbageclunk: i've updated https://github.com/juju/juju/pull/11071 with the correct fix, just re-testing now | 02:47 |
| wallyworld | would be awesome if you could take another look | 02:47 |
| wallyworld | tlm: for QA steps, you should also run juju controller-config to check the value and also test updating it | 02:53 |
| tlm | wallyworld: ta, just found a bug with it as well. Will update all in a sec | 03:02 |
| wallyworld | ok | 03:02 |
| thumper | wallyworld: package move https://github.com/juju/juju/pull/11077 | 03:05 |
| wallyworld | ok | 03:05 |
| wallyworld | thumper: can't see any issues | 03:13 |
| thumper | wallyworld: I'm running the tests locally as well to ensure everything builds and passes | 03:13 |
| wallyworld | sgtm | 03:14 |
| tlm | wallyworld: all updated | 03:17 |
| wallyworld | righto | 03:17 |
| wallyworld | tlm: i've left a couple of things to look at, let me know if anything is unclear | 03:25 |
| tlm | cheers | 03:26 |
| tlm | wallyworld: sent you a private message just in case notifications are still off | 03:33 |
| babbageclunk | wallyworld: ok looking | 03:50 |
| wallyworld | ta | 03:50 |
| wallyworld | kelvinliu: i have a trivial forward port of 2.7 into develp https://github.com/juju/juju/pull/11078 | 03:57 |
| kelvinliu | wallyworld: lgtm ty | 03:58 |
| wallyworld | ta | 03:58 |
| wallyworld | tlm: also forgot to mention sorry, the controller name PR should be targetted against develop not 2.7 | 03:59 |
| tlm | np will shift it around | 04:01 |
| thumper | wallyworld: have we fixed the bootstrap / upgrade issue that we were hitting with the build number? | 04:04 |
| wallyworld | thumper: that's the PR that babbageclunk is looking at currently | 04:05 |
| thumper | sweet | 04:05 |
| stickupkid | getting nothing past hml, yesterday - haha | 09:15 |
| zeestrat | Any news on a new libjuju release? | 10:09 |
| stickupkid | zeestrat, doing it now | 10:10 |
| stickupkid | :D | 10:10 |
| zeestrat | Yay :D | 10:10 |
| stickupkid | zeestrat, just catching up on what the changes are | 10:11 |
| stickupkid | manadart, you around? | 11:01 |
| manadart | stickupkid: Yep. | 11:01 |
| stickupkid | quick ho? | 11:01 |
| manadart | Sure. | 11:01 |
| stickupkid | manadart, CR please https://github.com/juju/python-libjuju/pull/378 | 11:05 |
| stickupkid | zeestrat, hopefully we can verify that everything is fine so that 2.7.0 can go out https://github.com/juju/python-libjuju/pull/378 | 11:19 |
| stickupkid | zeestrat, I ran against this, but manage to mitigate this issue luckily https://github.com/juju/python-libjuju/issues/377 | 11:20 |
| stickupkid | manadart, when you get time https://github.com/juju/juju/pull/11067 | 11:40 |
| manadart | stickupkid: Yep. | 11:40 |
| manadart | stickupkid: Got a sec to HO? | 12:34 |
| nammn_de | anyone quick cr? https://github.com/juju/juju/pull/11079 | 12:55 |
| manadart | nammn_de: Done. | 12:55 |
| nammn_de | manadart: thankso! | 12:56 |
| stickupkid | manadart, i'm in daily | 13:25 |
| stickupkid | watching you eat | 13:26 |
| === narindergupta is now known as narinderguptamac | ||
| roadmr | hey folks, how do I get a leaderless applicatin to get a leader? | 14:26 |
| roadmr | [INFO] canonical-livepatch does not have a leader <- appears repeatedly and juju status indeed shows no unit for that application with the nice asterisk * | 14:27 |
| rick_h | roadmr: hmmm, can you check the logs to see if there's a reason no one's winning leadership? | 14:35 |
| rick_h | roadmr: the cases we've seen with this in the past has been more leader stomping where it's taken more than the time allotted for the leader hook to run and someone else tries to get it and the first one gives up | 14:35 |
| stickupkid | zeestrat: 2.7.0 is released https://discourse.jujucharms.com/t/pylibjuju-2-7-0-release-notes/2489 | 14:47 |
| stickupkid | zeestrat, we had to bump the facades to 2.7.0, which was a bit of a pain, but now it's done the release went smoothly :D | 14:48 |
| stickupkid | let us/me know if you hit any issues | 14:48 |
| rick_h | gnuoy: ^ | 14:48 |
| rick_h | thedac: ^ /me can't recall who wanted it | 14:48 |
| gnuoy | rick_h, me ! it was me ! | 14:48 |
| gnuoy | thanks | 14:48 |
| zeestrat | stickupkid: ty very much! We'll let you know :) | 14:50 |
| thedac | rick_h: stickupkid: thanks, I'll spread that knowledge to my team. | 15:32 |
| cmars | roadmr: wondering if https://bugs.launchpad.net/juju/+bug/1853055 is related to your issue | 15:40 |
| mup | Bug #1853055: "ERROR could not determine leader" but `juju status` says there is a leader <juju:New> <https://launchpad.net/bugs/1853055> | 15:40 |
| cmars | although your status shows a leader? | 15:40 |
| cmars | one thing i've had to do to work around that, is scrape the json status for the unit that shows "leader" and address that unit specifically | 15:41 |
| cmars | maybe see what juju status --format yaml (or json) shows | 15:41 |
| roadmr | cmars: no, we see no leader :( | 15:43 |
| roadmr | rick_h: 2020-01-07 15:28:36 INFO juju.worker.leadership tracker.go:217 canonical-livepatch leadership for canonical-livepatch/2 denied (no further explanation as to why) | 15:43 |
| cmars | anarchy! | 15:44 |
| roadmr | cmars: exactly what I said :) | 15:45 |
| roadmr | on some units I see "2020-01-06 14:28:25 DEBUG juju.worker.leadership tracker.go:125 canonical-livepatch/1827 making initial claim for canonical-livepatch leadership" right before the above | 15:46 |
| stickupkid | hml, I've updated https://github.com/juju/juju/pull/11072 | 15:48 |
| hml | stickupkid: approved | 15:54 |
| stickupkid | ta | 15:54 |
| nammn_de | achilleasa: quick cr? https://github.com/juju/charmrepo/pull/158 | 16:02 |
| achilleasa | nammn_de: looking | 16:02 |
| nammn_de | rick_h: i have made the patch as small as possible. This now should mostly enhance bash completion ux https://github.com/juju/juju/pull/11032/ | 16:35 |
| rick_h | nammn_de: k, let's try it | 16:36 |
| rick_h | roadmr: can you file a bug please? | 16:36 |
| nammn_de | rick_h: thanks! something special needed to put that into edge beside just merging it? I did quite a lot of local testing and I don;t see any reason why this should make it worse... :D | 16:41 |
| rick_h | nammn_de: no, just land it in develop and it'll go into edge when our builds get back happy | 16:41 |
| rick_h | nammn_de: and just keep an eye out for any issues | 16:41 |
| achilleasa | nammn_de: can I get a quick CR on https://github.com/juju/charm/pull/301? | 16:54 |
| nammn_de | rick_h: will do! | 16:59 |
| achilleasa | nammn_de: or stickupkid can someone take a look at https://github.com/juju/juju/pull/11080? | 17:21 |
| nammn_de | achilleasa: approved | 17:36 |
| stickupkid | hml, there is fallout from my PR around handling outputs correctly, i.e. we put the empty value in stderr and not stdout, can you review my PR https://github.com/juju/cmd/pull/75 | 17:51 |
| stickupkid | achilleasa, or can you look into it (see above) | 17:52 |
| achilleasa | stickupkid: looking | 17:53 |
| achilleasa | stickupkid: done | 17:56 |
| verterok | rick_h: Hi! regarding roadmr leaderless issue, we have https://paste.ubuntu.com/p/Cp8tbkgHkv/ in the logs | 18:09 |
| verterok | cmars: ^ | 18:09 |
| verterok | even after adding a new unit | 18:09 |
| verterok | that logs is from a fresh unit | 18:10 |
| rick_h | verterok: do you know if you're using legacy leases or raft leases? | 18:10 |
| verterok | rick_h: this is a IS managed controller, the main one...let me check | 18:10 |
| rick_h | verterok: I'm not seeing "leadership for" in the current code so getting a bug with juju version, where it's at (prodstack I thought moved off leagacy leases but axino can correct me), and model version details would be good please | 18:11 |
| rick_h | verterok: I'm a bit distracted atm but if we can get the details pulled together I can ask someone to investigate | 18:11 |
| verterok | rick_h: 2.6.10 in client and model | 18:12 |
| rick_h | verterok: ok, good to know. on prodstack 4.5 you're saying? | 18:12 |
| verterok | yup | 18:12 |
| axino | features [] | 18:13 |
| axino | no legacy leases ! | 18:13 |
| verterok | axino: thx | 18:13 |
| rick_h | axino: ok cool ty for the confirmation there | 18:13 |
| hml | stickupkid: back, reviewing now. | 19:22 |
| hml | stickupkid: or not. :-) | 19:22 |
| verterok | roadmr, cmars, rick_h: this looks pretty similar to what we are seeing: https://bugs.launchpad.net/juju/+bug/1656275 | 20:26 |
| mup | Bug #1656275: unit leadership gets confused <leadership> <logging> <model-migration> <juju:Fix Released by thumper> <https://launchpad.net/bugs/1656275> | 20:26 |
| verterok | but in a version that should have that fix...which points to a regression of some kind | 20:27 |
| roadmr | 😱 | 20:27 |
| _thumper_ | verterok: I know that there was additional work later on that | 20:27 |
| _thumper_ | but I would have thought that it would be in 2.6.10 | 20:28 |
| === _thumper_ is now known as thumper | ||
| verterok | thumper: right, we are seeing a similar symptom: leadership claim -> denied | 20:30 |
| verterok | only difference I can guess is that in our case is a subordinate...in case that makes any difference | 20:30 |
| thumper | verterok: I am wondering about whether this could be scale and timeout related... | 20:30 |
| roadmr | 🦎 <- lots of scales here | 20:32 |
| thumper | I just see 🦎 | 20:32 |
| thumper | that is a rectangle... | 20:32 |
| verterok | heh | 20:33 |
| roadmr | thumper: haha you copy-pasted the rectangle and I see exactly what I posted :) | 20:33 |
| verterok | thumper: the model itself is nothing crazy, 18 machines with same number of canonical-livepatch subordinates | 20:33 |
| thumper | verterok: but this is on the shared ps4.5 controller? | 20:33 |
| verterok | thumper: yup | 20:34 |
| thumper | that has scale | 20:34 |
| verterok | indeed | 20:34 |
| roadmr | only two things to mention: this env has been up for a loong time and it sees a lot of churn (applications being created/removed very frequently) | 20:34 |
| roadmr | since we add a new application on every new code version rollout | 20:34 |
| roadmr | that tends to make juju unhappy | 20:34 |
| verterok | thumper: anything we could do to recover? is there any way to force a leader? | 20:34 |
| verterok | this is blocking rollouts to staging, as we use mojo and it doesn't like to have applications without a leader :) | 20:35 |
| thumper | verterok: it looks like raft thinkgs there is probably a leader, but mongo doesn't | 20:35 |
| thumper | verterok: we need to work out who raft thinks is the leader | 20:36 |
| thumper | you can probably do that with juju run over the app with 'is-leader' | 20:36 |
| thumper | then shut down that unit for a minute to force expiration | 20:36 |
| verterok | thumper: all return false | 20:36 |
| thumper | ah... | 20:36 |
| thumper | wat? | 20:36 |
| roadmr | verterok: did you file the bug already? | 20:37 |
| roadmr | verterok: please please make the bug title "ANARCHY!!!!!!" | 20:37 |
| verterok | thumper: https://paste.ubuntu.com/p/RwbDQFyYf3/ | 20:37 |
| thumper | babbageclunk: thoughts ? ^^^ | 20:37 |
| verterok | roadmr: didn't file one as I found 1656275 | 20:37 |
| roadmr | ahh... bummer :( | 20:38 |
| thumper | verterok: please file a new one | 20:38 |
| verterok | thumper: will do | 20:38 |
| thumper | verterok: it is probably a different issue | 20:38 |
| roadmr | it is anarchy!!! hahah | 20:40 |
| verterok | roadmr: can let you the honors | 20:40 |
| verterok | *do | 20:41 |
| * roadmr won't pass up the chance to file a silly bug | 20:41 | |
| roadmr | verterok: hm - we tried restarting juju agents for that application but not the machine-X agents, think it might help? | 20:46 |
| verterok | roadmr: I can restart all agents | 20:46 |
| babbageclunk | thumper: missed this - reading back | 20:47 |
| roadmr | https://bugs.launchpad.net/juju/+bug/1858693 has a summary | 20:48 |
| mup | Bug #1858693: ANARCHY!!!!!!! Entirely leaderless application spotted in the wild <juju:New> <https://launchpad.net/bugs/1858693> | 20:48 |
| verterok | roadmr: all machine agents restarted, no changes | 20:53 |
| babbageclunk | verterok: if you deploy a new application, does the new unit become the leader? | 20:54 |
| verterok | babbageclunk: a new application or add-unit? | 20:55 |
| babbageclunk | verterok: trying to determine whether all of leadership is broken, or whether there's some kind of pinning happening for that application specifically | 20:56 |
| babbageclunk | new application (can be an existing charm but with a new name) | 20:56 |
| verterok | babbageclunk: leadership for other applications seems to be fine | 20:56 |
| babbageclunk | do you mean, you can see leaders for other applications? Or you've seen leadership change for other applications when the problem has been happening? | 20:57 |
| babbageclunk | because the former might not indicate it's fine | 20:57 |
| babbageclunk | it might just be stuck in a non-visible way | 20:57 |
| verterok | babbageclunk: let me try deploying a trivial ubuntu unit, would that be enough? | 20:57 |
| babbageclunk | yup | 20:57 |
| babbageclunk | thanks | 20:58 |
| verterok | running: juju deploy cs:ubuntu -n 2 | 20:58 |
| verterok | babbageclunk: I get the leadership * as expected in juju status | 21:01 |
| verterok | babbageclunk: and juju run --aplication ubuntu is-leader return True and False as expected | 21:01 |
| babbageclunk | verterok: ok - so it sounds like the canonical-livepatch application has leadership pinned? | 21:02 |
| babbageclunk | Was there an upgrade-series done at some point? | 21:02 |
| verterok | babbageclunk: no upgrade-series AFAIK, the env is old but always in xenial | 21:03 |
| babbageclunk | verterok: oh, another useful thing might be to remove the leader unit of the new ubuntu and make sure that the other one eventually claims leadership | 21:04 |
| babbageclunk | hmm, that's the only thing that would do a pin that I know of | 21:04 |
| verterok | babbageclunk: we do remove units and deploy new ones on each new code revision | 21:04 |
| verterok | babbageclunk: which also means killing and spawning new suboardinates (which includes canonical-livepatch) | 21:05 |
| roadmr | verterok: 18 units in all, you said? each add/remove has a 20% or so chance of removing the current leader | 21:05 |
| babbageclunk | right - which would explain why the leader went away? But not why there's not a new one | 21:05 |
| roadmr | (we add 4 units, then remove the 4 old ones) | 21:05 |
| verterok | roadmr: right, 18 | 21:05 |
| babbageclunk | It might be that the raft time isn't being updated, so it's not expiring the old one. | 21:06 |
| roadmr | well if elections happened every 3 weeks eventually you'd also want there to not be a leader, right? | 21:06 |
| roadmr | (j/k not helping haha) | 21:06 |
| babbageclunk | if you kill the leader of the new ubuntu, does leadership switch to the other unit? | 21:07 |
| babbageclunk | verterok: ^ | 21:07 |
| verterok | checking | 21:07 |
| verterok | gimme 2'...because I need to deploy them again (killed them too soon :P) | 21:08 |
| babbageclunk | ha doh | 21:08 |
| roadmr | babbageclunk: where can I check the controller logs? (I assume it's you who asked for those) | 21:08 |
| babbageclunk | yeah, that's me | 21:08 |
| verterok | roadmr: IS can | 21:08 |
| roadmr | ah ok | 21:08 |
| verterok | babbageclunk: how long could it take to elect a new leader? | 21:09 |
| verterok | ok, after a brief moment of panic, it worked | 21:10 |
| babbageclunk | should happen in a minute or less | 21:10 |
| verterok | babbageclunk: /4 was leader, and after remove-unit the * is now in /3 | 21:11 |
| babbageclunk | huh. | 21:11 |
| babbageclunk | it's a pain but I'm kind of tempted to suggest you remove and re-add canonical-livepatch. | 21:12 |
| verterok | babbageclunk: that was our plan if we needed to unblock deploys | 21:13 |
| roadmr | is that a non-suggestion suggestion? :) | 21:13 |
| babbageclunk | I mean, it's the tactical nuclear fallback suggestion ;) | 21:14 |
| roadmr | if there's nothing else to be gleaned from the current live environment, we might as well | 21:14 |
| roadmr | we need to wait for controller logs anyway,if that's where more clues might be | 21:14 |
| babbageclunk | Well, I could definitely do with the logs | 21:15 |
| babbageclunk | as long as you're ok with waiting | 21:15 |
| roadmr | filing the RT | 21:17 |
| babbageclunk | but I'm not sure how it could get into this state without the application being pinned. | 21:18 |
| roadmr | babbageclunk: do you have a sec to drop by #is in case Alexandre needs more specifics on what to dig for in the logs? | 21:21 |
| babbageclunk | sure | 21:21 |
| roadmr | (I can triangulate if needed but might be more efficient if you're there) | 21:21 |
| verterok | thanks for the help babbageclunk | 21:29 |
| * verterok EODs | 21:29 | |
| kelvinliu | hi cory_fu: the current 2.7 edge `2.7.1+2.7-ec91b32` should be working fine for CaaS cmr now. | 22:10 |
| cory_fu | kelvinliu: Great, thanks | 22:10 |
| cory_fu | I'll give it a try | 22:10 |
| kelvinliu | cory_fu: ty, | 22:10 |
| wallyworld | babbageclunk: i just saw the bug - if you remove a unit that is leader, juju does not immediately elect a new leader, the current lease needs to time out. this is a known issue / works as designed. there are bugs like bug 1469731 raised back in 1.23 days | 22:19 |
| mup | Bug #1469731: Leadership must be dropped before removing lead unit <canonical-is> <charm> <charmers> <leadership> <teardown> <juju:Triaged> <postgresql (Juju Charms Collection):New> <https://launchpad.net/bugs/1469731> | 22:19 |
| babbageclunk | wallyworld: yeah, but in this case it's been ages | 22:20 |
| wallyworld | ah, ok, longer than the lease timeout | 22:20 |
| roadmr | 👴 ages :) | 22:22 |
| babbageclunk | wallyworld: yup as in hours | 22:22 |
| wallyworld | :-( | 22:22 |
| babbageclunk | wallyworld: but other applications seem to be ok - leadership changes fine | 22:22 |
| wallyworld | i guess we'll need the raft logs, engine reports etc | 22:23 |
| babbageclunk | yeah, although none of that really helps if leases seem to be changing ok for other applications | 22:26 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!