Friday, 2022-03-04

-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 831876: Trigger pipeline runs when semaphores release https://review.opendev.org/c/zuul/zuul/+/83187600:02
@jim:acmegating.comClark: ^00:02
@clarkb:matrix.orgcorvus: thanks lgtm. I guess if we a rolling restart puts that event in a queue and the old scheduler doesn't know what to do with it we log an "error" but continue on otherwise00:09
@jim:acmegating.comyep00:10
@jim:acmegating.comsame as with postconfig event00:10
@clarkb:matrix.org * corvus: thanks lgtm. I guess if during a rolling restart we put that event in a queue and the old scheduler doesn't know what to do with it we log an "error" but continue on otherwise00:10
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 831876: Trigger pipeline runs when semaphores release https://review.opendev.org/c/zuul/zuul/+/83187600:12
@jim:acmegating.comClark: ^ missed a call00:12
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 831876: Trigger pipeline runs when semaphores release https://review.opendev.org/c/zuul/zuul/+/83187600:27
@jim:acmegating.comClark: ^ and one last one00:27
@jim:acmegating.commy local test run is done, so that should be it :)00:27
@jim:acmegating.comzuul-maint: if anyone has a minute to +3 https://review.opendev.org/831876 i think that's worth rolling into the opendev zuul rolling restart (which is in progress, but paused between executors and schedulers/web right now and i plan to resume it tomorrow)01:50
-@gerrit:opendev.org- Ian Wienand proposed: [zuul/nodepool] 831895: functional-test : ensure glean started https://review.opendev.org/c/zuul/nodepool/+/83189503:27
-@gerrit:opendev.org- Ian Wienand proposed:03:41
- [zuul/nodepool] 788684: Switch to using openstackclient https://review.opendev.org/c/zuul/nodepool/+/788684
- [zuul/nodepool] 788876: Switch functional test to Focal nodeset https://review.opendev.org/c/zuul/nodepool/+/788876
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com:04:36
- [zuul/zuul] 831246: Add more pipeline processing stats https://review.opendev.org/c/zuul/zuul/+/831246
- [zuul/zuul] 831249: Add even more pipeline processing stats https://review.opendev.org/c/zuul/zuul/+/831249
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 831609: Add a tenant reconfiguration metric https://review.opendev.org/c/zuul/zuul/+/83160904:36
-@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 831906: Ignore deleted nodes when looking for lost params https://review.opendev.org/c/zuul/zuul/+/83190609:01
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 831876: Trigger pipeline runs when semaphores release https://review.opendev.org/c/zuul/zuul/+/83187609:05
-@gerrit:opendev.org- Zuul merged on behalf of Albin Vass: [zuul/zuul] 831737: Simplify reportBuildEnd call https://review.opendev.org/c/zuul/zuul/+/83173709:05
-@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 831925: Inject bundle id to inventroy zuul.items https://review.opendev.org/c/zuul/zuul/+/83192510:45
-@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 831925: Inject bundle id to inventroy zuul.items https://review.opendev.org/c/zuul/zuul/+/83192510:47
-@gerrit:opendev.org- Albin Vass proposed: [zuul/zuul] 831933: gerritdriver: enable filtering on wip state https://review.opendev.org/c/zuul/zuul/+/83193312:40
-@gerrit:opendev.org- Albin Vass proposed: [zuul/zuul] 831933: gerritdriver: enable filtering on wip state https://review.opendev.org/c/zuul/zuul/+/83193312:43
-@gerrit:opendev.org- Albin Vass proposed: [zuul/zuul] 830840: Make it possible to configure job retries with zuul_return https://review.opendev.org/c/zuul/zuul/+/83084013:13
-@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 830995: Correctly reset failing cycle behind failing item https://review.opendev.org/c/zuul/zuul/+/83099513:26
-@gerrit:opendev.org- yatin proposed on behalf of Slawek Kaplonski: [zuul/zuul-jobs] 762650: [multi-node-bridge] Add script to configure connectivity https://review.opendev.org/c/zuul/zuul-jobs/+/76265013:44
-@gerrit:opendev.org- Michal Nasiadka proposed on behalf of Tobias Henkel: [zuul/zuul] 757452: WIP: Support ansible 2.10 https://review.opendev.org/c/zuul/zuul/+/75745214:04
-@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul-client] 831946: Add web authentication https://review.opendev.org/c/zuul/zuul-client/+/83194614:10
-@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 831948: Document how to enable web authentication with zuul-client https://review.opendev.org/c/zuul/zuul/+/83194814:11
@mhuin:matrix.orgzuul-maint: 831946 and 831948 might make it more straightforward for CLI users when handling authentication14:12
-@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul-client] 819118: Support for "basic" authentication https://review.opendev.org/c/zuul/zuul-client/+/81911814:15
-@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul-client] 831946: Add web authentication https://review.opendev.org/c/zuul/zuul-client/+/83194614:16
-@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 796270: Tests: add non-voting unit testing for Python 3.10 https://review.opendev.org/c/zuul/zuul/+/79627014:20
-@gerrit:opendev.org- Michal Nasiadka proposed on behalf of Tobias Henkel: [zuul/zuul] 757452: WIP: Support ansible 2.10 https://review.opendev.org/c/zuul/zuul/+/75745214:34
@jpew:matrix.orgI'm trying to get diskimage builder working with Zuul and nodepool; It seems like the suggested method is to build very bare images using DIB; whats the recommended method to install all the extra software we need (`build-essentials`, `git`, `docker` etc.)14:39
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 831959: Clarify anonymous nodeset docs https://review.opendev.org/c/zuul/zuul/+/83195914:53
@clarkb:matrix.org> <@jpew:matrix.org> I'm trying to get diskimage builder working with Zuul and nodepool; It seems like the suggested method is to build very bare images using DIB; whats the recommended method to install all the extra software we need (`build-essentials`, `git`, `docker` etc.)15:04
OpenDev does this with a dedicated element that lists the packages out. https://opendev.org/openstack/project-config/src/branch/master/nodepool/elements/infra-package-needs
@jpew:matrix.orgClark: Ok, so you do install that stuff when you build the image15:05
@clarkb:matrix.orgWe don't pre install docker or build essential. But do install git and some other stuff15:06
-@gerrit:opendev.org- yatin proposed on behalf of Slawek Kaplonski: [zuul/zuul-jobs] 762650: [multi-node-bridge] Add script to configure connectivity https://review.opendev.org/c/zuul/zuul-jobs/+/76265015:06
@clarkb:matrix.orgA lot of what we install is to bootstrap Ansible and zuul. Python, git, rsync, ssh, etc15:08
@clarkb:matrix.orgThen if jobs need a compiler or container runtime we expect them to pull that in using Ansible in the job15:09
@jpew:matrix.orgYou give the user blanked sudo permissions, then revoke them?15:10
@jpew:matrix.org * You give the user ALL sudo permissions, then revoke them?15:10
@jpew:matrix.org * You give the user all sudo permissions, then revoke them?15:11
@clarkb:matrix.orgSome jobs revoke them yes. Particularly those like unittests and linting jobs where we set the expectation that root shouldn't be required to execute the actual tests. Others don't drop the extra permissions because they are required.15:12
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 831873: Use JobData for Build result data https://review.opendev.org/c/zuul/zuul/+/83187315:19
@blaisep-sureify:matrix.orgJust FYI, the latest quickstart docker swarm ran better on my fedora server, but ZK still showed one DB issue over night. https://paste.opendev.org/show/bUu7FEEmKlXNmDprQHrP/15:26
No action required and I'm happy to help investigate if you need me to.
@blaisep-sureify:matrix.org * Hi Clark , Just FYI, the latest quickstart docker-compose tutorial ran better on my fedora server, but ZK still showed one DB issue over night. 15:28
https://paste.opendev.org/show/bUu7FEEmKlXNmDprQHrP/
No action required and I'm happy to help investigate if you need me to.
@jim:acmegating.comBlaise Pabon: that's expected for an idle system; it should not be a problem.15:30
@blaisep-sureify:matrix.orgOh!! thank you!!15:30
@jim:acmegating.com(basically just the idle db connection hanging up; it'll get re-established when used)15:31
-@gerrit:opendev.org- yatin proposed on behalf of Slawek Kaplonski: [zuul/zuul-jobs] 762650: [multi-node-bridge] Add script to configure connectivity https://review.opendev.org/c/zuul/zuul-jobs/+/76265016:12
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com:16:27
- [zuul/zuul] 831106: Update patternfly-react https://review.opendev.org/c/zuul/zuul/+/831106
- [zuul/zuul] 831107: Add option to show overall duration in buildset table https://review.opendev.org/c/zuul/zuul/+/831107
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com marked as active: [zuul/zuul] 831873: Use JobData for Build result data https://review.opendev.org/c/zuul/zuul/+/83187317:20
@y2kenny:matrix.orgI think I have asked this before but I am still not very clear about it... when there is a burst of events (either from a single push of large amount of patches, or from a Timer trigger on a repo with massive number of branch/ref), which Zuul component is responsible for processing those events to determine if those events are no-op or not?  Is it the scheduler, mergers or executors?17:52
@clarkb:matrix.orgKenny Ho: the scheduler and mergers work together to determine that17:53
@clarkb:matrix.orgKenny Ho: the scheduler is the brain but it needs information from the code review system and the merged config data to make decisions.17:53
@y2kenny:matrix.orgClark: ok so would the scheduler be a bottle neck if there's like a burst of 500 events? or does the scheduler has some kind of load balance thing within itself also?17:59
@y2kenny:matrix.orgwhat I am seeing is that when the nightly Timer triggers, basically the whole world stops for Zuul and I am wondering if scaling the Merger will fixes that.18:00
@y2kenny:matrix.orgbasically the Timer pipeline will have a large number of events accumulated but then other pipelines are essentially frozen until the Timer related pipeline clears up18:01
@y2kenny:matrix.orgdoes each pipeline has their own "queue" within the scheduler for the scheduler to round-robin between the pipelines? (I am just guessing the scheduler internals...please let me know if I am not having the right mental picture.)18:03
@clarkb:matrix.orgKenny Ho: scaling the mergers will likely help. There are also throughput bottlenecks in the scheduler, but I'm not sure that adding more scheduelrs will help with this. THe scheduler can only handle one pipeline at a time and it sounds like these are all ending up in the same pipeline18:10
@clarkb:matrix.orgif you scale up the schedulers then multiple pipelines can be handled concurrently18:10
@clarkb:matrix.orgI guess if you need other pipelines to run as well then adding a scheduler will help too18:10
@y2kenny:matrix.orgok so scaling scheduler is an option... I didn't know about that.  Does the association of pipeline to the scheduler something that needs to be manually configured or will the schedulers talk among themselves?18:12
-@gerrit:opendev.org- yatin proposed on behalf of Slawek Kaplonski: [zuul/zuul-jobs] 762650: [multi-node-bridge] Add script to configure connectivity https://review.opendev.org/c/zuul/zuul-jobs/+/76265018:15
@y2kenny:matrix.orgsounds like I need to scale both merger and scheduler because I really don't want the periodic pipeline to block my check pipeline (I am seeing that only at midnight right now but I also only have a single periodic job with lots of refs.  And midnight my time would be working hour in other timezone when we have more users.)18:16
@clarkb:matrix.orgKenny Ho: the scheduelrs negotiate that amongst themselves18:19
@clarkb:matrix.orgyou need zuulv5 or newer to run multiple schedulers18:19
@clarkb:matrix.orgalso might be worth reconsidering what time of day you trigger the timer jobs. We try to adjust them for our quiestest period18:20
@y2kenny:matrix.orgok cool.  I am on v5, I will give that a try.18:22
@fungicide:matrix.orgalso if you don't already have performance graphing for zuul, it can be a huge help to see where different services are spending their time in such situations18:23
@fungicide:matrix.orgzuul has a lot of moving parts, so resource and performance trending can help you identify what to scale out18:24
@y2kenny:matrix.orgyea... that is definitely something on my mind... proper monitoring not just for Zuul but other things as well18:24
@fungicide:matrix.orgfor example, opendev's deployment is tracked in various zuul and nodepool dashboards here: https://grafana.opendev.org/?orgId=1&viewPanel=3&search=open18:26
@fungicide:matrix.orgthe general "nodepool," "zuul performance metrics" and "zuul status" dashboards have most of what we look at18:27
@y2kenny:matrix.orgoh and I can see the corresponding dashboard config source... I will definitely reference that18:28
@fungicide:matrix.orgyeah, we use https://pypi.org/project/grafyaml to tyrn yaml definitions in a git repository into grafana configs18:29
@fungicide:matrix.org * yeah, we use https://pypi.org/project/grafyaml to turn yaml definitions in a git repository into grafana configs18:30
@jim:acmegating.comthere's a container image for it you can run too18:31
@jim:acmegating.comhttps://hub.docker.com/r/opendevorg/grafyaml18:31
@fungicide:matrix.orgnote that the "zuul performance metrics" dashboard is a bit of a work in progress, that's just been added in the last day or so and is still in flux18:34
@fungicide:matrix.orghas some data which will presumably be included in 5.1.018:35
@fungicide:matrix.orgwe're still trying to nail down how best to represent it18:36
@y2kenny:matrix.orgunderstood.18:38
@jim:acmegating.comit's about to 1.x in size as soon as the next deployment job runs....18:39
-@gerrit:opendev.org- Dr. Jens Harbott proposed on behalf of Ian Wienand: [zuul/nodepool] 826541: [WIP] Revert "Remove openstacksdk from siblings job" https://review.opendev.org/c/zuul/nodepool/+/82654121:07
@vlotorev:matrix.orgHi, before removing node label from nodepool I'd like to make sure no jobs are using this label.21:32
Is there any way to find if there are any jobs using this label?
@clarkb:matrix.orgvlotorev: we use a search across our code repositories to do that sort of thing21:36
@clarkb:matrix.orgIt may be possible to scan the zuul api's job listings for the info though21:36
@vlotorev:matrix.org> we use a search across our code repositories to do that sort of thing21:38
That would mean I need to clone all repos in tenant...
Do you have any public scrawlers scripts to do that?
@vlotorev:matrix.orgScanning over zuul api looks more prominent though.21:40
@clarkb:matrix.orgvlotorev: no scripts, we just use https://codesearch.opendev.org21:41
@clarkb:matrix.orgif it returns results then we know it is in use21:41
@vlotorev:matrix.org * > we use a search across our code repositories to do that sort of thing21:41
That would mean I need to clone all repos in tenant...
Do you have any public scrawlers scripts to do that?
@clarkb:matrix.orgif not then its probably safe enough to proceed21:41
@vlotorev:matrix.orgSearching the code via codesearch is briliant :)21:45
I don't have on for local repos, there are no way to search the code in non-public Gerrit with custom ACL for different projects.
@vlotorev:matrix.org * Searching the code via codesearch is briliant :)21:45
I don't have one for local repos, there are no way to search the code in non-public Gerrit with custom ACL for different projects.
@vlotorev:matrix.org * Searching the code via codesearch is briliant :)21:46
I don't have one for local repos, there is no way to search the code in non-public Gerrit with custom ACL for different projects.
@vlotorev:matrix.orgClark: Thanks for ideas, zuul API is a good way to go.21:48
@clarkb:matrix.orgvlotorev: if you jobs consistently use centrally defined nodesets you may also be able to push a change removing the related nodesets and see if that results in error21:51
@vlotorev:matrix.orgClark: nope, it's not my case.21:53
@jim:acmegating.comzuul or nodepool logs or monitoring might tell you the last time they were used too22:17

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!