Skip to content

Log Hub response in llama-server auto update failure path#92

Merged
p1-0tr merged 1 commit into
mainfrom
ps-extra-log-in-auto-update
Jun 25, 2025
Merged

Log Hub response in llama-server auto update failure path#92
p1-0tr merged 1 commit into
mainfrom
ps-extra-log-in-auto-update

Conversation

@p1-0tr
Copy link
Copy Markdown

@p1-0tr p1-0tr commented Jun 25, 2025

In case Hub responds with an unexpected tag (or other unexpected content) when querying for the desired llama-server tag during the auto update process, we return an ok error, but extra info would be nice for debugging. So add a log entry in case the Hub response does not contain the desired tag.

In case Hub responds with an unexpected tag (or other unexpected
content) when querying for the desired llama-server tag during the auto
update process, we return an ok error, but extra info would be nice for
debugging. So add a log entry in case the Hub response does not contain
the desired tag.

Signed-off-by: Piotr Stankiewicz <piotr.stankiewicz@docker.com>
@p1-0tr p1-0tr requested review from doringeman and xenoscopic June 25, 2025 10:45
@p1-0tr p1-0tr merged commit 5d31399 into main Jun 25, 2025
4 checks passed
@p1-0tr p1-0tr deleted the ps-extra-log-in-auto-update branch June 25, 2025 12:27
doringeman pushed a commit to doringeman/model-runner that referenced this pull request Sep 23, 2025
Signed-off-by: Emily Casey <emily.casey@docker.com>
doringeman pushed a commit to doringeman/model-runner that referenced this pull request Sep 23, 2025
Signed-off-by: Emily Casey <emily.casey@docker.com>
doringeman pushed a commit to doringeman/model-runner that referenced this pull request Sep 24, 2025
Signed-off-by: Emily Casey <emily.casey@docker.com>
doringeman pushed a commit to doringeman/model-runner that referenced this pull request Sep 24, 2025
Signed-off-by: Emily Casey <emily.casey@docker.com>
doringeman pushed a commit to doringeman/model-runner that referenced this pull request Oct 2, 2025
Bump Moby-related projects to v28.2.2.
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