10000 feat: Add Ollama integration with Docker examples and CI tests by aksg87 · Pull Request #62 · google/langextract · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@aksg87
Copy link
Collaborator
@aksg87 aksg87 commented Aug 5, 2025

Description

Add Ollama integration support with Docker examples and documentation for LangExtract. This PR introduces a complete example setup for using Ollama with LangExtract, including containerized deployment options and automated testing.

Fixes #7
Fixes #22
Fixes #56

Type: Feature

Changes

Added

  • examples/ollama/ directory with:
    • Docker configuration (Dockerfile, .dockerignore, docker-compose.yml)
    • Quickstart Python example (quickstart.py)
    • Documentation (README.md)
  • tests/test_ollama_integration.py - Integration test for Ollama functionality

Modified

  • .github/workflows/ci.yaml - Updated CI pipeline to include Ollama tests
  • .github/workflows/validate_pr_template.yaml - PR template validation updates
  • README.md - Added Ollama integration documentation
  • tox.ini - Added Ollama test configuration

How Has This Been Tested?

Tested using the new integration test suite and manual verification of Docker examples:

$ python -m pytest tests/test_ollama_integration.py
$ docker-compose -f examples/ollama/docker-compose.yml up --build

Checklist:

  • I have read and acknowledged Google's Open Source Code of conduct
  • I have read the Contributing page and signed the appropriate CLA
  • I have discussed my proposed solution with code owners in the linked issue(s)
  • I have made necessary documentation changes
  • I have added tests to cover the new Ollama integration
  • I have followed Google's Python Style Guide and ran pylint over the affected code

@aksg87 aksg87 force-pushed the feat/ollama-quickstart branch 8 times, most recently from c02035f to fa69d64 Compare August 5, 2025 12:14
- Add quickstart example and documentation for local LLM usage
- Include Docker setup with health checks and docker-compose
- Add integration tests and update CI pipeline
- Secure setup: localhost-only binding, containerized deployment

Signed-off-by: Akshay Goel <goelak@google.com>
@aksg87 aksg87 force-pushed the feat/ollama-quickstart branch from fa69d64 to 8786522 Compare August 5, 2025 12:15
@aksg87 aksg87 self-assigned this Aug 5, 2025
@aksg87 aksg87 merged commit 337beee into main Aug 5, 2025
7 checks passed
aksg87 added a commit that referenced this pull request Aug 21, 2025
- Add quickstart example and documentation for local LLM usage
- Include Docker setup with health checks and docker-compose
- Add integration tests and update CI pipeline
- Secure setup: localhost-only binding, containerized deployment

Signed-off-by: Akshay Goel <goelak@google.com>
sinnaj referenced this pull request in sinnaj/langextract Sep 3, 2025
- Add quickstart example and documentation for local LLM usage
- Include Docker setup with health checks and docker-compose
- Add integration tests and update CI pipeline
- Secure setup: localhost-only binding, containerized deployment

Signed-off-by: Akshay Goel <goelak@google.com>
aksg87 added a commit that referenced this pull request Sep 12, 2025
- Add quickstart example and documentation for local LLM usage
- Include Docker setup with health checks and docker-compose
- Add integration tests and update CI pipeline
- Secure setup: localhost-only binding, containerized deployment

Signed-off-by: Akshay Goel <goelak@google.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

1 participant

0