Skip to content

Try fixing docker#570

Closed
JelteF wants to merge 13 commits intomainfrom
fix-docker
Closed

Try fixing docker#570
JelteF wants to merge 13 commits intomainfrom
fix-docker

Conversation

@JelteF
Copy link
Copy Markdown
Collaborator

@JelteF JelteF commented Jan 31, 2025

Our docker build has been broken for ~a week. Noticed because the user in #564
was still getting images the old read_parquet syntax.

@wuputah
Copy link
Copy Markdown
Collaborator

wuputah commented Feb 3, 2025

This is curious. I'm not seeing an error in the logs, and I wasn't able to reproduce locally (on main) with Postgres 17 and arm64. I'm going to try an amd64 build next. This is the command I used.

POSTGRES_VERSION=17 docker buildx bake --set '*.platform=linux/arm64' --load postgres

I wonder if this the build hitting a resource constraint of the worker.

In mostly unrelated news, public arm64 workers are here, though how to get the build to properly split across platforms and push to Dockerhub correctly is not trivial based on our previous attempts.
https://github.com/orgs/community/discussions/148648

Edit: example (complicated) workflow that does just that

@JelteF JelteF added this to the 0.3.0 milestone Feb 4, 2025
@wuputah
Copy link
Copy Markdown
Collaborator

wuputah commented Feb 5, 2025

one last idea, could try disabling ninja e.g. with DUCKDB_GEN=make make -j$(nproc). I had to do that for pgxman, though I don't remember the error that resulted in me doing so.

@JelteF
Copy link
Copy Markdown
Collaborator Author

JelteF commented Feb 6, 2025

Closing this in favor of #578

@JelteF JelteF closed this Feb 6, 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