Repository navigation
fix(apps): remove the Headlamp demo app - #270
Conversation
Headlamp is retired from the platform (devantler-tech/platform#2496) and the KSail web UI is taking over its role (devantler-tech/ksail#5827), so new platforms should not start with it. Remove the app with its Dex callback, network allowances, KEDA grant, security-exception namespaces, replica variable, local host name and docs. The Coroot observability components edit Dex callbacks and exception namespaces by list position, so their guarded indices move with the removed entries, and the OpenCost plugin patch that only targeted Headlamp goes. The observability check now requires a real apps render before it accepts the absence of OpenCost references. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
@coderabbitai review 🤖 Generated with Claude Code |
✅ Action performed
|
|
No actionable comments were generated in the recent review.
 |
| Check name | Status | Explanation |
|---|---|---|
| Title check |  |
The title "fix(apps): remove the Headlamp demo app" matches the main change. The changeset deletes the Headlamp demo app's manifests, HelmRelease, network policy, ExternalSecret, HTTPRoute, scaling ob… |
| Description check | |
The description explains why Headlamp is removed (platform retirement and the KSail web UI replacing it). It lists what is removed, including the sign-in callback, network and autoscaling allowances, … |
| Docstring Coverage | |
Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 1 files. (13 skipped: 1… |
| Linked Issues check | |
Check skipped because no linked issues were found for this pull request. |
| Out of Scope Changes check | |
Check skipped because no linked issues were found for this pull request. |
Full details: Title check
-
Explanation
The title "fix(apps): remove the Headlamp demo app" matches the main change. The changeset deletes the Headlamp demo app's manifests, HelmRelease, network policy, ExternalSecret, HTTPRoute, scaling object, Dex callback, and related docs and settings. The title is specific and concise. It uses a conventional-commit prefix, which is consistent with the repository style.
Full details: Description check
-
Explanation
The description explains why Headlamp is removed (platform retirement and the KSail web UI replacing it). It lists what is removed, including the sign-in callback, network and autoscaling allowances, security-scan exceptions, settings, host name, and docs. It also states that the Coroot profile should keep its behavior and that existing platforms keep their own Headlamp settings. This matches the changeset.
Full details: Docstring Coverage
-
Explanation
Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 1 files. (13 skipped: 13 unsupported.)
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Warning
Some tools did not complete. Review the errors below.
🔧 Checkov (3.3.25)
-
k8s/components/observability-coroot/remove-kps-ui/kustomization.yaml
-
Checkov retry budget exhausted before isolating this batch
k8s/components/observability-coroot/remove-opencost-references/kustomization.yaml
-
Checkov retry budget exhausted before isolating this batch
k8s/providers/hetzner/infrastructure/cluster-issuers/cloudflare-origin-ca-trust-bundle.yaml
-
Checkov retry budget exhausted before isolating this batch
-
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.
Comment @coderabbitai help to get the list of available commands.
Why
Headlamp is being retired from the platform (devantler-tech/platform#2496), and the KSail web UI is taking over as the cluster UI (devantler-tech/ksail#5827). New platforms made from this template should not start with a UI the suite no longer uses.
What
Removes the Headlamp demo app and everything that existed only for it: its sign-in callback, network and autoscaling allowances, security-scan exceptions, settings, local host name and docs. The optional Coroot observability profile still renders and removes exactly what it did before.
Part of devantler-tech/platform#2496
👉 After merge/promotion: platforms already created from this template keep their own Headlamp settings and host names until they remove them.
🤖 Generated with Claude Code