feat(cohere): upgrade integration from ai to gen_ai #6718
shellmayrtest-integrations-cloud.yml
on: pull_request
Matrix: Cloud
All Cloud tests passed
3s
Annotations
12 errors and 29 warnings
|
Cloud (3.10, ubuntu-22.04)
Process completed with exit code 1.
|
|
Cloud (3.6, ubuntu-22.04)
Process completed with exit code 1.
|
|
Cloud (3.12, ubuntu-22.04)
Process completed with exit code 1.
|
|
Cloud (3.7, ubuntu-22.04)
Process completed with exit code 255.
|
|
Cloud (3.14, ubuntu-22.04)
Process completed with exit code 1.
|
|
Cloud (3.14t, ubuntu-22.04)
Process completed with exit code 1.
|
|
Cloud (3.11, ubuntu-22.04)
Process completed with exit code 1.
|
|
Cloud (3.8, ubuntu-22.04)
Process completed with exit code 1.
|
|
Cloud (3.13, ubuntu-22.04)
Process completed with exit code 1.
|
|
Cloud (3.9, ubuntu-22.04)
Process completed with exit code 1.
|
|
Circular import prevents integration from loading:
sentry_sdk/integrations/cohere_v2.py#L22
Lines 22-26 import `CohereIntegration`, `COLLECTED_CHAT_PARAMS`, and `_capture_exception` from `cohere.py` at module level. However, `cohere.py` imports `setup_v2` from `cohere_v2.py` at line 24, before those names are defined. When Python loads `cohere.py`, it triggers loading `cohere_v2.py`, which then tries to import names that don't exist yet in the partially-initialized `cohere.py` module. This will raise `ImportError: cannot import name 'CohereIntegration' from partially initialized module` at runtime, completely breaking the Cohere integration.
|
|
All Cloud tests passed
Process completed with exit code 1.
|
|
Cloud (3.10, ubuntu-22.04)
❌ Patch coverage check failed: 8.23% < target 80%
|
|
Cloud (3.6, ubuntu-22.04)
❌ Patch coverage check failed: 8.28% < target 80%
|
|
Cloud (3.10, ubuntu-22.04)
Failed to upload coverage artifact: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run
|
|
Cloud (3.6, ubuntu-22.04)
Failed to upload test artifact: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run
|
|
Cloud (3.12, ubuntu-22.04)
❌ Patch coverage check failed: 8.23% < target 80%
|
|
Cloud (3.12, ubuntu-22.04)
Failed to upload coverage artifact: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run
|
|
Cloud (3.12, ubuntu-22.04)
Failed to upload test artifact: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run
|
|
Cloud (3.7, ubuntu-22.04)
❌ Patch coverage check failed: 8.28% < target 80%
|
|
Cloud (3.7, ubuntu-22.04)
Failed to upload coverage artifact: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run
|
|
Cloud (3.7, ubuntu-22.04)
Failed to upload test artifact: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run
|
|
Cloud (3.14, ubuntu-22.04)
❌ Patch coverage check failed: 8.23% < target 80%
|
|
Cloud (3.14, ubuntu-22.04)
Failed to upload coverage artifact: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run
|
|
Cloud (3.14, ubuntu-22.04)
Failed to upload test artifact: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run
|
|
Cloud (3.14t, ubuntu-22.04)
❌ Patch coverage check failed: 8.23% < target 80%
|
|
Cloud (3.14t, ubuntu-22.04)
Failed to upload coverage artifact: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run
|
|
Cloud (3.14t, ubuntu-22.04)
Failed to upload test artifact: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run
|
|
Cloud (3.11, ubuntu-22.04)
❌ Patch coverage check failed: 0.00% < target 80%
|
|
Cloud (3.11, ubuntu-22.04)
Failed to upload coverage artifact: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run
|
|
Cloud (3.11, ubuntu-22.04)
Failed to upload test artifact: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run
|
|
Cloud (3.8, ubuntu-22.04)
❌ Patch coverage check failed: 0.00% < target 80%
|
|
Cloud (3.8, ubuntu-22.04)
Failed to upload coverage artifact: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run
|
|
Cloud (3.8, ubuntu-22.04)
Failed to upload test artifact: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run
|
|
Cloud (3.13, ubuntu-22.04)
❌ Patch coverage check failed: 0.00% < target 80%
|
|
Cloud (3.13, ubuntu-22.04)
Failed to upload coverage artifact: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run
|
|
Cloud (3.13, ubuntu-22.04)
Failed to upload test artifact: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run
|
|
Cloud (3.9, ubuntu-22.04)
❌ Patch coverage check failed: 0.00% < target 80%
|
|
Cloud (3.9, ubuntu-22.04)
Failed to upload coverage artifact: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run
|
|
Cloud (3.9, ubuntu-22.04)
Failed to upload test artifact: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run
|
|
Test asserts wrong span data attribute - will fail at runtime:
tests/integrations/cohere/test_cohere.py#L319
The test `test_v2_nonstreaming_chat` asserts `SPANDATA.GEN_AI_RESPONSE_MODEL` on line 319, but the implementation in `cohere_v2.py` sets `SPANDATA.GEN_AI_REQUEST_MODEL` at line 188. This is a semantic difference: REQUEST_MODEL is what was requested, RESPONSE_MODEL is what the model actually used. The test will fail because the span data does not contain the `gen_ai.response.model` key.
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
codecov-coverage-results-shellmayr-feat-upgrade-cohere-ai-integration-from-ai-to-genai-test-cloud
|
107 KB |
sha256:5ae4235ffbc459f6e74ee12470fd72c3bc57e2ae1063e1f48b1de54f3fd2f708
|
|
|
codecov-test-results-shellmayr-feat-upgrade-cohere-ai-integration-from-ai-to-genai-test-cloud
|
1.08 KB |
sha256:d6f60137e4f5d576d42ba29bdef6e60a11a5596fe6b774b77f9646973d14b5b3
|
|