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] added Template subcollections to SystemTemplate
Date Thu, 28 Mar 2013 00:19:53 GMT
On Thu, 2013-03-28 at 11:11 +1100, diesk@fast.au.fujitsu.com wrote:
> From: Dies Koper <diesk@fast.au.fujitsu.com>
> 
> ---
>  server/lib/cimi/collections/systems.rb             |  58 +++++++++--
>  server/lib/cimi/models.rb                          |  12 ++-
>  server/lib/cimi/models/machine_template.rb         |   6 +-
>  server/lib/cimi/models/network_template.rb         |   4 +-
>  server/lib/cimi/models/system.rb                   |  24 ++---
>  server/lib/cimi/models/system_template.rb          |  20 ++--
>  server/lib/cimi/models/volume_template.rb          |   4 +-
>  server/lib/cimi/service.rb                         |   2 +
>  server/lib/deltacloud/drivers/fgcp/fgcp_client.rb  |   4 +
>  .../drivers/fgcp/fgcp_driver_cimi_methods.rb       | 116 +++++++++++++++++----
>  .../mock/data/cimi/system_template/template1.json  |  31 +++++-
>  .../drivers/mock/mock_driver_cimi_methods.rb       |  64 ++++++++++++
>  .../cimi/collections/system_templates_test.rb      |  47 +++++++++
>  13 files changed, 329 insertions(+), 63 deletions(-)

NAK; this is missing a whole bunch of files:

> index 8eeb96c..c8f138c 100644
> --- a/server/lib/cimi/models.rb
> +++ b/server/lib/cimi/models.rb
> @@ -79,7 +79,15 @@ require_relative './models/address_template_create'
>  require_relative './models/address_create'
>  require_relative './models/forwarding_group'
>  require_relative './models/forwarding_group_template'
> -require_relative './models/system_template'
> -require_relative './models/system'
>  require_relative './models/network_template'
>  require_relative './models/network_create'
> +require_relative './models/system_template'
> +require_relative './models/system_network_port'
> +require_relative './models/system_network'
> +require_relative './models/system_address'
> +require_relative './models/system_forwarding_group'
> +require_relative './models/system_machine'
> +require_relative './models/system_volume'
> +require_relative './models/system_credential'
> +require_relative './models/system_system'
> +require_relative './models/system'

All the system_* files are missing.

David



Mime
View raw message