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

Replace cachix seed workflow with new daemon mode #3208

Merged
merged 2 commits into from
Feb 6, 2024

Conversation

wolfgangwalther
Copy link
Member

To be able to merge #3194, I need to seed cachix first, otherwise the full rebuild of all dependencies makes the jobs time out. This can only happen from the postgrest repo and not from a fork, because otherwise the cachix push secret is not passed to the workflow.

Investigating the best way to do this, I found out that just a few weeks ago the cachix-action we are using has been given a new "daemon mode". This daemon mode keeps pushing new derivations as soon as they are build. It should be enabled by default, already. This should easily be able to replace the "seed cachix" workflow, which was designed to push all derivations in pieces to avoid the "nothing pushed because of timeout problem".

In this MR, I took everything from #3194 and then added a commit to pass the cachix token to all jobs in the CI workflow. Those jobs should now push new derivations continuously - and in case of a timeout, we just run that job again. In theory that job should then pick up where it left off.

cachix-action v14 added a new daemon mode, which pushes new derivations to the store
as soon as they have been built. This replaces the seed cachix workflow nicely by just
pushing from all jobs directly.
@wolfgangwalther
Copy link
Member Author

Seeding cachix this way seems to have worked fine, will merge this, too.

@wolfgangwalther wolfgangwalther merged commit 868913d into main Feb 6, 2024
13 of 14 checks passed
@wolfgangwalther wolfgangwalther deleted the no-static-references branch February 6, 2024 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant