Skip to content

Commit ebdf660

Browse files
committed
moving OBC to odf-internal
OBC (ObjectBucketClaim) is a CR managed by ODF. So, unless we have installed ODF, the Manifest will never work. This makes Observability Policy to depend on existing CM and Secret with the S3 info. If you have ODF these are created by the OBC, if not the user needs to provide them. Otherwise we are making ACM dependent on ODF. Now, ACM depends on a CM/Secret.
1 parent a1e722d commit ebdf660

6 files changed

Lines changed: 6 additions & 6 deletions

File tree

telco-hub/configuration/reference-crs-kube-compare/metadata.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@ parts:
5555
- path: optional/odf-internal/odfNS.yaml
5656
- path: optional/odf-internal/odfOperatorGroup.yaml
5757
- path: optional/odf-internal/odfSubscription.yaml
58+
- path: optional/odf-internal/observabilityOBC.yaml
5859
- path: optional/odf-internal/storageCluster.yaml
5960
config:
6061
ignore-unspecified-fields: true
@@ -85,7 +86,6 @@ parts:
8586
config:
8687
ignore-unspecified-fields: true
8788
- path: required/acm/observabilityNS.yaml
88-
- path: required/acm/observabilityOBC.yaml
8989
config:
9090
ignore-unspecified-fields: true
9191
- path: required/acm/observabilityMCSB.yaml

telco-hub/configuration/reference-crs-kube-compare/required/acm/observabilityOBC.yaml renamed to telco-hub/configuration/reference-crs-kube-compare/optional/odf-internal/observabilityOBC.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
apiVersion: objectbucket.io/v1alpha1
33
kind: ObjectBucketClaim
44
metadata:
5-
name: observability-obc
5+
name: observability-ob
66
annotations:
77
argocd.argoproj.io/sync-wave: "-10"
88
argocd.argoproj.io/sync-options: SkipDryRunOnMissingResource=true

telco-hub/configuration/reference-crs/optional/odf-internal/kustomization.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,3 +7,4 @@ resources:
77
- odfSubscription.yaml
88
- storageCluster.yaml
99
- odfReady.yaml
10+
- observabilityOBC.yaml

telco-hub/configuration/reference-crs/required/acm/observabilityOBC.yaml renamed to telco-hub/configuration/reference-crs/optional/odf-internal/observabilityOBC.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
apiVersion: objectbucket.io/v1alpha1
33
kind: ObjectBucketClaim
44
metadata:
5-
name: observability-obc
5+
name: observability-ob
66
annotations:
77
argocd.argoproj.io/sync-wave: "-10"
88
argocd.argoproj.io/sync-options: SkipDryRunOnMissingResource=true

telco-hub/configuration/reference-crs/required/acm/kustomization.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ resources:
1212
- acmPerfSearch.yaml
1313
- acmProvisioning.yaml
1414
- observabilityNS.yaml
15-
- observabilityOBC.yaml
1615
- observabilityRoutePolicy.yaml
1716
- observabilityRoutePlacement.yaml
1817
- observabilityRoutePlacementBinding.yaml

telco-hub/configuration/reference-crs/required/acm/thanosSecretPolicy.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,8 @@ spec:
2727
severity: high
2828
object-templates-raw: |
2929
{{- /* read the bucket data and noobaa endpoint access data */ -}}
30-
{{- $objBucket := (lookup "v1" "ConfigMap" "open-cluster-management-observability" "observability-obc") }}
31-
{{- $awsAccess := (lookup "v1" "Secret" "open-cluster-management-observability" "observability-obc") }}
30+
{{- $objBucket := (lookup "v1" "ConfigMap" "open-cluster-management-observability" "observability-object-bucket") }}
31+
{{- $awsAccess := (lookup "v1" "Secret" "open-cluster-management-observability" "observability-object-bucket") }}
3232
{{- /* create the thanos config file as a template */ -}}
3333
{{- $thanosConfig := `
3434
type: s3

0 commit comments

Comments
 (0)