Skip to content

Commit

Permalink
chore: removing Prometheus from Karpenter scope
Browse files Browse the repository at this point in the history
  • Loading branch information
jfavellar90 committed Sep 19, 2023
1 parent feec55a commit e245c7d
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 10 deletions.
5 changes: 0 additions & 5 deletions charts/harmony-chart/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -48,11 +48,6 @@ dependencies:
condition: opensearch.enabled
repository: https://opensearch-project.github.io/helm-charts

- name: prometheus
version: "19.7.2"
repository: https://prometheus-community.github.io/helm-charts
condition: prometheus.enabled

- name: karpenter
version: "0.16.3"
repository: https://charts.karpenter.sh
Expand Down
4 changes: 0 additions & 4 deletions charts/harmony-chart/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -184,10 +184,6 @@ opensearch:
".opendistro-asynchronous-search-response*",
]
prometheus:
# add prometheus to the chart by default. Cluster operators who have already
# installed prometheus should change this to 'false'.
enabled: false
karpenter:
# add Karpenter node management for AWS EKS clusters. See: https://karpenter.sh/
enabled: false
Expand Down
2 changes: 1 addition & 1 deletion infra-examples/aws/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ AWS EKS has grown more complex over time. This reference implementation is preco
- Key Management Service (KMS) is enabled, encrypting all Kubernetes Secrets
- cluster access via aws-auth/configMap is enabled
- a karpenter.sh/discovery resource tag is added to the EKS instance
- various AWS EKS add-ons that are required by Open edX and/or Karpenter and/or its supporting systems (metrics-server, vpa, prometheus) are included
- various AWS EKS add-ons that are required by Open edX and/or Karpenter and/or its supporting systems (metrics-server, vpa) are included
- additional cluster node security configuration is added to allow node-to-node and pod-to-pod communication using internal DNS resolution
- a managed node group is added containing custom labels, IAM roles, and resource tags; all of which are required by Karpenter
- adds additional resources required by AWS EBS CSI Driver add-on, itself required by EKS since 1.22
Expand Down

0 comments on commit e245c7d

Please sign in to comment.