Alluxio / k8s-operator

An operator for managing Alluxio system on Kubernetes cluster
https://www.alluxio.io/
Apache License 2.0
11 stars 9 forks source link

[DONT MERGE] Refactor #78

Closed ssz1997 closed 10 months ago

ssz1997 commented 1 year ago

Two change:

ssz1997 commented 1 year ago

Planning on using the default values in the helm chart for default values instead of using the annotations. Since we already have a set of default values in helm chart and we are using it anyways, there's no reason to copy all the default values to operator. So the plan is to read the file to the struct, then override the fields from user-provided config file.

Now opening separate PRs to sync the structs between helm chart and operator and give all fields explicit default values in helm chart.