Skip to content

fix(helm): harden workload defaults - #2469

Draft
kevkantes wants to merge 2 commits into
mainfrom
fix/helm-security-defaults
Draft

fix(helm): harden workload defaults#2469
kevkantes wants to merge 2 commits into
mainfrom
fix/helm-security-defaults

Conversation

@kevkantes

@kevkantes kevkantes commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator

Description

Hardens the Helm defaults surfaced by admission-policy checks and removes the values-type collision around additional resources.

  • Replace the list-based extraDeploy contract with the stable-key extraDeployMap contract. This is an intentional breaking change with no compatibility shim.
  • Add inherited CPU/memory defaults for MCP servers, OpenFGA hooks, Keycloak reconciliation jobs, and the Milvus components that previously rendered without limits.
  • Disable Kubernetes API token automount by default where workloads do not use the API, and add least-privilege security contexts for Keycloak, Milvus, etcd, Neo4j, and the skill scanner.
  • Refresh generated chart documentation for the current 0.6.0 release.

Type of Change

  • Bugfix
  • New Feature
  • Breaking Change
  • Refactor
  • Documentation
  • Other (please describe)

Pre-release Helm Charts (Optional)

This PR will use the helm-prerelease label so downstream profiles can validate the published chart before merge.

Validation

  • helm lint charts/rag-stack
  • helm lint charts/ai-platform-engineering
  • Full helm template renders for two map-based deployment profiles
  • npm run build in docs/
  • git diff --check

Checklist

  • I have read the contributing guidelines
  • Existing issues have been referenced (where applicable)
  • I have verified this change is not present in other open pull requests
  • Functionality is documented
  • All code style checks pass
  • New code contribution is covered by automated tests
  • All new and existing tests pass

@github-actions

Copy link
Copy Markdown
Contributor

✅ No proprietary content detected. This PR is clear for review!

@github-actions github-actions Bot added the dev Normal PR to main; uses dev prerelease versioning label Aug 24, 2026
@caipe-io-ci-release

caipe-io-ci-release Bot commented Aug 24, 2026

Copy link
Copy Markdown
Prebuild Artifacts for `b18cb07` (archived)

Prebuild Artifacts for b18cb07

Branch: fix/helm-security-defaults
Commit: b18cb07

Helm Charts

Chart Registry Version Status CI
ai-platform-engineering ghcr.io/caipe-io/charts 0.6.0-dev.6-chart.1-fix-helm-security-defaults-2 Published CI
rag-stack ghcr.io/caipe-io/charts 0.6.0-dev.6-chart.1-fix-helm-security-defaults-2 Published CI
Helm install commands
helm upgrade --install ai-platform oci://ghcr.io/caipe-io/charts/ai-platform-engineering --version 0.6.0-dev.6-chart.1-fix-helm-security-defaults-2
helm upgrade --install rag oci://ghcr.io/caipe-io/charts/rag-stack --version 0.6.0-dev.6-chart.1-fix-helm-security-defaults-2

These prebuild artifacts will be automatically cleaned up when the PR is closed or merged.

@kevkantes
kevkantes marked this pull request as draft August 24, 2026 20:47
@github-project-automation github-project-automation Bot moved this to Todo in CAIPE Aug 24, 2026
Signed-off-by: kevkantes <kkantesa@cisco.com>
@kevkantes
kevkantes force-pushed the fix/helm-security-defaults branch from b18cb07 to d7e0527 Compare August 25, 2026 14:15
@kevkantes
kevkantes removed the request for review from sriaradhyula August 25, 2026 14:17
@caipe-io-ci-release

Copy link
Copy Markdown

Prebuild Artifacts for 81ed862

Branch: fix/helm-security-defaults
Commit: 81ed862

Helm Charts

Chart Registry Version Status CI
ai-platform-engineering ghcr.io/caipe-io/charts 1.0.0-dev.2-chart.1-fix-helm-security-defaults-2 Published CI
rag-stack ghcr.io/caipe-io/charts 1.0.0-dev.2-chart.1-fix-helm-security-defaults-2 Published CI
Helm install commands
helm upgrade --install ai-platform oci://ghcr.io/caipe-io/charts/ai-platform-engineering --version 1.0.0-dev.2-chart.1-fix-helm-security-defaults-2
helm upgrade --install rag oci://ghcr.io/caipe-io/charts/rag-stack --version 1.0.0-dev.2-chart.1-fix-helm-security-defaults-2

These prebuild artifacts will be automatically cleaned up when the PR is closed or merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev Normal PR to main; uses dev prerelease versioning helm-chart-bump helm-prerelease

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants