Hi Chris, sorry for the delay -

at first I was sceptical about the usefulness of exposing the rule URI since it doesn't make sense to 'retrieve' (GET) a single rule. However I agree that it makes sense to expose the rule URI as we have a 'DELETE rule_id' operation. I'll send an updated patch and will commit as soon as you ack that,

marios

On 30/06/11 23:44, Chris Lalancette wrote:
On 06/27/11 - 05:22:18PM, [email protected] wrote:
From: marios<[email protected]>

DELETE /api/firewalls/:firewall/:rule_id

Signed-off-by: marios<[email protected]>
---
  server/lib/deltacloud/drivers/ec2/ec2_driver.rb |    2 +-
  server/server.rb                                |   31 ++++++++++------------
  server/views/firewalls/show.html.haml           |    3 +-
  3 files changed, 16 insertions(+), 20 deletions(-)

Hey Marios,
      Sorry for the long delay here.  This patch is most of what I would like
to see, though there is one thing missing.  In the XML for the firewall rule,
I think we should export the URL for the rule as well, similar to how blobs
have a full URL.  Something like this (untested) patch?


Reply via email to