Skip to content

Conversation

@ryanmrodriguez
Copy link

@ryanmrodriguez ryanmrodriguez commented May 28, 2025

When either customTLSSecret.name or customReplicationTLSSecret.name are provided, .Values.customRootCATLSSecret.name is evaluated in an if condition, but no default is provided for 'customRootCATLSSecret', leading to this error:

Error: template: ...: executing "...yaml" at <.Values.customRootCATLSSecret.name>: nil pointer evaluating interface {}.name

Current defaults:

# customRootCATLSSecret:
#   name: cluster1-ca-cert
#   items:
#     - key: "tls.crt"
#       path: "root.crt"
#     - key: "tls.key"
#       path: "root.key"
customTLSSecret:
  name: ""
customReplicationTLSSecret:
  name: ""

On a side-note, I think all of these options are not technically mutually exclusive? Therefore the entry conditional seems like it could be adjusted: {{- if or (.Values.customTLSSecret.name) (.Values.customReplicationTLSSecret.name) }} ?

@it-percona-cla
Copy link

it-percona-cla commented May 28, 2025

CLA assistant check
All committers have signed the CLA.

@jvpasinatto
Copy link
Contributor

Hello @ryanmrodriguez, thanks for the contribution and sorry for the delay.
Could you please change the base branch to pg-2.8.0 so we can have this in next PG operator charts release?
Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants