search

seaweedfs / seaweedfs-csi-driver

SeaweedFS CSI Driver https://github.com/seaweedfs/seaweedfs
Apache License 2.0
210 stars 50 forks source link

Avoid null set on optional cli dataLocality arg #144

Closed Jackbennett closed 10 months ago

Jackbennett commented 10 months ago

This is actually a change from Date: Tue May 9 15:11:51 2023 on top of v1.1.3 which I don't entirely recall but it would have stopped my csi deploy without it.

So I don't know if its now been made redundant but at the time my "optional" dataLocality flag didn't like being "none" instead of actually being absent.

This PR could may need to be chucked out, or probably fix the not "none" string logic, I don't use helm much.