Mortom123 commented on PR #387:
URL: 
https://github.com/apache/pulsar-helm-chart/pull/387#issuecomment-1892170158

   Thanks for the work @lhotari. I am trying to implement #424  and stumbled 
upon your recent work. 
   I understand separating PSP related stuff from the rest, as it will be 
deprecated soon. However, as far as I understand, when using RBAC one still 
needs to define `Role`s and `RoleBinding`s. Is there any reason why one needs 
to toggle PSP for these things to become active (e.g. 
[here](https://github.com/apache/pulsar-helm-chart/blob/8061a8b7aa777d14634be32b6aa3b92b24520255/charts/pulsar/templates/broker-psp.yaml#L20))?
 
   Shouldn't we split the old setup into 3 parts:
   1. PSP-related stuff (soon to be removed)
   2. Service Account creation
   3. `Role`s and `RoleBinding`s, used by 1. and 2.


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to