Andrew Bayer
Andrew Bayer
I just retitled the PR to start with `WIP` and marked it as ready for review, so that all the CI jobs launch properly/automatically.
This code path still exists in HEAD - makes me think this is likely not something fixed since 0.35.1.
/assign My first thought is that `rpt.Skip(facts).SkippingReason == v1beta1.WhenExpressionsSkip` should be replaced with `rpt.skipBecauseWhenExpressionsEvaluatedToFalse(facts)`, since the former is basically a roundabout way of determining if the latter is true. I'm...
@RafaeLeal Any chance you could provide a `PipelineRun` that hit this problem? Or the `Pipeline` for one of those `PipelineRun`s? I _think_ this is somehow related to parameter behavior behind...
Thanks, I appreciate it - feel free to scrub any information you need to, of course. I think a `Pipeline` is more of what we need than a `PipelineRun`. It...
Ok, @RafaeLeal sent me a dump of the `PipelineRun`s, and so far I'm not seeing anything obvious. Right now, my theory is that it's something specific to changing to `alpha`...
Yeah, the root problem here is some behavior going weird when changing `enable-api-fields` to `alpha` while a `PipelineRun` is running, with the result that a possible infinite loop when calling...
Ok, I've gotten a bit closer to reproducing this. I dove into the dump @RafaeLeal gave me, which is all `PipelineRun`s with embedded pipeline and task specs, and found a...