Skip to content

Docker - Fix publish docker image in GitHub Registry #235

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

Merged
merged 2 commits into from
Mar 20, 2023

Conversation

bernatvadell
Copy link
Contributor

@bernatvadell bernatvadell commented Mar 17, 2023

In the PR that was resolved (#132), the action defined to publish the packages used the user and token of the author of the commit in master.

In this case, I have been yours, but I do not have permissions for such an action.

That is why I have modified the file, so that the owner's token is included as a secret with the name GHCR_PAT.

It is important to select the read:packages & write:packages permissions.

NOTE: Checks will fail until the secret is included.

@j-f1
Copy link
Collaborator

j-f1 commented Mar 17, 2023

Would github.repository_owner / secrets.GITHUB_TOKEN work? (with the appropriate permissions, namely packages: write)

@gjmulder gjmulder added enhancement New feature or request good first issue Good for newcomers build Compilation issues labels Mar 17, 2023
@ggerganov
Copy link
Member

Let me know if I need to do something here - not familiar with Docker

@bernatvadell
Copy link
Contributor Author

Would github.repository_owner / secrets.GITHUB_TOKEN work? (with the appropriate permissions, namely packages: write)

I think this should work, I changed it and the login at least went well. It would be necessary to test if we have the appropriate permissions during the push of the image.

@ggerganov can you authorize this PR? Ty!

Copy link
Collaborator

@j-f1 j-f1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems reasonable to me but we can’t fully test until merging.

@bernatvadell bernatvadell merged commit 0f1b21c into ggml-org:master Mar 20, 2023
@bernatvadell
Copy link
Contributor Author

Seems reasonable to me but we can’t fully test until merging.

it's working! ty

Deadsg pushed a commit to Deadsg/llama.cpp that referenced this pull request Dec 19, 2023
Decrement CUDA version and bump Ubuntu
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Compilation issues enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants