incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Marcus Sorensen <shadow...@gmail.com>
Subject Re: extractVolume broken, api refactor?
Date Sat, 12 Jan 2013 21:23:52 GMT
Its just the test_volume.py , there are a couple of things not quite right
with the existing one in master and Ryan has mostly been playing with it.
Ryan will submit the fixes to review board when it works, and then a
separate patch for it to apply the added tests for my resizevolume branch.
On Jan 12, 2013 1:35 PM, "Min Chen" <min.chen@citrix.com> wrote:

> Marcus,
>     Is your volume integration test in Marvin that we can also run?
>
>     Thanks
>     -min
>
>
>
> Sent from my iPhone
>
> On Jan 11, 2013, at 7:51 PM, "Marcus Sorensen" <shadowsor@gmail.com>
> wrote:
>
> > Thanks. It was our playing with making the volume integration test work
> > that brought the issue up. Will be nice to have these in place.
> > On Jan 11, 2013 7:50 PM, "Rohit Yadav" <rohit.yadav@citrix.com> wrote:
> >
> >>
> >> On 11-Jan-2013, at 5:23 PM, Marcus Sorensen <shadowsor@gmail.com>
> wrote:
> >>
> >>> I was playing with the latest master and noticed that extractVolume was
> >>> throwing a null pointer exception on getting the zone id. I'm headed
> >>> somewhere but thought I'd mention it because it looked like it could
> >>> involve the identity mapper and that made me wonder if it was related
> to
> >>> the api refactoring.
> >>
> >> Fixed!
> >>
> >>
> https://git-wip-us.apache.org/repos/asf?p=incubator-cloudstack.git;a=commit;h=40779975d33c554ce488a21d84e63e635e95984e
> >>
> >>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message