DaanHoogland commented on a change in pull request #2791: router: Fixes #2789
fix proper mark based packet routing across interfaces
URL: https://github.com/apache/cloudstack/pull/2791#discussion_r207847398
##########
File path: systemvm/debian/opt/cloud/bin/cs/CsNetfilter.py
##########
@@ -189,7 +189,8 @@ def compare(self, list):
def add_chain(self, rule):
""" Add the given chain if it is not already present """
if not self.has_chain(rule.get_table(), rule.get_chain()):
- CsHelper.execute("iptables -t %s -N %s" % (rule.get_table(),
rule.get_chain()))
+ if rule.get_chain() != "":
Review comment:
so when rule.get_chain() != self.chain && rule.get_chain() != "", we still
add it to self. chain?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
