Skip to content

chore: update chart versions for cloudprem, console-v3, formance, and… - #411

Closed
reslene wants to merge 1 commit into
mainfrom
feat/platform-ui_v3.1.0
Closed

chore: update chart versions for cloudprem, console-v3, formance, and…#411
reslene wants to merge 1 commit into
mainfrom
feat/platform-ui_v3.1.0

Conversation

@reslene

@reslene reslene commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

… portal

Bumped the following chart versions:

  • cloudprem: 4.10.3 -> 5.0.0
  • console-v3: 3.7.2 -> 4.0.0
  • formance: 1.18.2 -> 2.0.0
  • portal: 3.7.2 -> 4.0.0

Updated app versions accordingly to maintain consistency across dependencies.

… portal

Bumped the following chart versions:
- cloudprem: 4.10.3 -> 5.0.0
- console-v3: 3.7.2 -> 4.0.0
- formance: 1.18.2 -> 2.0.0
- portal: 3.7.2 -> 4.0.0

Updated app versions accordingly to maintain consistency across dependencies.
@reslene
reslene requested a review from a team as a code owner August 13, 2026 14:45
@github-actions github-actions Bot added the docs label Aug 13, 2026
@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 604ecda3-79a9-41b5-ade6-cac65809c819

📥 Commits

Reviewing files that changed from the base of the PR and between a8ff1ea and 75c099e.

⛔ Files ignored due to path filters (6)
  • charts/cloudprem/Chart.lock is excluded by !**/*.lock, !**/*.lock
  • charts/cloudprem/Chart.yaml is excluded by !**/*.yaml
  • charts/console-v3/Chart.yaml is excluded by !**/*.yaml
  • charts/formance/Chart.lock is excluded by !**/*.lock, !**/*.lock
  • charts/formance/Chart.yaml is excluded by !**/*.yaml
  • charts/portal/Chart.yaml is excluded by !**/*.yaml
📒 Files selected for processing (5)
  • README.md
  • charts/cloudprem/README.md
  • charts/console-v3/README.md
  • charts/formance/README.md
  • charts/portal/README.md

Walkthrough

The pull request updates Helm chart README metadata for Cloudprem, Console-V3, Formance, and Portal. It changes chart and application versions in badges, updates documented dependency constraints, and refreshes the root chart listing.

Changes

Chart release metadata

Layer / File(s) Summary
Chart and application version badges
charts/cloudprem/README.md, charts/console-v3/README.md, charts/formance/README.md, charts/portal/README.md
Updated chart badges and the Console-V3 and Portal application versions.
Catalog and dependency references
README.md, charts/cloudprem/README.md, charts/formance/README.md
Updated the root chart listing and documented dependency versions for Cloudprem and Formance.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Mergeability Score: ⚪ Minimal · up to 75c09

This PR updates chart and app versions across the listed components; no actionable merge-blocking risk remains beyond normal checks and review.

Possibly related PRs

Suggested reviewers: gfyrag

Poem

I’m a rabbit with charts in my tray,
Updating release signs today.
Cloudprem hops to five,
New app versions arrive,
README badges now point the way.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the chart version updates for cloudprem, console-v3, formance, and portal.
Description check ✅ Passed The description clearly lists the chart and app version updates covered by the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/platform-ui_v3.1.0

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.

@NumaryBot

Copy link
Copy Markdown
Contributor

✅ Approve — automated review

The chart, dependency, lockfile, app, and generated README versions are updated consistently, with no stale version references found.

No findings.

Comment thread charts/portal/Chart.yaml
# follow Semantic Versioning. They should reflect the version the application is using.
# It is recommended to use it with quotes.
appVersion: "v2.6.2"
appVersion: "v3.1.0"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

appVersion jumps from v2.6.2 to v3.1.0, skipping v3.0.0. is it intentional? why there was no v3.0.0? the same for console-v3

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Cloudprem is already on v3.1.0

We missed some update on helm repo...
If it's better, i can create all the PR missed until v3.1.0 ?

the versions that is not in helm history are:
v2.7.0
v3.0.0

what do you prefer ?

@sofiasimdianova

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

it's not a blocker, but it's not best practice to skip version :(

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

i'm gonna make the missing PR, it's better this way

Comment thread charts/portal/Chart.yaml

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Portal was migrated from Remix to Next.js in platform-ui PR #1351. The old Remix app exposed /_info via apps/portal/app/routes/[_info].ts, but the new Next.js app no longer has apps/portal/app/_info/route.ts.
However, both readiness and liveness probes in https://github.com/formancehq/helm/blob/main/charts/portal/templates/deployment.yaml#L48
still use: GET /_info
With new version this will return 404, causing the pod to fail its health checks and potentially restart. can you recheck that please on your side ?
Helm expects: /_info
Next.js provides: /api/health

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@sofiasimdianova oh very good point, i do not know if the migration of @BrieucCaillot is having the healthcheck route, let me check this on my side.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

ok @sofiasimdianova i've just checked you were right, i'm gonna have to make a patch i'm afraid !

Comment thread charts/console-v3/Chart.yaml
@reslene
reslene marked this pull request as draft August 14, 2026 09:46
@reslene reslene closed this Aug 14, 2026
@reslene
reslene deleted the feat/platform-ui_v3.1.0 branch August 14, 2026 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants