Skip to content

nixos/tests/ente: migrate test from minio to garage#509708

Open
oddlama wants to merge 3 commits intoNixOS:masterfrom
oddlama:fix/ente-tests-garage
Open

nixos/tests/ente: migrate test from minio to garage#509708
oddlama wants to merge 3 commits intoNixOS:masterfrom
oddlama:fix/ente-tests-garage

Conversation

@oddlama
Copy link
Copy Markdown
Member

@oddlama oddlama commented Apr 13, 2026

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.

@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: 0 This PR does not cause any packages to rebuild on Darwin. 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 8.has: module (update) This PR changes an existing module in `nixos/` labels Apr 13, 2026
@nixpkgs-ci nixpkgs-ci bot requested review from nicegamer7 and pinpox April 13, 2026 20:23
@nixpkgs-ci nixpkgs-ci bot added 11.by: package-maintainer This PR was created by a maintainer of all the package it changes. and removed 9.needs: reviewer This PR currently has no reviewers requested and needs attention. labels Apr 13, 2026
@herbetom
Copy link
Copy Markdown
Member

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 509708 --additional-package nixosTests.ente
Commit: ede42c75f752e1a4f43084881633e22a1a86c641


x86_64-linux

⏩ 2 packages blacklisted:
  • nixos-install-tools
  • tests.nixos-functions.nixos-test
✅ 1 test built:
  • nixosTests.ente

Copy link
Copy Markdown
Member

@herbetom herbetom left a comment

Choose a reason for hiding this comment

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

Not so sure about documenting quite a lot of garage specifics inside ente docs. Seems like something that should probably be just a link to either some nixos garage docs or some wiki.

In general: i don't think the md is reachable in any other way then then the GitHub Code Browser? So maybe this whole thing would be better served with beeing inside wiki.nixos.org? 🙈

@oddlama
Copy link
Copy Markdown
Member Author

oddlama commented Apr 14, 2026

Not so sure about documenting quite a lot of garage specifics inside ente docs. Seems like something that should probably be just a link to either some nixos garage docs or some wiki.

I agree, it's generally more to set up with garage. But admittedly, I am struggling a bit to remove information without making the guide hard to follow. Any pointers on what you would like to omit?

In general: i don't think the md is reachable in any other way then then the GitHub Code Browser? So maybe this whole thing would be better served with beeing inside wiki.nixos.org? 🙈

Good point! Seems like I forgot to include the meta attribute on the module a while back 😅 I added that in now.

@nixpkgs-ci nixpkgs-ci bot requested a review from GetPsyched April 14, 2026 17:57
@nixpkgs-ci nixpkgs-ci bot added the 8.has: documentation This PR adds or changes documentation label Apr 14, 2026
@herbetom
Copy link
Copy Markdown
Member

I would just keep it at something along the lines of "you need an S3 compatible object storage. You can self host that with software like Garage (here is a link with instructions on how to set that up).

And then put the garage specifics into https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/services/web-servers/garage.md or the like.

And then continue with now you've got a hostname, bucket name, access key and secret key. Put them into these places.

That way it's also easy to follow when you don't want to use garage because you make the split exactly where both of those services meet in a standardized way.

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 8.has: documentation This PR adds or changes documentation 8.has: module (update) This PR changes an existing module in `nixos/` 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 11.by: package-maintainer This PR was created by a maintainer of all the package it changes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants