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.

Sign up for GitHub

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

Jump to bottom

Downgrade Kube Prometheus Stack to 12.3.0 and 0.43.2 CRD version for be… #1329

Merged
merged 1 commit into from
Jan 13, 2021
Merged

Downgrade Kube Prometheus Stack to 12.3.0 and 0.43.2 CRD version for be… #1329

merged 1 commit into from
Jan 13, 2021

Conversation

perk-sumo
Copy link
Contributor

@perk-sumo perk-sumo commented Jan 13, 2021 •
edited
Loading

…tter OpenShift support

Description

12.3.0 is the last one to use Prometheus Operator 0.43 with is compatible with OpenShift 4.6 at the moment.
sjentzsch/helm-charts@e46dc63

Testing performed
  • Redeploy fluentd and fluentd-events pods
  • Confirm events, logs, and metrics are coming in

@perk-sumo perk-sumo added this to the v2.0 milestone Jan 13, 2021
@perk-sumo perk-sumo requested a review from a team January 13, 2021 15:03
pmalek-sumo
pmalek-sumo suggested changes Jan 13, 2021
View reviewed changes
Copy link
Contributor

@pmalek-sumo pmalek-sumo left a comment

Choose a reason for hiding this comment

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

We need to update docs with CRDs links

@perk-sumo
Copy link
Contributor Author

Good catch!

@pmalek-sumo
Copy link
Contributor

We need to update this

sumologic-kubernetes-collection/deploy/docs/v2_migration_doc.md

Lines 172 to 179 in cefb933

kubectl apply -f https://raw.githubusercontent.com/prometheus-operator/prometheus-operator/v0.44.0/example/prometheus-operator-crd/monitoring.coreos.com_probes.yaml
kubectl apply -f https://raw.githubusercontent.com/prometheus-operator/prometheus-operator/v0.44.0/example/prometheus-operator-crd/monitoring.coreos.com_alertmanagers.yaml
kubectl apply -f https://raw.githubusercontent.com/prometheus-operator/prometheus-operator/v0.44.0/example/prometheus-operator-crd/monitoring.coreos.com_prometheuses.yaml
kubectl apply -f https://raw.githubusercontent.com/prometheus-operator/prometheus-operator/v0.44.0/example/prometheus-operator-crd/monitoring.coreos.com_prometheusrules.yaml
kubectl apply -f https://raw.githubusercontent.com/prometheus-operator/prometheus-operator/v0.44.0/example/prometheus-operator-crd/monitoring.coreos.com_servicemonitors.yaml
kubectl apply -f https://raw.githubusercontent.com/prometheus-operator/prometheus-operator/v0.44.0/example/prometheus-operator-crd/monitoring.coreos.com_podmonitors.yaml
kubectl apply -f https://raw.githubusercontent.com/prometheus-operator/prometheus-operator/v0.44.0/example/prometheus-operator-crd/monitoring.coreos.com_thanosrulers.yaml
kubectl apply -f https://raw.githubusercontent.com/prometheus-operator/prometheus-operator/v0.44.0/example/prometheus-operator-crd/monitoring.coreos.com_alertmanagerconfigs.yaml

and this

sumologic-kubernetes-collection/deploy/docs/Non_Helm_Installation.md

Lines 107 to 114 in be1c431

kubectl apply -f https://raw.githubusercontent.com/prometheus-operator/prometheus-operator/v0.44.0/example/prometheus-operator-crd/monitoring.coreos.com_probes.yaml
kubectl apply -f https://raw.githubusercontent.com/prometheus-operator/prometheus-operator/v0.44.0/example/prometheus-operator-crd/monitoring.coreos.com_alertmanagers.yaml
kubectl apply -f https://raw.githubusercontent.com/prometheus-operator/prometheus-operator/v0.44.0/example/prometheus-operator-crd/monitoring.coreos.com_prometheuses.yaml
kubectl apply -f https://raw.githubusercontent.com/prometheus-operator/prometheus-operator/v0.44.0/example/prometheus-operator-crd/monitoring.coreos.com_prometheusrules.yaml
kubectl apply -f https://raw.githubusercontent.com/prometheus-operator/prometheus-operator/v0.44.0/example/prometheus-operator-crd/monitoring.coreos.com_servicemonitors.yaml
kubectl apply -f https://raw.githubusercontent.com/prometheus-operator/prometheus-operator/v0.44.0/example/prometheus-operator-crd/monitoring.coreos.com_podmonitors.yaml
kubectl apply -f https://raw.githubusercontent.com/prometheus-operator/prometheus-operator/v0.44.0/example/prometheus-operator-crd/monitoring.coreos.com_thanosrulers.yaml
kubectl apply -f https://raw.githubusercontent.com/prometheus-operator/prometheus-operator/v0.44.0/example/prometheus-operator-crd/monitoring.coreos.com_alertmanagerconfigs.yaml

@perk-sumo perk-sumo force-pushed the perk-downgrade-kube-prometheus-stack-to-12.3.0 branch from 4b07158 to dbff97c Compare January 13, 2021 15:09
@perk-sumo perk-sumo requested a review from pmalek-sumo January 13, 2021 15:09
@perk-sumo perk-sumo force-pushed the perk-downgrade-kube-prometheus-stack-to-12.3.0 branch 2 times, most recently from 6f499a1 to 9a6c27b Compare January 13, 2021 15:12
@perk-sumo perk-sumo force-pushed the perk-downgrade-kube-prometheus-stack-to-12.3.0 branch from 9a6c27b to ebacc18 Compare January 13, 2021 15:14
@perk-sumo perk-sumo requested a review from pmalek-sumo January 13, 2021 15:16
@perk-sumo perk-sumo changed the title Downgrade Kube Prometheus Stack to 12.3.0 and 0.43 CRD version for be… Downgrade Kube Prometheus Stack to 12.3.0 and 0.43.2 CRD version for be… Jan 13, 2021
@perk-sumo perk-sumo merged commit 9548460 into main Jan 13, 2021
@perk-sumo perk-sumo deleted the perk-downgrade-kube-prometheus-stack-to-12.3.0 branch January 13, 2021 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sumo-drosiek sumo-drosiek sumo-drosiek approved these changes

@pmalek-sumo pmalek-sumo pmalek-sumo approved these changes

@andrzej-stencel andrzej-stencel Awaiting requested review from andrzej-stencel

@frankreno frankreno Awaiting requested review from frankreno

@kasia-kujawa kasia-kujawa Awaiting requested review from kasia-kujawa

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v2.0
Development

Successfully merging this pull request may close these issues.
3 participants
@perk-sumo @pmalek-sumo @sumo-drosiek