On Oct 1, 2023, at 00:36, Ramana Radhakrishnan ramana.gcc@googlemail.com wrote:
- linaro-toolchain as I don't understand the CI issues on patchwork.
...
Ok if no regressions but as you might get nagged by the post commit CI ...
I don't see any pre-commit failures for this patch, but regardless of what results are for pre-commit CI, there's always a chance to identify problems in post-commit CI -- simply because we test wa-a-ay more configurations in post-commit CI than in pre-commit CI.
While it is not policy yet to look at these bots but given the enthusiasm at Cauldron for patchwork and pre-commit CI and because all my armhf boxes are boxed up, I decided to do something a bit novel !
I tried reviewing this via patchwork
https://patchwork.sourceware.org/project/gcc/patch/PAWPR08MB8982A6AA40749B74...
and notice that
https://ci.linaro.org/job/tcwg_gcc_build--master-arm-precommit/2393/artifact... says nothing could be built.
Um, no. This says ... === Results changed to # reset_artifacts: -10 # true: 0 # build_abe gcc: 1
From # reset_artifacts: -10 # true: 0 # build_abe gcc: 1 === ... i.e., build succeeded both before and after patch. We'll change the boilerplate intro for successful builds from ... "Dear contributor, our automatic CI has detected problems related to your patch(es)." ... to ... "Dear contributor, you are awesome, no CI failures related to your patch(es)".
One things that is strange -- testsuite builds were not triggered, we have only 2 reports from build tests, but are missing another 2 reports from testsuite tests.
Possibly worth double checking the status for it being a false negative as to why the build failed.
Pre-commit CI is happy with the patch, albeit testsuite checks didn't run for some reason. Regardless, we'll quickly catch and report any fallout in the post-commit CI once the patch is merged.
It was green on patchwork but remembering that Green is not Green for CI in patchwork I clicked on the afore mentioned ci.linaro.org link and see that it's actually broken.
Unfortunately, I seem to have confused developers about green and red at my Cauldron presentation. "Green/Red" in patchwork mean the usual PASS/FAIL. It's only in post-commit CI in jenkins interface green and red mean something different.
-- Maxim Kuvyrkov https://www.linaro.org