Skip to content

Fix Build and Push Container Image#1633

Merged
ikawrakow merged 12 commits intoikawrakow:mainfrom
sourceupcode:main
Apr 16, 2026
Merged

Fix Build and Push Container Image#1633
ikawrakow merged 12 commits intoikawrakow:mainfrom
sourceupcode:main

Conversation

@yadirhb
Copy link
Copy Markdown
Contributor

@yadirhb yadirhb commented Apr 14, 2026

Here again with a few changes to hopefully fix the issues we have:

  • Updated the failing cleanup to depend on different action.
  • Added triggers for the workflow: push and pull_request so it triggers automatically. This was removed while I was fighting with rebase conflicts last week.
  • Fixed build number fixed to 100 due to checkout action always pulling only the last 100 commits.

@yadirhb
Copy link
Copy Markdown
Contributor Author

yadirhb commented Apr 14, 2026

@ikawrakow here some fixes for #1611. Still please check if in your package Settings page you have your repo ik_llama.cpp listed with Write permission. It should look like this:

image

... as I see:

ERROR: target server: failed to solve: failed to push ghcr.io/ikawrakow/ik-llama-cpp:cpu-server-4418: denied: installation not allowed to Write organization package
Build references
Check build summary support
  Build summary supported!

Comment thread .github/workflows/build-container.yml
@ikawrakow
Copy link
Copy Markdown
Owner

Here is what I have in the settings

Screenshot 2026-04-14 at 8 18 44 AM

@yadirhb yadirhb marked this pull request as draft April 14, 2026 18:25
yadirhb and others added 4 commits April 14, 2026 18:17
Explicitly pass GITHUB_TOKEN to the delete-package-versions action to ensure it has sufficient authorization for package deletion.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@yadirhb
Copy link
Copy Markdown
Contributor Author

yadirhb commented Apr 15, 2026

Here is what I have in the settings

Screenshot 2026-04-14 at 8 18 44 AM

Ok, just updated the workflow again and also tested the permissions. Write permission is not enough, it needs to be Admin so the cleanup step can Delete untagged packages from the registry.

@yadirhb yadirhb marked this pull request as ready for review April 15, 2026 17:09
@ikawrakow ikawrakow merged commit 8df5cbc into ikawrakow:main Apr 16, 2026
markaalonzo pushed a commit to markaalonzo/ik_llama.cpp that referenced this pull request Apr 17, 2026
* fix: Updating cleanup step

* fix: Updated trigger for build-container

* fix: Unset token for cleanup step

* fix: Set build-container cleanup step without run-dry

* fix: Removed 100 commits from checkout actions

* fix: Enable the whole history

* test: Suggestion from mcm007 on build-container

* fix: add token to package cleanup step

Explicitly pass GITHUB_TOKEN to the delete-package-versions action to ensure it has sufficient authorization for package deletion.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix: - Error: delete version API failed. Package not found.

* fix: Deleted LLAMA_COMMIT from build

* fix: Removed id-token permission

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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.

2 participants