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

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

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

    https://github.com/apache/incubator-brooklyn/pull/707#discussion_r32935680
  
    --- Diff: 
locations/jclouds/src/main/java/brooklyn/location/jclouds/JcloudsLocation.java 
---
    @@ -365,6 +365,16 @@ public boolean isWindows(NodeMetadata node, ConfigBag 
config) {
                     : (OsFamily.WINDOWS == confFamily);
         }
     
    +    public boolean isLocationFirewalldEnabled(SshMachineLocation location) 
{
    +        int result = location.execCommands("checking if firewalld is 
active", 
    +                
ImmutableList.of(IptablesCommands.firewalldServiceIsActive()));
    +        if (result == 0) {
    +            return true;
    +        }
    +        
    --- End diff --
    
    @neykov here is addressed your previous comment regarding the firewall-cmd 
existence 


> 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