tazjin / nixery

Container registry which transparently builds images using the Nix package manager. Canonical repository is https://cs.tvl.fyi/depot/-/tree/tools/nixery
https://nixery.dev/
Apache License 2.0
1.83k stars 69 forks source link

feat(ci): don't mount /var/cache/nixery from tmpfs into docker container #128

Closed flokli closed 3 years ago

flokli commented 3 years ago

With https://github.com/google/nixery/pull/127, nixery will use extended attributes to store metadata (when using local storage).

Right now, our integration test mounts a tmpfs to /var/cache/nixery. However, user xattrs aren't supported with tmpfs 1, so setting xattrs would fail.

To workaround this, use a folder in the current working directory and hope it's backed by something supporting user xattrs (which is the case for GitHub Actions).

flokli commented 3 years ago

This should be merged before https://github.com/google/nixery/pull/127.

flokli commented 3 years ago

I updated the PR to include your proposed changes - and also added some code removing var-cache-nixery if it already exists (leaving it around after the run for inspection). It doesn't matter much in CI, but for interactive tinkering.