deltacloud-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Koper, Dies" <di...@fast.au.fujitsu.com>
Subject RE: system_machines not working
Date Thu, 16 May 2013 03:49:43 GMT
Hi Michal,

I can't get the cimi tests to work anymore. I'm not sure what caused the
regression. Could you take a look?

With Joe's PR applied, I get:

d:/sources/OSS/cloud/deltacloud/server/lib/cimi/models/schema.rb:20:in
`<top (required)>': uninitialized constant CIMI::Model (NameError)
        from
d:/sources/OSS/cloud/deltacloud/server/lib/cimi/models.rb:19:in
`require_relative'
        from
d:/sources/OSS/cloud/deltacloud/server/lib/cimi/models.rb:19:in `<top
(required)>'
        from
d:/sources/OSS/cloud/deltacloud/tests/cimi/test_helper.rb:25:in
`require_relative'

With Joe's PR and the two commits I mentioned in DTACLOUD-559 reverted,
I get:

d:/sources/OSS/cloud/deltacloud/tests/cimi/test_helper.rb:152:in
`get_a': undefined method `[]' for nil:NilClass (NoMethodError)
        from
d:/sources/OSS/cloud/deltacloud/tests/cimi/machex1_test.rb:51:in
`<class:CaptureAMachine>'
        from
d:/sources/OSS/cloud/deltacloud/tests/cimi/machex1_test.rb:23:in `<top
(required)>'
        from
D:/Programs/Ruby193/lib/ruby/site_ruby/1.9.1/rubygems/custom_require.rb:
36:in `require'

When I revert more commits (everything after the network revamp was
added, i.e. April 17 with f69f3ddffdc471036b75b0223ec8ed268663753f), the
tests pass again.

Cheers,
Dies Koper


> -----Original Message-----
> From: Joseph VLcek [mailto:jvlcek@redhat.com]
> Sent: Thursday, 16 May 2013 11:00 AM
> To: dev@deltacloud.apache.org
> Subject: Re: system_machines not working
> 
> 
> On May 15, 2013, at 1:04 AM, Koper, Dies wrote:
> 
> > Hi Joe, Michal,
> >
> > I'd like to bring your attention to this regression:
> >
> > https://issues.apache.org/jira/browse/DTACLOUD-559
> >
> > When I revert your following two commits, it works again, so you may
> > have a better idea at what's going wrong?
> >
> > pick 959cc62 Changes needed to adapt to using a hash to contain all
> root
> > entry point, vs the acts_as_root_entity method. Refactor
> > service/cloud_entry_point.rb b
> > ased on input from Michal Fojtik. Huge Thanks! Michal Fojtik!
> > pick aa9a232 CIMI - Remove credentialTemplates from CEP COLLECTIONS.
> > Use ruby style hash iteration on CEP COLLECTIONS.        Fix typos
> > uncovered during
> > testing.
> >
> > I'll put the addition of unit tests to catch system related
regressions
> > higher on my to-do list.
> >
> > Cheers,
> > Dies Koper
> >
> >
> >
> >
> 
> 
> Dies,
> 
> I''ve posted a pull request for review.
> 
> https://github.com/deltacloud/deltacloud-core/pull/27
> 
> Sorry for any inconvenience!
> 
> Joe



Mime
View raw message