Skip to content

Conversation

LGLO
Copy link
Contributor

@LGLO LGLO commented Jun 27, 2025

Description

Revives building docker image locally. Build image was tested on arm64 Mac - Docker Desktop with Rosetta enabled and on linux with AMD CPU.

  • added "runner" image, to avoid re-installing things, also it nicely separates this part from part relevant to our demo node
  • added --platform=linux/amd64 , so it is like we user in local-environment (works with rosetta)

On a rebuild, extra time spend for building docker images is less than recompilation of runtime and node crates alone. We can test our changes in local-environment every few minutes now

@LGLO LGLO force-pushed the fix-building-docker-image branch from 487251f to d08b3ae Compare June 27, 2025 08:58
@tgunnoe
Copy link
Member

tgunnoe commented Jun 27, 2025

a9e1cc5
we did the same work. sorry I didn't share. I shared with Tomasz

@LGLO
Copy link
Contributor Author

LGLO commented Jun 30, 2025

a9e1cc5 we did the same work. sorry I didn't share. I shared with Tomasz

I think there is more in my PR:

  • we use recent builder image
  • mount with cache type for target, CARGO_HOME and RUSTUP_HOME makes rebuilds really fast

@LGLO LGLO enabled auto-merge (squash) June 30, 2025 09:25
@LGLO LGLO merged commit e9a8b9a into master Jun 30, 2025
37 checks passed
@LGLO LGLO deleted the fix-building-docker-image branch June 30, 2025 09:49
iquerejeta pushed a commit that referenced this pull request Jul 21, 2025
Klapeyron pushed a commit that referenced this pull request Jul 22, 2025
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.

3 participants