@fungicide:matrix.org | > <@jreed:matrix.org> Good point, I had not considered it. I thought it was a stand alone tool that people just use if needed. Is there some sort of deprecation procedure StarlingX follows? | 00:57 |
---|---|---|
i'm not really a starlingx maintainer, i just hang around in case people have opendev infrastructure questions, so hopefully someone else here more familiar with the project will pipe up | ||
@jreed:matrix.org | Does anyone know why Zuul isn't triggering for this repo anymore? It ran for the first patch set but it won't run for the most recent patch set. | 18:06 |
https://review.opendev.org/c/starlingx/app-gen-tool/+/899896/4 | ||
@brunomuniz:matrix.org | 🤔 | 18:10 |
@brunomuniz:matrix.org | You made changes to the .zuul.yaml file on the second patchset... could this have something to do with it? | 18:14 |
@jreed:matrix.org | I only changed the folder that it points to. | 18:15 |
@jreed:matrix.org | Not sure how that would break zuul. | 18:15 |
@jreed:matrix.org | I wonder if I have to first merge this change, and then change zuul in a later commit? | 18:16 |
@jreed:matrix.org | * I wonder if I have to first merge this change, and then change zuul folder in a later commit? | 18:16 |
@brunomuniz:matrix.org | Well, that's a good test: remove changes to `.zuul.yaml` from this Review to see if Zuul picks it up again. | 18:18 |
@brunomuniz:matrix.org | Create a separate Review for the `.zuul.yaml` changes. | 18:18 |
@brunomuniz:matrix.org | Of course, more info would be preferable. haha | 18:18 |
@jreed:matrix.org | alright... will try that. | 18:19 |
@jreed:matrix.org | okay... I put it back to what it was... | 18:20 |
@jreed:matrix.org | and its queued up now. | 18:20 |
@jreed:matrix.org | https://zuul.opendev.org/t/openstack/status#starlingx | 18:20 |
@jreed:matrix.org | Can see it on zuul status board | 18:21 |
@jreed:matrix.org | that's... odd | 18:21 |
@brunomuniz:matrix.org | Well, it kinda makes sense that you need to go through the approval process + merge to get something new to run, right? | 18:23 |
@brunomuniz:matrix.org | The odd part is that you don't get the "current " check when you're changing to a new one, right? | 18:24 |
@jreed:matrix.org | I honestly don't know... it's somewhat makes sense and the rest is voo doo. | 18:25 |
@jreed:matrix.org | ... and it ran to finish. | 18:38 |
@jreed:matrix.org | will probably have to make another pass at it to update. And that's okay, I have another task to improve use of paths and whatnot. | 18:39 |
@brunomuniz:matrix.org | I'm taking a cursory look at the Review. | 18:41 |
@brunomuniz:matrix.org | daniel.caires will also take a look tomorrow (he's not here today) | 18:41 |
@fungicide:matrix.org | zuul speculatively executes changes you propose for its configuration, so the change where you alter the configuration will normally be tested as if that configuration were already changed | 19:20 |
@fungicide:matrix.org | also `./*` is somewhat nonsensical as regular expressions go | 19:21 |
@fungicide:matrix.org | it looks like you wanted it to trigger those jobs on changes to any file in the repository, but used a glob instead of a regex (or instead of just not including a `files` list at all, zuul's default behavior is to always run specified jobs, so using `files` and `irrelevant-files` patterns are basically ways of telling it when not to run jobs) | 19:23 |
@fungicide:matrix.org | also, if zuul doesn't run any jobs on a change, that usually indicates that you've told it that none of its jobs should be run | 19:24 |
@fungicide:matrix.org | i had recommended that you look at and try to understand what the zuul configuration in other starlingx projects are doing, but it seems like there's a bit of blind copy-paste going on there without understanding what the configuration means. https://zuul-ci.org/docs/zuul/ includes detailed documentation for its configuration language | 19:26 |
@fungicide:matrix.org | i definitely recommend not just copying from random configs you find lying around, since you'll be copying the mistakes of others who also possibly didn't understand what they were doing | 19:27 |
@fungicide:matrix.org | it's a very powerful system, but if you treat it as an inscrutable black box it will become frustrating very fast | 19:28 |
@fungicide:matrix.org | https://zuul-ci.org/docs/zuul/latest/config/job.html#attr-job.files covers the `files` job attribute | 19:32 |
@fungicide:matrix.org | and if there's ever a question as to why something's not running, https://zuul-ci.org/docs/zuul/latest/config/project.html#attr-project.%3Cpipeline%3E.debug is a great way to get detailed information about zuul's job selection decisions | 19:43 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!