Friday, 2022-02-25

-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 830708: Add a "dirty" flag to pipeline processing https://review.opendev.org/c/zuul/zuul/+/83070800:35
-@gerrit:opendev.org- Zuul merged on behalf of Tobias Henkel: [zuul/zuul] 755765: Call manageLoad during pause and unpause https://review.opendev.org/c/zuul/zuul/+/75576501:42
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 829829: Don't reconfigure after every gitlab merge https://review.opendev.org/c/zuul/zuul/+/82982901:54
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed:02:34
- [zuul/zuul] 830924: Rename MERGER_FAILURE to MERGE_CONFLICT https://review.opendev.org/c/zuul/zuul/+/830924
- [zuul/zuul] 830925: Add MERGE_FAILURE buildset result https://review.opendev.org/c/zuul/zuul/+/830925
-@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 830781: debug tests https://review.opendev.org/c/zuul/zuul/+/83078106:19
-@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 830781: debug tests https://review.opendev.org/c/zuul/zuul/+/83078106:20
-@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 830628: Items in extra paths should be loaded in dependent changes https://review.opendev.org/c/zuul/zuul/+/83062806:24
-@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 830781: debug tests https://review.opendev.org/c/zuul/zuul/+/83078108:02
-@gerrit:opendev.org- Benjamin Schanzel proposed: [zuul/zuul] 829867: Report gross/total tenant resource usage stats https://review.opendev.org/c/zuul/zuul/+/82986708:29
-@gerrit:opendev.org- Tobias Henkel proposed: [zuul/zuul] 830902: Simplify _saveRepoState https://review.opendev.org/c/zuul/zuul/+/83090208:52
-@gerrit:opendev.org- Benjamin Schanzel proposed: [zuul/zuul] 829867: Report gross/total tenant resource usage stats https://review.opendev.org/c/zuul/zuul/+/82986709:08
-@gerrit:opendev.org- Benjamin Schanzel proposed: [zuul/zuul] 829867: Report gross/total tenant resource usage stats https://review.opendev.org/c/zuul/zuul/+/82986709:57
-@gerrit:opendev.org- Florian Haas proposed: [zuul/zuul-jobs] 830992: tox: Include ansible_env in tox_environment https://review.opendev.org/c/zuul/zuul-jobs/+/83099214:15
-@gerrit:opendev.org- Florian Haas proposed: [zuul/zuul-jobs] 830992: tox: Include ansible_env in tox_environment https://review.opendev.org/c/zuul/zuul-jobs/+/83099214:46
-@gerrit:opendev.org- Florian Haas marked as active: [zuul/zuul-jobs] 830992: tox: Include ansible_env in tox_environment https://review.opendev.org/c/zuul/zuul-jobs/+/83099214:53
-@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 830995: Correctly reset failing cycle behind failing item https://review.opendev.org/c/zuul/zuul/+/83099515:09
-@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 830995: Correctly reset failing cycle behind failing item https://review.opendev.org/c/zuul/zuul/+/83099515:11
-@gerrit:opendev.org- Florian Haas proposed: [zuul/zuul-jobs] 830992: tox: Include ansible_env in tox_environment https://review.opendev.org/c/zuul/zuul-jobs/+/83099215:41
-@gerrit:opendev.org- Florian Haas proposed: [zuul/zuul-jobs] 830992: tox: Include ansible_env in tox_environment https://review.opendev.org/c/zuul/zuul-jobs/+/83099215:43
@clarkb:matrix.orgcorvus: thinking about landing that zuul-registry update. If it were to berak the buildset-registry what sort of revert process would we have? Does it make sense to tag zuul-registry now so that we have a tag in dockerhub that we can refer to in the zuul-jobs run-buildset-registry role?17:32
@clarkb:matrix.orgbasically a checkpoint at a known good?17:32
@clarkb:matrix.orgI guess we have a couple tags already /me checks how old they are17:32
@clarkb:matrix.orglooks like the 1.1.0 is before we updated to bullseye and my chagne so that should actually work as a valid fallback. Seems to exist on docker hub too: https://hub.docker.com/layers/zuul/zuul-registry/1.1.0/images/sha256-611ac3ce39a9ffa47ff58257118d59b32a9e5c2f9f774d3efb5c0da49ef67356?context=explore17:33
@clarkb:matrix.orgI'll approve that change shortly after some breakfast then17:33
@jim:acmegating.comClark: sounds good.  then if we're happy this would be a good one to tag again17:34
-@gerrit:opendev.org- Zuul merged on behalf of Clark Boylan: [zuul/zuul-registry] 830905: Atomically concatenate blob objects https://review.opendev.org/c/zuul/zuul-registry/+/83090518:22
@clarkb:matrix.orgI'm double checking ^ with 829975 in the openstack tenant. That makes use of the buildset registry and was tripping over the problem I hope that change addresses18:29
@clarkb:matrix.orgIn particular the large image layer of our gerrit image seems to be more likely to hit it18:29
-@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul-registry] 831054: Revert "Atomically concatenate blob objects" https://review.opendev.org/c/zuul/zuul-registry/+/83105421:38
@clarkb:matrix.orgI'm not sure that revert is necessary yet. It seems to have shifted the issue from a manifest integrity problem to an error when docker tries to HEAD the half written blob. That is now a 404 (I think haven't gotten access to the logs yet to verify) rather than a 200 with short size response21:39
@clarkb:matrix.orgBut I figured pushing the revert was a good idea so that it was ready if we device that going back to the old behavior is more reliable.21:39
@clarkb:matrix.orgFeel free to land that if we feel that the current situation is less good than the previous one. I suspect that we are currently at least as bad as before, but hard to say if we are worse. But being at least as bad means the revert should be safe21:40
@clarkb:matrix.orgcorvus: thinking about it on the walk to get the kids from school and I think we should revert in case the intermediate registry is affected differently or worse. 22:36
@clarkb:matrix.orgI can hit +A when I get home in half an hour or maybe you can do it when you return?22:37
@clarkb:matrix.orgOh wait I forgot it uses swift backend again22:41
@clarkb:matrix.orgIn that case I think we can wait and see unless buildset registry gets worse or someone sees something in the logs that indicate worse behavior22:41

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