| -@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 816433: WIP: debugging https://review.opendev.org/c/zuul/zuul/+/816433 | 05:26 | |
| -@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 816433: WIP: debugging https://review.opendev.org/c/zuul/zuul/+/816433 | 06:26 | |
| -@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 816433: WIP: debugging https://review.opendev.org/c/zuul/zuul/+/816433 | 06:35 | |
| -@gerrit:opendev.org- Felix Edel proposed: [zuul/zuul] 816783: Implement autohold endpoints directly in zuul-web https://review.opendev.org/c/zuul/zuul/+/816783 | 06:42 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: | 08:05 | |
| - [zuul/zuul] 816690: Store branch cache minimum ltimes in layout state https://review.opendev.org/c/zuul/zuul/+/816690 | ||
| - [zuul/zuul] 816691: Use branch cache min. ltime when loading a tenant https://review.opendev.org/c/zuul/zuul/+/816691 | ||
| -@gerrit:opendev.org- Simon Westphahl proposed on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 816767: Don't clear branch cache https://review.opendev.org/c/zuul/zuul/+/816767 | 08:05 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: | 08:16 | |
| - [zuul/zuul] 815787: Refresh pipelines in tests when settled https://review.opendev.org/c/zuul/zuul/+/815787 | ||
| - [zuul/zuul] 815278: DNM: execute tests with two schedulers https://review.opendev.org/c/zuul/zuul/+/815278 | ||
| -@gerrit:opendev.org- Felix Edel proposed: | 08:49 | |
| - [zuul/zuul] 814996: Make the ConfigLoader work independently of the Scheduler https://review.opendev.org/c/zuul/zuul/+/814996 | ||
| - [zuul/zuul] 816361: Load system config and tenant layouts in zuul-web https://review.opendev.org/c/zuul/zuul/+/816361 | ||
| - [zuul/zuul] 816362: Implement job freezing API in zuul-web https://review.opendev.org/c/zuul/zuul/+/816362 | ||
| - [zuul/zuul] 816514: Implement managenet events directly in zuul-web https://review.opendev.org/c/zuul/zuul/+/816514 | ||
| - [zuul/zuul] 816783: Implement autohold endpoints directly in zuul-web https://review.opendev.org/c/zuul/zuul/+/816783 | ||
| @westphahl:matrix.org | corvus, Clark, tobiash: updated 816690 and 816691 | 08:50 |
|---|---|---|
| -@gerrit:opendev.org- Felix Edel proposed: [zuul/zuul] 816807: Split up registerScheduler() and onLoad() methods https://review.opendev.org/c/zuul/zuul/+/816807 | 09:40 | |
| -@gerrit:opendev.org- Felix Edel proposed: [zuul/zuul] 816817: Fix missing statsd in zuul times library https://review.opendev.org/c/zuul/zuul/+/816817 | 10:37 | |
| @felixedel:matrix.org | corvus: ^ A small fix for zuul/lib/times.py | 10:38 |
| -@gerrit:opendev.org- Felix Edel proposed: [zuul/zuul] 816817: Don't rely on statsd client in zuul times library https://review.opendev.org/c/zuul/zuul/+/816817 | 10:39 | |
| -@gerrit:opendev.org- Felix Edel proposed: [zuul/zuul] 816807: Split up registerScheduler() and onLoad() methods https://review.opendev.org/c/zuul/zuul/+/816807 | 12:11 | |
| -@gerrit:opendev.org- Felix Edel proposed: | 12:18 | |
| - [zuul/zuul] 814996: Make the ConfigLoader work independently of the Scheduler https://review.opendev.org/c/zuul/zuul/+/814996 | ||
| - [zuul/zuul] 816361: Load system config and tenant layouts in zuul-web https://review.opendev.org/c/zuul/zuul/+/816361 | ||
| - [zuul/zuul] 816362: Implement job freezing API in zuul-web https://review.opendev.org/c/zuul/zuul/+/816362 | ||
| - [zuul/zuul] 816514: Implement managenet events directly in zuul-web https://review.opendev.org/c/zuul/zuul/+/816514 | ||
| - [zuul/zuul] 816783: Implement autohold endpoints directly in zuul-web https://review.opendev.org/c/zuul/zuul/+/816783 | ||
| -@gerrit:opendev.org- Andre Aranha proposed: [zuul/zuul-jobs] 816385: Add fips version of jobs needed for OpenStack https://review.opendev.org/c/zuul/zuul-jobs/+/816385 | 14:18 | |
| @jim:acmegating.com | swest: thanks; tobiash, Clark: if you want to take a look at those two: https://review.opendev.org/816690 and child, i think that will open the door for some (hopefully) longer-term 2-scheduler testing this weekend | 14:49 |
| @tobias.henkel:matrix.org | corvus: q on 816767 | 14:57 |
| @mordred:inaugust.com | wow - what does ```defaultdict(lambda: -1)``` do? | 14:59 |
| @jim:acmegating.com | tobiash: good call | 15:00 |
| @tobias.henkel:matrix.org | mordred: that's creating a defaultdict with -1 as the default value (the lambda is the constructor of the default value) | 15:00 |
| @avass:vassast.org | mordred: creates a dict and instead of throwing a keyerror it will return `-1` | 15:01 |
| @jim:acmegating.com | since it's a callable you can have it do neat things like have it construct nested dictionaries or objects as the default too. | 15:01 |
| @mordred:inaugust.com | neat! TIL | 15:02 |
| @mordred:inaugust.com | didn't there used to be a default dict where you could just pass a non-factory value? | 15:02 |
| @tobias.henkel:matrix.org | it requires a callable which might be a little bit counter intuitive for such simple usages | 15:03 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 816767: Don't clear branch cache https://review.opendev.org/c/zuul/zuul/+/816767 | 15:03 | |
| @mordred:inaugust.com | yeah. like - I could have sworn I used something like default dict with just a static value before - but maybe I'm just getting old and senile | 15:03 |
| @mordred:inaugust.com | OOOOHHHHHHHHH. I see what happened to my brain | 15:04 |
| @jim:acmegating.com | mordred: you can use "defaultdict(list)" because list is a callable | 15:04 |
| @mordred:inaugust.com | I'm thinking about usage like "defaultdict(set)" ... which of course is a callable. I'm just an idiot :) | 15:04 |
| @mordred:inaugust.com | yeah | 15:04 |
| @mordred:inaugust.com | I blame being on a plane - which I'm no longer used to - and is also weird | 15:05 |
| @jim:acmegating.com | mordred: oof, good luck! | 15:05 |
| @jim:acmegating.com | try not to get assaurted, diverted, or infected! | 15:05 |
| @jim:acmegating.com | * try not to get assaulted, diverted, or infected! | 15:06 |
| @mordred:inaugust.com | I'll do my best! | 15:06 |
| @tobias.henkel:matrix.org | corvus: just to get my understanding right in the branch cache, the remainder are all branches minus the protected branches (aka all unprotected unprotected)? | 15:11 |
| @tobias.henkel:matrix.org | * corvus: just to get my understanding right in the branch cache, the remainder are all branches minus the protected branches (aka all unprotected branches)? | 15:11 |
| @jim:acmegating.com | tobiash: yes; and i think there's still a bug in 767 (the tests say so) | 15:12 |
| @tobias.henkel:matrix.org | that's why I'm asking | 15:12 |
| @tobias.henkel:matrix.org | commented | 15:13 |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 816208: WIP Use AuthProvider https://review.opendev.org/c/zuul/zuul/+/816208 | 15:13 | |
| @tobias.henkel:matrix.org | I wonder if that split is premature optimization and we'd need have just lists for protected branches and all branches | 15:14 |
| @jim:acmegating.com | tobiash: yes that's it! :) | 15:14 |
| @jim:acmegating.com | tobiash: it's a lot easier with 2 lists, but i don't really want to store it in zk 2x. | 15:15 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 816767: Don't clear branch cache https://review.opendev.org/c/zuul/zuul/+/816767 | 15:15 | |
| @jim:acmegating.com | this feels like something we ought to be able to get right :) | 15:16 |
| @tobias.henkel:matrix.org | k, since that logic is abstracted it's likely ok after we got it right | 15:16 |
| @tobias.henkel:matrix.org | now lgtm | 15:18 |
| -@gerrit:opendev.org- Zuul merged on behalf of Felix Edel: [zuul/zuul] 816817: Don't rely on statsd client in zuul times library https://review.opendev.org/c/zuul/zuul/+/816817 | 15:34 | |
| @clarkb:matrix.org | corvus: couple of nits on https://review.opendev.org/c/zuul/zuul/+/816767 I'll let you decide if you want to go ahead and approve it and do a follow on (or not) or update it | 16:17 |
| @jim:acmegating.com | agree, i'll push up a new ps | 16:17 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 816767: Don't clear branch cache https://review.opendev.org/c/zuul/zuul/+/816767 | 16:20 | |
| @jim:acmegating.com | Clark: i did 80% of your suggestion on the log msg; i englished it to make it equivalent; that work? | 16:21 |
| @jim:acmegating.com | (if it 'changed' it must have been different) | 16:21 |
| @clarkb:matrix.org | Yup. I think what is useful about the message that is that there was a state change and the old message didn't really communicate that. This works | 16:25 |
| @jim:acmegating.com | i'm going to carry tobiash's +2 on that and +3 it | 16:26 |
| @jim:acmegating.com | there's that zk disconnect test error again https://zuul.opendev.org/t/zuul/build/76a430f303d84caf8de81daf85f78ed5/artifacts | 17:14 |
| @jim:acmegating.com | i'll try to dig into that more while i recheck | 17:15 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 816867: Add some more debug logs at shutdown. https://review.opendev.org/c/zuul/zuul/+/816867 | 17:41 | |
| @jim:acmegating.com | i suspect it's an issue with the stats election after the simulated zk disconnect; that may help narrow it down ^ | 17:41 |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: | 18:31 | |
| - [zuul/zuul] 816690: Store branch cache minimum ltimes in layout state https://review.opendev.org/c/zuul/zuul/+/816690 | ||
| - [zuul/zuul] 816691: Use branch cache min. ltime when loading a tenant https://review.opendev.org/c/zuul/zuul/+/816691 | ||
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 816767: Don't clear branch cache https://review.opendev.org/c/zuul/zuul/+/816767 | 18:32 | |
| @clarkb:matrix.org | corvus: ^ woot. note I noticed that the infra-prod-run-zuul job failed in the hourly run in opendev and havent had a cahnce to look at it yet. Not sure if that implies something is sad with deploying those updates or not | 18:33 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 816867: Add some more debug logs at shutdown. https://review.opendev.org/c/zuul/zuul/+/816867 | 19:47 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 816208: WIP Use AuthProvider https://review.opendev.org/c/zuul/zuul/+/816208 | 23:41 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 816208: WIP Use AuthProvider https://review.opendev.org/c/zuul/zuul/+/816208 | 23:49 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!