Skip to content

docs: fix Ollama engine name from ollama_engine to ollama#12362

Merged
ZoranPandovski merged 1 commit intomindsdb:releases/26.1.0from
singhvishalkr:fix/docs-ollama-engine-name
Apr 8, 2026
Merged

docs: fix Ollama engine name from ollama_engine to ollama#12362
ZoranPandovski merged 1 commit intomindsdb:releases/26.1.0from
singhvishalkr:fix/docs-ollama-engine-name

Conversation

@singhvishalkr
Copy link
Copy Markdown
Contributor

@singhvishalkr singhvishalkr commented Apr 4, 2026

Summary

  • The Ollama integration docs reference ollama_engine as the ML engine name, but the handler registers itself as ollama.
  • Users following the docs verbatim get an engine mismatch error.

Changes

  • docs/integrations/ai-engines/ollama.mdx: Replace all 5 occurrences of ollama_engine with ollama so the examples work out of the box.

Fixes #12011

The Ollama integration docs used 'ollama_engine' as the ML engine name, but the handler registers itself as 'ollama'. Updated all references so the examples work without modification.

Fixes mindsdb#12011
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 4, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@singhvishalkr
Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA

github-actions Bot added a commit that referenced this pull request Apr 4, 2026
@singhvishalkr
Copy link
Copy Markdown
Contributor Author

This fixes #12011 — the Ollama docs reference ollama_engine as the engine name, but the handler actually registers as ollama. Users following the docs verbatim would get an engine mismatch. All 5 occurrences in the file are updated consistently.

@singhvishalkr singhvishalkr force-pushed the fix/docs-ollama-engine-name branch from 0f4b361 to b42f1bd Compare April 5, 2026 12:55
Copy link
Copy Markdown
Member

@ZoranPandovski ZoranPandovski left a comment

Choose a reason for hiding this comment

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

Thanks

@singhvishalkr
Copy link
Copy Markdown
Contributor Author

Thanks for the review, @ZoranPandovski! If #12361 (the CREATE MODEL/PREDICTOR typo fix) is also worth a quick look, it's a similar 2-line docs correction.

@ZoranPandovski ZoranPandovski changed the base branch from main to releases/26.1.0 April 8, 2026 13:39
@ZoranPandovski ZoranPandovski merged commit e7399d6 into mindsdb:releases/26.1.0 Apr 8, 2026
9 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 8, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Docs]: error in Ollama documentation

2 participants