Skip to content

[kubernetes-dashboard] Add kubernetes-dashboard helm chart with resource limits#7307

Open
mani-thumu wants to merge 10 commits intomainfrom
kubernetes-dashboard-helm-chart
Open

[kubernetes-dashboard] Add kubernetes-dashboard helm chart with resource limits#7307
mani-thumu wants to merge 10 commits intomainfrom
kubernetes-dashboard-helm-chart

Conversation

@mani-thumu
Copy link
Copy Markdown
Collaborator

@mani-thumu mani-thumu commented Jan 26, 2026

Description
Added kubernetes-dashboard helm chart (v7.3.2) with resource limits based on actual usage monitoring from dev-cluster. Fixes critical issue where Kong gateway has no resource limits (BestEffort QoS).

Benefits

  • Fixes Kong BestEffort QoS by adding resource limits for the first time
  • Reduces total resource requests from 500m CPU / 1000Mi memory to 90m CPU / 437Mi memory
  • Provides appropriate headroom (10-25x) over actual usage (6m CPU, 171Mi memory)
  • Prevents resource over-provisioning while maintaining stability

Possible drawbacks

  • Currently configured based on dev-cluster usage patterns
  • Other clusters may need adjustments based on their specific workloads

Applicable issues

  • Related to infrastructure resource optimization
  • Addresses Kong missing resource limits issue

Additional information
Chart: kubernetes-dashboard v1.0.0 | App: v7.3.2

Resource Changes:

  • Auth/API/Web/Metrics: 100m/200Mi → 10m/64Mi requests, 250m/400Mi → 100m/256Mi limits
  • Kong: None → 50m/181Mi requests, None → 100m/256Mi limits (CRITICAL FIX)

Checklist

  • Chart version bumped in Chart.yaml according to semver (1.0.0 - initial version)
  • Variables are documented in the README.md
  • Title of the PR starts with chart name ([kubernetes-dashboard])

@mani-thumu mani-thumu self-assigned this Jan 26, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 26, 2026

Great PR! Please pay attention to the following items before merging:

Files matching charts/*/values.yaml:

  • Is the PR adding a new container? Please reviewer, add it to the models (internal process)
  • Is the PR adding a new parameter? Please, ensure it’s documented in the README.md

This is an automatically generated QA checklist based on modified files.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants