Code review comment for lp:~morita-kazutaka/nova/snapshot-volume

Revision history for this message
Mark Washenberger (markwash) wrote :

Thanks for adding the tests. I'm not sure that the database teardown steps in those tests are necessary, but it appears to be the convention for the tests nearby. . . hmm. . .

> Yes, we can remove DB messaging in this proposal.
> I think of starting the work after Vish finishes his work on nova-volume:
> https://code.launchpad.net/~vishvananda/nova/no-db-messaging/+merge/61189

I agree--the db messaging proposal is not as far along as I had initially suspected, so I would agree we should worry about addressing this concern later.

« Back to merge proposal