Skip to content

nixos/atticd: use garage in the test instead of minio#492528

Merged
Defelo merged 1 commit intoNixOS:masterfrom
LeSuisse:attic-use-rclone-instead-minio
Mar 23, 2026
Merged

nixos/atticd: use garage in the test instead of minio#492528
Defelo merged 1 commit intoNixOS:masterfrom
LeSuisse:attic-use-rclone-instead-minio

Conversation

@LeSuisse
Copy link
Copy Markdown
Member

Related #490996, minio is not supported upstream anymore.

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

@LeSuisse LeSuisse force-pushed the attic-use-rclone-instead-minio branch from c4018e6 to 8b9e408 Compare February 20, 2026 17:16
@LeSuisse LeSuisse changed the title nixos/atticd: use `rclone serve s3' in the tes instead of Minio nixos/atticd: use `rclone serve s3' in the test instead of Minio Feb 20, 2026
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. 9.needs: reviewer This PR currently has no reviewers requested and needs attention. 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS labels Feb 20, 2026
@nixpkgs-ci nixpkgs-ci bot added 12.approvals: 1 This PR was reviewed and approved by one person. and removed 9.needs: reviewer This PR currently has no reviewers requested and needs attention. labels Feb 20, 2026
@LeSuisse LeSuisse force-pushed the attic-use-rclone-instead-minio branch from 8b9e408 to 365bb01 Compare February 21, 2026 09:30
Comment thread nixos/tests/atticd.nix Outdated
@LeSuisse LeSuisse marked this pull request as draft February 21, 2026 15:11
Related NixOS#490996, `minio` is not supported upstream anymore.
@LeSuisse LeSuisse force-pushed the attic-use-rclone-instead-minio branch from 365bb01 to 846b1a7 Compare February 26, 2026 12:01
@LeSuisse LeSuisse changed the title nixos/atticd: use `rclone serve s3' in the test instead of Minio nixos/atticd: use garage' in the test instead of minio` Feb 26, 2026
@LeSuisse LeSuisse marked this pull request as ready for review February 26, 2026 12:02
@LeSuisse LeSuisse changed the title nixos/atticd: use garage' in the test instead of minio` nixos/atticd: use garage in the test instead of minio Feb 26, 2026
@britter
Copy link
Copy Markdown
Contributor

britter commented Mar 22, 2026

@LeSuisse can we merge this?

@LeSuisse
Copy link
Copy Markdown
Member Author

Ideally I try to not self-merge patches especially on stuff I do not maintain directly. That's said given it only impacts a test I will merge by the end of the week if there is no progress.

Copy link
Copy Markdown
Member

@Defelo Defelo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, test passes on x86_64-linux and aarch64-linux. Thanks!

@Defelo Defelo added this pull request to the merge queue Mar 23, 2026
Merged via the queue into NixOS:master with commit 3758e71 Mar 23, 2026
31 of 33 checks passed
@LeSuisse LeSuisse deleted the attic-use-rclone-instead-minio branch March 24, 2026 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. 12.approvals: 1 This PR was reviewed and approved by one person.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants