Skip to content

Commit

Permalink
[stable/polaris] remove default limits for polaris chart (#1453)
Browse files Browse the repository at this point in the history
* remove default limits for polaris chart

* helm-docs --sort-values-order=file

---------

Co-authored-by: Robert Brennan <contact@rbren.io>
  • Loading branch information
mikutas and rbren authored Mar 29, 2024
1 parent ea8730c commit 653847d
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 9 deletions.
2 changes: 1 addition & 1 deletion stable/polaris/Chart.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
apiVersion: v1
description: Validation of best practices in your Kubernetes clusters
name: polaris
version: 5.17.0
version: 5.17.1
appVersion: "8.5"
kubeVersion: ">= 1.22.0-0"
icon: https://raw.githubusercontent.com/FairwindsOps/polaris/master/pkg/dashboard/assets/favicon-32x32.png
Expand Down
4 changes: 2 additions & 2 deletions stable/polaris/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ the 0.10.0 version of this chart will only work on kubernetes 1.14.0+
| dashboard.logLevel | string | `"Info"` | Set the logging level for the Dashboard command |
| dashboard.podAdditionalLabels | object | `{}` | Custom additional labels on dashboard pods. |
| dashboard.deploymentAnnotations | object | `{}` | Custom additional annotations on dashboard Deployment. |
| dashboard.resources | object | `{"limits":{"cpu":"150m","memory":"512Mi"},"requests":{"cpu":"100m","memory":"128Mi"}}` | Requests and limits for the dashboard |
| dashboard.resources | object | `{"limits":{},"requests":{"cpu":"100m","memory":"128Mi"}}` | Requests and limits for the dashboard |
| dashboard.extraContainers | list | `[]` | allows injecting additional containers. |
| dashboard.service.type | string | `"ClusterIP"` | Service Type |
| dashboard.service.annotations | object | `{}` | Service annotations |
Expand Down Expand Up @@ -97,7 +97,7 @@ the 0.10.0 version of this chart will only work on kubernetes 1.14.0+
| webhook.defaultRules | list | `[{"apiGroups":["apps"],"apiVersions":["v1","v1beta1","v1beta2"],"operations":["CREATE","UPDATE"],"resources":["daemonsets","deployments","statefulsets"],"scope":"Namespaced"},{"apiGroups":["batch"],"apiVersions":["v1","v1beta1"],"operations":["CREATE","UPDATE"],"resources":["jobs","cronjobs"],"scope":"Namespaced"},{"apiGroups":[""],"apiVersions":["v1"],"operations":["CREATE","UPDATE"],"resources":["pods","replicationcontrollers"],"scope":"Namespaced"}]` | An array of rules for common types for the ValidatingWebhookConfiguration |
| webhook.podAdditionalLabels | object | `{}` | Custom additional labels on webhook pods. |
| webhook.deploymentAnnotations | object | `{}` | Custom additional annotations on webhook Deployment. |
| webhook.resources | object | `{"limits":{"cpu":"100m","memory":"128Mi"},"requests":{"cpu":"100m","memory":"128Mi"}}` | Requests and limits for the webhook. |
| webhook.resources | object | `{"limits":{},"requests":{"cpu":"100m","memory":"128Mi"}}` | Requests and limits for the webhook. |
| webhook.priorityClassName | string | `nil` | Priority Class name to be used in deployment if provided. |
| webhook.disallowExemptions | bool | `false` | Disallow any exemption |
| webhook.disallowConfigExemptions | bool | `false` | Disallow exemptions that are configured in the config file |
Expand Down
8 changes: 2 additions & 6 deletions stable/polaris/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -54,9 +54,7 @@ dashboard:
requests:
cpu: 100m
memory: 128Mi
limits:
cpu: 150m
memory: 512Mi
limits: {}
# dashboard.extraContainers -- allows injecting additional containers.
extraContainers: []
# extraContainers:
Expand Down Expand Up @@ -234,9 +232,7 @@ webhook:
requests:
cpu: 100m
memory: 128Mi
limits:
cpu: 100m
memory: 128Mi
limits: {}
# webhook.priorityClassName -- Priority Class name to be used in deployment if provided.
priorityClassName:
# webhook.disallowExemptions -- Disallow any exemption
Expand Down

0 comments on commit 653847d

Please sign in to comment.