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

ASF GitHub Bot commented on CLOUDSTACK-8799:
--------------------------------------------

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

    https://github.com/apache/cloudstack/pull/784#discussion_r39033655
  
    --- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py ---
    @@ -95,9 +95,17 @@ def get_control_if(self):
                     return ip
             return None
     
    +    def check_if_link_up(self,dev):
    +        cmd="ip link show dev %s | tr '\n' ' ' | cut -d ' ' -f 9"%dev
    --- End diff --
    
    Hi, 
    
    the output of Ip link show command is not expected to changes so in this 
case i think hardcoding the limiters is not an issue. 


> fix CsRedundant.py to handle public interfaces and default routes when 
> changing state.
> --------------------------------------------------------------------------------------
>
>                 Key: CLOUDSTACK-8799
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8799
>             Project: CloudStack
>          Issue Type: Bug
>      Security Level: Public(Anyone can view this level - this is the 
> default.) 
>          Components: Virtual Router
>    Affects Versions: 4.6.0
>            Reporter: Bharat Kumar
>            Assignee: Bharat Kumar
>            Priority: Critical
>
> When the Vr changes state to backup we need bring all the public interfaces 
> down. Similarly when it changes state to master we have bring all the public 
> interfaces up and add the default routes.



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

Reply via email to