Skip to content

selfci: init at 0-unstable-2026-01-17#481172

Merged
doronbehar merged 1 commit intoNixOS:masterfrom
dvn0:dvn0/selfci-init
Jan 19, 2026
Merged

selfci: init at 0-unstable-2026-01-17#481172
doronbehar merged 1 commit intoNixOS:masterfrom
dvn0:dvn0/selfci-init

Conversation

@dvn0
Copy link
Contributor

@dvn0 dvn0 commented Jan 17, 2026

Git is not a hard dependency but some primary functionality will be broken without it.
Four tests fail, so tests are disabled. Did not spend much time investigating.

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.

Add a 👍 reaction to pull requests you find important.

@dvn0
Copy link
Contributor Author

dvn0 commented Jan 17, 2026

cc @dpc

@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. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 9.needs: reviewer This PR currently has no reviewers requested and needs attention. labels Jan 17, 2026
@dvn0 dvn0 force-pushed the dvn0/selfci-init branch from 08cf998 to e8a36d2 Compare January 18, 2026 02:51
@dvn0 dvn0 requested review from MatthewCroughan and doronbehar and removed request for doronbehar January 18, 2026 02:52
version = "0-unstable-2026-01-17";

src = fetchgit {
url = "https://radicle.dpc.pw/z2tDzYbAXxTQEKTGFVwiJPajkbeDU.git";
Copy link
Contributor

Choose a reason for hiding this comment

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

How important realistically this host becomes? It's a $5 Hetzner VPS, but if it ever has issues, I will not be breaking NixOS, right? :D

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Heh, if it goes down then it will simply break the package build. I don't really understand how radicle works, but is it possible to pull it from another "node" if your VPS were to go down?

Copy link
Contributor

Choose a reason for hiding this comment

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

Are there any plans to add to Nixpkgs a fetchRadicle function that will actually use this protocol directly to download sources?

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

@dpc dpc Jan 19, 2026

Choose a reason for hiding this comment

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

Heh, if it goes down then it will simply break the package build. I don't really understand how radicle works, but is it possible to pull it from another "node" if your VPS were to go down?

While the radicle is p2p and the source code is redundantly stored in the network, the git link here is just a wrapper exposing git interface running on my $5 VPS. So when my VPS is down the link won't work. One would need to use radicle itself rad clone rad:z2tDzYbAXxTQEKTGFVwiJPajkbeDU to benefit from the redundancy directly. NixOS/nix#10567 would allow Nix to use rad directly, which would be sweet.

BTW. I thought NixOS / hydra / something was actually caching all the building sources so once this builds all NixOS users would be covered effectively forever, right? Just curious. I think that VPS easily has 99.9% uptime anyway.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

While the radicle is p2p and the source code is redundantly stored in the network, the git link here is just a wrapper exposing git interface running on my $5 VPS.

Why aren't there more of these "wrappers" among the nodes seeding the repo though? Is that just not how it works?

NixOS/nix#10567 would allow Nix to use rad directly, which would be sweet.

That would be nice to see merged.

BTW. I thought NixOS / hydra / something was actually caching all the building sources so once this builds all NixOS users would be covered effectively forever, right? Just curious. I think that VPS easily has 99.9% uptime anyway.

I'm not sure what the details are regarding caching, but regardless the VPS will need to be up on successive version bumps, of course.

Copy link
Contributor

Choose a reason for hiding this comment

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

Why aren't there more of these "wrappers" among the nodes seeding the repo though? Is that just not how it works?

Well, git uses centralized URL, but what your comment made me realize, that any public node can serve the same URL, the same way my does. So e.g.

So my instance is:

But here is a public node hosted by Radicle team:

will show in "Clone -> Git":

git clone https://iris.radicle.xyz/z2tDzYbAXxTQEKTGFVwiJPajkbeDU.git selfci

I suspect their node might be a bit more capable and taken care of.

And it seems fetchGit can do urls, so actually we could have redundancy.

Copy link
Contributor

Choose a reason for hiding this comment

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

The solution would be to have a fetchFromRadicle function that uses the node configured in your Nix config. This would be analog to a fetchFromIpfs function.

I also believe we need an issue to track the subject of content addressing vs location addressing with Nix fetchers

@nixpkgs-ci nixpkgs-ci bot removed the 9.needs: reviewer This PR currently has no reviewers requested and needs attention. label Jan 18, 2026
@dvn0 dvn0 force-pushed the dvn0/selfci-init branch from e8a36d2 to 971ad07 Compare January 18, 2026 16:53
@dvn0
Copy link
Contributor Author

dvn0 commented Jan 18, 2026

Can I get an approval @doronbehar?

@doronbehar doronbehar added this pull request to the merge queue Jan 19, 2026
Merged via the queue into NixOS:master with commit b47158f Jan 19, 2026
33 of 35 checks passed
@dvn0
Copy link
Contributor Author

dvn0 commented Jan 19, 2026

Merge is even better - thanks

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

Labels

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. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants