Skip to content

redis: disable aarch64-linux failing test#422352

Merged
vcunat merged 1 commit into
masterfrom
unknown repository
Jul 4, 2025
Merged

redis: disable aarch64-linux failing test#422352
vcunat merged 1 commit into
masterfrom
unknown repository

Conversation

@ghost

@ghost ghost commented Jul 4, 2025

Copy link
Copy Markdown

Resolves #403628 (comment)

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • Nixpkgs 25.11 Release Notes (or backporting 25.05 Nixpkgs Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
  • NixOS 25.11 Release Notes (or backporting 25.05 NixOS Release notes)
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other contributing documentation in corresponding paths.

Add a 👍 reaction to pull requests you find important.

@nix-owners nix-owners Bot requested a review from globin July 4, 2025 09:28
@nixpkgs-ci nixpkgs-ci Bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. labels Jul 4, 2025
@vcunat

vcunat commented Jul 4, 2025

Copy link
Copy Markdown
Member

This is very similar to #422377
Let me mark at least one as channel blocker, as nixos-unstable is blocked by this, three failed attempts on Hydra already.

@vcunat vcunat added the 1.severity: channel blocker Blocks a channel label Jul 4, 2025
@ghost

ghost commented Jul 4, 2025

Copy link
Copy Markdown
Author

I've added the optionalString as suggested.

@vcunat vcunat merged commit 5c724ed into NixOS:master Jul 4, 2025
25 of 27 checks passed
@vcunat

vcunat commented Jul 4, 2025

Copy link
Copy Markdown
Member

nixos-unstable is on a 4 days old commit, so let's not wait long with blockers.

@ghost

ghost commented Jul 4, 2025

Copy link
Copy Markdown
Author

Would you mind explaining/sending a link with information on how a channel gets blocked?
I would like to understand how one (less used lets say) program can block a whole channel which is for both x86_64 and aarch64.

@ghost ghost deleted the redis branch July 4, 2025 14:01
@vcunat

vcunat commented Jul 4, 2025

Copy link
Copy Markdown
Member

@vcunat

vcunat commented Jul 4, 2025

Copy link
Copy Markdown
Member

Why not hide behind lib.optionalString stdenv.hostPlatform.isAarch64? The test doesn't fail on amd64

And... this time that test fails on x86_64-linux:
https://hydra.nixos.org/build/301760014/nixlog/1/tail

@ghost

ghost commented Jul 4, 2025

Copy link
Copy Markdown
Author

Something doesn't make sense here.
I would understand if aarch64-linux would fail after the redis update if there is something that differs from x86_64-linux, but I did build and ran redis after making the above commit.
How does it fail on x86_64-linux now when it didn't before and how does it fail when locally it works.
Does hydra differ somehow from a normal pc?
Like... I don't understand how this is possible.

Also:
wolfgangwalther said about removing this disabled test in the next release because of some commit
How can one know that the above problem won't happen again by doing so if building locally is not a way to verify it?

@vcunat

vcunat commented Jul 4, 2025

Copy link
Copy Markdown
Member

Often the difference is that locally you don't have too much other stuff building at the same time.

@Prince213

Prince213 commented Jul 4, 2025

Copy link
Copy Markdown
Member

Yeah system speed is not considered for reproducibility. I guess it can fail when the system is under heavy load?

@vcunat

vcunat commented Jul 4, 2025

Copy link
Copy Markdown
Member

Even with this the aarch64 build needed a few manual retries:
https://hydra.nixos.org/build/301760015#tabs-buildsteps

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.severity: channel blocker Blocks a channel 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants