Skip to content

Commit d0738e6

Browse files
j-zimnowodaMatthias Erll
and
Matthias Erll
authored
feat: remove deprecated storage option for loki and GCS (#1556)
Co-authored-by: Matthias Erll <[email protected]>
1 parent c8b5c36 commit d0738e6

File tree

3 files changed

+17
-12
lines changed

3 files changed

+17
-12
lines changed

values-changes.yaml

+5
Original file line numberDiff line numberDiff line change
@@ -124,3 +124,8 @@ changes:
124124
- 'apps.argocd.enabled'
125125
- version: 20
126126
networkPoliciesMigration: true
127+
additions:
128+
- 'apps.loki.storage.gcs.serviceAccount'
129+
deletions:
130+
- 'apps.loki.storage.gcs.project'
131+
- 'apps.loki.storage.gcs.instance'

values-schema.yaml

+2-3
Original file line numberDiff line numberDiff line change
@@ -2552,6 +2552,7 @@ properties:
25522552
required:
25532553
- duration
25542554
storage:
2555+
description: 'Chunk storage'
25552556
properties:
25562557
azure:
25572558
properties:
@@ -2582,9 +2583,7 @@ properties:
25822583
properties:
25832584
bucket:
25842585
$ref: '#/definitions/wordCharacterPattern'
2585-
project:
2586-
$ref: '#/definitions/wordCharacterPattern'
2587-
instance:
2586+
serviceAccount:
25882587
$ref: '#/definitions/wordCharacterPattern'
25892588
type:
25902589
type: string

values/loki/loki.gotmpl

+10-9
Original file line numberDiff line numberDiff line change
@@ -106,11 +106,7 @@ loki:
106106
schema_config:
107107
configs:
108108
- from: {{ $l | get "v11StartDate" "2023-01-01" }}
109-
{{- if eq $st.type "gcs" }}
110-
store: bigtable
111-
{{- else }}
112109
store: boltdb-shipper
113-
{{- end }}
114110
{{- if eq $st.type "minioLocal" }}
115111
object_store: s3
116112
{{- end }}
@@ -128,7 +124,6 @@ loki:
128124
prefix: index_
129125
period: 24h
130126
storage_config:
131-
{{- if ne $st.type "gcs" }}
132127
boltdb_shipper:
133128
{{- if eq $st.type "minioLocal" }}
134129
shared_store: s3
@@ -139,16 +134,16 @@ loki:
139134
{{- if eq $st.type "azure" }}
140135
shared_store: azure
141136
{{- end }}
137+
{{- if eq $st.type "gcs" }}
138+
shared_store: gcs
139+
{{- end }}
142140
{{- if eq $st.type "filesystem" }}
143141
shared_store: filesystem
144142
{{- end }}
145143
active_index_directory: /var/loki/index
146144
cache_location: /var/loki/cache
147145
cache_ttl: 168h
148-
{{- else }}
149-
bigtable:
150-
instance: {{ $st.gcs.instance }}
151-
project: {{ $st.gcs.project }}
146+
{{- if eq $st.type "gcs" }}
152147
gcs:
153148
bucket_name: {{ $st.gcs.bucket }}
154149
{{- end }}
@@ -182,6 +177,12 @@ loki:
182177
container_name: {{ $st.azure.containerName }}
183178
{{- end }}
184179

180+
{{- if and (eq $st.type "gcs") $st.gcs.serviceAccount }}
181+
serviceAccount:
182+
annotations:
183+
iam.gke.io/gcp-service-account: {{ $st.gcs.serviceAccount }}
184+
{{- end }}
185+
185186
ingester:
186187
resources: {{- $l.resources.ingester | toYaml | nindent 4 }}
187188
podAnnotations:

0 commit comments

Comments
 (0)