Open Stack

Hi Flavio,

Could you pls take a review on https://review.openstack.org/#/c/29245/

thanks,
Zhi Yan

On Thu, May 16, 2013 at 6:10 PM, lzy.dev at gmail.com <lzy.dev at gmail.com> wrote:
> thanks, I have replied your comment there.
>
> thanks,
> Zhi Yan
>
> On Thu, May 16, 2013 at 4:12 PM, Flavio Percoco <flavio at redhat.com> wrote:
>> On 16/05/13 14:56 +0800, lzy.dev at gmail.com wrote:
>>>
>>> Hi guys,
>>>
>>> This issue be found in my environment and traced by:
>>> https://bugs.launchpad.net/glance/+bug/1180377
>>>
>>> I have proposed a fix on follow page, I have talked this change with
>>> Mark in IRC, current I need you to review this change to make sure it
>>> is good enough and will not make regress and keep the backwards
>>> compatibility. After that I'd like put the unit test there.
>>> https://review.openstack.org/#/c/29245/
>>
>>
>> Hi,
>>
>> I added a comment in your review. Pls, take a look there.
>>
>> Cheers,
>> FF
>>
>> --
>> { name: "Flavio Percoco",
>>   gpg: "87112EC1",   internal: "8261386",
>>   phone: "+390687502386",
>>   irc: ["fpercoco", "flaper87"]}
>>
>> _______________________________________________
>> OpenStack-dev mailing list
>> OpenStack-dev at lists.openstack.org
>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev