Replace obsoleted readfp() with read_file(), fixes gluster/glusterfs#4403

[hunter86bg]

As stated in the deprecation/removal warning, readfp() is removed in python 3.12 which leads georeplication setup to fail with a traceback.

Official docs clearly state that read_file() is the proper replacement.

[gluster-ant]

Can one of the admins verify this patch?

[gluster-ant]

Can one of the admins verify this patch?

[gluster-ant]

Can one of the admins verify this patch?

[hunter86bg]

This one should fix #4403

[hunter86bg]

As CentOS7 is dead, devrpm-el7 should not be required, right ?

[sulphur]

I've been searching for that solution for past few days. We are upgrading ubutnu to 24.04 and Gluster 11.1 and indeed Geo-replicaion is broken and i've tested by manually replacing these functions names from readfp to read_file and that seems to fix the problem. Thanks @hunter86bg for finding that. Hope it will get patched soon

[hunter86bg]

@sulphur , it was reported in the mailing list by a Fedora user but I guess evry modern distro will be affected of it.

[amarts]

/run regression

[gluster-ant]

1 test(s) failed
./tests/00-geo-rep/00-georep-verify-non-root-setup.t

0 test(s) generated core

1 test(s) needed retry
./tests/00-geo-rep/00-georep-verify-non-root-setup.t
https://build.gluster.org/job/gh_centos7-regression/3416/