[automated-testing] Usage of Function name in ib_utils

[Vijay Bhaskar Reddy Avuthu]

Hi All,

It looks like there is a confusion of using function name for firewall
library.

library : *glustolibs-gluster/glustolibs/gluster/lib_utils.py *

1) def *open_firewall*(nodes, firewall_service, permanent=False): --->
merged one

since above function will disable the services, In my opinion its better to
change the name to disable_firewall or remove_services_from_firewall


2) def *enable_firewall*(nodes, firewall_service, permanent=False): ---->
https://review.gluster.org/#/c/19481/7


Above function will enable the services, its better to have the same name
enable_firewall or add_services_to_firwall


Any thoughts or suggestions ?

Regards,
Vijay A
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.gluster.org/pipermail/automated-testing/attachments/20180605/60b945fe/attachment-0001.html>