Skip to content

Conversation

@tisnik
Copy link
Contributor

@tisnik tisnik commented Jan 26, 2026

Description

LCORE-1216: 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-1216

Summary by CodeRabbit

Release Notes

  • Chores
    • Updated dependencies: llama-stack upgraded to 0.4.2, wcwidth updated to 0.4.0, setuptools bumped to 80.10.2
    • Added new dependencies: einops and cachetools
    • Reorganized package lists and removed llama-stack-client dependency
    • Added tornado to package set

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

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 26, 2026

Walkthrough

This pull request updates project dependencies, upgrading llama-stack from 0.3.5 to 0.4.2, adding new dependencies (cachetools, einops, multidict, llama-stack-api, llama-stack-client), introducing tornado, upgrading wcwidth to 0.4.0, and bumping setuptools to 80.10.2 with corresponding hash file updates.

Changes

Cohort / File(s) Summary
Tekton configuration
.tekton/lightspeed-stack-pull-request.yaml, .tekton/lightspeed-stack-push.yaml
Removed cachetools from pull-request config package list; added tornado and removed llama-stack-client from push config package list with minor reordering
Build requirements
requirements-build.txt
Bumped setuptools from 80.10.1 to 80.10.2; added commented entries for cachetools, llama-stack-api, llama-stack-client, multidict, einops; removed commented dunamai entry
Source hashes
requirements.hashes.source.txt
Added cachetools 6.2.5, einops 0.8.2, multidict 6.7.1, llama-stack-api 0.4.2, llama-stack-client 0.4.2; upgraded llama-stack from 0.3.5 to 0.4.2; upgraded wcwidth from 0.3.5 to 0.4.0; added corresponding hashes for all entries
Wheel hashes
requirements.hashes.wheel.txt
Removed cachetools, einops, llama-stack-client, multidict entries; added tornado with hashes

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 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 actual changes involve updating Python package dependencies (cachetools, llama-stack, einops, multidict, wcwidth, tornado, setuptools) across multiple configuration and requirements files, with no observable Konflux-related changes in the provided diffs. Update the title to accurately reflect the main change, such as 'Update Python package dependencies' or 'Bump Python dependencies and setuptools version' to match the actual changeset.
✅ 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 8aa52c6 into lightspeed-core:main Jan 26, 2026
22 checks passed
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