gluster / glusterfs

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

tests: replace localhost and 127.0.01 with valid ip #2945

Closed Shwetha-Acharya closed 1 year ago

Shwetha-Acharya commented 2 years ago

Problem: From Fedora 34 it is mandatory to use valid ip instead of localhost, 127.0.0.1 or loopback address (0.0.0.0 to 0.255.255.255).

Solution: use $hostname -I | awk '{print $1}' instead

Fixes: #2944 Change-Id: I282a0a519c650c6848ffa668ade86a4f35f9de42 Signed-off-by: Shwetha K Acharya sacharya@redhat.com

gluster-ant commented 1 year ago

1 test(s) failed ./tests/bugs/glusterd/bug-824753.t

0 test(s) generated core

3 test(s) needed retry ./tests/000-flaky/basic_afr_split-brain-favorite-child-policy.t ./tests/basic/ctime/ctime-rep-rebalance.t ./tests/bugs/glusterd/bug-824753.t https://build.gluster.org/job/gh_centos7-regression/2583/

mscherer commented 1 year ago

/run regression

gluster-ant commented 1 year ago

1 test(s) failed ./tests/bugs/glusterd/bug-824753.t

0 test(s) generated core

4 test(s) needed retry ./tests/000-flaky/basic_afr_split-brain-favorite-child-policy.t ./tests/basic/ec/ec-1468261.t ./tests/bugs/bitrot/bug-1227996.t ./tests/bugs/glusterd/bug-824753.t https://build.gluster.org/job/gh_centos7-regression/2584/

Shwetha-Acharya commented 1 year ago

/run regression

Shwetha-Acharya commented 1 year ago

/run regression

gluster-ant commented 1 year ago

1 test(s) failed ./tests/bugs/posix/bug-765380.t

0 test(s) generated core

4 test(s) needed retry ./tests/000-flaky/glusterd-restart-shd-mux.t ./tests/basic/ec/ec-1468261.t ./tests/bugs/glusterd/remove-brick-testcases.t ./tests/bugs/posix/bug-765380.t

1 flaky test(s) marked as success even though they failed ./tests/000-flaky/glusterd-restart-shd-mux.t https://build.gluster.org/job/gh_centos7-regression/2595/

Shwetha-Acharya commented 1 year ago

/run regression

gluster-ant commented 1 year ago

1 test(s) failed ./tests/bugs/rpc/bug-921072.t

0 test(s) generated core

6 test(s) needed retry ./tests/000-flaky/basic_afr_split-brain-favorite-child-policy.t ./tests/00-geo-rep/georep-basic-dr-rsync-arbiter.t ./tests/basic/afr/ta-shd.t ./tests/basic/quick-read-with-upcall.t ./tests/bugs/replicate/do-not-reopen-fd.t ./tests/bugs/rpc/bug-921072.t https://build.gluster.org/job/gh_centos7-regression/2598/

mscherer commented 1 year ago

Shouldn't we merge ? It seems some tests are unstable, but they shouldn't be affected by this change.

Shwetha-Acharya commented 1 year ago

/run regression

Shwetha-Acharya commented 1 year ago

Shouldn't we merge ? It seems some tests are unstable, but they shouldn't be affected by this change.

Let me just see if retriggering the regression works. Nature of failure of tests/bugs/rpc/bug-921072.t: mount.nfs: /mnt/nfs/0 is busy or already mounted seems be a transient unstability as you have suggested

gluster-ant commented 1 year ago

1 test(s) failed ./tests/bugs/rpc/bug-921072.t

0 test(s) generated core

7 test(s) needed retry ./tests/000-flaky/basic_afr_split-brain-favorite-child-policy.t ./tests/000-flaky/glusterd-restart-shd-mux.t ./tests/basic/afr/ta-read.t ./tests/basic/afr/ta-shd.t ./tests/basic/distribute/phase1-migration-fops.t ./tests/bugs/bitrot/bug-1245981.t ./tests/bugs/rpc/bug-921072.t

1 flaky test(s) marked as success even though they failed ./tests/000-flaky/glusterd-restart-shd-mux.t https://build.gluster.org/job/gh_centos7-regression/2605/

Shwetha-Acharya commented 1 year ago

/run regression

gluster-ant commented 1 year ago

1 test(s) failed ./tests/bugs/rpc/bug-921072.t

0 test(s) generated core

5 test(s) needed retry ./tests/000-flaky/basic_afr_split-brain-favorite-child-policy.t ./tests/000-flaky/glusterd-restart-shd-mux.t ./tests/basic/ec/self-heal.t ./tests/bugs/distribute/bug-961615.t ./tests/bugs/rpc/bug-921072.t https://build.gluster.org/job/gh_centos7-regression/2610/

Shwetha-Acharya commented 1 year ago

/run regression

Shwetha-Acharya commented 1 year ago

As all the tests are passing, this PR is ready for merge @mscherer @xhernandez