Skip to content

Commit

Permalink
improve naming scheme for extrernal credential config
Browse files Browse the repository at this point in the history
Co-authored-by: saisatish karra <[email protected]>
Signed-off-by: Wayne Starr <[email protected]>
  • Loading branch information
Racer159 and saisatishkarra committed Aug 7, 2024
1 parent 8b50eed commit 1490b45
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 3 deletions.
2 changes: 1 addition & 1 deletion charts/fulcio/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ helm uninstall [RELEASE_NAME]
| server.affinity | object | `{}` | |
| server.args.aws_hsm_root_ca_path | string | `nil` | |
| server.args.certificateAuthority | string | `"fileca"` | |
| server.args.creds | string | `""` | |
| server.args.cloud_credential_config | string | `""` | |
| server.args.ct_log_url | string | `""` | |
| server.args.disable_ct_log | bool | `false` | |
| server.args.gcp_private_ca_parent | string | `"projects/test/locations/us-east1/caPools/test"` | |
Expand Down
4 changes: 3 additions & 1 deletion charts/fulcio/templates/fulcio-configmap.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,4 +20,6 @@ data:
{{- if (eq .Values.server.args.certificateAuthority "kmsca")}}
chain.pem: {{.Values.server.args.kms_cert_chain | quote }}
{{- end }}
cloud_credentials: {{.Values.server.args.creds | quote }}
{{- if .Values.server.args.cloud_credential_config }}
cloud_credential_config: {{.Values.server.args.cloud_credential_config | quote }}
{{- end }}
2 changes: 1 addition & 1 deletion charts/fulcio/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ server:
port: 5555
grpcPort: 5554
# valid values: GCP workload identity config json for trusted external cloud providers
creds: ""
cloud_credential_config: ""
# Valid values: googleca, pkcs11ca, aws-hsm-root-ca-path, fileca, kmsca
certificateAuthority: fileca
# kms_resource: gcpkms://....
Expand Down

0 comments on commit 1490b45

Please sign in to comment.