~soren/nova/rs_auth_fixes : changes

~soren/nova/rs_auth_fixes » Changes from revision 285

From Revision 278 to 259

Rev	Summary	Authors	Date
278	Refactored the auth branch based on review feedback Refactored the auth branch based on review feedback	Cerberus	13 years ago
277	Replaced the existing Rackspace Auth Mechanism with one that... Replaced the existing Rackspace Auth Mechanism with one that mirrors the implementation in the design document.	Cerberus	13 years ago
276	Better error message on the failure of a spawned process, an... Better error message on the failure of a spawned process, and it's a ProcessExecutionException irrespective of how the process is run (twisted or not)	Vishvananda Ishaya	13 years ago
275	Proposing merge to get feedback on orm refactoring. I am ve... Proposing merge to get feedback on orm refactoring. I am very interested in feedback to all of these changes. This is a huge set of changes, that touches almost all of the files. I'm sure I have broken quite a bit, but better to take the plunge now than to postpone this until later. The idea is to allow for pluggable backends throughout the code. Brief Overview For compute/volume/network, there are multiple classes service - responsible for rpc this currently uses the existing cast and call in rpc.py and a little bit of magic to call public methods on the manager class. each service also reports its state into the database every 10 seconds manager - responsible for managing respective object classes all the business logic for the classes go here db (db_driver) - responsible for abstracting database access driver (domain_driver) - responsible for executing actual shell commands and implementation Compute hasn't been fully cleaned up, but to get an idea of how it works, take a look at volume and network Known issues/Things to be done: * nova-api accesses db objects directly It seems cleaner to have only the managers dealing with their respective objects. This would mean code for 'run_instances' would move into the manager class and it would do the initial setup and cast out to the remote service * db code uses flat methods to define its interface In my mind this is a little prettier as an abstract base class, but driver loading code can load a module or a class. It works, so I'm not sure it needs to be changed but feel free to debate it. * Service classes have no code in them Not sure if this is a problem for people, but the magic of calling the manager's methods is done in the base class. We could remove the magic from the base class and explicitly wrap methods that we want to make available via rpc if this seems nasty. * AuthManager Projects/Users/Roles are not integrated into this system. In order for everything to live happily in the backend, we need some type of adaptor for LDAP * Context is not passed properly across rabbit Context should probably be changed to a simple dictionary so that it can be passed properly through the queue * No authorization checks on access to objects We need to decide on which layer auth checks should happen. * Some of the methods in ComputeManager need to be moved into other layers/managers * Compute driver layer should be abstracted more cleanly * Flat networking is untested and may need to be reworked * Some of the api commands are not working yet * Nova Swift Authentication needs to be refactored(Todd is working on this)	Vishvananda Ishaya	13 years ago
274	When destroying a VM using the XenAPI backend, if the VM is ... When destroying a VM using the XenAPI backend, if the VM is still running (the usual case) the destroy fails. It needs to be powered-off first.	Ewan Mellor	13 years ago
273	Remove power state constants that have ended up duplicated f... Remove power state constants that have ended up duplicated following a bad merge. They were moved from nova.compute.node.Instance into nova.compute.power_state at the same time that Instance was moved into nova.compute.service. We've ended up with these constants in both places. Remove the ones from service, in favour of the ones in power_state.	Ewan Mellor	13 years ago
272	Move vol.destroy() call out of the _check method in Move vol.destroy() call out of the _check method in test_multiple_volume_race_condition test and into a callback of the DeferredList. This should fix the intermittent failure of that test. I /think/ test_too_many_volumes's failure was caused by test_multiple_volume_race_condition failure, since I have not been able to reproduce its failure after fixing this one.	Soren Hansen	13 years ago
271	Adds 'shell run' to nova manage, which spawns a shell with f... Adds 'shell run' to nova manage, which spawns a shell with flags properly imported.	Jesse Andrews	13 years ago
270	It's annoying and confusing to have to set PYTHONPATH to poi... It's annoying and confusing to have to set PYTHONPATH to point to your development tree before you run any of the scripts. If you're lucky, it just fails (because you don't have nova in the default search path (i.e. don't have them installed in /usr/lib/blah)) so that you can fix it up. If you're less lucky, you spend at least a couple of minutes wondering why the cool changes you made to one of the nova modules don't take effect until you realise it's because it's using the nova stuff in /usr/lib. So, to save myself (and probably others) a bit of time, this patch adds a snippet to each of the scripts in bin/ to detect this and set up the PYTHONPATH.	Soren Hansen	13 years ago
269	Just a couple of UML-only fixes: Just a couple of UML-only fixes: * Due to an issue with libvirt, we need to chown the disk image to root. * Just point UML's console directly at a file, and don't bother with the pty. It was only used for debugging.	Soren Hansen	13 years ago
268	Replace an if/else with a dict lookup to a factory method. Replace an if/else with a dict lookup to a factory method.	Michael Gundlach	13 years ago
267	Create console.log ahead of time. This ensures that the user... Create console.log ahead of time. This ensures that the user running nova-compute maintains read privileges.	Soren Hansen	13 years ago
266	This improves the changelog generated as part of "setup.py s... This improves the changelog generated as part of "setup.py sdist". If you look at it now, it says that Tarmac has done everything and every little commit is listed. With this patch, it only logs the "top-most" commit and credits the author rather than the committer. Example from the current output format (from current trunk): ########################################## 2010-08-31 Tarmac Adjust setup.py to match nova-rsapi -> nova-api-new rename. 2010-08-31 Soren Hansen <soren.hansen@rackspace.com> Fix up setup.py to match nova-rsapi -> nova-api-new rename. 2010-08-30 Tarmac Reconnect to libvirt on broken connection. 2010-08-30 Soren Hansen <soren.hansen@rackspace.com> Detect if libvirt connection has been broken and reestablish it. 2010-08-30 Tarmac pylint fixes for /nova/virt/connection.py 2010-08-30 jaypipes@gmail.com Merge trunk and resolve conflicts ########################################## With this patch, this is reduced to: ########################################## 2010-08-31 Soren Hansen <soren.hansen@rackspace.com> Adjust setup.py to match nova-rsapi -> nova-api-new rename. 2010-08-30 Soren Hansen <soren.hansen@rackspace.com> Reconnect to libvirt on broken connection. 2010-08-30 jaypipes@gmail.com pylint fixes for /nova/virt/connection.py ##########################################	Soren Hansen	13 years ago
265	Moved API tests into a sub-folder of the tests/ and added a ... Moved API tests into a sub-folder of the tests/ and added a stubbed-out test declarations to mirror existing API tickets. Resubmission from earlier because of my improper file move.	Cerberus	13 years ago
264	Call getInfo() instead of getVersion() on the libvirt connec... Call getInfo() instead of getVersion() on the libvirt connection object. virConnectGetVersion was not exposed properly in the python bindings until quite recently, so this makes us rather more backwards compatible.	Soren Hansen	13 years ago
263	Adjust setup.py to match nova-rsapi -> nova-api-new rename. Adjust setup.py to match nova-rsapi -> nova-api-new rename.	Soren Hansen	13 years ago
262	Reconnect to libvirt on broken connection. Reconnect to libvirt on broken connection.	Soren Hansen	13 years ago
261	pylint fixes for /nova/virt/connection.py pylint fixes for /nova/virt/connection.py	jaypipes at gmail	13 years ago
260	pylint fixes for nova/objectstore/handler.py pylint fixes for nova/objectstore/handler.py	jaypipes at gmail	13 years ago
259	Add Flavors controller supporting Add Flavors controller supporting GET /flavors GET /flavors/detail GET /flavors/<id> Also add GET /images/detail Turn the RackspaceAPIImageIdTranslator into a RackspaceAPIIdTranslator, so that it can be used to translate IDs for other rackspace API components as well (servers, backup schedules.) I thought I'd need it for flavors but it turns out flavors are so simple I could hard code their ids into compute.instance_types.INSTANCE_TYPES.	Michael Gundlach	13 years ago

Older »