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

GitHub Actions eval doesn’t fail on evaluation warnings #371223

Closed
emilazy opened this issue Jan 5, 2025 · 4 comments
Closed

GitHub Actions eval doesn’t fail on evaluation warnings #371223

emilazy opened this issue Jan 5, 2025 · 4 comments
Labels
0.kind: bug Something is broken 6.topic: continuous integration Affects continuous integration (CI) in Nixpkgs, including Ofborg and GitHub Actions

Comments

@emilazy
Copy link
Member

emilazy commented Jan 5, 2025

As reported in #370661 (comment), ofborg-eval would fail when evaluation of the Nixpkgs jobset produced warnings, but the GitHub Actions eval doesn’t. This can lead to the channel being unexpectedly blocked by the release-checks job.

cc @infinisil @Mic92 @JohnRTitor @wolfgangwalther (not sure exactly who to ping for this)

@emilazy emilazy added the 0.kind: bug Something is broken label Jan 5, 2025
@wolfgangwalther
Copy link
Contributor

(not sure exactly who to ping for this)

All the eval stuff is still mostly a mystery to me, so I can't help much here :)

@Mic92
Copy link
Member

Mic92 commented Jan 5, 2025

There is a abort-on-warn flag we can enable in nix.

@emilazy
Copy link
Member Author

emilazy commented Jan 5, 2025

That’s not quite enough; pkgs/top-level/nixpkgs-basic-release-checks.nix also aborts on non‐warning traces. I’d recommend following the logic there, as there are some other subtle channel‐blocking checks that aren’t related to warnings that I’m guessing the current eval CI doesn’t catch.

@FliegendeWurst FliegendeWurst added the 6.topic: continuous integration Affects continuous integration (CI) in Nixpkgs, including Ofborg and GitHub Actions label Jan 31, 2025
@infinisil
Copy link
Member

PR with the fix: #381266

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.kind: bug Something is broken 6.topic: continuous integration Affects continuous integration (CI) in Nixpkgs, including Ofborg and GitHub Actions
Projects
None yet
Development

No branches or pull requests

5 participants