feat(engine-v2): add platform configuration UI by pggb25 · Pull Request #2817 · Qovery/console

pggb25 · 2026-07-13T15:56:39Z

Summary

Here is a PR for self-managed clusters within the scope of the engine V2 POC.
All changes is under feature flag.

Preview can be found here
https://p80-z9693eb24-z7d62c7cc-gtw.zc531a994.rustrocks.cloud/

Existing test cluster is here
https://p80-z9693eb24-z7d62c7cc-gtw.zc531a994.rustrocks.cloud/organization/460616f0-94da-4d35-b631-6fa4ed08eb9a/cluster/8a494a2b-ce45-4499-a3c2-f5c75fbd38dd/settings/platform

What we reuse from the Service Catalog is the normalized field vocabulary and the generic form renderer: field types, defaults, constraints, sensitivity, local validation, and widget selection.
The change on the Service Catalog side was mainly an extraction: Blueprint-specific field utilities and rendering were moved into shared catalog primitives. Blueprints now adapt their DTOs to those primitives, and Platform Configuration does the same with its own q-core DTOs.
The goal is to share the configuration mechanism while keeping the two domains independent.

Screenshots / Recordings

Adding new "Platform configurations" panel in the edit and creation of a self-managed cluster

The idea of the review is to check the godd separation and the good utilisation of the existing service catalogue feature

Testing

Changes tested locally in the relevant Console's pages and Storybooks
yarn test or yarn test -u (if you need to regenerate snapshots)
yarn format
yarn lint

PR Checklist

I followed naming, styling, and TypeScript rules (see .cursor/rules)
I performed a self-review (diff inspected, dead code removed)
I titled the PR using Conventional Commits with a scope when possible (e.g. feat(service): add new Terraform service) - required for semantic-release
I only kept necessary comments, written in English (watch for useless AI comments)
I involved a designer to validate UI changes if I am not a designer
I covered new business logic with tests (unit)
I confirmed CI is green (Codecov red can be accepted)
I reviewed and executed locally any AI-assisted code

codecov · 2026-07-15T12:22:22Z

Codecov Report

❌ Patch coverage is 66.56891% with 228 lines in your changes missing coverage. Please review.
✅ Project coverage is 49.80%. Comparing base (f39f461) to head (d0a6fbc).

Files with missing lines	Patch %	Lines
...ster-creation-flow/step-platform/step-platform.tsx	61.53%	8 Missing and 22 partials ⚠️
.../platform-configuration/platform-configuration.tsx	57.89%	15 Missing and 9 partials ⚠️
...red/util-js/src/lib/catalog-configuration-field.ts	54.54%	12 Missing and 8 partials ⚠️
...n-guide-modal/cluster-installation-guide-modal.tsx	37.03%	4 Missing and 13 partials ⚠️
...form-configuration/platform-configuration-utils.ts	70.68%	11 Missing and 6 partials ⚠️
...onfiguration-input/catalog-configuration-input.tsx	67.50%	5 Missing and 8 partials ⚠️
...atform-configuration/hooks/use-cluster-operator.ts	0.00%	12 Missing ⚠️
.../catalog-variable-input/catalog-variable-input.tsx	45.45%	5 Missing and 7 partials ⚠️
...luster-creation-flow/step-summary/step-summary.tsx	52.17%	5 Missing and 6 partials ⚠️
...platform-configuration/cluster-operator-status.tsx	57.69%	8 Missing and 3 partials ⚠️
... and 19 more

Additional details and impacted files

@@             Coverage Diff             @@
##           staging    #2817      +/-   ##
===========================================
+ Coverage    49.52%   49.80%   +0.27%     
===========================================
  Files         1286     1355      +69     
  Lines        27799    29335    +1536     
  Branches      8300     8679     +379     
===========================================
+ Hits         13767    14609     +842     
- Misses       11740    12277     +537     
- Partials      2292     2449     +157

Flag	Coverage Δ
unittests	`49.80% <66.56%> (+0.27%)`	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

RemiBonnet

Thanks @pggb25! I added few comments and I think you can probably share with @TheoGrandin74 to check if the UI is ok for him. I added after my comments:

I'm not sure why I've an error when I click on the Qovery stack in the creation flow, is it something expected?

Label aren't supported right now for this kind of cluster, could we hide it?

RemiBonnet · 2026-07-20T12:18:33Z

+}
+
+// These endpoints are not generated by qovery-typescript-axios yet. Keeping the raw
+// calls behind this data-access module avoids leaking their transport details into UI code.


When do you want to add it in the qovery-typescript-axios package?

RemiBonnet · 2026-07-20T12:29:12Z

+  type PlatformComponentConfigurationPreviewRequest,
+  PlatformConfigurationApi,
+} from 'qovery-typescript-axios'
+import { isHttpStatus } from '../http/is-http-status'


I'm not sure to understand why do you need to return null if it's a 404? Same for cluster-operator.ts file

RemiBonnet · 2026-07-20T12:48:18Z

