k8s-at-home / charts

⚠️ Deprecated : Helm charts for applications you run at home
https://docs.k8s-at-home.com
Apache License 2.0
1.45k stars 623 forks source link

[mosquitto] Revert: Remove storage fields from configinc value (#1710) #1731

Closed morremeyer closed 2 years ago

morremeyer commented 2 years ago

Description of the change

This reverts commit 8d8dba0f15606778d2a8d550585c0a746bcb66cb which was breaking backwards compatibility.

Benefits

Fix for breaking compatibility in a minor version. If we want to go ahead and remove those defaults, that should be a 5.0.0 release.

Possible drawbacks

None known

Applicable issues

Additional information

Checklist

bjw-s commented 2 years ago

Hi, thanks for taking the time to create this PR, but as explained in the issue we do not want to set these defaults. Instead users should configure the storage configuration they require.