[ 
https://issues.apache.org/jira/browse/BROOKLYN-137?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14597346#comment-14597346
 ] 

ASF GitHub Bot commented on BROOKLYN-137:
-----------------------------------------

Github user neykov commented on a diff in the pull request:

    https://github.com/apache/incubator-brooklyn/pull/707#discussion_r33020040
  
    --- Diff: 
utils/common/src/main/java/brooklyn/util/ssh/IptablesCommands.java ---
    @@ -89,6 +89,38 @@ public static String iptablesServiceStatus() {
             return iptablesService("status");
         }
     
    +    @Beta // implementation not portable across distros
    +    public static String firewalldService(String cmd) {
    +        return sudo(alternatives(
    +                BashCommands.ifExecutableElse1("systemctl", "systemctl " + 
cmd + " firewalld"),
    +                "/usr/bin/systemctl " + cmd + " firewalld"));
    --- End diff --
    
    Does this mean that `/usr/bin` might not be on the `PATH`? In what cases?


> Stop Iptables and open iptables don't work on centOS 7
> ------------------------------------------------------
>
>                 Key: BROOKLYN-137
>                 URL: https://issues.apache.org/jira/browse/BROOKLYN-137
>             Project: Brooklyn
>          Issue Type: Bug
>            Reporter: Andrea Turli
>            Assignee: Andrea Turli
>
> RHEL7 introduces firewalld as a replacement to iptables
> http://www.certdepot.net/rhel7-get-started-firewalld/



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to