| @iwienand:matrix.org | Clark: https://review.opendev.org/c/zuul/zuul/+/794688 lgtm -- i left a comment you might like to consider that gives a little more depth to the examples? but if you feel it's too much i think it's still a good improvement | 00:29 |
|---|---|---|
| -@gerrit:opendev.org- daniel.pawlik https://matrix.to/#/@dpawlik:matrix.org proposed: [zuul/zuul-jobs] 894755: Add feature to set --vm-driver name for minikube https://review.opendev.org/c/zuul/zuul-jobs/+/894755 | 07:21 | |
| -@gerrit:opendev.org- daniel.pawlik https://matrix.to/#/@dpawlik:matrix.org proposed: [zuul/zuul-jobs] 894755: Add feature to set --vm-driver name for minikube https://review.opendev.org/c/zuul/zuul-jobs/+/894755 | 07:33 | |
| -@gerrit:opendev.org- daniel.pawlik https://matrix.to/#/@dpawlik:matrix.org proposed: [zuul/zuul-jobs] 895597: Bump cri-o default version to 1.26 https://review.opendev.org/c/zuul/zuul-jobs/+/895597 | 07:45 | |
| -@gerrit:opendev.org- daniel.pawlik https://matrix.to/#/@dpawlik:matrix.org proposed: [zuul/zuul-jobs] 894755: Add feature to set --vm-driver name for minikube https://review.opendev.org/c/zuul/zuul-jobs/+/894755 | 07:46 | |
| -@gerrit:opendev.org- Benjamin Schanzel proposed on behalf of Benedikt Löffler: [zuul/nodepool] 802255: Use optional upload script for uploading an image https://review.opendev.org/c/zuul/nodepool/+/802255 | 07:46 | |
| -@gerrit:opendev.org- daniel.pawlik https://matrix.to/#/@dpawlik:matrix.org proposed: [zuul/zuul-jobs] 895597: Bump cri-o default version to 1.26 https://review.opendev.org/c/zuul/zuul-jobs/+/895597 | 07:50 | |
| -@gerrit:opendev.org- daniel.pawlik https://matrix.to/#/@dpawlik:matrix.org proposed: [zuul/zuul-jobs] 895597: Bump cri-o default version to 1.28 https://review.opendev.org/c/zuul/zuul-jobs/+/895597 | 08:09 | |
| @bb_erg:matrix.org | requesting re-review on: https://review.opendev.org/c/zuul/zuul-jobs/+/887917 | 08:19 |
| -@gerrit:opendev.org- daniel.pawlik https://matrix.to/#/@dpawlik:matrix.org proposed: [zuul/zuul-jobs] 895604: Change CoreDNS configuration for Minikube https://review.opendev.org/c/zuul/zuul-jobs/+/895604 | 08:38 | |
| -@gerrit:opendev.org- daniel.pawlik https://matrix.to/#/@dpawlik:matrix.org proposed: [zuul/zuul-jobs] 895606: Fix tox job when stestr is used https://review.opendev.org/c/zuul/zuul-jobs/+/895606 | 08:55 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 895660: dnm: Temporarily disable early failure https://review.opendev.org/c/zuul/zuul/+/895660 | 13:21 | |
| @clarkb:matrix.org | corvus: do you have an opinion on ianw's more indepth suggestion on https://review.opendev.org/c/zuul/zuul/+/794688 ? | 15:41 |
| @clarkb:matrix.org | I'm happy to update to include the suggested expansion (replacing merge request with pull request since github does pull requests). You had style input previuosly so want ot make sure you are happy with it too before pushing | 15:42 |
| @jim:acmegating.com | Clark: no objections. i might s/specific branch of a GitHub project/the contents of the current branch of a GitHub project/ but that's a bit of a nit. | 15:56 |
| @jim:acmegating.com | and i guess if we wanted to really polish it, we should probably linkify that with `:attr:\'job.required-projects\` | 15:57 |
| @jim:acmegating.com | * and i guess if we wanted to really polish it, we should probably linkify that with `:attr:\'job.required-projects\`` | 15:57 |
| @jim:acmegating.com | * and i guess if we wanted to really polish it, we should probably linkify that with ```:attr:'job.required-projects` ``` | 15:57 |
| -@gerrit:opendev.org- Clark Boylan proposed on behalf of Ian Wienand: [zuul/zuul] 794688: github: fallback to api_token when can't find installation https://review.opendev.org/c/zuul/zuul/+/794688 | 16:29 | |
| @clarkb:matrix.org | that should do it. I included the fancy link to required-projects too | 16:30 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 895708: Revert "Disable base role testing that runs code on localhost" https://review.opendev.org/c/zuul/zuul-jobs/+/895708 | 17:18 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 895253: Prevent exception when getting namespace PIDs https://review.opendev.org/c/zuul/zuul/+/895253 | 17:19 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 17:36 | |
| - [zuul/zuul-jobs] 895708: Revert "Disable base role testing that runs code on localhost" https://review.opendev.org/c/zuul/zuul-jobs/+/895708 | ||
| - [zuul/zuul-jobs] 895715: Pin python-subunit in fetch-subunit-output test https://review.opendev.org/c/zuul/zuul-jobs/+/895715 | ||
| - [zuul/zuul-jobs] 895716: Unpin stestr and python-subunit in fetch-subunit-output test https://review.opendev.org/c/zuul/zuul-jobs/+/895716 | ||
| @jim:acmegating.com | Clark: that's a pleasant revert of a 3 year old change from you ^ :) i think we can merge all 3 of those now, and close a hole in our testing. | 19:30 |
| @clarkb:matrix.org | corvus: ack I'll look shortyl | 19:34 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 895518: Load config errors less often https://review.opendev.org/c/zuul/zuul/+/895518 | 22:32 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 895742: Add tenant_status web endpoint https://review.opendev.org/c/zuul/zuul/+/895742 | 22:46 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!