Skip to content

Upgrade charts to latest versions#79

Merged
andrest50 merged 1 commit intomainfrom
andrest50/bump-charts
Nov 11, 2025
Merged

Upgrade charts to latest versions#79
andrest50 merged 1 commit intomainfrom
andrest50/bump-charts

Conversation

@andrest50
Copy link
Contributor

Summary

This PR upgrades chart dependencies to their latest versions:

  • lfx-v2-fga-sync: 0.2.3 -> 0.2.4
  • lfx-v2-indexer-service: 0.4.4 -> 0.4.6

The lfx-platform chart version has been bumped to 0.3.7.

🤖 Generated with Claude Code

Copilot AI review requested due to automatic review settings November 11, 2025 01:16
@andrest50 andrest50 requested review from a team and emsearcy as code owners November 11, 2025 01:16
@coderabbitai
Copy link

coderabbitai bot commented Nov 11, 2025

Walkthrough

Bumps charts/lfx-platform Chart version to 0.3.7 and updates three chart dependency version constraints: lfx-v2-fga-sync, lfx-v2-indexer-service, and lfx-v2-meeting-service.

Changes

Cohort / File(s) Summary
Helm chart metadata & dependencies
charts/lfx-platform/Chart.yaml
Incremented chart version to 0.3.7; updated dependency constraints: lfx-v2-fga-sync~0.2.4, lfx-v2-indexer-service~0.4.6, lfx-v2-meeting-service~0.5.3.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: upgrading chart dependencies to their latest versions, which matches the changeset.
Description check ✅ Passed The description is directly related to the changeset, detailing the specific dependency version upgrades and the chart version bump.
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
  • Commit unit tests in branch andrest50/bump-charts

📜 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 ebf0b1a and c2d13ef.

⛔ Files ignored due to path filters (1)
  • charts/lfx-platform/Chart.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • charts/lfx-platform/Chart.yaml (2 hunks)
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: mauriciozanettisalomao
Repo: linuxfoundation/lfx-v2-helm PR: 49
File: charts/lfx-platform/Chart.yaml:8-8
Timestamp: 2025-08-29T16:53:12.710Z
Learning: The maintainer mauriciozanettisalomao prefers to keep chart version increments conservative, choosing 0.2.7 over suggested 0.3.0 even for changes that could be considered behavior-altering (like new middleware and JSON Content-Type enforcement in Heimdall).
📚 Learning: 2025-08-29T16:53:12.710Z
Learnt from: mauriciozanettisalomao
Repo: linuxfoundation/lfx-v2-helm PR: 49
File: charts/lfx-platform/Chart.yaml:8-8
Timestamp: 2025-08-29T16:53:12.710Z
Learning: The maintainer mauriciozanettisalomao prefers to keep chart version increments conservative, choosing 0.2.7 over suggested 0.3.0 even for changes that could be considered behavior-altering (like new middleware and JSON Content-Type enforcement in Heimdall).

Applied to files:

  • charts/lfx-platform/Chart.yaml
⏰ 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
🔇 Additional comments (1)
charts/lfx-platform/Chart.yaml (1)

8-8: PR description omits lfx-v2-meeting-service update; verify minor version compatibility.

The PR description lists three dependency updates, but the actual changes include a fourth: lfx-v2-meeting-service 0.4.22 → 0.5.3. This is a minor version bump (0.4 → 0.5), which differs from the patch-level updates of the other dependencies and may introduce breaking changes or significant behavioral shifts.

The chart version bump from 0.3.6 to 0.3.7 appropriately reflects patch-level dependency updates, though the meeting-service upgrade warrants closer scrutiny.

Please verify the following:

  1. Was the lfx-v2-meeting-service update to 0.5.3 intentional? If so, the PR description should be updated to reflect this.
  2. Review the release notes for lfx-v2-meeting-service 0.5.x to confirm backward compatibility and identify any breaking changes.
  3. Test or validate that the platform integrations with this service remain functional after the upgrade.

If this is a breaking change or introduces incompatibilities, consider whether the chart version should be bumped to 0.4.0 instead of 0.3.7, and whether dependent services need corresponding updates.

Also applies to: 65-65, 73-73, 81-81


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 PR upgrades chart dependencies to their latest versions and bumps the lfx-platform chart version to 0.3.7.

  • Upgrades lfx-v2-fga-sync from 0.2.3 to 0.2.4
  • Upgrades lfx-v2-indexer-service from 0.4.4 to 0.4.6
  • Updates Chart.lock with new dependency versions and digest

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 Updates main chart version to 0.3.7 and dependency versions for fga-sync and indexer-service
charts/lfx-platform/Chart.lock Updates locked versions for dependencies and regenerates digest to match Chart.yaml changes

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

@andrest50 andrest50 force-pushed the andrest50/bump-charts branch from 13c16cb to ebf0b1a Compare November 11, 2025 01:21
Updated the following chart dependencies:
- lfx-v2-fga-sync: 0.2.3 -> 0.2.4
- lfx-v2-indexer-service: 0.4.4 -> 0.4.6

Bumped lfx-platform chart version to 0.3.7

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

Signed-off-by: Andres Tobon <andrest2455@gmail.com>
@andrest50 andrest50 force-pushed the andrest50/bump-charts branch from ebf0b1a to c2d13ef Compare November 11, 2025 01:28
@andrest50 andrest50 merged commit 0f83a50 into main Nov 11, 2025
4 checks passed
@andrest50 andrest50 deleted the andrest50/bump-charts branch November 11, 2025 01:36
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.

3 participants