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

fixes k3s image pulling #109

Closed
wants to merge 1 commit into from
Closed

Conversation

antoinerg
Copy link
Contributor

@antoinerg antoinerg commented Jan 20, 2024

Closes issue #102 by upgrading k3s to a recent version that supports the --image-service-endpoint flag. Then, we use that flag to point k3s to nix-snapshotter.

Comment on lines +19 to +27
_module.args.pkgs = let
# Creates a k3s overlay to get a recent version with the --image-service-endpoint flag
nixpkgs-unstable = import inputs.nixpkgs-unstable {inherit system;};
k3sOverlay = final: prev: {k3s = nixpkgs-unstable.k3s;};
in import inputs.nixpkgs {
inherit system;
# Apply default overlay to provide nix-snapshotter for NixOS tests &
# configurations.
overlays = [ self.overlays.default ];
overlays = [ self.overlays.default k3sOverlay];
Copy link
Contributor Author

Choose a reason for hiding this comment

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

In #108, @RobbieBuxton brings in a newer version of k3s in a different way. I'm not sure which is best 🤷

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It turns out that the approach in #108 isn't working. Please see #108 (comment) for details!

@antoinerg
Copy link
Contributor Author

#108 might be better than this PR once it adds the "--image-service-endpoint unix:///run/nix-snapshotter/nix-snapshotter.sock" flag!

@antoinerg antoinerg marked this pull request as ready for review January 22, 2024 22:02
@antoinerg antoinerg closed this Jan 24, 2024
@antoinerg antoinerg deleted the fix-k3s branch January 24, 2024 02:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant