Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: stacklok/codegate
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v0.1.18
Choose a base ref
...
head repository: stacklok/codegate
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v0.1.19
Choose a head ref
  • 15 commits
  • 28 files changed
  • 5 contributors

Commits on Feb 11, 2025

  1. Bump ruff from 0.9.5 to 0.9.6 (#1007)

    Bumps [ruff](https://github.com/astral-sh/ruff) from 0.9.5 to 0.9.6.
    - [Release notes](https://github.com/astral-sh/ruff/releases)
    - [Changelog](https://github.com/astral-sh/ruff/blob/main/CHANGELOG.md)
    - [Commits](astral-sh/ruff@0.9.5...0.9.6)
    
    ---
    updated-dependencies:
    - dependency-name: ruff
      dependency-type: direct:development
      update-type: version-update:semver-patch
    ...
    
    Signed-off-by: dependabot[bot] <[email protected]>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    dependabot[bot] authored Feb 11, 2025

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    Copy the full SHA
    7a3e50f View commit details
  2. Add Kodu to the list of clients with supported code patterns (#1003)

    * Add Kodu to the list of clients with supported code patterns
    
    Extracting patterns is necessary for appropriately mux between
    providers depending on the client. This PR enables Kodu.
    
    * fixing copy paste errors
    aponcedeleonch authored Feb 11, 2025

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    Copy the full SHA
    c542d83 View commit details
  3. Force ANSI character output when running integration tests (#1011)

    The intention is to output everything, since currently we're missing
    stderr.
    
    Signed-off-by: Juan Antonio Osorio <[email protected]>
    JAORMX authored Feb 11, 2025

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    Copy the full SHA
    9f542c3 View commit details
  4. Remove code for mapping origin to destination in muxing (#1010)

    The initial muxing implementation assumed that we would need to
    transform the input body from one provider to another. e.g.
    from athropic to openai. This changed. The final implementation
    is assuming that is receiving OpenAI format and will respond with
    OpenAI format. This PR is removing unused code.
    aponcedeleonch authored Feb 11, 2025

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    Copy the full SHA
    0bec802 View commit details
  5. Add codegate package path to poetry (#1012)

    * Fix codegate not a package when poetry install
    
    Signed-off-by: Radoslav Dimitrov <[email protected]>
    
    * Add a log message when an integration test has started
    
    Signed-off-by: Radoslav Dimitrov <[email protected]>
    
    ---------
    
    Signed-off-by: Radoslav Dimitrov <[email protected]>
    rdimitrov authored Feb 11, 2025

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    Copy the full SHA
    46a5fd0 View commit details
  6. Pass down base URL and API key to completion handler (#1002)

    * Pass down base URL and API key to completion handler
    
    In order to make Ollama work with API Keys, I needed to change the
    completion handler to take a base URL and also leverage a given API key
    (if available).
    
    Signed-off-by: Juan Antonio Osorio <[email protected]>
    Co-Authored-by: Alejandro Ponce de Leon <[email protected]>
    
    * fix unit tests and formatting
    
    * fix integration tests
    
    * Pass API key to ollama calls
    
    Signed-off-by: Juan Antonio Osorio <[email protected]>
    
    * Handle optional authorization header in ollama endpoints.
    
    Signed-off-by: Juan Antonio Osorio <[email protected]>
    
    ---------
    
    Signed-off-by: Juan Antonio Osorio <[email protected]>
    Co-authored-by: Alejandro Ponce de Leon <[email protected]>
    JAORMX and aponcedeleonch authored Feb 11, 2025

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    Copy the full SHA
    6938347 View commit details
  7. Switch to qwen2.5-coder:1.5b (#1013)

    Signed-off-by: Radoslav Dimitrov <[email protected]>
    rdimitrov authored Feb 11, 2025

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    Copy the full SHA
    8cb658d View commit details
  8. Enable the integration tests for openrouter (#1016)

    rdimitrov authored Feb 11, 2025

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    Copy the full SHA
    d86a965 View commit details

Commits on Feb 12, 2025

  1. Enable provider model updating when updating provider itself (#1014)

    This now calls the model updating logic when updating the provider
    itself. Thus allowing us to have a way to update the model list.
    
    Signed-off-by: Juan Antonio Osorio <[email protected]>
    JAORMX authored Feb 12, 2025

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    Copy the full SHA
    f361178 View commit details
  2. Bump litellm from 1.60.8 to 1.61.1 (#1018)

    Bumps [litellm](https://github.com/BerriAI/litellm) from 1.60.8 to 1.61.1.
    - [Release notes](https://github.com/BerriAI/litellm/releases)
    - [Commits](BerriAI/litellm@v1.60.8...v1.61.1)
    
    ---
    updated-dependencies:
    - dependency-name: litellm
      dependency-type: direct:production
      update-type: version-update:semver-minor
    ...
    
    Signed-off-by: dependabot[bot] <[email protected]>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    dependabot[bot] authored Feb 12, 2025

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    Copy the full SHA
    f4e252a View commit details
  3. Bump cryptography from 44.0.0 to 44.0.1 (#1019)

    Bumps [cryptography](https://github.com/pyca/cryptography) from 44.0.0 to 44.0.1.
    - [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst)
    - [Commits](pyca/cryptography@44.0.0...44.0.1)
    
    ---
    updated-dependencies:
    - dependency-name: cryptography
      dependency-type: direct:production
      update-type: version-update:semver-patch
    ...
    
    Signed-off-by: dependabot[bot] <[email protected]>
    Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
    dependabot[bot] authored Feb 12, 2025

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    Copy the full SHA
    1b5595f View commit details
  4. Pin pyproject to require python 3.12 (#1009)

    * Pin pyproject to require python 3.12
    
    Some dependencies (namely presidio) don't support python 3.13 yet.
    
    Signed-off-by: Juan Antonio Osorio <[email protected]>
    
    * updating lock file
    
    ---------
    
    Signed-off-by: Juan Antonio Osorio <[email protected]>
    Co-authored-by: Alejandro Ponce <[email protected]>
    JAORMX and aponcedeleonch authored Feb 12, 2025

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    Copy the full SHA
    a7e14f6 View commit details
  5. Fix FIM on Continue. Have specific formatters for chat and FIM (#1015)

    * Fix FIM on Continue. Have specific formatters for chat and FIM
    
    Until now we had a general formatter on the way out of muxing.
    This is wrong since sometimes the pipelines respond with different
    format for chat or FIM. Such is the case for Ollama.
    
    This PR separates the formatters and declares them explicitly so
    that they're easier to adjust in the future.
    
    * Adressed comments from review
    aponcedeleonch authored Feb 12, 2025

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    Copy the full SHA
    8897318 View commit details
  6. Fix anthropic format in integration tests (#1021)

    jhrozek authored Feb 12, 2025

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    Copy the full SHA
    58512fd View commit details
  7. Fix ollama support for Kodu when muxing (#1022)

    Muixing was failing for 2 reasons:
    
    1. Sometimes we return OpenAI format
    from ollama provider. Before we were assuming that everything that
    was returned from ollama provider had ollama format.
    2. The OpenAI format returned from ollama provider had an invalid
    `created` field.
    aponcedeleonch authored Feb 12, 2025

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature.
    Copy the full SHA
    e6600f6 View commit details
13 changes: 8 additions & 5 deletions .github/workflows/integration-tests.yml
Original file line number Diff line number Diff line change
@@ -15,7 +15,9 @@ on:
anthropic-key:
description: 'The Anthropic key to use for integration tests'
required: true

openrouter-key:
description: 'The Openrouter key to use for integration tests'
required: true
jobs:
integration-tests:
name: Test
@@ -24,11 +26,12 @@ jobs:
fail-fast: false # Continue running other tests if one fails
matrix:
python-version: [ "3.12" ]
test-provider: [ "copilot", "openai", "anthropic", "ollama", "vllm", "llamacpp" ]
test-provider: [ "copilot", "openai", "anthropic", "ollama", "vllm", "llamacpp", "openrouter" ]
env:
ENV_COPILOT_KEY: ${{ secrets.copilot-key }}
ENV_OPENAI_KEY: ${{ secrets.copilot-key }} # We use the same key for OpenAI as the Copilot tests
ENV_ANTHROPIC_KEY: ${{ secrets.anthropic-key }}
ENV_OPENROUTER_KEY: ${{ secrets.openrouter-key }}
CA_CERT_FILE: "/home/runner/work/codegate/codegate/codegate_volume/certs/ca.crt"
CODEGATE_CONTAINER_NAME: "codegate"
CODEGATE_MOUNT_PATH_CERT_FILE: "/app/codegate_volume/certs/ca.crt"
@@ -144,7 +147,7 @@ jobs:
run: |
function check_model_ready() {
response=$(curl -s http://localhost:11434/api/generate -d '{
"model": "qwen2.5-coder:0.5b",
"model": "qwen2.5-coder:1.5b",
"prompt": "Hello",
"stream": false
}' 2>&1)
@@ -187,7 +190,7 @@ jobs:
fi
echo "Starting model download/initialization..."
docker exec -d ollama ollama run qwen2.5-coder:0.5b
docker exec -d ollama ollama run qwen2.5-coder:1.5b
# Monitor container and model status
monitor_count=0
@@ -273,7 +276,7 @@ jobs:
env:
CODEGATE_PROVIDERS: ${{ matrix.test-provider }}
run: |
poetry run python tests/integration/integration_tests.py
poetry run --ansi python tests/integration/integration_tests.py
- name: Logs - CodeGate container
if: always()
1 change: 1 addition & 0 deletions .github/workflows/run-on-pr.yml
Original file line number Diff line number Diff line change
@@ -28,3 +28,4 @@ jobs:
secrets:
copilot-key: ${{ secrets.INTEGRATION_TESTS_COPILOT_KEY }}
anthropic-key: ${{ secrets.INTEGRATION_TESTS_ANTHROPIC_KEY }}
openrouter-key: ${{ secrets.INTEGRATION_TESTS_OPENROUTER_KEY }}
1 change: 1 addition & 0 deletions .github/workflows/run-on-push.yml
Original file line number Diff line number Diff line change
@@ -27,6 +27,7 @@ jobs:
secrets:
copilot-key: ${{ secrets.INTEGRATION_TESTS_COPILOT_KEY }}
anthropic-key: ${{ secrets.INTEGRATION_TESTS_ANTHROPIC_KEY }}
openrouter-key: ${{ secrets.INTEGRATION_TESTS_OPENROUTER_KEY }}
openapi:
name: Generate the OpenAPI docs
needs: [ci]
112 changes: 58 additions & 54 deletions poetry.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading