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 4/6] CIMI: Added MachineTemplate database model
Date Fri, 30 Nov 2012 08:55:36 GMT
On 11/29, David Lutterkort wrote:
> On Thu, 2012-11-29 at 14:05 +0100, mfojtik@redhat.com wrote:
> > From: Michal Fojtik <mfojtik@redhat.com>
> > 
> > * Added BaseEntity as a parent model for Entity
> > 
> > Signed-off-by: Michal fojtik <mfojtik@redhat.com>
> 
> ACK. Nice. Though do we need to call it MachineTemplateEntity ? Isn't
> Deltacloud::Database::MachineTemplate enough ? If we do need the extra
> distinction, I suggest we use 'Record' instead of 'Entity', since the
> latter is so overloaded, i.e. call this MachineTemplateRecord

Sure, I renamed it to just MachineTemplate, hope the ruby namespacing works
well :-)

I was a bit afraid about confusion caused by having two classes with the
same name, but not a big deal :-)

  -- Michal


-- 
Michal Fojtik <mfojtik@redhat.com>
Deltacloud API, CloudForms

Mime
View raw message