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

fix(base-cluster/monitoring): only roll out alertmanager oauth-proxy when alertmanager is enabled 🤣 #1180

Merged
merged 1 commit into from
Oct 31, 2024

Conversation

cwrau
Copy link
Member

@cwrau cwrau commented Sep 19, 2024

No description provided.

@teutonet-bot
Copy link
Contributor

teutonet-bot commented Sep 19, 2024

🤖 I have diffed this beep boop

"/$namespace/$kind/$name.yaml" for normal resources
"/$namespace/HelmRelease/$name/$namespace/$kind/$name.yaml" for HelmReleases <- this is recursive
'null' means it's either cluster-scoped or it's in the default namespace for the HelmRelease

charts/base-cluster/ci/monitoring-ingress-unauthenticated-values.yaml

charts/base-cluster/ci/rbac-values.yaml has no changes

charts/base-cluster/ci/pagerduty-values.yaml

charts/base-cluster/ci/velero-backupStorageLocations-gen-values.yaml

charts/base-cluster/ci/flux-gitrepositories-gen-values.yaml

charts/base-cluster/ci/imagepullsecrets-values.yaml has no changes

charts/base-cluster/values.yaml has no changes

charts/base-cluster/ci/artifacthub-values.yaml has no changes

charts/base-cluster/ci/disabled-ingress-values.yaml

charts/base-cluster/ci/priorityclasses-values.yaml

charts/base-cluster/ci/deadmansswitch-values.yaml

charts/base-cluster/ci/monitoring-oidc-values.yaml

charts/base-cluster/ci/basic-values.yaml has no changes

charts/base-cluster/ci/monitoring-oidc-ingress-disabled-values.yaml

charts/base-cluster/ci/limitrange-resourcequota-values.yaml

@cwrau cwrau force-pushed the fix/base-cluster/alertmanager-oauth-proxy-rollout branch 2 times, most recently from 5f5985a to 6547d5d Compare September 24, 2024 07:42
@cwrau cwrau force-pushed the fix/base-cluster/alertmanager-oauth-proxy-rollout branch from 6547d5d to 166fe22 Compare October 31, 2024 08:33
@cwrau cwrau merged commit a1d8888 into main Oct 31, 2024
16 checks passed
@cwrau cwrau deleted the fix/base-cluster/alertmanager-oauth-proxy-rollout branch October 31, 2024 08:53
teutonet-bot added a commit that referenced this pull request Nov 11, 2024
🤖 I have created a release *beep* *boop*
---


##
[7.0.0](base-cluster-v6.7.0...base-cluster-v7.0.0)
(2024-11-11)


### ⚠ BREAKING CHANGES

* **base-cluster/rbac:** allow to use the k8s default ClusterRoles
([#1230](#1230))
* **base-cluster/dependencies:** update helm release descheduler to
0.31.x
([#1173](#1173))

### Features

* **base-cluster/rbac:** allow to use the k8s default ClusterRoles
([#1230](#1230))
([a3b24e4](a3b24e4))


### Bug Fixes

* **base-cluster/monitoring:** only roll out alertmanager oauth-proxy
when alertmanager is enabled 🤣
([#1180](#1180))
([a1d8888](a1d8888))
* **base-cluster:** curl images imagePullPolicy
([#1168](#1168))
([bb1942d](bb1942d))


### Miscellaneous Chores

* **base-cluster/dependencies:** update docker.io/bitnami/kubectl docker
tag to v1.31.2
([#1191](#1191))
([e3ffc13](e3ffc13))
* **base-cluster/dependencies:** update docker.io/curlimages/curl docker
tag to v8.10.1
([#1193](#1193))
([54966a8](54966a8))
* **base-cluster/dependencies:** update docker.io/fluxcd/flux-cli docker
tag to v2.4.0
([#1212](#1212))
([6519edf](6519edf))
* **base-cluster/dependencies:** update helm release descheduler to
0.31.x
([#1173](#1173))
([784958c](784958c))
* **base-cluster/dependencies:** update helm release
kube-prometheus-stack to v65
([#1214](#1214))
([5f701ed](5f701ed))
* **base-cluster:** formatting
([#1198](#1198))
([b9bd4a4](b9bd4a4))
* **base-cluster:** this is now supported 🥳
([#1135](#1135))
([515ce2d](515ce2d))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants