RE: [PATCH] added Template subcollections to SystemTemplate

2013-03-27 Thread Koper, Dies
A bit of background on this patch: It contains my latest work on implementing cimi systems and system templates for mock and fgcp. It is not complete and contains some debugging statements. Its main purpose is to show you what I have and what issues I'm seeing. It applies to master (fcffad13e66175

[PATCH] added Template subcollections to SystemTemplate

2013-03-27 Thread diesk
From: Dies Koper --- 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/model

Re: [PATCH] added Template subcollections to SystemTemplate

2013-03-27 Thread David Lutterkort
On Thu, 2013-03-28 at 11:11 +1100, di...@fast.au.fujitsu.com wrote: > From: Dies Koper > > --- > server/lib/cimi/collections/systems.rb | 58 +-- > server/lib/cimi/models.rb | 12 ++- > server/lib/cimi/models/machine_template.rb | 6 +- >

[PATCH] added Template subcollections to SystemTemplate

2013-03-27 Thread diesk
From: Dies Koper --- 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/model

[PATCH] CIMI schema: tolerate nil hash_map attributes

2013-03-27 Thread diesk
From: David Lutterkort --- server/lib/cimi/models/schema.rb | 28 ++-- 1 file changed, 22 insertions(+), 6 deletions(-) diff --git a/server/lib/cimi/models/schema.rb b/server/lib/cimi/models/schema.rb index 5a2049b..8dd309c 100644 --- a/server/lib/cimi/models/schema.rb +

Re: [PATCH] added Template subcollections to SystemTemplate

2013-03-27 Thread David Lutterkort
On Thu, 2013-03-28 at 09:27 +1100, Koper, Dies wrote: > The error message is the same, but the line number it occurs in has changed > (was schema.rb:273:in > `to_xml'). After adding extract(model) to it there it works fine. Thanks! > > Not sure where to move the "def extract" method declaration t

RE: [PATCH] added Template subcollections to SystemTemplate

2013-03-27 Thread Koper, Dies
Hi David, The error message is the same, but the line number it occurs in has changed (was schema.rb:273:in `to_xml'). After adding extract(model) to it there it works fine. Thanks! Not sure where to move the "def extract" method declaration to make it accessible from both classes. I tried movi

Re: [PATCH] added Template subcollections to SystemTemplate

2013-03-27 Thread David Lutterkort
Hi Dies, it's a little hard to diagnose what's going on there; it seems to happen because the code makes an assumption that an attribute of type hash_map is never nil (i.e., will be initialized at some point) I think that assumption is nonsense. See if the attached patch fixes things for you - if

Re: Fun with Bundler vs rubygems - ideas

2013-03-27 Thread jvlcek
On 03/26/2013 10:18 AM, Michal Fojtik wrote: > Hi everyone, > > First. Sorry for this long email :-) > > I started working on refactoring of how we handle gems loading in > Deltacloud API. This is what I discover: > > 1. In 'server', we don't use Bundler. If you run Deltacloud using: > > './bin/del

RE: [PATCH core 2/2] CIMI: Revamp all collections to use generated operations (where possible)

2013-03-27 Thread Koper, Dies
ACK. Nice, I'd been contemplating writing something like this when writing all the boilerplate code for system subcollections. This will show me how to do that. One nit: +description "Delete specified Credential entity" Doesn't need "Credential". Regards, Dies Koper

[ANNOUNCE]: CANCELLED Deltacloud Community Call - 27th March 2013

2013-03-27 Thread mar...@redhat.com
Due to the ongoing EU-US daylight savings time difference and the resulting scheduling conflicts, we have to cancel today's Deltacloud Community Call. EU gets the time change this weekend (31st March) so next week things are back to normal. If there's some aspect of Deltacloud you'd like to hear a

[PATCH core 1/2] CIMI: Added 'generate_XXX_route' helpers for CIMI

2013-03-27 Thread mfojtik
From: Michal Fojtik All CIMI entities share the same code in Rabbit operations. These helpers should merge them all to avoid code duplication. Signed-off-by: Michal fojtik --- server/lib/cimi/helpers/cimi_rabbit_helper.rb | 59 +++ 1 file changed, 59 insertions(+) crea

[PATCH core 2/2] CIMI: Revamp all collections to use generated operations (where possible)

2013-03-27 Thread mfojtik
From: Michal Fojtik Signed-off-by: Michal fojtik --- server/lib/cimi/collections/address_templates.rb | 46 ++- server/lib/cimi/collections/addresses.rb | 48 +++- server/lib/cimi/collections/base.rb| 6 ++- server/lib/cimi/collectio