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

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

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

    https://github.com/apache/cloudstack/pull/784#discussion_r39176487
  
    --- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py ---
    @@ -105,8 +121,17 @@ def process(self):
                 ip = CsIP(dev, self.config)
     
                 for address in self.dbag[dev]:
    +                if(address["nw_type"]!="public"):
    +                    continue
    +
    +                #check if link is up
    +                if (not self.check_if_link_exists(dev)):
    +                    logging.info("link %s dose not exist, so not 
processing"%dev)
    --- End diff --
    
    Typo in the logs here. dose = does


> 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: Blocker
>
> 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