You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/01/26 19:58:25 UTC

[GitHub] [pulsar-helm-chart] miecio45 opened a new pull request #99: Fix namespace handling and missing dnsNames

miecio45 opened a new pull request #99:
URL: https://github.com/apache/pulsar-helm-chart/pull/99


   Fixes for wrong namespace handling in some RBAC and missing dnsNames for TLS
   
   ### Motivation
   
   Fixes old unused handling of namespace name in RBAC for autorecovery and bookkeeper.
   Fixes Helm exception of missing key when not defining TLS dnsNames
   
   ### Modifications
   
   Use namespace template in RBAC definitions for bookkeeper and autorecovery. Add if around every `toYaml .Values.tls.bookie.dnsNames` clause in TLS certs definitions.
   
   ### Verifying this change
   
   - [ ] Make sure that the change passes the CI checks.
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar-helm-chart] sijie merged pull request #99: Fix namespace handling and missing dnsNames

Posted by GitBox <gi...@apache.org>.

sijie merged pull request #99:
URL: https://github.com/apache/pulsar-helm-chart/pull/99


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org