dragonflyoss / helm-charts

Dragonfly Helm Charts
https://d7y.io
Apache License 2.0
30 stars 49 forks source link

feat: remove the unnecessary dependency on Dragonfly chart #288

Closed gaius-qi closed 3 months ago

gaius-qi commented 3 months ago

Description

Related Issue

Motivation and Context

kakkoyun commented 3 months ago

I noticed that these API changes are quite significant. Given the extent of the changes, it might have been more appropriate to release them as a minor or even major update.

Our team uses auto-upgrade for Helm releases, and we didn’t expect such API changes in a patch release. This unexpected update requires us to change our values.yaml file.

It would be helpful if changes of this magnitude were communicated differently in the future to better manage our upgrade processes.