summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Move default_instance_type into nova.computeMark McLoughlin2013-01-094-6/+10
| | | | | | | | Apart from the tests, this option is only used in nova.compute.instance_types so move it there. blueprint: scope-config-opts Change-Id: Iadf4011739e50455b7ad4214a778a02e244fef35
* Move osapi_compute_unique_server_name_scope to dbMark McLoughlin2013-01-092-5/+8
| | | | | | | This strange little option is only used within the DB layer. blueprint: scope-config-opts Change-Id: I10b0fef02a5fffd44094fe351cde683fc5c31442
* Move api_class options to where they are usedMark McLoughlin2013-01-095-10/+28
| | | | | | | | The compute_api_class, network_api_class and volume_api_class options are only used in their respective modules, so move them there. blueprint: scope-config-opts Change-Id: Ib0497f46b44c4b3af7319550cc64c09d629ea362
* Move manager options into nova.serviceMark McLoughlin2013-01-0912-27/+26
| | | | | | | | | | | | Move the compute_manager, console_manager, cert_manager, network_manager and scheduler_manager options into nova.service. Apart from the tests, these options are mostly just used in nova.service so it makes sense for them to live there. Also, metadata_manager already lives there. blueprint: scope-config-opts Change-Id: I180859817b57c081b83d6a6f075a6ff76bb0ef6d
* Move compute_topic into nova.compute.rpcapiMark McLoughlin2013-01-0910-12/+13
| | | | | | | | Even though the compute_topic opt is used outside nova.compute.rpcapi, it makes sense for the RPC module to "own" the topic option. blueprint: scope-config-opts Change-Id: I453951825481b9a5ba1865f6bf1223f7b1e25f98
* Merge "Move scheduler_topic into nova.scheduler.rpcapi"Jenkins2013-01-096-8/+10
|\
| * Move scheduler_topic into nova.scheduler.rpcapiMark McLoughlin2013-01-086-8/+10
| | | | | | | | | | | | | | | | The scheduler_topic opt is only used in nova.scheduler.rpcapi and it makes sense for the RPC module to "own" the topic option. blueprint: scope-config-opts Change-Id: Iafb8705eb716b650b63e060fe1dd36666bcee812
* | Merge "Move console_topic into nova.console.rpcapi"Jenkins2013-01-096-8/+10
|\|
| * Move console_topic into nova.console.rpcapiMark McLoughlin2013-01-086-8/+10
| | | | | | | | | | | | | | | | Even though the console_topic opt is used outside nova.console.rpcapi, it makes sense for the RPC module to "own" the topic option. blueprint: scope-config-opts Change-Id: I0f00927ca1c0bba7b21315240165e83e7075ebb7
* | Merge "Move network_topic into nova.network.rpcapi"Jenkins2013-01-094-6/+8
|\|
| * Move network_topic into nova.network.rpcapiMark McLoughlin2013-01-084-6/+8
| | | | | | | | | | | | | | | | The network_topic opt is only used in nova.network.rpcapi and it makes sense for the RPC module to "own" the topic option. blueprint: scope-config-opts Change-Id: Icd779f34df47ac6dd048b3e41995e872d8859872
* | Merge "Move cert_topic into nova.cert.rpcapi"Jenkins2013-01-094-6/+8
|\|
| * Move cert_topic into nova.cert.rpcapiMark McLoughlin2013-01-084-6/+8
| | | | | | | | | | | | | | | | The cert_topic opt is only used in nova.cert.rpcapi and it makes sense for the RPC module to "own" the topic option. blueprint: scope-config-opts Change-Id: If3584b8e1984e0425d9f33133f9d291739b04d49
* | Merge "Move global s3 opts into nova.image.s3"Jenkins2013-01-093-10/+9
|\|
| * Move global s3 opts into nova.image.s3Mark McLoughlin2013-01-083-10/+9
| | | | | | | | | | | | | | s3_host and s3_port are only used in nova.image.s3. blueprint: scope-config-opts Change-Id: Ia841406ebe7c03b2a58392e2453118e2646ad7ce
* | Merge "Verify the disk file exists before running qemu-img on it."Jenkins2013-01-094-0/+38
|\ \
| * | Verify the disk file exists before running qemu-img on it.Michael Still2012-12-254-0/+38
| | | | | | | | | | | | | | | | | | | | | Should resolve bug 955788, although it is a little hard to tell because the bug is so old. Change-Id: Ic0c47f4b6181f56a98cf58d4ebe2cc926d06d524
* | | Merge "attach/detach_volume() take instance as a parameter"Jenkins2013-01-0913-57/+46
|\ \ \
| * | | attach/detach_volume() take instance as a parameterArata Notsu2013-01-0813-57/+46
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously the methods take instance['name'] as a parameter. With this change, ComputeDriver can lookup informations about the instance from hypervisor (in bare-metal driver, it is a local DB) by any other attributes of the instance. blueprint general-bare-metal-provisioning-framework Change-Id: Ibd0567f34ed5053909ce1a408f9cbf87516ba597
* | | | Merge "Remove availability_zones from service table"Jenkins2013-01-0924-93/+291
|\ \ \ \
| * | | | Remove availability_zones from service tableJoe Gordon2013-01-0824-93/+291
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is the final step in enabling availability_zones using aggregate metadata. Previously all services had an availability_zone, but the availability_zone is only used for nova-compute. Services such as nova-scheduler, nova-network, nova-conductor have always spanned all availability_zones. After this change only compute nodes (nova-compute), will have an availability_zone. In order to preserve current APIs, when running: * nova host-list (os-hosts) * euca-describe-availability-zones verbose * nova-manage service list Internal services will appear in there own internal availability_zone (CONF.internal_service_availability_zone) Internal zone is hidden in euca-describe-availability_zones (non-verbose) CONF.node_availability_zone has been renamed to CONF.default_availability_zone and is only used by the nova-api and nova-scheduler. CONF.node_availability_zone still works but is deprecated DocImpact Completes blueprint aggregate-based-availability-zones Change-Id: Ib772df5f9ac2865f20df479f8ddce575a9ce3aff
* | | | | Merge "Import latest timeutils from oslo-incubator"Jenkins2013-01-091-5/+28
|\ \ \ \ \
| * | | | | Import latest timeutils from oslo-incubatorFlaper Fesp2013-01-081-5/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug #1097464 Changes synced are: a08daf1 Use basestring instead of str for type check. 615d769 Account for tasks duration in LoopingCall delay ce80aac Convenience wrapper for datetime.timedelta.total_seconds() Change-Id: If4ab7ccd2f4e815e01c41f25fdd541ffcd7693d5
* | | | | | Merge "Enable Aggregate based availability zones"Jenkins2013-01-0939-191/+355
|\ \ \ \ \ \ | | |/ / / / | |/| | | |
| * | | | | Enable Aggregate based availability zonesJoe Gordon2013-01-0839-191/+355
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead of implementing availability zones in the service table, availability zones are implemented using general aggregate metadata. This patch does not remove availability zones from the service table, a latter patch will do that. * In theory supports a single compute node in multiple availability zones * Drop availability_zone column from Aggregate table (is now a property) * map aggregate metadata 'availability_zone' so API does not change Implements blueprint aggregate-based-availability-zones Change-Id: I2a2ac5bfaa526d639dff5efa392c051347dbd9bb
* | | | | | Merge "Added sample tests to FlavorRxtx API."Jenkins2013-01-0917-1/+421
|\ \ \ \ \ \
| * | | | | | Added sample tests to FlavorRxtx API.Giampaolo Lauria2013-01-0817-1/+421
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Partially implements blueprint nova-api-samples Fixes bug 1091743 Change-Id: I400e627cfc499b3e7bb1c88d6ce7d24b4d222b84
* | | | | | | Merge "Move global glance opts into nova.image.glance"Jenkins2013-01-0911-69/+69
|\ \ \ \ \ \ \ | | |_|_|_|_|/ | |/| | | | |
| * | | | | | Move global glance opts into nova.image.glanceMark McLoughlin2013-01-0811-69/+69
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move the glance_host, glance_port and glance_protocol options into the nova.image.glance module. Also move the generate_glance_url() and generate_image_url() utility methods there since they're the only uses of the options outside of the module. blueprint: scope-config-opts Change-Id: I3beca1a65cdd902a94ddc1c512d79a4de635ef38
* | | | | | | Merge "Adds to manager init_host validation for instances location"Jenkins2013-01-0910-1/+396
|\ \ \ \ \ \ \
| * | | | | | | Adds to manager init_host validation for instances locationKravchenko Pavel2013-01-0810-1/+396
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While the compute was down the instances running on it could be evacuated to another host. The method checks that instance host identical to current host. Otherwise destroying it. part of blueprint rebuild-for-ha DocImpact Change-Id: Ic90df2b2887ee203e6d8261084e3f97773c5d81c Co-authored-by: Oshrit Feder <oshritf@il.ibm.com>
* | | | | | | | Merge "Clarify the DBApi object in cells fakes"Jenkins2013-01-091-2/+7
|\ \ \ \ \ \ \ \
| * | | | | | | | Clarify the DBApi object in cells fakesChris Behrens2013-01-081-2/+7
| | |_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds a docstring and makes instance_get_by_uuid() raise by default. It's only used as a stubbing point, but returning 'None' is not really valid and having it raise might save someone some confusion in the future. Change-Id: Ie3fa333288f9c0263ead69c2d307101b8583dea7
* | | | | | | | Merge "Add to libvirt driver instance_on_disk method"Jenkins2013-01-093-0/+27
|\ \ \ \ \ \ \ \ | | |/ / / / / / | |/| | | | | |
| * | | | | | | Add to libvirt driver instance_on_disk methodKravchenko Pavel2013-01-083-0/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added method to driver layer to validate that instance directory accessible from the host. part of libvirt driver support for blueprint rebuild-for-ha DocImpact Change-Id: I9c6a3aec99f61bd4fb6bc4b4264771e077aae96a Co-authored-by: Oshrit Feder <oshritf@il.ibm.com>
* | | | | | | | Merge "add to driver option to keep disks when instance destroyed"Jenkins2013-01-0912-75/+161
|\| | | | | | |
| * | | | | | | add to driver option to keep disks when instance destroyedKravchenko Pavel2013-01-0812-75/+161
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adding delete_disks flag to destroy instance method. In case it's false the driver keeps disks. *required for blueprint rebuild-for-ha DocImpact Change-Id: I38c7833114d8414dff831c9cc0c04c4610dbd33a Co-authored-by: Oshrit Feder <oshritf@il.ibm.com>
* | | | | | | | Merge "Fix serialization in impl_zmq."Jenkins2013-01-091-9/+32
|\ \ \ \ \ \ \ \
| * | | | | | | | Fix serialization in impl_zmq.Russell Bryant2013-01-081-9/+32
| | |_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sync rpc from oslo-incubator to include some fixes to impl_zmq: https://review.openstack.org/#/c/18913/ Change-Id: I5f5c157e81026c108df04bd385776a9985d4a497
* | | | | | | | Merge "fix N401 errors, stop ignoring all N4* errors"Jenkins2013-01-0939-128/+130
|\ \ \ \ \ \ \ \
| * | | | | | | | fix N401 errors, stop ignoring all N4* errorsSean Dague2013-01-0739-128/+130
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We had previously been ignoring all our custom N4xx hacking.py errors. This fixes all the N401 errors "doc strings should not start with a space" and reduces the ignore set down to N402 only "single line docstrings should end with period". It also fixes the N401 parser to catch only docstrings, and not tripple quoted string blocks used later on in a function. Clean up a few of the more crazy uses of """ in our code Clean up additional funky comments to make indents a bit more consistent, and pull in lines when possible. Change-Id: I9040a1d2ca7efda83bd5e425b95d1408b5b63577
* | | | | | | | | Merge "Fix baremetal VIFDriver"Jenkins2013-01-081-2/+1
|\ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / |/| | | | | | | |
| * | | | | | | | Fix baremetal VIFDriverDevananda van der Veen2013-01-081-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit Id287f7e661 removed the nova/virt/vif.py file and the VIFDriver base class, and erroneously stated in its commit message that baremetal driver did not use it. This patch repeats the work of that earlier patch by removing the import of nova.virt.vif from baremetal/vif_driver.py. blueprint general-bare-metal-provisioning-framework Change-Id: Ifac9510a72a028dc5d3574148f3c4a6410b1d249
* | | | | | | | | Merge "Remove unused osapi_path option"Jenkins2013-01-081-3/+0
|\ \ \ \ \ \ \ \ \ | | |_|_|_|/ / / / | |/| | | | | | |
| * | | | | | | | Remove unused osapi_path optionMark McLoughlin2013-01-081-3/+0
| | |_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This has been unused since deprecated auth was removed. It looks like we intended to remove this in commit ea06eff7. Change-Id: I7a5868c42300ba92d5c8f528a447cfe3fbace297
* | | | | | | | Merge "Add api extension to get and reset password"Jenkins2013-01-0822-0/+356
|\ \ \ \ \ \ \ \
| * | | | | | | | Add api extension to get and reset passwordVishvananda Ishaya2013-01-0722-0/+356
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Password is stored in system_instance_metadata as four items with the keys 'password_0' through 'password_3'. The extension adds a resource endpoint with two methods: GET v2/servers/<uuid>/os-server-password # get_password DELETE v2/servers/<uuid>/os-server-password # reset_password get_password retuns: JSON: {"password": "xxx"} XML: <?xml version='1.0' encoding='UTF-8'?><password>xxx</password> Note that this is intended to be an encrypted password set by the guest. Includes api tests for json and xml. Also includes api samples. Part of blueprint get-password Change-Id: I6c03f9c3bc8a2c70403bbb3e57917ab9522d75bd
* | | | | | | | | Merge "Refresh instance metadata in-place"Jenkins2013-01-082-5/+31
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / |/| | | | | | | |
| * | | | | | | | Refresh instance metadata in-placeDan Smith2013-01-082-5/+31
| | |_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For some reason that isn't entirely clear to me, the attempt to update instance metadata in _instance_update() results in a stale sqlalchemy object state. This is a similar issue to the one fixed in commit: 9fdf7552779d518af9cda4e366bf81fddb0cb6f2 For some reason, session.refresh() doesn't work for instance['metadata'] in the same way, which also raises suspicion that perhaps it's not the right fix for system_metadata either. This patch adds an _instance_update_metadata_in_place() method that updates the actual instance['metadata'] list _and_ the database, mirroring the high-level behavior of instance_metadata_update(), but without requiring us to refresh or re-fetch the metadata or the whole instance (as a workaround). Fixes bug 1096653 Change-Id: Ic5a205631b1b7dce3744960ed4201dcc7b4b2ae6
* | | | | | | | Merge "Add exception handler for previous deleted flavor."Jenkins2013-01-083-1/+43
|\ \ \ \ \ \ \ \