Pin LoongSuite core repo SHA to 0.62-compatible commit#168
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Pins LoongSuite CI’s default upstream core checkout (CORE_REPO_SHA) to a known 0.62-compatible opentelemetry-python commit to prevent dependency-resolution breakages when upstream main advances beyond the semconv line LoongSuite currently targets.
Changes:
- Pin default
CORE_REPO_SHAin tox configs and generated GitHub Actions workflows to a specific upstream core commit. - Cap
loongsuite-instrumentation-mem0supportedmem0aiversions to<2.0.0and pin mem0 test environments tomem0ai==1.0.11. - Adjust mem0 VCR tests to skip reranker assertions when reranker spans are not emitted under the current fake/VCR setup.
Reviewed changes
Copilot reviewed 23 out of 23 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| tox.ini | Pins default CORE_REPO_SHA used by tox to a 0.62-compatible commit. |
| tox-loongsuite.ini | Pins default CORE_REPO_SHA used by LoongSuite tox matrix to the same commit. |
| .github/workflows/test_0.yml | Pins default CORE_REPO_SHA in generated CI workflow env. |
| .github/workflows/test_1.yml | Same as above for another test shard. |
| .github/workflows/test_2.yml | Same as above for another test shard. |
| .github/workflows/lint_0.yml | Pins default CORE_REPO_SHA in generated lint workflow env. |
| .github/workflows/misc_0.yml | Pins default CORE_REPO_SHA in generated misc workflow env. |
| .github/workflows/loongsuite_test_0.yml | Pins default CORE_REPO_SHA for LoongSuite test workflow. |
| .github/workflows/loongsuite_lint_0.yml | Pins default CORE_REPO_SHA for LoongSuite lint workflow. |
| .github/workflows/loongsuite_misc_0.yml | Pins default CORE_REPO_SHA for LoongSuite misc workflow. |
| .github/workflows/generate_workflows_lib/src/generate_workflows_lib/test.yml.j2 | Updates workflow template to use pinned CORE_REPO_SHA. |
| .github/workflows/generate_workflows_lib/src/generate_workflows_lib/lint.yml.j2 | Updates workflow template to use pinned CORE_REPO_SHA. |
| .github/workflows/generate_workflows_lib/src/generate_workflows_lib/misc.yml.j2 | Updates workflow template to use pinned CORE_REPO_SHA. |
| .github/workflows/generate_workflows_lib/src/generate_workflows_lib/loongsuite_test.yml.j2 | Updates LoongSuite workflow template to use pinned CORE_REPO_SHA. |
| .github/workflows/generate_workflows_lib/src/generate_workflows_lib/loongsuite_lint.yml.j2 | Updates LoongSuite workflow template to use pinned CORE_REPO_SHA. |
| .github/workflows/generate_workflows_lib/src/generate_workflows_lib/loongsuite_misc.yml.j2 | Updates LoongSuite workflow template to use pinned CORE_REPO_SHA. |
| instrumentation-loongsuite/loongsuite-instrumentation-mem0/src/opentelemetry/instrumentation/mem0/package.py | Caps declared supported mem0ai range to <2.0.0. |
| instrumentation-loongsuite/loongsuite-instrumentation-mem0/pyproject.toml | Caps optional mem0ai dependency to <2.0.0. |
| instrumentation-loongsuite/loongsuite-instrumentation-mem0/tests/requirements.oldest.txt | Pins mem0 test env dependency mem0ai to 1.0.11. |
| instrumentation-loongsuite/loongsuite-instrumentation-mem0/tests/requirements.latest.txt | Pins mem0 test env dependency mem0ai to 1.0.11. |
| instrumentation-loongsuite/loongsuite-instrumentation-mem0/tests/test_memory_operations_vcr.py | Skips reranker assertions when reranker spans aren’t emitted under current setup. |
| instrumentation-loongsuite/loongsuite-instrumentation-mem0/CHANGELOG.md | Documents mem0ai <2.0.0 cap and test pin rationale. |
| CHANGELOG-loongsuite.md | Documents CI core SHA pin + mem0ai cap at LoongSuite level. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
ralf0131
approved these changes
Apr 23, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR pins the default
CORE_REPO_SHAused by LoongSuite CI to a0.62-compatible upstream core commit.
This avoids CI breakage caused by upstream core
mainmoving from the0.62b0.devline to0.63b0.devwhile local LoongSuite packages are stilltested against the
0.62line.Fixes #167
Type of change
How Has This Been Tested?
tox -c tox-loongsuite.ini -e lint-util-genai,lint-loongsuite-instrumentation-litellmLoongSuite Lint 0: passed forutil-genaiandloongsuite-instrumentation-litellmLoongSuite Test 0: passed for watchedutil-genaiandloongsuite-instrumentation-litellmjobsDoes This PR Require a Core Repo Change?
Checklist: