search

gluster / glusterfs

Gluster Filesystem : Build your distributed storage in minutes
https://www.gluster.org
GNU General Public License v2.0
4.51k stars 1.07k forks source link

mount.glusterfs: Increase the max characters allow in the hostnames of glusterservers #4331

Open spotzero opened 1 month ago

spotzero commented 1 month ago

The mount scripts validate hostnames by checking that they are less than 64 characters long. Host names on many cloud environments are generally much longer than this, but are still valid.

This change increases the hostname length to allow for these valid hostname.

gluster-ant commented 1 month ago

Can one of the admins verify this patch?

gluster-ant commented 1 month ago

Can one of the admins verify this patch?

gluster-ant commented 1 month ago

Can one of the admins verify this patch?

aravindavk commented 1 month ago

/run regression

gluster-ant commented 1 month ago

0 test(s) failed

1 test(s) generated core ./tests/bugs/glusterd/mgmt-handshake-and-volume-sync-post-glusterd-restart.t

2 test(s) needed retry ./tests/00-geo-rep/georep-basic-dr-rsync.t ./tests/basic/afr/ta-shd.t https://build.gluster.org/job/gh_centos7-regression/3392/

spotzero commented 1 month ago

@aravindavk I've amended the commit to include the sign-off. Thanks!

mohit84 commented 1 month ago

/run regression