search

termie / nova-migration-demo

Nova is a cloud computing fabric controller (the main part of an IaaS system). It is written in Python.
http://openstack.org/projects/compute/
Apache License 2.0
2 stars 0 forks source link

nova-compute dependency on crypto #516

Open termie opened 13 years ago

termie commented 13 years ago

There doesn't seem to be a good reason for nova-compute to depend on nova.crypto (via AuthManager). nova.crypto also contains md5 which isn't technically crypto and can perhaps be moved to nova.utils.

The dependency on M2Crypto is a major bottleneck for Hyper-V since it is quite non-trivial to build it on Windows 2008 64-bit

Imported from Launchpad using lp2gh.

termie commented 13 years ago

(by ttx) Could you precise where nova-compute depends on AuthManager ? I quickly searched for it but couldn't see where the link was. Also are you proposing moving md5 out of nova.crypto so that you can more easily use it in HyperV-related code ?

termie commented 13 years ago

(by chiradeep) nova-compute loads a virt driver from nova.virt. All virt drivers import nova.auth.manager since they need to pass some credentials to the s3 service. But all they need is the user secret and access key. The crypto functions aren't required.

http://bazaar.launchpad.net/~hudson-openstack/nova/trunk/annotate/head%3A/nova/virt/images.py#L48

Re: md5, the dependency goes nova.virt.images -> nova.objectstore.image -> nova.objectstore.bucket -> nova.objectstore.stored -> nova.crypto.compute_md5 Once again, this causes a dependency on M2Crypto

On 12/30/2010 03:48 AM, Thierry Carrez wrote:

Could you precise where nova-compute depends on AuthManager ? I quickly searched for it but couldn't see where the link was. Also are you proposing moving md5 out of nova.crypto so that you can more easily use it in HyperV-related code ?

\ Changed in: nova Status: New => Incomplete

termie commented 13 years ago

(by ttx) I see now. Is this blocking your Hyper-V branch merge proposal ? Could you propose another branch that would implement this crypto refactoring ?