Hey,
The attached patch maps InstanceId.1 parameter as id, when using
DescribeInstances EC2 action.
I saw that some drivers take it into account, and some don't.
Didn't do massive testing there, only manually tested it with the rhevm driver,
and also ran all the testers.
Can someone review it?
Thank you,
Oved
From ee73d42f2841782385ec9379cb09ce6f8a75a1fe Mon Sep 17 00:00:00 2001
From: Oved Ourfali <[email protected]>
Date: Tue, 2 Oct 2012 16:28:16 +0200
Subject: [PATCH] EC2: passing InstanceId.1 as id in DescribeInstances
---
server/lib/ec2/query_parser.rb | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/server/lib/ec2/query_parser.rb b/server/lib/ec2/query_parser.rb
index fc2d5be..d0585e3 100644
--- a/server/lib/ec2/query_parser.rb
+++ b/server/lib/ec2/query_parser.rb
@@ -20,7 +20,7 @@ module Deltacloud::EC2
MAPPINGS = {
:describe_availability_zones => { :method => :realms, :params => { 'ZoneName.1' => :id } },
:describe_images => { :method => :images, :params => { 'ImageId.1' => :id }},
- :describe_instances => { :method => :instances, :params => {} },
+ :describe_instances => { :method => :instances, :params => {'InstanceId.1' => :id } },
:describe_key_pairs => { :method => :keys, :params => {} },
:create_key_pair => { :method => :create_key, :params => { 'KeyName' => :key_name }},
:delete_key_pair => { :method => :destroy_key, :params => { 'KeyName' => :id }},
--
1.7.11.2
