Skip to content

Conversation

@mauriciozanettisalomao
Copy link
Contributor

Overview

Jira Ticket: https://linuxfoundation.atlassian.net/browse/LFXV2-791

This pull request updates the version of the lfx-v2-committee-service dependency in the Helm chart to ensure the platform uses the latest compatible release.

Dependency updates:

  • Increased the version constraint for lfx-v2-committee-service from ~0.2.10 to ~0.2.13 in charts/lfx-platform/Chart.yaml to pull in newer bug fixes and features.

Jira Ticket: https://linuxfoundation.atlassian.net/browse/LFXV2-791

Signed-off-by: Mauricio Zanetti Salomao <mauriciozanetti86@gmail.com>
Copilot AI review requested due to automatic review settings November 24, 2025 17:21
@coderabbitai
Copy link

coderabbitai bot commented Nov 24, 2025

Walkthrough

Bumped the Helm chart version in charts/lfx-platform/Chart.yaml and updated its lfx-v2-committee-service dependency constraint; added GOMEMLIMIT to the spell-check dictionary in .cspell.json.

Changes

Cohort / File(s) Summary
Helm chart and dependency version updates
charts/lfx-platform/Chart.yaml
Bumped chart version from 0.3.13 to 0.3.14; updated lfx-v2-committee-service dependency constraint from ~0.2.10 to ~0.2.13.
Spell-check dictionary addition
.cspell.json
Added the word GOMEMLIMIT to the cspell words list.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Files to spot-check: charts/lfx-platform/Chart.yaml, .cspell.json

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning The PR updates chart dependencies but does not implement the coding requirements outlined in LFXV2-791, which calls for structural and schema changes to the committee-service codebase. This PR should either implement the LFXV2-791 objectives (adding org ID to structs, schema attributes, Tags() method, tests, and documentation) or be linked to a different issue focused solely on dependency bumping.
Out of Scope Changes check ⚠️ Warning The spell-check dictionary update (.cspell.json) appears tangential to the stated objective of bumping chart dependencies and is unrelated to LFXV2-791's requirements. Remove the .cspell.json change or provide justification for its inclusion. It should either be in a separate PR or clearly related to the dependency update objectives.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately reflects the main change: bumping the committee-service chart dependency to a newer version in the Helm chart.
Description check ✅ Passed The description clearly explains the purpose of the PR, references the Jira ticket, and details the specific dependency version update being made.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Jira integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 1ac36a8 and 94c5617.

📒 Files selected for processing (1)
  • .cspell.json (1 hunks)
✅ Files skipped from review due to trivial changes (1)
  • .cspell.json
⏰ 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). (1)
  • GitHub Check: MegaLinter

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

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request updates the Helm chart dependency constraint for lfx-v2-committee-service from ~0.2.10 to ~0.2.13, allowing the platform to use newer patch versions of the committee service. The Chart.lock file was regenerated to reflect this change along with automatic updates to other dependencies that satisfy their existing version constraints.

Key Changes

  • Updated committee-service version constraint from ~0.2.10 to ~0.2.13 in Chart.yaml
  • Regenerated Chart.lock with committee-service resolved to 0.2.13 and updated digest
  • Chart.lock also reflects automatic updates to openfga (0.2.48→0.2.49) and lfx-v2-project-service (0.5.1→0.5.2) within existing constraints

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
charts/lfx-platform/Chart.yaml Updated committee-service version constraint to ~0.2.13
charts/lfx-platform/Chart.lock Regenerated lock file with updated dependency versions and digest

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Jira Ticket: https://linuxfoundation.atlassian.net/browse/LFXV2-791

Signed-off-by: Mauricio Zanetti Salomao <mauriciozanetti86@gmail.com>
Jira Ticket: https://linuxfoundation.atlassian.net/browse/LFXV2-791

Generated with [Claude Code](https://claude.com/claude-code)

Signed-off-by: Mauricio Zanetti Salomao <mauriciozanetti86@gmail.com>
@mauriciozanettisalomao mauriciozanettisalomao merged commit fd6584c into linuxfoundation:main Nov 24, 2025
4 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.

2 participants