Skip to content

feat: push docker image also to Github registry#605

Merged
kemingy merged 4 commits intotensorchord:mainfrom
tobiasdiez:ghcr
Feb 24, 2025
Merged

feat: push docker image also to Github registry#605
kemingy merged 4 commits intotensorchord:mainfrom
tobiasdiez:ghcr

Conversation

@tobiasdiez
Copy link
Contributor

Currently the docker image is only pushed to the Docker Hub repository. In certain countries dockerhub is not working very reliable, thus this PR proposes to also publish the images to the Github Packages repository.

The code is essentially taken from the cloudnative variant: https://github.com/tensorchord/cloudnative-pgvecto.rs/blob/main/.github/workflows/docker.yml

Test run currently fails already at the build step, so I don't know how to test this: https://github.com/tobiasdiez/pgvecto.rs/actions/runs/11326123877/job/31494425115

@gaocegege
Copy link
Member

LGTM

Thanks for your contribution! 🎉 👍

cc @VoVAllen

@usamoi
Copy link
Collaborator

usamoi commented Oct 15, 2024

Test run currently fails already at the build step

Should be fixed now.

@tobiasdiez
Copy link
Contributor Author

Test run currently fails already at the build step

Should be fixed now.

Thanks! I've rerun the workflow, but it's now failing because I (obviously) don't have the correct docker hub credentials: https://github.com/tobiasdiez/pgvecto.rs/actions/runs/11426048592/job/31788570283

So I have no idea how to properly test these changes. Sorry.

@kemingy kemingy mentioned this pull request Jan 8, 2025
Signed-off-by: Keming <kemingyang@tensorchord.ai>
@kemingy
Copy link
Member

kemingy commented Feb 24, 2025

CI failures are not related to this PR.

@kemingy kemingy merged commit 0dccfb4 into tensorchord:main Feb 24, 2025
1 of 8 checks passed
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.

4 participants