Github user ygy commented on the pull request:
https://github.com/apache/incubator-brooklyn/pull/707#issuecomment-114064705
Thanks for the feedback.
@bostko: adding rules with firewall-cmd requires root privileges. it
follows the implementation of addIptablesRule
@andreaturli @tbouron: actually, the initial implementation was just as
your proposal. i've discussed it with @bostko already.
I've decided to go with the current one, because we are days before a major
release. I didn't wanted to change the interface or introduce big changes which
will require comprehensive testing. It will be good to have firewalld support
in 0.7 so we can get Riak running on CentOS 7 too.
If you agree, I can get it implemented with proper abstraction post 0.7
release?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
