incubator-deltacloud-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From David Lutterkort <lut...@redhat.com>
Subject Re: [PATCH core 4/4] Removed local mappings and switched to storing mappings in remote datastore.
Date Wed, 20 Jul 2011 22:29:10 GMT
On Tue, 2011-07-19 at 16:19 +0200, mfojtik@redhat.com wrote:
> From: Michal Fojtik <mfojtik@redhat.com>

I have one nit with this patch series:

> +    # You can use 'user_data' feature to set 'user_data' parameter when creating
> +    # a new instance where this parameter can hold gzipped CDROM iso which will
> +    # be mounted into created instance after boot
>      feature :instances, :user_data

We can't call this ISO upload 'user_data' in the API - there are very
different expectations around what that data is for vSphere than for
EC2/Euca.

Instead, we should call this feature something else, say 'user_iso', and
accept a different parameter for it.

Ideally, we could also support the feature user_data straightup, with
the effect that we will create an ISO from it, place the user-supplied
data inside that ISO with a fixed file name, and attach that as the
CD-ROM.

David



Mime
View raw message