[
https://issues.apache.org/jira/browse/CLOUDSTACK-8799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14738964#comment-14738964
]
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_r39176449
--- Diff: systemvm/patches/debian/config/opt/cloud/bin/cs/CsAddress.py ---
@@ -95,9 +95,25 @@ def get_control_if(self):
return ip
return None
+ def check_if_link_exists(self,dev):
+ cmd="ip link show dev %s"%dev
+ result=CsHelper.execute(cmd)
+ if(len(result)!=0):
+ return True
+ else:
+ return False
+
+ def check_if_link_up(self,dev):
+ cmd="ip link show dev %s | tr '\n' ' ' | cut -d ' ' -f 9"%dev
--- End diff --
I still think that we should do the splitting in Python.
Execute ip with the -o flag so it is a one-liner and split with Python
afterwards.
> 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)
