Skip to content

Commit 4249ea7

Browse files
Add validation for authMethod value
1 parent 442713f commit 4249ea7

File tree

3 files changed

+20
-4
lines changed

3 files changed

+20
-4
lines changed

charts/connect/templates/operator-deployment.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ spec:
7272
value: "{{ .Values.operator.pollingInterval }}"
7373
- name: AUTO_RESTART
7474
value: "{{ .Values.operator.autoRestart }}"
75-
{{- if eq .Values.operator.authMethod "serviceaccount" }}
75+
{{- if eq .Values.operator.authMethod "service-account" }}
7676
- name: OP_SERVICE_ACCOUNT_TOKEN
7777
valueFrom:
7878
secretKeyRef:

charts/connect/values.schema.json

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
{
2+
"$schema": "https://json-schema.org/draft-07/schema#",
3+
"type": "object",
4+
"properties": {
5+
"operator": {
6+
"type": "object",
7+
"properties": {
8+
"authMethod": {
9+
"type": "string",
10+
"enum": ["connect", "service-account"],
11+
"description": "Denotes authentication method that 1Password Operator will use to access 1Password secrets."
12+
}
13+
}
14+
}
15+
}
16+
}

charts/connect/values.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -241,10 +241,10 @@ operator:
241241
# Denotes whether the 1Password Operator will be deployed
242242
create: false
243243

244-
# Denotes authentication method that 1Password Operator will use to access secrets
244+
# Denotes authentication method that 1Password Operator will use to access 1Password secrets
245245
# Valid values:
246-
# - connect: sets OP_CONNECT_TOKEN
247-
# - serviceaccount: sets OP_SERVICE_ACCOUNT_TOKEN
246+
# - connect: sets OP_CONNECT_HOST and OP_CONNECT_TOKEN
247+
# - service-account: sets OP_SERVICE_ACCOUNT_TOKEN
248248
authMethod: connect
249249

250250
# The number of replicas to run the 1Password Connect Operator deployment

0 commit comments

Comments
 (0)