azimuth-cloud / capi-helm-charts

Apache License 2.0
16 stars 20 forks source link

Do not enable optional addons #370

Closed waipeng closed 1 month ago

waipeng commented 3 months ago

The Nvidia and Mellanox addons are not necessary for a cluster; do not enable them for now.

waipeng commented 3 months ago

Should we do it here or in charts/openstack-cluster/values.yaml? Looking for advice on consistency. E.g.

For openstack addon, it is cluster-addons - enabled: false openstack-cluster - enabled: true

For nvidiaGPUOperator addon, it is cluster-addons - enabled: true openstack-cluster - not defined

pawcykca commented 2 months ago

I think the best place would be the values ​​object generated by the Magnum CAPI Helm driver. This way we will avoid changes coming from this or another repository providing Helm Charts for Magnum CAPI Helm driver.