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
prometheus: disable tests #44864
prometheus: disable tests #44864
Conversation
Success on aarch64-linux (full log) Attempted: prometheus Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: prometheus Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: prometheus Partial log (click to expand)
|
I don't quite understand the rationale here. If tests fail, the solution should be to fix the tests, not disable them. If you don't want to fix the tests yourself, contact upstream. Disabling the tests only seems to remove value of Nixpkgs. |
That's not helpful. Tests in Nixpkgs are disabled due to failure, breakage, etc. all the time, search for |
I know about the practice and I don't agree with it. |
I'm not sure I understand the issue here; I may be overlooking something or misunderstanding. prometheus_2 builds (seemingly) finelatest build on hydra as of now, but does have an ignored failure in the log:
You say:
How are they unable to rebuild their systems if the package ( I have verified, also, that the package builds locally by forcing a rebuild, and it does build without failure (with the silently failing test). IF the issue is that prometheus fails to run because of the issue causing the silently failing test, the better approach would be to ensure the test fails the build and not to disable all tests, right? Additionally, when there is a failure in one test from upstream, I think the more common approach is to disable that test only through a project or framework specific method (e.g. a patch, |
Thanks, @samueldr! It looks like this has been resolved in the meantime. |
Motivation for this change
Tests silently fail on Prometheus 2.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)