| @jim:acmegating.com | yeah, i don't *remember* that with docker is why i suggested it might be the case | 00:00 |
|---|---|---|
| @jim:acmegating.com | we could also think about doing "-d" and then running "... compose logs" for the interesting containers | 00:01 |
| @clarkb:matrix.org | internet says using something like setsid to detach the process might help too. But I wonder if that affects podman's ability to manage the process through signalling for stopping the container | 00:02 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 926221: Avoid recursive delete in forceUnlockNode https://review.opendev.org/c/zuul/nodepool/+/926221 | 00:55 | |
| @bennetefx:matrix.org | > <@clarkb:matrix.org> We recently updated zuul (I can't remember if this was before or after 10.0.0) to use a regular user token for when the app isn't installed if you configure a regular token too. This gives you significantly higher quota than doing unauthenticated requests. I would check that a) 10.0.0 is new enough to include that improvement and that b) you configure a regular token too for this. | 04:38 |
| Hi Clark , the regular user token can you be more specific regarding which particular token to use. I've created a Personal Access Token and gave it as the api_token in github connection configuration in zuul.conf, but I am still facing the issue of unauthenticated requests and hitting the rate limit. | ||
| @bennetefx:matrix.org | > <@clarkb:matrix.org> We recently updated zuul (I can't remember if this was before or after 10.0.0) to use a regular user token for when the app isn't installed if you configure a regular token too. This gives you significantly higher quota than doing unauthenticated requests. I would check that a) 10.0.0 is new enough to include that improvement and that b) you configure a regular token too for this. | 07:04 |
| * Hi Clark , the regular user token can you be more specific regarding which particular token to use. I've created a Personal Access Token and gave it as the api\_token in github connection configuration in zuul.conf, but I am still facing the issue of unauthenticated requests and hitting the rate limit. I am going the github app way for connecting zuul and github. | ||
| [connection "github-bennet"]driver=github | ||
| #app_id=980478 | ||
| #app_key=/etc/zuul/connections/bennet_github/bennet-zuul-app | ||
| webhook_token=secret-token | ||
| api_token=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 | ||
| @bennetefx:matrix.org | * Hi Clark , the regular user token can you be more specific regarding which particular token to use. I've created a Personal Access Token and gave it as the api\_token in github connection configuration in zuul.conf, but I am still facing the issue of unauthenticated requests and hitting the rate limit. I am going the github app way for connecting zuul and github. | 07:04 |
| \[connection "github-bennet"\]driver=github | ||
| #app\_id=980478 | ||
| #app\_key=/etc/zuul/connections/bennet\_github/bennet-zuul-app | ||
| webhook\_token=secret-token | ||
| api\_token=ghp\ | ||
| @bennetefx:matrix.org | * Hi Clark , the regular user token can you be more specific regarding which particular token to use. I've created a Personal Access Token and gave it as the api\_token in github connection configuration in zuul.conf, but I am still facing the issue of unauthenticated requests and hitting the rate limit. I am going the github app way for connecting zuul and github. | 07:05 |
| \[connection "github"\] | ||
| driver=github | ||
| app\_id= | ||
| #app\_key= | ||
| webhook\_token= | ||
| api\_token=ghp\\ | ||
| @bennetefx:matrix.org | * Hi Clark , the regular user token can you be more specific regarding which particular token to use. I've created a Personal Access Token and gave it as the api\_token in github connection configuration in zuul.conf, but I am still facing the issue of unauthenticated requests and hitting the rate limit. I am going the github app way for connecting zuul and github. | 07:06 |
| \[connection "github"\] | ||
| driver=github | ||
| app\_id=<GITHUB_APP_ID> | ||
| app\_key=<GITHUB_APP_KEY> | ||
| webhook\_token=<PERSONAL_ACCESS_TOKEN> | ||
| api\_token=ghp\\ | ||
| @bennetefx:matrix.org | * Hi Clark , the regular user token can you be more specific regarding which particular token to use. I've created a Personal Access Token and gave it as the api\_token in github connection configuration in zuul.conf, but I am still facing the issue of unauthenticated requests and hitting the rate limit. I am going the github app way for connecting zuul and github. | 07:07 |
| \[connection "github"\] | ||
| driver=github | ||
| app\_id=\<GITHUB\_APP\_ID> | ||
| app\_key=\<GITHUB\_APP\_KEY> | ||
| webhook\_token=<WEBHOOK_SECRET_TOKEN> | ||
| api\_token=<PERSONAL\_ACCESS\_TOKEN> | ||
| @bennetefx:matrix.org | * Hi Clark , the regular user token can you be more specific regarding which particular token to use. I've created a Personal Access Token and gave it as the api\_token in github connection configuration in zuul.conf, but I am still facing the issue of unauthenticated requests and hitting the rate limit. I am going the github app way for connecting zuul and github. | 07:07 |
| This is my github connection configuration in zuul.conf | ||
| \[connection "github"\] | ||
| driver=github | ||
| app\_id=\<GITHUB\_APP\_ID> | ||
| app\_key=\<GITHUB\_APP\_KEY> | ||
| webhook\_token=\<WEBHOOK\_SECRET\_TOKEN> | ||
| api\_token=\<PERSONAL\_ACCESS\_TOKEN> | ||
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 927376: Complete preliminary provider node lifecycle https://review.opendev.org/c/zuul/zuul/+/927376 | 10:41 | |
| @bennetefx:matrix.org | * Hi Clark , the regular user token can you be more specific regarding which particular token to use. I've created a Personal Access Token and gave it as the api\_token in github connection configuration in zuul.conf, but I am still facing the issue of unauthenticated requests and hitting the rate limit. I am going the github app way for connecting zuul and github. | 10:41 |
| This is my github connection configuration in zuul.conf | ||
| \[connection "github"\] | ||
| driver=github | ||
| app\_id=\<GITHUB\_APP\_ID> | ||
| app\_key=\<GITHUB\_APP\_KEY> | ||
| webhook\_token=\<WEBHOOK\_SECRET\_TOKEN> | ||
| api\_token=\<PERSONAL\_ACCESS\_TOKEN> | ||
| I followed this documentation to integrate github as a zuul connection: | ||
| https://zuul-ci.org/docs/zuul/latest/drivers/github.html | ||
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 927376: Complete preliminary provider node lifecycle https://review.opendev.org/c/zuul/zuul/+/927376 | 10:43 | |
| @bennetefx:matrix.org | * Hi Clark , the regular user token can you be more specific regarding which particular token to use. I've created a Personal Access Token and gave it as the api\_token in github connection configuration in zuul.conf, but I am still facing the issue of unauthenticated requests and hitting the rate limit. I am going the github app way for connecting zuul and github. I am also facing issue when using "Depends-On" feature, scheduler can't get the dependent pull request. | 10:43 |
| This is my github connection configuration in zuul.conf | ||
| \[connection "github"\] | ||
| driver=github | ||
| app\_id=\<GITHUB\_APP\_ID> | ||
| app\_key=\<GITHUB\_APP\_KEY> | ||
| webhook\_token=\<WEBHOOK\_SECRET\_TOKEN> | ||
| api\_token=\<PERSONAL\_ACCESS\_TOKEN> | ||
| I followed this documentation to integrate github as a zuul connection: | ||
| https://zuul-ci.org/docs/zuul/latest/drivers/github.html | ||
| Please do tell if I missed any step. | ||
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 927376: Complete preliminary provider node lifecycle https://review.opendev.org/c/zuul/zuul/+/927376 | 11:17 | |
| @praveen08:matrix.org | corvus: I have a doubt regarding basic usage of periodic pipeline? Can you explain the periodic pipeline? | 11:24 |
| @jangutter:matrix.org | praveen08: sorry, but can you clarify, do you perhaps mean you have a question about the periodic pipeline? | 12:08 |
| @jangutter:matrix.org | There is nothing "special" about the periodic pipeline (an example definition is here: https://opendev.org/zuul/project-config/src/branch/master/zuul.d/pipelines.yaml#L119 ) | 12:20 |
| The definition of pipelines is explained more in-depth here: https://zuul-ci.org/docs/zuul/latest/config/pipeline.html | ||
| The model I use to understand it is like so: | ||
| * a pipeline merely defines triggers and some actions to take. | ||
| * triggers can be things like "someone uploaded a new patchset", "someone commented on a patchset", "a git tag got created", or "the clock reached a specific hour" | ||
| * actions can be "add a comment to the patchset", "merge the patchset", or various other things. | ||
| But, to associate a job with a pipeline, you need a project config. So a pipeline definition is global, but each project can select which jobs to use in which pipelines. | ||
| In Zuul, for example, you can add the same job to multiple pipelines: | ||
| * in the check pipeline, the CI tells you whether it thinks it would fail when it merges. | ||
| * in the gate pipeline, the CI ensures that nothing changed between the last time you ran check and the time you want to merge. | ||
| * in the periodic pipeline, the CI tells you whether your "job is healthy". In a well-managed Zuul setup, a failure in the periodic pipeline is a signal that something unforeseen changed and the job has developed false negatives. | ||
| @jangutter:matrix.org | (these are all super high level generalisations - but I've found it as a useful way to remember how Zuul's definition of 'pipeline', 'job' and 'project' interact. | 12:23 |
| -@gerrit:opendev.org- Felix Edel proposed: | 12:34 | |
| - [zuul/zuul] 916284: Implement new status page https://review.opendev.org/c/zuul/zuul/+/916284 | ||
| - [zuul/zuul] 916285: Make helper functions available to other components https://review.opendev.org/c/zuul/zuul/+/916285 | ||
| - [zuul/zuul] 916286: Implement QueueItemPopover https://review.opendev.org/c/zuul/zuul/+/916286 | ||
| - [zuul/zuul] 916287: Implement PipelineDetails view https://review.opendev.org/c/zuul/zuul/+/916287 | ||
| - [zuul/zuul] 916744: Visualize branches in ChangeQueues https://review.opendev.org/c/zuul/zuul/+/916744 | ||
| - [zuul/zuul] 916867: Implement admin actions (promote, dequeue) in new QueueItem component https://review.opendev.org/c/zuul/zuul/+/916867 | ||
| - [zuul/zuul] 916973: Show queue lengths and fetching state on status page https://review.opendev.org/c/zuul/zuul/+/916973 | ||
| - [zuul/zuul] 921790: Hide empty pipelines and queues on status page https://review.opendev.org/c/zuul/zuul/+/921790 | ||
| - [zuul/zuul] 917952: Show last reconfiguration time on status page https://review.opendev.org/c/zuul/zuul/+/917952 | ||
| - [zuul/zuul] 923112: Implement auto-reload on new status page https://review.opendev.org/c/zuul/zuul/+/923112 | ||
| - [zuul/zuul] 925096: Make QueueItem cards in ChangeQueue more concise https://review.opendev.org/c/zuul/zuul/+/925096 | ||
| - [zuul/zuul] 926404: Port "Fix web dequeue of periodic items" to new status UI https://review.opendev.org/c/zuul/zuul/+/926404 | ||
| - [zuul/zuul] 926471: Hide red cross icon next to progress bar for pre-fail jobs https://review.opendev.org/c/zuul/zuul/+/926471 | ||
| - [zuul/zuul] 926514: Show user-created pipeline description in Tooltip https://review.opendev.org/c/zuul/zuul/+/926514 | ||
| - [zuul/zuul] 926537: Use queue.name to differentiate between different queue visualizations https://review.opendev.org/c/zuul/zuul/+/926537 | ||
| - [zuul/zuul] 926570: Overwrite job result label colors to match the old status page design https://review.opendev.org/c/zuul/zuul/+/926570 | ||
| - [zuul/zuul] 926612: Show job waiting status in tooltip https://review.opendev.org/c/zuul/zuul/+/926612 | ||
| - [zuul/zuul] 926642: Hide skipped jobs on new status page by default https://review.opendev.org/c/zuul/zuul/+/926642 | ||
| - [zuul/zuul] 926722: Show window size and queue length per queue on new status page https://review.opendev.org/c/zuul/zuul/+/926722 | ||
| - [zuul/zuul] 926989: Show failure reason via tooltip on progress stepper icon https://review.opendev.org/c/zuul/zuul/+/926989 | ||
| - [zuul/zuul] 927255: Add share icon to QueueItem actions https://review.opendev.org/c/zuul/zuul/+/927255 | ||
| - [zuul/zuul] 927282: New animation for starting jobs https://review.opendev.org/c/zuul/zuul/+/927282 | ||
| -@gerrit:opendev.org- Felix Edel proposed on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 12:34 | |
| - [zuul/zuul] 926426: Implement tile scrubbing in status overview https://review.opendev.org/c/zuul/zuul/+/926426 | ||
| - [zuul/zuul] 923778: Add ability to expand pipeline contents in overview https://review.opendev.org/c/zuul/zuul/+/923778 | ||
| - [zuul/zuul] 925922: Expand queue items to the full width of column https://review.opendev.org/c/zuul/zuul/+/925922 | ||
| - [zuul/zuul] 927028: Use InfoIcon for non-live items https://review.opendev.org/c/zuul/zuul/+/927028 | ||
| -@gerrit:opendev.org- Felix Edel proposed on behalf of Benjamin Schanzel: | 12:34 | |
| - [zuul/zuul] 924067: Add filtering support for pipeline status views https://review.opendev.org/c/zuul/zuul/+/924067 | ||
| - [zuul/zuul] 926467: Add a switch to expand all jobs in PipelineDetails https://review.opendev.org/c/zuul/zuul/+/926467 | ||
| -@gerrit:opendev.org- Benjamin Schanzel proposed on behalf of Felix Edel: | 12:48 | |
| - [zuul/zuul] 926471: Hide red cross icon next to progress bar for pre-fail jobs https://review.opendev.org/c/zuul/zuul/+/926471 | ||
| - [zuul/zuul] 926514: Show user-created pipeline description in Tooltip https://review.opendev.org/c/zuul/zuul/+/926514 | ||
| - [zuul/zuul] 926537: Use queue.name to differentiate between different queue visualizations https://review.opendev.org/c/zuul/zuul/+/926537 | ||
| - [zuul/zuul] 926570: Overwrite job result label colors to match the old status page design https://review.opendev.org/c/zuul/zuul/+/926570 | ||
| - [zuul/zuul] 926612: Show job waiting status in tooltip https://review.opendev.org/c/zuul/zuul/+/926612 | ||
| - [zuul/zuul] 926642: Hide skipped jobs on new status page by default https://review.opendev.org/c/zuul/zuul/+/926642 | ||
| - [zuul/zuul] 926722: Show window size and queue length per queue on new status page https://review.opendev.org/c/zuul/zuul/+/926722 | ||
| - [zuul/zuul] 926989: Show failure reason via tooltip on progress stepper icon https://review.opendev.org/c/zuul/zuul/+/926989 | ||
| - [zuul/zuul] 927255: Add share icon to QueueItem actions https://review.opendev.org/c/zuul/zuul/+/927255 | ||
| - [zuul/zuul] 927282: New animation for starting jobs https://review.opendev.org/c/zuul/zuul/+/927282 | ||
| -@gerrit:opendev.org- Benjamin Schanzel proposed: | 12:48 | |
| - [zuul/zuul] 926467: Add a switch to expand all jobs in PipelineDetails https://review.opendev.org/c/zuul/zuul/+/926467 | ||
| - [zuul/zuul] 927385: Store "Show all pipelines" state in localStorage https://review.opendev.org/c/zuul/zuul/+/927385 | ||
| -@gerrit:opendev.org- Benjamin Schanzel proposed on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 927028: Use InfoIcon for non-live items https://review.opendev.org/c/zuul/zuul/+/927028 | 12:48 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 927388: Handle create state-machine errors as node failure https://review.opendev.org/c/zuul/zuul/+/927388 | 12:59 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: | 13:31 | |
| - [zuul/zuul] 927376: Complete preliminary provider node lifecycle https://review.opendev.org/c/zuul/zuul/+/927376 | ||
| - [zuul/zuul] 927388: Handle create state-machine errors as node failure https://review.opendev.org/c/zuul/zuul/+/927388 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 927411: Avoid fstrings in test-prepare-workspace-git https://review.opendev.org/c/zuul/zuul-jobs/+/927411 | 16:15 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 927412: Synchronize test-prepare-workspace-git to prepare-workspace-git https://review.opendev.org/c/zuul/zuul-jobs/+/927412 | 16:17 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 16:22 | |
| - [zuul/zuul-jobs] 927411: Avoid fstrings in test-prepare-workspace-git https://review.opendev.org/c/zuul/zuul-jobs/+/927411 | ||
| - [zuul/zuul-jobs] 927412: Synchronize test-prepare-workspace-git to prepare-workspace-git https://review.opendev.org/c/zuul/zuul-jobs/+/927412 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 16:33 | |
| - [zuul/zuul-jobs] 927411: Avoid fstrings in test-prepare-workspace-git https://review.opendev.org/c/zuul/zuul-jobs/+/927411 | ||
| - [zuul/zuul-jobs] 927412: Synchronize test-prepare-workspace-git to prepare-workspace-git https://review.opendev.org/c/zuul/zuul-jobs/+/927412 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 16:35 | |
| - [zuul/zuul-jobs] 927411: Avoid fstrings in test-prepare-workspace-git https://review.opendev.org/c/zuul/zuul-jobs/+/927411 | ||
| - [zuul/zuul-jobs] 927412: Synchronize test-prepare-workspace-git to prepare-workspace-git https://review.opendev.org/c/zuul/zuul-jobs/+/927412 | ||
| @clarkb:matrix.org | corvus: in https://review.opendev.org/c/zuul/zuul-jobs/+/927411/4/zuul-tests.d/general-roles-jobs.yaml do you need to change the playbook that the new test jobs run so that we can run the different role? | 16:51 |
| @clarkb:matrix.org | otherwise it isn't claer to me why/how the test role is selected instead of the main role | 16:51 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 16:52 | |
| - [zuul/zuul-jobs] 927411: Avoid fstrings in test-prepare-workspace-git https://review.opendev.org/c/zuul/zuul-jobs/+/927411 | ||
| - [zuul/zuul-jobs] 927412: Synchronize test-prepare-workspace-git to prepare-workspace-git https://review.opendev.org/c/zuul/zuul-jobs/+/927412 | ||
| @jim:acmegating.com | Clark: indeed :) | 16:52 |
| @jim:acmegating.com | running those jobs is a bit more complex/duplicative than i hoped... not sure i love it, but figured it's worth a try. | 16:53 |
| @jim:acmegating.com | maybe we should add variable switches to the playbooks and have a combined set; i could see them getting out of sync pretty easily | 16:54 |
| @clarkb:matrix.org | ya being explicit is wordy but probably straightforward to understand in the future | 16:55 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 16:56 | |
| - [zuul/zuul-jobs] 927411: Avoid fstrings in test-prepare-workspace-git https://review.opendev.org/c/zuul/zuul-jobs/+/927411 | ||
| - [zuul/zuul-jobs] 927412: Synchronize test-prepare-workspace-git to prepare-workspace-git https://review.opendev.org/c/zuul/zuul-jobs/+/927412 | ||
| @jim:acmegating.com | Clark: fungi ^ those are green now | 17:09 |
| @jim:acmegating.com | Albin Vass: fyi ^ | 17:09 |
| @fungicide:matrix.org | thanks! both lgtm | 17:17 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul-jobs] 927411: Avoid fstrings in test-prepare-workspace-git https://review.opendev.org/c/zuul/zuul-jobs/+/927411 | 17:39 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 927422: DNM: test base-test https://review.opendev.org/c/zuul/zuul-jobs/+/927422 | 17:40 | |
| -@gerrit:opendev.org- Tristan Cacqueray https://matrix.to/#/@tristanc_:matrix.org proposed: [zuul/zuul] 926803: Add support for log_url bigger than 255 characters https://review.opendev.org/c/zuul/zuul/+/926803 | 18:05 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed on behalf of Tobias Henkel: [zuul/zuul] 922450: Add spec for OIDC Workload Identity Federation https://review.opendev.org/c/zuul/zuul/+/922450 | 18:13 | |
| @sylvass:albinvass.se | > <@jim:acmegating.com> Albin Vass: fyi ^ | 19:42 |
| I take f-strings for granted, now I know I probably shouldn't for opendev related things :) | ||
| +2 | ||
| @jim:acmegating.com | heh, well, it's technically out of the support window for everything, including opendev, but for this case, since it's so fundamental, i thought it worth fixing and best-effort supporting :) | 19:43 |
| @fungicide:matrix.org | yeah, i mean, i was perfectly fine with dropping the xenial test in opendev instead (or in addition) | 20:11 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul-jobs] 927412: Synchronize test-prepare-workspace-git to prepare-workspace-git https://review.opendev.org/c/zuul/zuul-jobs/+/927412 | 20:20 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/nodepool] 926363: Add extra safety to metastatic bnr cleanup https://review.opendev.org/c/zuul/nodepool/+/926363 | 21:05 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Dong Zhang: [zuul/nodepool] 924932: Add a test for backing node deletion errors https://review.opendev.org/c/zuul/nodepool/+/924932 | 21:05 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Ruisi Jian: [zuul/zuul] 927298: fix(kafka): fix config key of gerritkafka https://review.opendev.org/c/zuul/zuul/+/927298 | 21:23 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!