Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Bump kube-prometheus-stack to 48.3.2 #1555

Closed
wants to merge 1 commit into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 15 additions & 4 deletions licenses.d2iq.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -112,6 +112,12 @@ resources:
notice_path: NOTICE.md
ref: v${image_tag}
url: https://github.com/grafana/grafana
- container_image: docker.io/grafana/grafana:10.0.3
sources:
- license_path: LICENSE
notice_path: NOTICE.md
ref: v${image_tag}
url: https://github.com/grafana/grafana
- container_image: docker.io/grafana/loki:2.8.2
sources:
- license_path: LICENSE
Expand Down Expand Up @@ -533,6 +539,11 @@ resources:
- license_path: LICENSE
ref: ${image_tag}
url: https://github.com/kiwigrid/k8s-sidecar
- container_image: quay.io/kiwigrid/k8s-sidecar:1.24.6
sources:
- license_path: LICENSE
ref: ${image_tag}
url: https://github.com/kiwigrid/k8s-sidecar
- container_image: quay.io/kubernetes-multicluster/kubefed:v0.9.1
sources:
- license_path: LICENSE
Expand All @@ -543,13 +554,13 @@ resources:
- license_path: LICENSE
ref: nfs-provisioner-${image_tag}
url: https://github.com/kubernetes-retired/external-storage
- container_image: quay.io/prometheus-operator/prometheus-config-reloader:v0.65.2
- container_image: quay.io/prometheus-operator/prometheus-config-reloader:v0.66.0
sources:
- license_path: LICENSE
notice_path: NOTICE
ref: ${image_tag}
url: https://github.com/prometheus-operator/prometheus-operator
- container_image: quay.io/prometheus-operator/prometheus-operator:v0.65.2
- container_image: quay.io/prometheus-operator/prometheus-operator:v0.66.0
sources:
- license_path: LICENSE
notice_path: NOTICE
Expand All @@ -567,7 +578,7 @@ resources:
notice_path: NOTICE
ref: ${image_tag}
url: https://github.com/prometheus/alertmanager
- container_image: quay.io/prometheus/node-exporter:v1.5.0
- container_image: quay.io/prometheus/node-exporter:v1.6.0
sources:
- license_path: LICENSE
notice_path: NOTICE
Expand All @@ -579,7 +590,7 @@ resources:
notice_path: NOTICE
ref: ${image_tag}
url: https://github.com/prometheus/prometheus
- container_image: quay.io/prometheus/prometheus:v2.44.0
- container_image: quay.io/prometheus/prometheus:v2.45.0
sources:
- license_path: LICENSE
notice_path: NOTICE
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
apiVersion: v1
kind: ConfigMap
metadata:
name: kube-prometheus-stack-46.8.0-d2iq-defaults
name: kube-prometheus-stack-48.3.2-d2iq-defaults
namespace: ${releaseNamespace}
data:
values.yaml: |
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ spec:
wait: true
interval: 6h
retryInterval: 1m
path: ./services/kube-prometheus-stack/46.8.0/etcd-metrics-proxy
path: ./services/kube-prometheus-stack/48.3.2/etcd-metrics-proxy
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's use 46.8.1 now that it has been published

sourceRef:
kind: GitRepository
name: management
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ spec:
wait: true
interval: 6h
retryInterval: 1m
path: ./services/kube-prometheus-stack/46.8.0/helmrelease
path: ./services/kube-prometheus-stack/48.3.2/helmrelease
sourceRef:
kind: GitRepository
name: management
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ spec:
kind: HelmRepository
name: mesosphere.github.io-charts-staging
namespace: kommander-flux
version: 46.8.0
version: 48.3.2
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am just worried about jumping 2 major versions in a dkp patch version. in the past we have created "release branches" for the helm charts in m/charts but I would have to remember how that process works, it was a little manual and a little risky when publishing the version

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried this here mesosphere/charts#1449 backporting the fix to the KPS version we released in DKP 2.6.0 -- some more details in the PR description

interval: 15s
install:
timeout: 5m0s
Expand All @@ -27,7 +27,7 @@ spec:
releaseName: kube-prometheus-stack
valuesFrom:
- kind: ConfigMap
name: kube-prometheus-stack-46.8.0-d2iq-defaults
name: kube-prometheus-stack-48.3.2-d2iq-defaults
- kind: ConfigMap
name: kube-prometheus-stack-mgmt-overrides
optional: true
Expand All @@ -46,7 +46,7 @@ data:
docsLink: "https://prometheus.io/docs/"
# Prometheus app version can be found at prometheus.prometheusSpec.image.tag:
# https://github.com/mesosphere/charts/blob/master/staging/kube-prometheus-stack/values.yaml#L2147
version: "2.44.0"
version: "2.45.0"
---
apiVersion: v1
kind: ConfigMap
Expand Down Expand Up @@ -77,4 +77,4 @@ data:
# Since Grafana is a subchart of kube-prometheus-stack, get the version of the Grafana chart dependency at:
# https://github.com/mesosphere/charts/tree/master/staging/kube-prometheus-stack/charts
# Then, check https://artifacthub.io/packages/helm/grafana/grafana/ for app version
version: "9.5.3"
version: "10.0.3"
Loading