On 2024-09-01 08:34, Cyril Brulebois wrote:
Hey Kibi, thanks for your prompt response!
Reinhard Tartler <siret...@tauware.de> (2024-09-01):
While working on updating the docker.io package in experimental, I've
noticed an autopkgtest failure on arm64 that did not happen on amd64:
https://ci.debian.net/packages/c/crowdsec/unstable/arm64/51171995/
274s === RUN TestOneShot
274s journalctl_test.go:172: Expected log output 'journalctl:
invalid option' but got nothing !
274s --- FAIL: TestOneShot (0.05s)
Looking at the code in question, I don't see anything obvious that
could be attributed into changes of the docker.io package:
https://sources.debian.org/src/crowdsec/1.4.6-9/pkg/acquisition/modules/journalctl/journalctl_test.go/?hl=104#L104-L178
Please have a look at the code and form an opinion whether this
indicates a flaky test (in which case disabling it it might be a good
way to proceed), or whether code changes in either the docker.io
package or crowdsec would be appropriate.
I'd suggest taking this specific issue out of your todo list before
considering an upload to unstable: I'd rather avoiding diving into
unstable + experimental overlay with different results per arch at
this point, and deal with whatever happens in unstable.
I'm not really familiar with the crowdsec package. I just found the
article at
https://debamax.com/blog/2023/06/12/packaging-crowdsec-for-debian-bullseye/
and it looks really interesting. I'll have to take a closer look.
As for this issue, I was hoping you as maintainer could help by
forming an opinion on how to proceed.
(Of course letting this bug report get an upgraded severity, possibly
resulting in crowdsec's getting out of testing for a little while.)
That'd be a shame if it came to that, because that would also delay
testing migration for other packages (such as podman, etc.)
Would that approach be sufficient to let you make more progress?
I currently suspect that this is actually a timing issue indicating a
flaky
test. I've retriggered the autopkgtest to see if that theory holds.
best,
-rt