-
Notifications
You must be signed in to change notification settings - Fork 67
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: create helm template #891
base: main
Are you sure you want to change the base?
Conversation
As I am not familiar with Kube, I need some time for this. The overall sentiment is that this is a lot of code that I would need to support forever. Ideally, it would have been nice that either the docker-compose setup could be translated to Helm or the other way to avoid duplication. Notably, if I need to update the docker-compose part, I need to be able to also update the Helm chart. In the meantime, people are welcome to get your work through this PR. |
I still have a lot of work to do here. The current implementation for the shared PV/PVC for the ipinfo data does not work out of the box, instead of using the current approach I will change to use a NFS endpoint. |
Hi, Some thoughts after a first look:
|
In my pocs the strimzi kafka operator and the altinity clickhouse operator worked really well. |
I created a NFS Server that is used to share the files between the inlet and the ipinfo containers. I'm also in the final tests of the reloader. I'm adding a clickchouse and a Kafka in the helm template in the next commits. |
The reloader is working very well for Orchestrator and Console. But I'm having a little problem with the rollout of the inlet. |
Stumbling upon that, with #1059 it removes the need to have ipinfo/geoip data shared between the inlet, as only the orchestrator requires it. |
This is the first commit that creates a helm template for deployment of Akvorado.
Todo:
Add probes
Improve README(add all variables)