<div dir="ltr">Hi All,<br><br>It looks like there is a confusion of using function name for firewall library.<br><br>library : <b>glustolibs-gluster/glustolibs/gluster/lib_utils.py </b><br><br>1) def <b>open_firewall</b>(nodes, firewall_service, permanent=False): ---&gt; merged one<br><br>since above function will disable the services, In my opinion its better to change the name to disable_firewall or remove_services_from_firewall<br>        <br><br>2) def <b>enable_firewall</b>(nodes, firewall_service, permanent=False): ----&gt; <a href="https://review.gluster.org/#/c/19481/7">https://review.gluster.org/#/c/19481/7</a><br><br><br>Above function will enable the services, its better to have the same name enable_firewall or add_services_to_firwall<br><br><br>Any thoughts or suggestions ?<br><br>Regards,<br>Vijay A<br></div>