+  enabled?: boolean
+}
+
+export function useClusterOperatorStatus({ organizationId, clusterId, enabled = true }: ClusterOperatorQueryProps) {


You need to have one file per query

RemiBonnet · 2026-07-20T12:50:08Z

+    <div className="rounded-lg border border-neutral bg-surface-neutral p-5">
+      <div className="mb-5 flex items-start justify-between gap-3">
+        <div>
+          <Heading level={2}>{formatCatalogKey(component.key)}</Heading>


You need to have <Section> component before your Heading

RemiBonnet · 2026-07-20T12:50:30Z

+          ) : null}
+
+          {fields.length > 0 ? (
+            <section className="flex flex-col gap-3">


This one should be <Section>

RemiBonnet · 2026-07-20T12:58:21Z

+  const { data: binding } = usePlatformBinding({ organizationId, clusterId, suspense: true })
+  const { mutate: updateBinding, isLoading: isSaving } = useUpdatePlatformBinding()
+
+  const [state, setState] = useState<PlatformConfigurationState | null>(() => {


Could you rename it with a more explicit name? platformConfigurationState and setPlatformConfigurationState, in react state is generic

RemiBonnet · 2026-07-20T13:00:35Z

+  value: CatalogVariableValue | undefined
+}
+
+export function CatalogVariableInput({


This component is not enough generic to be in the UI shared library, could you add it in a domain?

RemiBonnet · 2026-07-20T13:12:34Z

+    )
+  }
+
+  const saveConfiguration = () =>


Inputs that are no longer required remain in customerProvidedInputs, could we filter them against preview.requirements, as we already do in the creation flow?

RemiBonnet · 2026-07-20T13:13:33Z

+        })}
+      </Accordion.Root>
+
+      <div className="flex justify-between border-t border-neutral pt-4">


Can users continue without validating the components of an enabled layer?

RemiBonnet · 2026-07-20T13:14:28Z

+  const { data: binding } = usePlatformBinding({ organizationId, clusterId, suspense: true })
+  const { mutate: updateBinding, isLoading: isSaving } = useUpdatePlatformBinding()
+
+  const [state, setState] = useState<PlatformConfigurationState | null>(() => {


The creation and settings flows duplicate template selection, preview, field updates and payload cleanup, could this shared controller logic live in a domain hook?

RemiBonnet · 2026-07-20T13:18:21Z

+  const isOperatorGuideLoading =
+    isFeatureFlagLoading ||
+    (canUseOperator && (isOperatorStatusLoading || (isOperatorManaged && isOperatorBootstrapLoading)))
+  // Only a bootstrap failure of a confirmed operator-managed cluster blocks the guide;


Should a status request failure fall back to the legacy guide? A transient API error could show incorrect instructions for an operator-managed cluster, I would expect an error state with Retry instead

RemiBonnet · 2026-07-20T13:18:47Z

+  templateId: string
+}
+
+export function PlatformConfiguration({


Could you add small test for this one?

nx-cloud · 2026-08-17T08:49:47Z

View your CI Pipeline Execution ↗ for commit 89cdf60

Command	Status	Duration	Result
`nx affected --target=lint --parallel=3`	✅ Succeeded	2m 29s	View ↗
`nx-cloud record -- yarn nx format:check`	✅ Succeeded	8s	View ↗

💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗

☁️ Nx Cloud last updated this comment at 2026-09-14 13:53:46 UTC

- guard operator queries against the undefined PostHog flag state (modal, create/settings platform routes) so flag-off orgs never hit operator endpoints and refreshes no longer bounce users off the platform step - make engine-v2 cluster creation idempotent on retry and redirect to the platform step when platform data is missing instead of failing silently - fix bool cluster-input requirements round-trip and keep requirement inputs mounted during preview refreshes (no more focus loss per keystroke) - keep cleared field values empty instead of resurrecting schema defaults; omit cleared markers from resolver and save payloads - make components without catalog fields openable so requirement-only components can be configured - exclude Stop / Upgrade K8s actions for self-managed clusters - restore bool-before-allowedValues precedence in CatalogVariableInput (blueprint regression) with a non-regression test - key PlatformConfiguration by cluster and re-seed when the template list changes; surface binding fetch errors instead of redirecting - add Back navigation to step-platform error/empty states - drop dead validationMode/componentsConfigurable props, use ts-pattern for enum mappers, import catalog types directly from util-js Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Use qovery-typescript-axios 1.1.972 for platform APIs and field schemas. Render nested objects and arrays, including per-row fields and validation. Preserve displayed defaults and existing component values when saving. Derive platform requests from generated types and share scalar rendering with Blueprint without changing its API contract.

pggb25 force-pushed the feat/engine-v2-slice4-platform-configuration branch 4 times, most recently from 84b9344 to 4234d9b Compare July 15, 2026 12:02

pggb25 force-pushed the feat/engine-v2-slice4-platform-configuration branch 8 times, most recently from e140985 to a2a0d1a Compare July 17, 2026 13:21

RemiBonnet reviewed Jul 20, 2026

View reviewed changes

pggb25 force-pushed the feat/engine-v2-slice4-platform-configuration branch from a2a0d1a to 2b24b90 Compare August 17, 2026 08:48

pggb25 force-pushed the feat/engine-v2-slice4-platform-configuration branch 2 times, most recently from 5d4a623 to dac7c40 Compare August 18, 2026 12:23

pggb25 force-pushed the feat/engine-v2-slice4-platform-configuration branch 2 times, most recently from d37438e to f63e8f1 Compare September 11, 2026 09:34

pggb25 and others added 5 commits September 14, 2026 10:44

feat(engine-v2): add platform configuration UI

33b0c95

feat(operator): add update action

cf244df

fix(cluster-logs): align step labels dynamically

c3927e5

pggb25 force-pushed the feat/engine-v2-slice4-platform-configuration branch from f63e8f1 to d0a6fbc Compare September 14, 2026 08:46

pggb25 added 2 commits September 14, 2026 15:45

fix(cluster-metrics): group self-managed Karpenter nodes by pool

89cdf60

feat(catalog): make allowed-value selectors searchable

daea175

Uh oh!

Conversation

pggb25 commented Jul 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Screenshots / Recordings

Testing

PR Checklist

Uh oh!

codecov Bot commented Jul 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

RemiBonnet left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

nx-cloud Bot commented Aug 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

pggb25 commented Jul 13, 2026 •

edited

Loading

codecov Bot commented Jul 15, 2026 •

edited

Loading

nx-cloud Bot commented Aug 17, 2026 •

edited

Loading