-
Notifications
You must be signed in to change notification settings - Fork 9
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/backup)!: only enable velero if a backupstoragelocation is set #763
Conversation
This looks fine to me. But i'm asking myself how the rollout will go. Most clusters currently aren't using velero, but some do. In those clusters the backupStorageLocation ist not set via the chart. At least not that i'm aware of. But im not sure how practical that would be. What do you think @cwrau @tasches ? |
bfbbd19
to
02e797d
Compare
You're missing some locations where
You can just search for |
e3806e1
to
04724b4
Compare
You should require diff --git a/charts/base-cluster/values.schema.json b/charts/base-cluster/values.schema.json
index d8f2f69..948cc3b 100644
--- a/charts/base-cluster/values.schema.json
+++ b/charts/base-cluster/values.schema.json
@@ -1290,6 +1290,25 @@
"additionalProperties": false
}
},
+ "oneOf": [
+ {
+ "properties": {
+ "backupStorageLocations": {
+ "minProperties": 1
+ }
+ },
+ "required": [
+ "defaultLocation"
+ ]
+ },
+ {
+ "properties": {
+ "backupStorageLocations": {
+ "maxProperties": 0
+ }
+ }
+ }
+ ],
"additionalProperties": false
},
"common": { In this case I'd also be fine with the diff --git a/charts/base-cluster/values.schema.json b/charts/base-cluster/values.schema.json
index d8f2f69..4ac7ee7 100644
--- a/charts/base-cluster/values.schema.json
+++ b/charts/base-cluster/values.schema.json
@@ -1290,6 +1290,18 @@
"additionalProperties": false
}
},
+ "if": {
+ "properties": {
+ "backupStorageLocations": {
+ "minProperties": 1
+ }
+ }
+ },
+ "then": {
+ "required": [
+ "defaultLocation"
+ ]
+ },
"additionalProperties": false
},
"common": { |
b73e9fd
to
a1253c5
Compare
865a8a3
to
0f4b3fa
Compare
🤖 I have diffed this beep boop"/$namespace/$kind/$name.yaml" for normal resources
|
🤖 I have created a release *beep* *boop* --- ## [5.0.0](base-cluster-4.12.0...base-cluster-v5.0.0) (2024-02-10) ### ⚠ BREAKING CHANGES * **base-cluster/backup:** only enable velero if a backupstoragelocation is set ([#763](#763)) ### Features * **base-cluster/grafana:** add dashboards for namespace monitoring ([#712](#712)) ([1651273](1651273)) * **base-cluster/limitrange-quotas:** add limitrange and resource quota ([#673](#673)) ([f0ac4c2](f0ac4c2)) ### Bug Fixes * **base-cluster/backup:** only enable velero if a backupstoragelocation is set ([#763](#763)) ([4dfec43](4dfec43)), closes [#752](#752) * **base-cluster/cert-manager:** set higher resource quota for cert-ma… ([#742](#742)) ([53a51bb](53a51bb)) ### Miscellaneous Chores * **base-cluster/artifacthub-images:** Update ArtifactHUB images ([#720](#720)) ([e77941e](e77941e)) * **base-cluster/artifacthub-images:** Update ArtifactHUB images ([#735](#735)) ([45842f3](45842f3)) * **base-cluster/artifacthub-images:** Update ArtifactHUB images ([#738](#738)) ([c83ff3e](c83ff3e)) * **base-cluster/artifacthub-images:** Update ArtifactHUB images ([#744](#744)) ([b8d4d2f](b8d4d2f)) * **base-cluster/artifacthub-images:** Update ArtifactHUB images ([#746](#746)) ([3da1a81](3da1a81)) * **base-cluster/artifacthub-images:** Update ArtifactHUB images ([#747](#747)) ([72978e3](72978e3)) * **base-cluster/artifacthub-images:** Update ArtifactHUB images ([#748](#748)) ([92b09a1](92b09a1)) * **base-cluster/artifacthub-images:** Update ArtifactHUB images ([#749](#749)) ([b53bd86](b53bd86)) * **base-cluster/artifacthub-images:** Update ArtifactHUB images ([#750](#750)) ([feb6630](feb6630)) * **base-cluster/artifacthub-images:** Update ArtifactHUB images ([#760](#760)) ([4b0dac4](4b0dac4)) * **base-cluster/artifacthub-images:** Update ArtifactHUB images ([#761](#761)) ([6515b38](6515b38)) * **base-cluster/artifacthub-images:** Update ArtifactHUB images ([#762](#762)) ([dd58442](dd58442)) * **base-cluster/artifacthub-images:** Update ArtifactHUB images ([#765](#765)) ([cf7f587](cf7f587)) * **base-cluster/artifacthub-images:** Update ArtifactHUB images ([#775](#775)) ([5837439](5837439)) * **base-cluster/artifacthub-images:** Update ArtifactHUB images ([#776](#776)) ([5c04406](5c04406)) * **base-cluster/dependencies:** update docker.io/bitnami/kubectl docker tag to v1.29.1 ([#726](#726)) ([11fa033](11fa033)) * **base-cluster/dependencies:** update helm release common to v2.14.1 ([#692](#692)) ([c570d70](c570d70)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
closes #752