incubator-deltacloud-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Michal Fojtik <mfoj...@redhat.com>
Subject Re: [PATCH core] EC2: Retrieve Windows administrator password and advertise it through instance show
Date Mon, 08 Aug 2011 09:15:26 GMT

On Aug 5, 2011, at 11:28 PM, David Lutterkort wrote:

> On Thu, 2011-08-04 at 16:10 +0200, mfojtik@redhat.com wrote:
>> From: Michal Fojtik <mfojtik@redhat.com>
>> 
>> ---
>> server/lib/deltacloud/drivers/ec2/ec2_driver.rb |   17 +++++++++++++++++
>> server/views/instances/show.xml.haml            |    6 +++---
>> 2 files changed, 20 insertions(+), 3 deletions(-)
> 
> ACK.
> 
> I am a little uneasy that we now have to make two requests to the AWS
> API for each instance; is there any way to use the 'platform' attribute
> that DescribeInstances sends back to avoid the second call ? Supposedly
> it gets set to 'windows' for Windows instances, and not at all for
> others.

Yes, you're right. I'll look on the was gem if there is a way how to fetch
this platform string out there. Then I'll rework this patch.

  -- Michal

> 
> David
> 
> 

------------------------------------------------------
Michal Fojtik, mfojtik@redhat.com
Deltacloud API: http://deltacloud.org


Mime
View raw message