[dev] libcloud pull request: LIBCLOUD-494: Add image extension methods for create...

2014-01-12 Thread cderamus
Github user cderamus closed the pull request at: https://github.com/apache/libcloud/pull/222

[dev] libcloud pull request: LIBCLOUD-494: Add image extension methods for create...

2014-01-12 Thread cderamus
GitHub user cderamus opened a pull request: https://github.com/apache/libcloud/pull/222 LIBCLOUD-494: Add image extension methods for create/copy This PR adds new extension methods to copy AMIs between Amazon regions and also adds the ability to create an image from an EBS backed i

Re: [dev] Proposal for security group promotion to the base compute API

2014-01-12 Thread Chris DeRamus
Ah, that's a good point John and I agree entirely. I've updated the gist and will keep that in mind moving forward. On Sun, Jan 12, 2014 at 11:49 AM, John Carr wrote: > Where AWS doesn’t have an explicit ID and our API would be rubbish without > one I have in the past fabricated one. Records in

Re: [dev] Proposal for security group promotion to the base compute API

2014-01-12 Thread John Carr
Where AWS doesn’t have an explicit ID and our API would be rubbish without one I have in the past fabricated one. Records in the Route53 API spring to mind. So for AWS i would consider setting the id property to “:”.join(self.source, self.proto, self.from) or similar. I think this is kind of th

[dev] libcloud pull request: LIBCLOUD-493: Add additional data into the Node.extr...

2014-01-12 Thread cderamus
Github user cderamus closed the pull request at: https://github.com/apache/libcloud/pull/221

Re: [dev] Proposal for security group promotion to the base compute API

2014-01-12 Thread Chris DeRamus
The gist has been updated at https://gist.github.com/cderamus/8182092 and is ready for review. If desired, we could add a direction parameter to the authorize/revoke methods to specify a direction (ingress vs. egress). The reason I kept them separate was mainly just because I thought calling out th

[dev] libcloud pull request: LIBCLOUD-493: Add additional data into the Node.extr...

2014-01-12 Thread cderamus
GitHub user cderamus opened a pull request: https://github.com/apache/libcloud/pull/221 LIBCLOUD-493: Add additional data into the Node.extra dictionary Note that this PR removes libcloud/test/compute/fixtures/ec2/describe_instances_with_tags.xml as it is no longer necessary. The t