Tuesday, 2023-06-06

@vlotorev:matrix.orgHi zuul-maint, could you have a look at https://review.opendev.org/c/zuul/zuul/+/878677 ?00:34
This is one-liner fix for improper linkification in ReST documentation.
-@gerrit:opendev.org- lotorev vitaly proposed: [zuul/zuul] 878677: doc: Add underscore for ReST external link https://review.opendev.org/c/zuul/zuul/+/87867707:27
@vlotorev:matrix.org> <@gerrit:opendev.org> lotorev vitaly proposed: [zuul/zuul] 878677: doc: Add underscore for ReST external link  https://review.opendev.org/c/zuul/zuul/+/87867707:33
Updated the commit, the URL was improper itself and was wrongly linkified (both issues are fixed now).
@yoctozepto:matrix.orghi Zuulers; I have a general Zuul question but let me ask it in the scope of OpenDev's deployment; it has occurred to me that I don't quite get how Zuul decides which tenant a particular project belongs to if it (the project)'s placed in multiple tenants; for example, like the zuul/zuul-jobs project which is in opendev and zuul (and others) but its builds are properly attributed to the zuul tenant and not the others12:41
@avass:matrix.vassast.org> <@yoctozepto:matrix.org> hi Zuulers; I have a general Zuul question but let me ask it in the scope of OpenDev's deployment; it has occurred to me that I don't quite get how Zuul decides which tenant a particular project belongs to if it (the project)'s placed in multiple tenants; for example, like the zuul/zuul-jobs project which is in opendev and zuul (and others) but its builds are properly attributed to the zuul tenant and not the others12:52
The project basically belongs to all of them but i think in the opendev case only the zuul tenant loads project stanzas so only that tenant runs jobs for changes to the project
@avass:matrix.vassast.orgThe zuul tenants adds zuul-test.d as an extra config path: https://opendev.org/openstack/project-config/src/commit/b5e236a37ece2a31959142b282cb24186439921e/zuul/main.yaml#L163312:55
@yoctozepto:matrix.orgAh, I see now. So, normally, it would try running one buildset per each tenant?12:56
@avass:matrix.vassast.orgYep12:57
@yoctozepto:matrix.orgAll right, then it all makes sense to me now. Thank you!12:58
@rancher:matrix.org> <@fungicide:matrix.org> i wonder if that's supposed to be `gitlab/root/zuul-config` instead?13:32
The opposite, the other line which returns the `GetProjectMetadata` error should be `root/zuul-config`. That's what I specified in my tenants config. I suppose `gitlab` is extracted from the `server` parameter in `zuul.conf` (the default is `gitlab.com`).
I see that Quick-Start guide is changed. I cloned it and installed `podman`, but I cannot seem to access Zuul web interface on `localhost:9000` anymore. Gerrit works.
@rancher:matrix.orgI tried entering the IP address without `http` in `server`, but it didn't work either. I'll try again if I resolve this problem with Zuul.13:35
@rancher:matrix.org * I tried entering the IP address without `http://` in `server`, but it didn't work either. I'll try again if I resolve this problem with Zuul.13:36
@jim:acmegating.comRancher: i just double checked the podman-compose deployment and it worked locally.  you might try running `podman-compose -p zuul-tutorial down` and `docker-compose -p zuul-tutorial down` and then use `podman ps` and `docker ps` to make sure nothing is running before trying again.  also, make sure nothing else is running locally on port 9000.14:30
-@gerrit:opendev.org- Jeremy Stanley https://matrix.to/#/@fungicide:matrix.org proposed: [zuul/zuul] 885155: Add config metadata to identify project-templates https://review.opendev.org/c/zuul/zuul/+/88515517:59
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 885380: Add manager/reporter support for config warnings https://review.opendev.org/c/zuul/zuul/+/88538018:34
@fungicide:matrix.orgif someone gets a moment to look at https://review.opendev.org/885155 and leave a review comment, i'm trying to understand why the metadata ends up with is_template=True even though i tried to make it so that's only made true by the configloader for actual project-templates. also please tell me if that approach is just completely bonkers!20:02
@jim:acmegating.comfungi: on it20:06
@jim:acmegating.comfungi: done20:09
-@gerrit:opendev.org- Jeremy Stanley https://matrix.to/#/@fungicide:matrix.org proposed: [zuul/zuul] 885155: Add config metadata to identify project-templates https://review.opendev.org/c/zuul/zuul/+/88515520:20
@fungicide:matrix.orgthanks corvus!21:25
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed:22:00
- [zuul/zuul] 885415: Attach loading_errors to the ParseContext https://review.opendev.org/c/zuul/zuul/+/885415
- [zuul/zuul] 885416: Add configuration warnings https://review.opendev.org/c/zuul/zuul/+/885416
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com:22:01
- [zuul/zuul] 884939: Web: convert config errors to table https://review.opendev.org/c/zuul/zuul/+/884939
- [zuul/zuul] 884940: Web: remove config error drawer https://review.opendev.org/c/zuul/zuul/+/884940
@jim:acmegating.comfungi: re 155 -- the first error (cache) does not appear related to your change and is probably a non-deterministic failure; the second one (with the big data structure) i think just needs the extra attr added to the expected output22:01
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 885418: Correct node cache lock handling https://review.opendev.org/c/zuul/nodepool/+/88541823:16
@jim:acmegating.comClark: ^ re opendev observed behavior23:19
@clarkb:matrix.orgIs that urgent or is the exisitng behavior going to work for us until that lands?23:20
@clarkb:matrix.orgFrom what I could see it looked like Nodepool was functioning after I restarted it23:20
@jim:acmegating.comClark: as long as it keeps failing in the way it's currently doing, it should be tolerable.  there's a chance (but i haven't been able to confirm a path for this) that the opposite could happen and we could accumulate used nodes that don't get deleted.23:22
@jim:acmegating.com(basically, current failure is "keep trying to lock nodes that we expect to delete but actually are still in use" opposite would be "don't bother trying to even lock nodes which are actually no longer in use and should be deleted")23:23
@clarkb:matrix.orggotcha23:23
@clarkb:matrix.orgI'm reviewing it anyway23:23
@clarkb:matrix.orgcorvus: quick question in the tests if you have a second to check shouldn't take long23:29
@jim:acmegating.comreplied23:33
@clarkb:matrix.org+2 thanks23:33

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