moosefs / moosefs-docker-cluster

Multiple node MooseFS cluster on Docker
https://moosefs.com
GNU General Public License v2.0
77 stars 27 forks source link

fix for overriding MFS_HDD_CONFIG by static content #19

Closed wfr3d closed 1 year ago

wfr3d commented 3 years ago

I think I found a bug in the code. Regardless of what we pass in the MFS_HDD_CONFIG environment variable, the value is overwritten by the "echo" command on lines 29 or 31. My solution: when the MFS_HDD_CONFIG variable is used, it leaves the user fully responsible for the content of the mfshdd.cfg file.

toomyem commented 1 year ago

:+1:

I didn't noticed that PR is already issued, and created separate issue for this :smile: https://github.com/moosefs/moosefs-docker-cluster/issues/20

toomyem commented 1 year ago

When can we expect to have this changes visible in docker image?