Skip to content

Conversation

@tisnik
Copy link
Contributor

@tisnik tisnik commented Jan 27, 2026

Description

LCORE-1108: Updated Konflux references

Type of change

  • Refactor
  • New feature
  • Bug fix
  • CVE fix
  • Optimization
  • Documentation Update
  • Configuration Update
  • Bump-up service version
  • Bump-up dependent library
  • Bump-up library or tool used for development (does not change the final image)
  • CI configuration change
  • Konflux configuration change
  • Unit tests improvement
  • Integration tests improvement
  • End to end tests improvement

Tools used to create PR

  • Assisted-by: N/A
  • Generated by: N/A

Related Tickets & Documents

  • Related Issue #LCORE-1108

Summary by CodeRabbit

  • Chores
    • Updated multiple project dependencies to their latest versions, including google-auth, llama-stack-api, sentence-transformers, wcwidth, and pathspec for improved compatibility and stability.
    • Applied minor formatting adjustments to build configuration files.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 27, 2026

Walkthrough

Minor formatting adjustments to Tekton pipeline YAMLs with corresponding dependency updates: pathspec bumped to 1.0.4, google-auth and wcwidth versions incremented, sentence-transformers added at 5.2.1, llama-stack-api updated, and hash entries adjusted accordingly with comment additions in build requirements.

Changes

Cohort / File(s) Summary
Tekton Pipeline Configuration
.tekton/lightspeed-stack-pull-request.yaml, .tekton/lightspeed-stack-push.yaml
Minor whitespace reformatting of the binary.packages string in prefetch input blocks; no semantic changes to package lists
Build Requirements
requirements-build.txt
Pathspec dependency bumped from 1.0.3 to 1.0.4; added comment notes for sentence-transformers in wheel and setuptools sections
Hash Source File
requirements.hashes.source.txt
Dependency version updates with new hashes: google-auth (2.47.0→2.48.0), llama-stack-api (0.4.2→0.4.3), wcwidth (0.4.0→0.5.0), sentence-transformers (5.2.1 added); expanded hash entries for greenlet
Hash Wheel File
requirements.hashes.wheel.txt
Removed sentence-transformers==5.2.0 entry (2 hash lines removed)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • radofuchs
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The title mentions 'Updated Konflux references' but the changes are primarily dependency updates (pathspec, google-auth, llama-stack-api, sentence-transformers, wcwidth) and formatting adjustments to Tekton YAML files, not Konflux configuration changes. Revise the title to accurately reflect the main changes, such as 'Bump dependencies and update Tekton YAML formatting' or provide a title that aligns with the actual Konflux-specific configuration modifications if they are not apparent.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@tisnik tisnik merged commit d3823b8 into lightspeed-core:main Jan 27, 2026
22 checks passed
@coderabbitai coderabbitai bot mentioned this pull request Jan 27, 2026
15 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant