Skip to content

ci: bump pinned ollama to v0.33.1 - #102

Merged
jakelorocco merged 1 commit into
generative-computing:mainfrom
jakelorocco:ci/bump-ollama-0.33.1
Oct 21, 2026
Merged

jakelorocco merged 1 commit into
generative-computing:mainfrom
jakelorocco:ci/bump-ollama-0.33.1

Conversation

@jakelorocco

Copy link
Copy Markdown
Contributor

v0.5.7 predates Jinja chat-template support. granite4.2 (mellea v0.8.0's default) ships its template only inside the GGUF, so v0.5.7 silently fell back to a bare {{ .Prompt }} template and the model just continued the prompt text, breaking JSON-output tests like test_top_k_selection.

Matches the version mellea's own CI uses. Release assets switched from .tgz to .tar.zst, so the extract step changes accordingly.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Jake LoRocco <jake.lorocco@ibm.com>
@jakelorocco
jakelorocco requested a review from a team as a code owner October 13, 2026 15:34
@jakelorocco
jakelorocco requested review from AngeloDanducci, markstur and planetf1 and removed request for a team October 13, 2026 15:34
@jakelorocco

jakelorocco commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor Author

@markstur @AngeloDanducci @planetf1, can I please get y'all's review on this? I think longterm we may want to automate this, but I'd like to move through a few contribs updates before doing that so that we can get a sense of what is actually needed.

@AngeloDanducci AngeloDanducci left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@jakelorocco
jakelorocco added this pull request to the merge queue Oct 21, 2026
Merged via the queue into generative-computing:main with commit bc28669 Oct 21, 2026
18 checks passed
@jakelorocco
jakelorocco deleted the ci/bump-ollama-0.33.1 branch October 21, 2026 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants