Mortom123 opened a new pull request, #438:
URL: https://github.com/apache/pulsar-helm-chart/pull/438

   This pull request implements #426. It also moves some files around for 
overall better grouping of resources. 
   Additionally, the deployed pulsar-manager is conditioned with given secret 
credentials so the web login works out of the box. 
   
   ### Motivation
   
   The project had some historical growth, such that there exist a lot of 
template files now each prefixed with a component. This PR aims to increase the 
overall readability of this repo by splitting these files into different 
directories. This also makes it easier to get a quick overview which components 
have a `PodMonitor`, a `ServiceAccount`, ... associated with them.
   
   Setting the password for the pulsar-manager in `values.yaml` does only set 
the internal database connection of the manager but not the outward-facing 
website credentials. This is now also implemented. 
   
   ### Modifications
   - moved files around
   - created an additional Batch Job which is along the lines of the 
`pulsar-cluster-initialize.yaml` but conditions the pulsar-manager, for this a 
new template function is introduced to keep it DRY. 
   
   ### 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.

To unsubscribe, e-mail: [email protected]

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

Reply via email to