Code review comment for lp:~nttdata/nova/live-migration

Revision history for this message
Masanori Itoh (itohm) wrote :

Hello Soren,

It looks like masumotok cannot reproduce the failure below, and
he is struggling to fix the issue as his highest priority.

I suggested to checkout his branch on the launchpad again,
and try to reproduce the issue.

Please give him some more time.

Thanks in advance,

Masanori (a.k.a thatsdone)

---
Masanori ITOH R&D Headquarters, NTT DATA CORPORATION
               e-mail: <email address hidden>
               phone : +81-50-5546-2301 (ext: 47-6278)

From: Soren Hansen <email address hidden>
Subject: Re: [Merge] lp:~nttdata/nova/live-migration into lp:nova
Date: Tue, 18 Jan 2011 21:44:52 +0900

> Review: Needs Fixing
> ======================================================================
> FAIL: test_authors_up_to_date (nova.tests.test_misc.ProjectTestCase)
> ----------------------------------------------------------------------
> Traceback (most recent call last):
> File "/home/soren/src/openstack/nova/nova/nova/tests/test_misc.py", line 53, in test_authors_up_to_date
> '%r not listed in Authors' % missing)
> AssertionError: set([u'<root@openstack2-api>', u'Masumoto<email address hidden>']) not listed in Authors
>
> ----------------------------------------------------------------------
> Ran 286 tests in 94.682s
>
> FAILED (failures=1)
>
>
> I've already explained *exactly* how to fix this problem.
>
>
> Before you submit things for review, please run the test suite.
> --
> https://code.launchpad.net/~nttdata/nova/live-migration/+merge/44940
> You are reviewing the proposed merge of lp:~nttdata/nova/live-migration into lp:nova.

« Back to merge proposal