Skip to content

Invalidate build cache less often#2558

Merged
yorickdowne merged 2 commits intoethstaker:mainfrom
yorickdowne:docker-build-cache
May 4, 2026
Merged

Invalidate build cache less often#2558
yorickdowne merged 2 commits intoethstaker:mainfrom
yorickdowne:docker-build-cache

Conversation

@yorickdowne
Copy link
Copy Markdown
Collaborator

@yorickdowne yorickdowne commented Apr 22, 2026

What I did

Users that ./ethd update frequently get to keep their build cache, until 50GB are reclaimable. Strike a balance between cleaning up Docker artefacts and constantly rebuilding from scratch.

Parsing the docker system df output is brittle. There are multiple safeguards in place to just return and do nothing, instead of failing ethd entirely.

@yorickdowne yorickdowne force-pushed the docker-build-cache branch 8 times, most recently from 8f0be1c to 5f94df5 Compare April 22, 2026 01:58
@yorickdowne yorickdowne marked this pull request as draft April 27, 2026 06:50
@yorickdowne
Copy link
Copy Markdown
Collaborator Author

Putting this to draft. I'm worried it'll keep the build from pulling in changed latest clients. Need to test more

@yorickdowne yorickdowne force-pushed the docker-build-cache branch from 5f94df5 to 553e3ce Compare May 4, 2026 06:49
@yorickdowne
Copy link
Copy Markdown
Collaborator Author

Out of draft. I build --pull which makes it OK. Specifically in ./ethd update. A user that runs ./ethd cmd build may miss an update, but if they run ./ethd cmd build --pull they will not.

@yorickdowne yorickdowne marked this pull request as ready for review May 4, 2026 07:07
@yorickdowne yorickdowne requested a review from pietjepuk2 May 4, 2026 07:07
@yorickdowne yorickdowne merged commit 4626b10 into ethstaker:main May 4, 2026
8 checks passed
@yorickdowne yorickdowne deleted the docker-build-cache branch May 4, 2026 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants