openova/.github
e3mrah 953ef8290f
fix(catalyst-build): stop auto-bumping contabo Kustomize-path image refs (#980)
* fix(catalyst-ui): drop stale params={{ deploymentId }} from clean-root Links (#975)

#976 collapsed `to="/provision/$deploymentId/<page>"` to clean root
paths (`to="/<page>"`) but left the `params={{ deploymentId }}` prop
on every callsite, breaking the Vite tsc build with TS2353. Fixes:

- Drop `params={{ deploymentId }}` from Links whose target is now a
  parameterless clean root path (StatusStrip, AppDetail, AppsPage,
  DecommissionPage, FlowPage, JobDetail, JobsPage, JobsTimeline,
  SettingsPage, DeploymentsList).
- For Links whose `to` still uses `$componentId`/`$jobId`, cast
  `params` with `as never` to match the existing pattern in
  cloud-compute/cloud-network/cloud-storage/Sidebar/UserAccess
  (the dual-mount under provisionRoute + consoleLayoutRoute defeats
  TS's strict params inference; the runtime path is correct).
- Drop `deploymentId` prop + interface field from JobCard / JobRow /
  JobsTable / AppCard now that the Links don't need it; update test
  fixtures + the JobsTable row-link assertion to match the new
  clean `/jobs/$jobId` href.
- Drop the unused ArchEdgeType import in k8sAdapter (TS6196).
- Dashboard navigateToApp uses `as never` casts to align with the
  same pattern.

* fix(catalyst-build): stop auto-bumping contabo Kustomize-path image refs

Two paths consume the catalyst-api / catalyst-ui images:
1. bp-catalyst-platform OCI chart (Sovereigns) — values.yaml driven, tag
   in values.yaml is rendered at helm install time by Sovereign Flux.
2. contabo Kustomize-path — literal image refs in templates/api-deployment.yaml
   and templates/ui-deployment.yaml. Flux kustomize-controller on contabo
   reconciles those files directly.

The CI deploy step was bumping BOTH on every PR, which auto-rolled
contabo every time anyone merged a catalyst-api code change. On
2026-05-05 PR #975's k8scache feature broke contabo startup on the
auto-roll because contabo has 27 dead-Sovereign kubeconfigs that the
new code iterates synchronously at startup, blocking readiness.

Fix: keep the values.yaml bump (Sovereigns auto-pick-up via OCI chart
which is the right behaviour for fresh provisions). Drop the
templates/*-deployment.yaml bump so contabo only rolls when an
operator manually commits a validated SHA into those files.

Closes the auto-deploy-to-contabo blast radius on every PR.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: hatiyildiz <hatiyildiz@users.noreply.github.com>
Co-authored-by: hatiyildiz <hatice.yildiz@openova.io>
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
2026-05-05 21:24:57 +04:00
..
workflows fix(catalyst-build): stop auto-bumping contabo Kustomize-path image refs (#980) 2026-05-05 21:24:57 +04:00
dependabot.yml chore(ci): add Dependabot for npm and GitHub Actions dependency updates 2026-03-19 13:42:02 +01:00