Skip to content

zrok2: rename caSecretName → ca_cert_configmap to match volume type and v1 convention#404

Merged
qrkourier merged 2 commits intozrok2from
copilot/sub-pr-401
Mar 18, 2026
Merged

zrok2: rename caSecretName → ca_cert_configmap to match volume type and v1 convention#404
qrkourier merged 2 commits intozrok2from
copilot/sub-pr-401

Conversation

Copy link

Copilot AI commented Mar 18, 2026

ziti.caSecretName was misnamed — the CA bundle is mounted via a configMap: volume in all three templates, not a Secret. The field name also diverged from the zrok v1 chart's ca_cert_configmap.

Changes

  • values.yaml: rename caSecretNameca_cert_configmap; fix comments referencing "secret" → "configmap"
  • templates/controller-deployment.yaml, frontend-deployment.yaml, pre-delete-hook.yaml: update .Values.ziti.caSecretName.Values.ziti.ca_cert_configmap
  • README.md / README.md.gotmpl: update --set example and values table entry
# before
ziti:
  caSecretName: ziti-controller-ctrl-plane-cas  # comment said "secret"

# after
ziti:
  ca_cert_configmap: ziti-controller-ctrl-plane-cas  # aligns with configMap: volume and zrok v1

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants