← Back to team overview

txawsteam team mailing list archive

Re: lp:~oubiwann/txaws/415486-reservation-object into lp:txaws


> > On Tue, 2009-08-18 at 20:34 +0000, Thomas Herve wrote:
> > > > As far as I could tell only one reservation is returned, not N?
> > >
> > > No, you get one reservation per call to CreateInstances. If you're calling
> > CreateInstances several times, you're getting several reservations.
> > > The reservations are what is holding the information about the security
> > groups used, for example.
> >
> > Thats 'yes' then - a call to CreateInstances returns one reservation.
> >
> > Having our thunk to create instances return a _list of reservations_
> > makes the API harder to work with.
> We're not *creating* instances, we're describing them. A call to
> CreateInstances may return a list of instances, but it's not what we're doing
> right now.

There was some confusion about this which got sorted out on IRC. Robert then proposed an excellent idea that we all liked. Here's the IRC chat:

[18:14] <therve`> lifeless: the API we're calling is DescribeInstances
[18:14] <therve`> this one returns a list of reservations
[18:14] <lifeless> ok
[18:14] <lifeless> its still a little ugly to work with but I can see the rationale
[18:14] <therve`> yeah, the problem is that the reservation actually holds important information
[18:15] <therve`> which we can't get anywhere else
[18:15] <lifeless> we could fold that down to the instance
[18:15] <lifeless> if the info is only a field or two
[18:15] <obiwann> lifeless: yeah, that was one of my thoughts too... still not sure how I feel about it
[18:15] <lifeless> that is
[18:15] <lifeless> rather than reservations[N].instances[N]
[18:15] <lifeless> have instances[N].reservation
[18:16] <therve`> this is a valid idea
[18:16] <obiwann> yeah, I kinda like that
[18:16] <obiwann> instances[N].reservation is better than what I was thinking
Your team txAWS Team is subscribed to branch lp:txaws.