| dpawlik | any chance to review https://review.opendev.org/c/zuul/zuul-jobs/+/997002 and merge that change? | 07:29 |
|---|---|---|
| *** ykarel_ is now known as ykarel | 09:13 | |
| fungi | i already did, but i try to avoid single-core approving changes in the zuul/zuul-jobs repo because the potential for impact to other zuul users outside opendev is significantly higher than for our own config repos | 12:34 |
| dpawlik | understand, thank you fungi | 15:25 |
| sean-k-mooney | so if i remember correctly queue in zuul are configure in the tenet config correct | 17:18 |
| sean-k-mooney | Change 994580,7 in project openstack/cyborg-tempest-plugin does not share a change queue with 997365 in project openstack/cyborg | 17:19 |
| sean-k-mooney | so if i want to fix that i need to add cybrog-tempet-plugin to the share integrated queue | 17:19 |
| sean-k-mooney | or move cyborg ot a sperate one | 17:19 |
| fungi | sean-k-mooney: yes, one of those options | 17:19 |
| sean-k-mooney | ack, depens on is workign fine in check just not in gate | 17:19 |
| fungi | note that a project can only belong to one queue in a pipeline | 17:20 |
| sean-k-mooney | yep addign it ot the same common integrated queue that nova ectra is in is probaly the simpler long term chioce | 17:20 |
| fungi | depends-on technically also works in the gate pipeline, but prevents the depending change from being enqueued if the change it depends-on hasn't merged | 17:20 |
| sean-k-mooney | even if that will slow merging | 17:20 |
| sean-k-mooney | ah ok, well in any case i can just reslvoe the merge order manually | 17:21 |
| sean-k-mooney | so that not a bit issue ill just rechcek this after the tempest plugin change merges | 17:21 |
| sean-k-mooney | its just nice to let zuul do it :) | 17:21 |
| sean-k-mooney | "use-nodepool: false" is that how you make a tenant opt into the zuul laucher? | 17:23 |
| sean-k-mooney | i kind of assumed you either deploy with zuul-launcher or nodepool but not both | 17:23 |
| fungi | we designed it to be possible to run different tenants on each, in order to ease our multi-tenant migration | 17:25 |
| fungi | so we moved from nodepool to zuul-launcher (and back in some cases) in different tenants at different times | 17:25 |
| sean-k-mooney | cool | 17:25 |
| sean-k-mooney | ok os https://github.com/openstack/project-config/blob/master/zuul.d/projects.yaml#L1715-L1732 is the problem | 17:26 |
| fungi | and yes i think it's the `use-nodepool` toggle, but documentation should confirm | 17:26 |
| sean-k-mooney | i either need remove queue:cyborg form the tempest plugin or add queue:integrated to cybrog and the plugin | 17:27 |
| sean-k-mooney | or put them all on queue: cyborg i guess | 17:28 |
| fungi | https://zuul-ci.org/docs/zuul/latest/tenants.html#attr-tenant.use-nodepool | 17:28 |
| fungi | so that's only for label selection | 17:28 |
| sean-k-mooney | it really just come dont to if i care to be able to gate with cross repo depencies or not | 17:28 |
| sean-k-mooney | fungi: on i sse | 17:29 |
| fungi | you can actually have different labels provided by zuul-launcher and nodepool in the same tenant too if memory serves, i think we did that during transitional periods | 17:29 |
| sean-k-mooney | that what the fallback implies | 17:29 |
| fungi | it's just that toggle determines which one wins the fallback determination | 17:29 |
| sean-k-mooney | presuablly nodepool will eventually be discontinued | 17:30 |
| sean-k-mooney | so it makes sense for the transition | 17:30 |
| sean-k-mooney | wheil both are supproted | 17:30 |
| fungi | yes, that's the plan. this is for backward-compatibility in existing deployments | 17:30 |
| fungi | but it's officially deprecated | 17:30 |
| fungi | https://zuul-ci.org/docs/zuul/latest/releasenotes.html#relnotes-14-0-0-deprecation-notes | 17:31 |
| fungi | at some point that option may default to false, or simply get ignored, prior to or when nodepool support is removed entirely | 17:32 |
| fungi | it'll definitely be on a major version release though, following zuul's deprecation policy | 17:32 |
| fungi | we just don't currently (afaik) have any solid plans for when and what the version number will be | 17:33 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!