Skip to content

Commit

Permalink
Merge pull request #2067 from 2i2c-org/bump-helm-deps/support/vtjc
Browse files Browse the repository at this point in the history
Bumping helm chart dependency versions: support
  • Loading branch information
consideRatio authored Jan 23, 2023
2 parents 7ab39e7 + a43e7c9 commit a6a0654
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 5 deletions.
13 changes: 13 additions & 0 deletions deployer/cluster.py
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,19 @@ def deploy_support(self, cert_manager_version):
)
print_colour("Done!")

# FIXME: Revert this addition after one upgrade is made
print_colour("Preparing for prometheus v16 v19 migration...")
subprocess.check_call(
[
"kubectl",
"delete",
"daemonset",
"--namespace=support",
"support-prometheus-node-exporter",
]
)
print_colour("Done!")

print_colour("Provisioning support charts...")

support_dir = (Path(__file__).parent.parent).joinpath("helm-charts", "support")
Expand Down
6 changes: 3 additions & 3 deletions helm-charts/support/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,20 +7,20 @@ dependencies:
# Prometheus for collection of metrics.
# https://github.com/prometheus-community/helm-charts/tree/main/charts/prometheus
- name: prometheus
version: 15.16.1
version: 19.3.1
repository: https://prometheus-community.github.io/helm-charts

# Grafana for dashboarding of metrics.
# https://github.com/grafana/helm-charts/tree/main/charts/grafana
- name: grafana
version: 6.42.2
version: 6.50.1
repository: https://grafana.github.io/helm-charts

# ingress-nginx for a k8s Ingress resource controller that routes traffic from
# a single IP entrypoint to various services exposed via k8s Ingress resources
# that references this controller.
- name: ingress-nginx
version: 4.1.4
version: 4.4.2
repository: https://kubernetes.github.io/ingress-nginx

# cluster-autoscaler for k8s clusters where it doesn't come out of the box (EKS)
Expand Down
4 changes: 2 additions & 2 deletions helm-charts/support/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ prometheus:
# Grafana also has alerts, and the UI of grafana is much better than that of alertmanager
# We also expose Grafana publicly behind auth anyway, so we can consolidate alerting with Grafana too
enabled: false
nodeExporter:
prometheus-node-exporter:
tolerations:
# Tolerate tainted jupyterhub user nodes
- key: hub.jupyter.org_dedicated
Expand All @@ -42,7 +42,7 @@ prometheus:
effect: NoSchedule
updateStrategy:
type: RollingUpdate
pushgateway:
prometheus-pushgateway:
enabled: false
server:
ingress:
Expand Down

0 comments on commit a6a0654

Please sign in to comment.