LCORE-2011: Updated Konflux deps#1831
Merged
tisnik merged 2 commits intoJun 2, 2026
Merged
Conversation
Contributor
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
WalkthroughUpdates pinned versions and SHA256 hashes in ChangesDependency Version and Hash Updates
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
tisnik
force-pushed
the
lcore-2011-updated-konflux-deps-
branch
6 times, most recently
from
d46a0e5 to
4478278
Compare
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.konflux/requirements.hashes.source.txt:
- Around line 682-699: The OpenTelemetry packages are pinned to mixed versions
(opentelemetry-api/exporters at 1.42.1 while opentelemetry-sdk and
opentelemetry-proto remain at 1.40.0), which breaks compatibility; update the
requirement pins so the core OTel packages and related packages
(opentelemetry-api, opentelemetry-sdk, opentelemetry-proto,
opentelemetry-exporter-otlp, opentelemetry-exporter-otlp-proto-http/grpc/common,
opentelemetry-distro, and any instrumentations/conventions) all target the same
compatible release (e.g., set opentelemetry-sdk and opentelemetry-proto to
1.42.1 or downgrade the api/exporters to 1.40.0), and ensure the corresponding
--hash entries are updated to match the chosen version set.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: d6d43648-2d00-4a41-ab23-2dafc0114d8e
📒 Files selected for processing (1)
.konflux/requirements.hashes.source.txt
📜 Review details
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (17)
- GitHub Check: list_outdated_dependencies
- GitHub Check: spectral
- GitHub Check: Pyright
- GitHub Check: Pylinter
- GitHub Check: unit_tests (3.13)
- GitHub Check: integration_tests (3.13)
- GitHub Check: integration_tests (3.12)
- GitHub Check: unit_tests (3.12)
- GitHub Check: build-pr
- GitHub Check: E2E: server mode / ci / group 3
- GitHub Check: E2E: library mode / ci / group 3
- GitHub Check: E2E: server mode / ci / group 2
- GitHub Check: E2E: library mode / ci / group 2
- GitHub Check: E2E: library mode / ci / group 1
- GitHub Check: E2E: server mode / ci / group 1
- GitHub Check: E2E Tests for Lightspeed Evaluation job
- GitHub Check: Konflux kflux-prd-rh02 / lightspeed-stack-on-pull-request
tisnik
force-pushed
the
lcore-2011-updated-konflux-deps-
branch
2 times, most recently
from
9f89a11 to
3a55e71
Compare
tisnik
force-pushed
the
lcore-2011-updated-konflux-deps-
branch
from
3a55e71 to
d0c07a8
Compare
Contributor
Author
|
/retest |
2 similar comments
Contributor
Author
|
/retest |
Contributor
Author
|
/retest |
tisnik
force-pushed
the
lcore-2011-updated-konflux-deps-
branch
from
d0c07a8 to
3213e70
Compare
Contributor
There was a problem hiding this comment.
♻️ Duplicate comments (1)
.konflux/requirements.hashes.source.txt (1)
682-702:⚠️ Potential issue | 🟠 MajorAlign OpenTelemetry pins to one compatible release set.
opentelemetry-api/OTLP exporters are pinned at1.42.1whileopentelemetry-sdkandopentelemetry-protoremain at1.40.0(Lines 703-708). This mixed core set is not supported; pin OTel core packages to the same compatible version line.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In @.konflux/requirements.hashes.source.txt around lines 682 - 702, The OpenTelemetry packages are pinned to mixed release lines (opentelemetry-api and exporters at 1.42.1 while opentelemetry-sdk/opentelemetry-proto are at 1.40.0), which is unsupported; update the package pins so all core OTel packages match a single compatible version line — e.g., set opentelemetry-sdk and opentelemetry-proto to 1.42.1 (or alternatively downgrade api/exporters to 1.40.0) and regenerate/update the corresponding hashes for opentelemetry-sdk and opentelemetry-proto entries so the requirements file is consistent with opentelemetry-api and opentelemetry-exporter-otlp.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Duplicate comments:
In @.konflux/requirements.hashes.source.txt:
- Around line 682-702: The OpenTelemetry packages are pinned to mixed release
lines (opentelemetry-api and exporters at 1.42.1 while
opentelemetry-sdk/opentelemetry-proto are at 1.40.0), which is unsupported;
update the package pins so all core OTel packages match a single compatible
version line — e.g., set opentelemetry-sdk and opentelemetry-proto to 1.42.1 (or
alternatively downgrade api/exporters to 1.40.0) and regenerate/update the
corresponding hashes for opentelemetry-sdk and opentelemetry-proto entries so
the requirements file is consistent with opentelemetry-api and
opentelemetry-exporter-otlp.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: 59a51007-bcd4-4e12-a2f7-fabbea4de73a
📒 Files selected for processing (1)
.konflux/requirements.hashes.source.txt
📜 Review details
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (15)
- GitHub Check: mypy
- GitHub Check: build-pr
- GitHub Check: unit_tests (3.13)
- GitHub Check: unit_tests (3.12)
- GitHub Check: Pylinter
- GitHub Check: integration_tests (3.12)
- GitHub Check: spectral
- GitHub Check: E2E: library mode / ci / group 1
- GitHub Check: E2E: library mode / ci / group 2
- GitHub Check: E2E: server mode / ci / group 1
- GitHub Check: E2E: server mode / ci / group 2
- GitHub Check: E2E: server mode / ci / group 3
- GitHub Check: E2E Tests for Lightspeed Evaluation job
- GitHub Check: E2E: library mode / ci / group 3
- GitHub Check: Konflux kflux-prd-rh02 / lightspeed-stack-on-pull-request
🔇 Additional comments (1)
.konflux/requirements.hashes.source.txt (1)
527-535: LGTM!Also applies to: 647-652
tisnik
force-pushed
the
lcore-2011-updated-konflux-deps-
branch
2 times, most recently
from
f84c6da to
4185cf3
Compare
tisnik
force-pushed
the
lcore-2011-updated-konflux-deps-
branch
from
4185cf3 to
a11443b
Compare
Contributor
Author
|
/retest |
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
1 participant
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
LCORE-2011: Updated Konflux deps
Type of change
Tools used to create PR
Related Tickets & Documents
Summary by CodeRabbit