Skip to content

Add extra nix.conf to consider using cachix in flake#230

Merged
kachick merged 1 commit intomainfrom
consider-use-of-cachix
Aug 8, 2025
Merged

Add extra nix.conf to consider using cachix in flake#230
kachick merged 1 commit intomainfrom
consider-use-of-cachix

Conversation

@kachick
Copy link
Owner

@kachick kachick commented Aug 8, 2025

Merging kachick/dotfiles#1235 made much slow to build container in the CI.(8 minutes -> 60 or more minutes)
In my understanding, the container flake definition does not require cachix, however results time-out.
I'm guessing, it came from the specified substituters in flake.nix confirmation in the steps.
If then, I should add extra conf likely kachick/dotfiles@ee8a24d

ref: kachick/dotfiles#754

@github-actions
Copy link
Contributor

github-actions bot commented Aug 8, 2025

⬢🦭 Staging container-image has been deployed 🚀
You can check in package URL

This image will be automatically 🤖 removed from ghcr.io 🗑️ if you merged/closed this PR 😌

@kachick kachick merged commit 63b48ef into main Aug 8, 2025
10 checks passed
@kachick kachick deleted the consider-use-of-cachix branch August 8, 2025 07:59
@github-actions
Copy link
Contributor

github-actions bot commented Aug 8, 2025

🤖 removed 🗑️ staging ⬢ from ghcr.io

481811822
481811846


RUN curl --proto '=https' --tlsv1.2 -sSf -L https://install.determinate.systems/nix | sh -s -- install linux \
--extra-conf "sandbox = false" \
--extra-conf "trusted-users = root user" \
Copy link
Owner Author

Choose a reason for hiding this comment

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

As I understand it, I can't use $username at here 🤔

@kachick
Copy link
Owner Author

kachick commented Aug 8, 2025

Fixed the container workflow in dotfiles repository with this PR.

https://github.com/kachick/dotfiles/actions/runs/16825558984/job/47661074028

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