BLasan commented on pull request #1623: URL: https://github.com/apache/fineract/pull/1623#issuecomment-800798473
> Note that there would be a lot more we could do... in particular the next item on my list is to implement a `helm test` hook, as well as making the `mysql` dependency optional, in case you already have an external instance running. > > This is really only a quickstart... > > @BLasan have you noticed this PR? It looks like something you could be interested in helping to review... smile > > I was thinking about something when reviewing #1653: That (resource limits) should be done here as well? More importantly, we're now maintaining Kubernetes YAML in 2 places (original raw Kube YAML, and Helm chart templates). That's perhaps not ideal... you guys should figure out how you want to do this going forward? Keep both? Replace the original YAML with the Helm templates, and subsequently remove the originals? Up to you - work together. > > Regarding the scripts vs Helm, I would say that Helm is probably the de-facto standard nowadays for deploying to k8s, but not everyone will have it, so it could make sense to keep supporting both. > > However, I would say that in the long run, once it's quite stable, the Helm chart probably should be in a separate repo so it can be versioned independently from the application, and then published into https://artifacthub.io/ +1 @vorburger I think this is ideal as we can maintain the helm chart implmentations separately and do releases accordingly ---------------------------------------------------------------- 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: [email protected]
