Friday, 2025-07-04

*** ykarel__ is now known as ykarel04:10
opendevreviewStephen Finucane proposed openstack/pbr master: Use importlib.metadata where possible  https://review.opendev.org/c/openstack/pbr/+/95404209:13
opendevreviewStephen Finucane proposed openstack/pbr master: Use packaging where possible  https://review.opendev.org/c/openstack/pbr/+/95404309:13
opendevreviewStephen Finucane proposed openstack/pbr master: Cleanup removal of 'build_sphinx' command doc  https://review.opendev.org/c/openstack/pbr/+/95404409:13
opendevreviewStephen Finucane proposed openstack/pbr master: Remove 'test' distutils command override  https://review.opendev.org/c/openstack/pbr/+/95404509:13
opendevreviewStephen Finucane proposed openstack/pbr master: Move fixtures to their own module  https://review.opendev.org/c/openstack/pbr/+/95404609:13
opendevreviewStephen Finucane proposed openstack/pbr master: Remove support for custom compilers  https://review.opendev.org/c/openstack/pbr/+/95404709:13
opendevreviewStephen Finucane proposed openstack/pbr master: Move distutils commands to _compat  https://review.opendev.org/c/openstack/pbr/+/95404809:13
opendevreviewStephen Finucane proposed openstack/pbr master: Move functional tests to their own directory  https://review.opendev.org/c/openstack/pbr/+/95404909:13
opendevreviewStephen Finucane proposed openstack/pbr master: docs: Update usage guide and README  https://review.opendev.org/c/openstack/pbr/+/94905109:13
opendevreviewStephen Finucane proposed openstack/pbr master: Deprecate '[entry_points]' section  https://review.opendev.org/c/openstack/pbr/+/94905209:13
opendevreviewStephen Finucane proposed openstack/pbr master: Deprecate '[files]' section  https://review.opendev.org/c/openstack/pbr/+/94905309:13
opendevreviewStephen Finucane proposed openstack/pbr master: Deprecate various '[metadata]' options  https://review.opendev.org/c/openstack/pbr/+/94905409:13
opendevreviewStephen Finucane proposed openstack/pbr master: Deprecate various '[backwards_compat]' options  https://review.opendev.org/c/openstack/pbr/+/94905509:13
opendevreviewStephen Finucane proposed openstack/pbr master: Deprecate 'tests_require' option  https://review.opendev.org/c/openstack/pbr/+/95405009:13
opendevreviewStephen Finucane proposed openstack/pbr master: tox: Stop calling 'setup.py'  https://review.opendev.org/c/openstack/pbr/+/94918809:13
opendevreviewStephen Finucane proposed openstack/pbr master: Use packaging where possible  https://review.opendev.org/c/openstack/pbr/+/95404311:00
opendevreviewStephen Finucane proposed openstack/pbr master: Cleanup removal of 'build_sphinx' command doc  https://review.opendev.org/c/openstack/pbr/+/95404411:00
opendevreviewStephen Finucane proposed openstack/pbr master: Remove 'test' distutils command override  https://review.opendev.org/c/openstack/pbr/+/95404511:00
opendevreviewStephen Finucane proposed openstack/pbr master: Move fixtures to their own module  https://review.opendev.org/c/openstack/pbr/+/95404611:00
opendevreviewStephen Finucane proposed openstack/pbr master: Remove support for custom compilers  https://review.opendev.org/c/openstack/pbr/+/95404711:00
opendevreviewStephen Finucane proposed openstack/pbr master: Move distutils commands to _compat  https://review.opendev.org/c/openstack/pbr/+/95404811:00
opendevreviewStephen Finucane proposed openstack/pbr master: Move functional tests to their own directory  https://review.opendev.org/c/openstack/pbr/+/95404911:00
opendevreviewStephen Finucane proposed openstack/pbr master: docs: Update usage guide and README  https://review.opendev.org/c/openstack/pbr/+/94905111:00
opendevreviewStephen Finucane proposed openstack/pbr master: Deprecate '[entry_points]' section  https://review.opendev.org/c/openstack/pbr/+/94905211:00
opendevreviewStephen Finucane proposed openstack/pbr master: Deprecate '[files]' section  https://review.opendev.org/c/openstack/pbr/+/94905311:00
opendevreviewStephen Finucane proposed openstack/pbr master: Deprecate various '[metadata]' options  https://review.opendev.org/c/openstack/pbr/+/94905411:00
opendevreviewStephen Finucane proposed openstack/pbr master: Deprecate various '[backwards_compat]' options  https://review.opendev.org/c/openstack/pbr/+/94905511:00
opendevreviewStephen Finucane proposed openstack/pbr master: Deprecate 'tests_require' option  https://review.opendev.org/c/openstack/pbr/+/95405011:00
opendevreviewStephen Finucane proposed openstack/pbr master: tox: Stop calling 'setup.py'  https://review.opendev.org/c/openstack/pbr/+/94918811:01
gibihi infra! Is there a tracker for the multi node job issue where nodepool allows cross provider worker placement preventing nova compute to talk to rabbit hosted in the other provider? I have a nova tracker for it https://bugs.launchpad.net/nova/+bug/211598012:56
gibithere was some chatter yesterday on openstack-nova about it https://bugs.launchpad.net/nova/+bug/211598012:58
gibihttps://meetings.opendev.org/irclogs/%23openstack-nova/%23openstack-nova.2025-07-03.log.html#openstack-nova.2025-07-03.log.html#t2025-07-03T18:20:1212:58
fungigibi: there's https://review.opendev.org/954064 which we should be auto-upgrading to within the next 24 hours12:59
gibifungi: thanks13:00
fungiour zuul servers upgrade to the latest master branch state through weekly rolling restarts that begin around midnight utc on saturday mornings13:01
gibigood to know 13:02
frickleriiuc this still doesn't guarantee locality 100%, though. maybe devstack needs to add a precheck to reject nodesets, possibly based on some latency cutoff?13:05
sean-k-mooneyfrickler: we discussed this a little on the nova channel. routableity is more imporant then latency but thats aslo a factor as is open port access13:44
sean-k-mooneyfrickler: to me this si just an infra bug as our exisaitng ci proivde always provide network connectivet and our jobs were built to depend on tha tas part of the contact13:45
sean-k-mooneywe can adapt the jobst compensate for the new capablites of zuul/node pools13:45
sean-k-mooneybut i would conder the current weaking of that guarentee to be a regression13:46
sean-k-mooneybut ya having a simpel role that is run as a preplaybook to reject the nodes and allow the job to be retired is a good first step13:47
sean-k-mooneyextendign it ot be abelto factor in latency or ports woudl be another good step. we generally rely on the default secuffity group in the provider openign up access to all port for vms in the same secuirty gorup13:48
opendevreviewCyril Roelandt proposed openstack/project-config master: Add centos-10-stream nodepool image  https://review.opendev.org/c/openstack/project-config/+/95413614:15
fungiand just to reiterate, the bug is fixed (unless you consider trying anyway instead of returning node_failure to be a buggy behavior as well), we're merely waiting for this weekend's automated upgrade to be running on it14:29
sean-k-mooneyfungi: i consider not requiring an explcity opt in at the node set  to be a regression14:39
sean-k-mooneybut i think you state that its not enable by defualt14:40
sean-k-mooneyand we ahve elected to turn it on in our config somewhere14:40
sean-k-mooneyso tis not nessiarly a zull bug14:40
sean-k-mooneybut rather a bug in the current config given our existign jobs cannot tolerate having nodes form diffent providers in the same nodeset14:41
fungisean-k-mooney: afaik it is not currently a configurable option. we're in the process of (nearly done) replacing nodepool-launcher with zuul-launcher. the former could only boot a nodeset within a single provider, which made things like mixed-architecture multi-node jobs impossible. zl doesn't have that limitation, but tries to boot all nodes from a nodeset within the same14:48
fungiprovider unless that's deemed impossible for some reason (e.g. different architectures, nodeset is larger than the total quota for the largest provider, et cetera)14:48
sean-k-mooneyfungi: yep but the fact that it could only boot a nodeset within a singel provier was also a feature not a bug14:53
sean-k-mooneyi like the idea of being able to have multi architecture jobs or have jobs that have a contaienr registry rom openshfit that is used with vms form openstack to test something14:54
sean-k-mooneybut i really dont think that shoudl be allowe by default and there shoudl be some way of expressing a policy around that wehn defining nodesets14:55
sean-k-mooneyim not saying we shoudl break the lable absraction by the way14:55
sean-k-mooneyjust a policy in the node set that can ssay if its ok for the nodes ot come form difent proviers or how to group them14:56
sean-k-mooneytoday if you wanted to do a multi architure job you woudl havd to crfeate to provider jobs taht pasused and a 3 djob that depened on both that then used the nodes form the provider jobs to execute the multi arch job14:57
sean-k-mooneyi.e an arm provider job, x86 provider job and a test execuitr job that depended on both and uses the nodes form both to do whatever14:58
sean-k-mooneyits nice that you culd do that more simplely now14:58
fungifrom a broader zuul userbase (not just opendev/openstack) it was a hindrance, particularly because static nodes get a separate provider and the old model made jobs incorporating a static node and ephemeral nodes into a nodeset together impossible15:48
clarkbit also meant you cannot mix kubernetes pods/namespaces and VMs17:27
clarkbI think we should see if this is actually a problem after the fix17:28
clarkbrather than jumping to assumptions and demanding changes beforehand17:28
clarkbthe current behavior was acknowledged as buggy, a fix was pushed, and will be deployed shortly17:28
opendevreviewStephen Finucane proposed openstack/pbr master: Move fixtures to their own module  https://review.opendev.org/c/openstack/pbr/+/95404617:59
opendevreviewStephen Finucane proposed openstack/pbr master: Remove support for custom compilers  https://review.opendev.org/c/openstack/pbr/+/95404717:59
opendevreviewStephen Finucane proposed openstack/pbr master: Move distutils commands to _compat  https://review.opendev.org/c/openstack/pbr/+/95404817:59
opendevreviewStephen Finucane proposed openstack/pbr master: Move functional tests to their own directory  https://review.opendev.org/c/openstack/pbr/+/95404917:59
opendevreviewStephen Finucane proposed openstack/pbr master: docs: Update usage guide and README  https://review.opendev.org/c/openstack/pbr/+/94905117:59
opendevreviewStephen Finucane proposed openstack/pbr master: Deprecate '[entry_points]' section  https://review.opendev.org/c/openstack/pbr/+/94905217:59
opendevreviewStephen Finucane proposed openstack/pbr master: Deprecate '[files]' section  https://review.opendev.org/c/openstack/pbr/+/94905317:59
opendevreviewStephen Finucane proposed openstack/pbr master: Deprecate various '[metadata]' options  https://review.opendev.org/c/openstack/pbr/+/94905417:59
opendevreviewStephen Finucane proposed openstack/pbr master: Deprecate various '[backwards_compat]' options  https://review.opendev.org/c/openstack/pbr/+/94905517:59
opendevreviewStephen Finucane proposed openstack/pbr master: Deprecate 'tests_require' option  https://review.opendev.org/c/openstack/pbr/+/95405017:59
opendevreviewStephen Finucane proposed openstack/pbr master: tox: Stop calling 'setup.py'  https://review.opendev.org/c/openstack/pbr/+/94918817:59
sean-k-mooneyclarkb: no one is demandieng changes but i am reporting a backwared incompatbale change in behavior that does violate one of the requirement we have for the networkign in our exisitng jobs.18:35
sean-k-mooneyclarkb: zuul is free ot evolve to meet the wirder needs of its users18:35
sean-k-mooneybut i would hope that not brekaign there users is also valued and that providing feedback when there are breakign changes is also18:36
sean-k-mooneyi have hit the exact limitation with mixing pods form openshfit and vms form opnestack and or static nodes18:37
sean-k-mooneyand i have also worked around tha tusing job depencies before i.e. where i spawned a pod to run a cli to provision bemetal host and then consomed those as static nodes18:38
sean-k-mooneyi.e. i spwaned a pod on openshift to run the beaker cli to provision static nodes becauses nodepool did not have a beaker driver18:38
sean-k-mooneyit takes a lot more understanding of how zuul and nodepool worked to be able ot leave multile providers in the past. as you had to intrally device  thigs up into multipel jobs. with depenices. 18:46
sean-k-mooney nodeset.alternatives seam cool18:46
sean-k-mooneyi feel like thats an underused feature18:47
opendevreviewStephen Finucane proposed openstack/pbr master: Move functional tests to their own directory  https://review.opendev.org/c/openstack/pbr/+/95404920:35
opendevreviewStephen Finucane proposed openstack/pbr master: docs: Update usage guide and README  https://review.opendev.org/c/openstack/pbr/+/94905120:35
opendevreviewStephen Finucane proposed openstack/pbr master: Deprecate '[entry_points]' section  https://review.opendev.org/c/openstack/pbr/+/94905220:35
opendevreviewStephen Finucane proposed openstack/pbr master: Deprecate '[files]' section  https://review.opendev.org/c/openstack/pbr/+/94905320:35
opendevreviewStephen Finucane proposed openstack/pbr master: Deprecate various '[metadata]' options  https://review.opendev.org/c/openstack/pbr/+/94905420:35
opendevreviewStephen Finucane proposed openstack/pbr master: Deprecate various '[backwards_compat]' options  https://review.opendev.org/c/openstack/pbr/+/94905520:35
opendevreviewStephen Finucane proposed openstack/pbr master: Deprecate 'tests_require' option  https://review.opendev.org/c/openstack/pbr/+/95405020:35
opendevreviewStephen Finucane proposed openstack/pbr master: tox: Stop calling 'setup.py'  https://review.opendev.org/c/openstack/pbr/+/94918820:35

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