jclouds-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ignasi Barrera <notificati...@github.com>
Subject Re: [jclouds-labs] CloudSigma2 ComputeService and SecurityGroupExtension (#70)
Date Fri, 17 Oct 2014 20:34:34 GMT
@demobox I've been discussing with the people at CloudSigma the best approach to get this pull
request merged, and we've agreed to remove for now the SecurityGroupExtension, and merge this
just with the ComputeService.

The extension can be added later, but it presents some design challenges, as the CloudSigma
API supports ACCEPT and DROP rules as long as "negated" IP and port ranges. That makes it
impossible to have one-to-one mappings between a jclouds IpPermission and a CloudSigma firewall
rule, and that complicates the extension considerably. We'd better discuss the best approach
in the mailing list, and implement it accordingly after that. Sounds good to you @demobox?

This said, @Kentzo, @shevchenator, there are still some comments that have to be addressed.
Could you check the list of comments in this PR, as there are several still unanswered and
that need changes, and fix them? I hope after that this will be ready to be merged. Agree
@demobox?

I look forward to having this in!

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/70#issuecomment-59571924
Mime
View raw message