some comments

[Frankkkkk]

Hi @matmair , thanks for your message!

The helm chart looks good!

some comments:

• PVC
  • hardcoded size.
  • Why ReadWriteOnce and not ReadWriteMany? The application doesn't support it?
  • Would be nice to be able to specify the name
• HPA: I guess it won't work if PVC is RWO

Cheers and congrats! I'll archive my repo

[munin92]

@Frankkkkk i think the pvc comments are obsolte now :) #31

[Frankkkkk]

Indeed! That MR looks great!

[matmair]

To be honest I am not sure if HPA would not lead to wired issues, we still have some components that depend heavy on storage for state

[Frankkkkk]

That makes sense, in this case we should be careful about exposing the replicas value: https://github.com/inventree/helm-charts/blob/main/charts/inventree/values.yaml#L6 so that not 2+ replicas are launched