Skip to content

Commit

Permalink
Fix missing registry field (#39)
Browse files Browse the repository at this point in the history
* Fix missing property
  • Loading branch information
alfespa17 authored Nov 24, 2022
1 parent 3220a9d commit 081151d
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 1 deletion.
1 change: 1 addition & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -247,6 +247,7 @@ Once you have completed the above steps you can complete the file values.yaml to
| registry.version | Yes | |
| registry.replicaCount | Yes | |
| registry.serviceType | Yes | ClusterIP/NodePort/LoadBalancer/ExternalName |
| registry.properties.DexClientId | Yes | Dex Client Id for authentication |
| registry.env | No | |
| registry.volumes | No | |
| registry.volumeMounts | No | |
Expand Down
4 changes: 3 additions & 1 deletion templates/secrets-registry.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,9 @@ stringData:
InternalSecret: '{{ .Values.security.internalSecret | b64enc }}'
TerrakubeEnableSecurity: 'true'
TerrakubeUiURL: 'https://{{ .Values.ingress.ui.domain }}'

AppClientId: '{{ .Values.registry.properties.DexClientId }}'
AppIssuerUri: 'https://{{ .Values.ingress.api.domain }}/dex'

{{- if and (.Values.storage.azure).storageAccountName (.Values.storage.azure).storageAccountAccessKey }}
#Azure Storage
RegistryStorageType: 'AzureStorageImpl'
Expand Down
10 changes: 10 additions & 0 deletions values.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -373,6 +373,16 @@
"type": "object"
}
}
},
"properties": {
"type": "object",
"required": ["DexClientId"],
"properties": {
"DexClientId": {
"description": "Dex client id for the terraform login",
"type": "string"
}
}
}
}
},
Expand Down

0 comments on commit 081151d

Please sign in to comment.