Skip to content

Conversation

@zimbatm
Copy link
Member

@zimbatm zimbatm commented Mar 16, 2019

/cc @nlewo

default.nix Outdated
{ nixpkgs ? <nixpkgs> }:
let
nix-container-images = builtins.fetchTarball {
url = "https://github.com/nlewo/nix-container-images/archive/7577da87f7249442b51359e36a0f0493949a14d6.tar.gz";
Copy link
Member

Choose a reason for hiding this comment

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

How do you intend to keep this up-to-date?

Copy link
Member Author

Choose a reason for hiding this comment

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

For now it's manual, but an updater script would be nice

Copy link
Member

Choose a reason for hiding this comment

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

What about using Niv?
I don't know if it is relevant but we could also use Niv to fetch NixOS releases which seem to be fetched by the CI. Correct me if I'm wrong.

Copy link
Member Author

Choose a reason for hiding this comment

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

yeah niv could work but the hard part is mostly around CI. make a cron-like job that updates and pushes updates if it works. Mic92 already did something like that for NUR so I might steal that.

Copy link
Member

Choose a reason for hiding this comment

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

Also have this project for automated updates:

https://github.com/Mic92/iana-etc/blob/master/ci/deploy.sh

Setting up secrets is sometimes a hassle in travis, when you mix up travis.org and travis.com keys.

This allows us to easily update it by running `niv update`.
@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/nixos-nix-docker-image/9101/2

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.

5 participants