Wednesday, 2025-06-04

*** ralonsoh_ is now known as ralonsoh12:36
dansmithremind me - there just has to be *something* (i.e. a job) in the gate queue for zuul to merge on +2+W right?13:42
fungiyes, though it can be the noop job (typically applied in-project with the noop-jobs template in your .zuul.yaml file or whatever)13:47
fungidansmith: ^13:48
fungiwhich, as the name implies, just does nothing very quickly and then returns success13:48
dansmithokay I didn't see a noop job int he zuul jobs interface13:48
fungiit's a (the only?) implicit built-in job, it has no actual definition13:48
dansmithah, okay13:48
fungidansmith: https://codesearch.opendev.org/?q=noop-jobs13:49
fungishould turn up some references and numerous exampls13:49
dansmithso "noop" is in the .zuul template, but I need "noop-jobs" ?13:49
fungithe noop-jobs template just applies the noop job in check and gate pipelines, definition for it can be found here: https://opendev.org/openstack/openstack-zuul-jobs/src/branch/master/zuul.d/project-templates.yaml#L18-L3113:50
fungimost projects apply noop-jobs as a template for convenience, rather than explicitly adding noop to multiple pipelines directly13:51
dansmiththis is the default it gave me: https://opendev.org/openstack/nova-contrib/src/branch/master/.zuul.yaml13:51
dansmithis that not the same as the template?13:51
fungiyeah, that should work exactly the same13:51
fungialso fine13:51
dansmithokay but +W and no gate action: https://review.opendev.org/c/openstack/nova-contrib/+/951684/113:52
fungimight that have been proposed before the noop jobs were added to .zuul.yaml?13:53
fungiif so, there was never a verified +1 from zuul on it, a recheck will solve that13:53
dansmiththey're in the tree for that commit13:53
fungimmm13:53
dansmithand git-review worked because it was pre-setup for me, and that's the same commit as the .zuul13:55
dansmithyeah that setup commit is the parent of mine13:55
fungiyeah, strangely the child change in that series did get a verified +113:55
dansmithwell, I added actual tests, so I figured that was why13:56
dansmithI could just squash those, but was trying to keep the sync commit to be exactly and only what is currently in nova (i.e. there are no tests there)13:56
fungishouldn't be necessary, looking still13:57
fungioh, i wonder if we have an unexpected catch-22 with the initial commit state13:58
fungithe manage-projects deployment script pushes that commit directly to gerrit rather than as a change for review...13:59
fungithough, no, zuul should still have received the branch update event and known to re-read the repo for any job configuration13:59
fungiat this point https://zuul.opendev.org/t/openstack/project/opendev.org/openstack/nova-contrib does indicate zuul expects to run the noop jobs on changes for it14:02
fungiso a recheck will likely "just work"14:02
fungibut i'm wondering what sort of race got it into that initial state wher zuul didn't run anything on the change14:02
dansmithI see jobs14:02
dansmithmaybe my fault for pushing to the repo minutes after it was created14:03
fungiit's possible there's a brief opportunity for a race if a change gets pushed after the project is created but before zuul has completed its own reconfig14:04
fungiwhich usually taks on the order of minutes14:04
dansmithah crap, I rechecked the wrong one14:05
dansmithheh, I rechecked the one with actual tests14:05
dansmithI don't see jobs for that one14:05
dansmiththere we go14:06
dansmithmerged14:06
dansmiththanks, sorry for the noise, I guess I should have tried rechecking my noop jobs first :)14:06
fungino worries, i'll see if i can think of a way to confirm my theory as to what happened initially14:09
fungidansmith: https://review.opendev.org/c/openstack/project-config/+/951570 deployed main.yaml to zuul in https://zuul.opendev.org/t/openstack/build/59be142897814009a0226cb621feb4a5 around 17:20 yesterday at which time a smart reconfig was triggered, but https://review.opendev.org/c/openstack/nova-contrib/+/951684 was uploaded about a minute earlier before zuul became aware of15:00
fungithe project15:00
fungiso that explains it15:00
dansmithack, thanks for the RCA15:01
* dansmith blows smoke from his guns and re-holsters15:01
opendevreviewClark Boylan proposed openstack/project-config master: Disable Nodepool's rax iad region  https://review.opendev.org/c/openstack/project-config/+/95179417:19
opendevreviewMerged openstack/project-config master: Disable Nodepool's rax iad region  https://review.opendev.org/c/openstack/project-config/+/95179417:42
opendevreviewMerged openstack/project-config master: Remove redundant editHashtags config  https://review.opendev.org/c/openstack/project-config/+/95160320:54
opendevreviewMerged openstack/project-config master: Remove editHashtags config  https://review.opendev.org/c/openstack/project-config/+/95160420:57
opendevreviewMerged openstack/project-config master: Disallow editHashtags in acl configs  https://review.opendev.org/c/openstack/project-config/+/95171520:58

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