Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Some x-ci-accept-failures appears to be ignored #19929

Closed
thierry-martinez opened this issue Nov 1, 2021 · 2 comments
Closed

Some x-ci-accept-failures appears to be ignored #19929

thierry-martinez opened this issue Nov 1, 2021 · 2 comments

Comments

@thierry-martinez
Copy link
Contributor

thierry-martinez commented Nov 1, 2021

Failures in #19824 are marked as x-ci-accept-failures, but surprisingly the grep on this field at the end of the CI script seems to miss them and still reports them as CI errors.

@thierry-martinez
Copy link
Contributor Author

The script executed by opam-ci only considers x-ci-accept-failures when opam fails with error-code 31 (line test "$res" != 31 && exit "$res";), which corresponds (according to opam --help) to Package script error.. However, in the case of #19824, the error is triggered by opam remove for a dependency problem, and opam fails with error-code 5 (Not found.).

It seems to me that considering x-ci-accept-failures in this case would make sense: why restricting x-ci-accept-failures to error code 31? Could it be applied at least for error code 5 as well?

@thierry-martinez
Copy link
Contributor Author

The error-code returned by opam is erroneous and is reported in ocaml/opam#4890 . The lack of covering of x-ci-accept-failures is handled here: ocaml/opam#4493 (thank you @kit-ty-kate for pointing this). Two open issues for this problem seems enough and I close this third one!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant