| *** dviroel|rover is now known as dviroel|out | 00:11 | |
| opendevreview | Ian Wienand proposed openstack/project-config master: nodepool: infra-package-needs; cleanup python https://review.opendev.org/c/openstack/project-config/+/872476 | 00:24 |
|---|---|---|
| opendevreview | Ian Wienand proposed openstack/project-config master: nodepool: infra-package-needs; remove lvm2 https://review.opendev.org/c/openstack/project-config/+/872477 | 00:24 |
| opendevreview | Ian Wienand proposed openstack/project-config master: nodepool: infra-package-needs; cleanup tox installs https://review.opendev.org/c/openstack/project-config/+/872478 | 00:24 |
| opendevreview | Ian Wienand proposed openstack/project-config master: nodepool: infra-package-needs; cleanup tox installs https://review.opendev.org/c/openstack/project-config/+/872478 | 00:46 |
| *** yadnesh|away is now known as yadnesh | 05:51 | |
| *** jpena|off is now known as jpena | 08:22 | |
| *** rlandy|out is now known as rlandy | 11:17 | |
| *** dviroel|out is now known as dviroel|rover | 11:37 | |
| *** blarnath is now known as d34dh0r53 | 12:08 | |
| opendevreview | daniel.pawlik proposed openstack/ci-log-processing master: Add timeout parameter to avoid freeze https://review.opendev.org/c/openstack/ci-log-processing/+/872425 | 13:08 |
| *** dasm|off is now known as dasm | 13:53 | |
| opendevreview | daniel.pawlik proposed openstack/ci-log-processing master: Add timeout parameter to avoid freeze https://review.opendev.org/c/openstack/ci-log-processing/+/872425 | 13:53 |
| *** dabukalam6 is now known as dabukalam | 15:26 | |
| *** yadnesh is now known as yadnesh|away | 15:51 | |
| ralonsoh | ianw, hi! can I ask you a question? By default when using a list of "required-projects", the project in this list is cloned and the "master" branch used | 16:51 |
| ralonsoh | what happens if this project doesn't have "master" branch? | 16:51 |
| ralonsoh | --> https://zuul.opendev.org/t/openstack/build/27d32914cf7a410fbadb3ce952e9899f | 16:51 |
| ralonsoh | it has "main" branch | 16:52 |
| fungi | ralonsoh: we can set a default branch explicitly | 16:53 |
| ralonsoh | how can I? | 16:53 |
| clarkb | ansible is an example of this iirc | 16:53 |
| ralonsoh | I'll check that | 16:53 |
| ralonsoh | ahh with override-checkout, right? | 16:54 |
| ralonsoh | clarkb, fungi thank you!! | 16:54 |
| clarkb | yes that appears to be what we use for ansible. I think there is a way to say ansible/s default branch is main in the zuul tenant config too though | 16:54 |
| clarkb | that might be more correct | 16:55 |
| fungi | ralonsoh: literally default-branch: https://opendev.org/openstack/project-config/src/branch/master/gerrit/projects.yaml#L176 | 16:55 |
| fungi | er, wait, wrong file | 16:55 |
| clarkb | fungi: ^ that config doesn't affect zuul | 16:55 |
| fungi | that's our jeepyb syntax | 16:55 |
| fungi | https://opendev.org/opendev/project-config/src/branch/master/zuul.d/projects.yaml#L10 | 16:56 |
| clarkb | aha its the zuul project config. | 16:56 |
| fungi | here's an example from the openstack tenant: https://opendev.org/openstack/project-config/src/branch/master/zuul.d/projects.yaml#L26 | 16:57 |
| fungi | and another: https://opendev.org/openstack/project-config/src/branch/master/zuul.d/projects.yaml#L48 | 16:57 |
| ralonsoh | thanks, I'll use that | 16:57 |
| fungi | yeah, for some reason i was initially looking for it in the tenant config rather than the projects | 16:58 |
| clarkb | I suspect the reason we didn't do that for ansible is we don't really want a project definition for it (which would report merge errors?) | 17:00 |
| fungi | ralonsoh: for reference, here's the documentation for that option: https://zuul-ci.org/docs/zuul/latest/config/project.html#attr-project.default-branch | 17:03 |
| ralonsoh | yeah, better than using override-checkout for all job definitions using this repo | 17:04 |
| opendevreview | Rodolfo Alonso proposed openstack/project-config master: Set "main" as default branch of "sqlalchemy/sqlalchemy" https://review.opendev.org/c/openstack/project-config/+/872591 | 17:08 |
| clarkb | ya this is the situation I was worried about. I'm not sure if that is safe? | 17:09 |
| clarkb | zuul will want to report merge errors? | 17:09 |
| clarkb | I guess if sqlalchmey doesn't add the app to their repo it will just silently fail? | 17:09 |
| ralonsoh | clarkb, should we override the branch per job definition instead? | 17:10 |
| ralonsoh | we are pushing now the patches (Neutron and stadium projects) | 17:10 |
| clarkb | I'm not sure. I'm mostly wondering why we treat ansible the way we do and thats the only possibility I can come up with | 17:11 |
| *** jpena is now known as jpena|off | 17:34 | |
| fungi | i see notifications from pypi about some projects where other collaborators are being removed from openstack projects, and one case so far where openstackci was promoted to owner. just a heads up that i'm moving the ones i read to our pypi folder, but also that it seems some teams are getting traction on pypi collaborator cleanup | 18:01 |
| clarkb | ralonsoh: thinking about it more I think your change is fine. You would need to add the github zuul app to the repo for it to create noise | 18:16 |
| clarkb | and as you say this ensures that all the jobs don't need an update, it will just work | 18:16 |
| clarkb | however there is an unknown configuration error so zuul -1'd | 18:17 |
| clarkb | I'm not sure I'll be able to look at that for a bit as I'm trying to correct docker 23 release problems | 18:17 |
| ralonsoh | clarkb, thanks for checking that! | 19:34 |
| opendevreview | Merged openstack/project-config master: nodepool: infra-package-needs; drop curl https://review.opendev.org/c/openstack/project-config/+/872473 | 22:08 |
| *** dviroel|rover is now known as dviroel|rout | 22:38 | |
| *** dasm is now known as dasm|off | 22:42 | |
| *** rlandy is now known as rlandy|out | 22:57 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!