Hi,
These were added for Yong. He requires them for the l3 work. My
apologies for not updating this in the commit messages.
Thanks
Gary
On 11/07/2012 07:10 PM, Mark McClain wrote:
All-
Playing catch-up this morning. Today we had three reviews that were merged.
https://review.openstack.org/#/c/15551/
https://review.openstack.org/#/c/15556/
https://review.openstack.org/#/c/15557/
While updates to OS-common are nice, why are we adding code that actually is
not used within Quantum? The reason I'm asking is that 15551 could not stand
on its own and would have failed any unit tests had it actually been used
within Quantum.
I'm fine with updating existing OS-common modules, but it seems like new
modules should actually have a Quantum use case when they are added. (Similar
to adding need project dependencies).
Thoughts?
mark
--
Mailing list: https://launchpad.net/~quantum-core
Post to : [email protected]
Unsubscribe : https://launchpad.net/~quantum-core
More help : https://help.launchpad.net/ListHelp
