summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | | | | | | | | | | | | | Add REST API to show availability_zone of instance.gtt1162013-02-0334-24/+715
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement one workitem for bp:show-availability-zone This show the availability zone and availability zone of its host in API: GET /servers/details, GET /server/{uuid}. When create instance if "availability_zone" doesn't specified, the "availability_zone" of the instance will be None. But actually the instance is in a zone which its host is in. So I think show both of them will be fine. Change-Id: I2d742ee2d291b514bf783fa79293785cca8ea2a5
* | | | | | | | | | | | | | | | | | | | Merge "Fix import for install_venv.py"Jenkins2013-02-051-0/+3
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / / / / / / / / / |/| | / / / / / / / / / / / / / / / / / | | |/ / / / / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | Fix import for install_venv.pyMatthew Treinish2013-02-041-0/+3
| | |_|_|_|_|_|/ / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On systems that don't use devstack the nova directory doesn't automatically get added to the python path. This breaks tools/install_venv_common.py which imports from nova.openstack.common. This fixes the issue by manually adding the nova root directory to the python path before tools/install_venv_common.py is imported. Change-Id: I8a3df02a51c2c777d8f47cb8bc72d0644ae47095
* | | | | | | | | | | | | | | | | | Merge "Allow for specifying nfs mount options"Jenkins2013-02-051-2/+11
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | Allow for specifying nfs mount optionsRafi Khardalian2013-02-031-2/+11
| | |_|/ / / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1113042 Adds a new config option: nfs_mount_options=None (Default) When not None, anything set here will be passed as -o <options> to the mount command. The default behavior is to do exactly as we are doing today and relying on the OS/kernel defaults. Change-Id: I935ae6f4d96c20904b7132fe8c6dfd3cbe089e51 Flags: DocImpact
* | | | | | | | | | | | | | | | | | Make get_dev_name_for_instance() use stashed instance_type infoDan Smith2013-02-041-3/+1
| |_|_|_|_|_|_|_|_|_|_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Related to blueprint no-db-compute Change-Id: I6a8e357b69e6cfbdfa5be846b74baf6df3b8eae4
* | | | | | | | | | | | | | | | | Merge "Remove remaining instance_types query from compute/manager"Jenkins2013-02-051-2/+1
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | Remove remaining instance_types query from compute/managerDan Smith2013-02-041-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Related to blueprint no-db-compute Change-Id: Ib4bf1731e408821f727f70dcc77b9883504ba960
* | | | | | | | | | | | | | | | | | Merge "Make cells_api fetch stashed instance_type info"Jenkins2013-02-051-2/+2
|\| | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | Make cells_api fetch stashed instance_type infoDan Smith2013-02-041-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Related to blueprint no-db-compute Change-Id: If87bdd6a29d511117ce5d58057e601a155c64959
* | | | | | | | | | | | | | | | | | Merge "Teach resource tracker about stashed instance types"Jenkins2013-02-052-23/+38
|\| | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | Teach resource tracker about stashed instance typesDan Smith2013-02-042-23/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This removes instance_type-related database queries from the resource_tracker. Related to blueprint no-db-compute Change-Id: Ia5dd407181fa733cc81d3ce03edc1c06c2aae762
* | | | | | | | | | | | | | | | | | Merge "Fix up instance types in sys meta for resizes"Jenkins2013-02-058-32/+200
|\| | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | Fix up instance types in sys meta for resizesDan Smith2013-02-048-32/+200
| | |/ / / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds code to the resize paths to update the system_metadata with new (or reverted) flavor information so that resized instances are consistent with normal ones. It also stashes the new and old instance_type information in system_metadata during resize, which allows us to avoid database lookups for type information at finish, confirm, and revert time. Related to blueprint no-db-compute Change-Id: I7f74ef8131be7a60e3a844fdf00fdeb3683f1317
* | | | | | | | | | | | | | | | | Merge "Fix incorrect device name being raised."Jenkins2013-02-053-26/+38
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | Fix incorrect device name being raised.Rick Harris2013-02-043-26/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `device` was being used as an index variable in a for-loop, overwriting the value that was passed in, causing the in-use exception to point to the wrong device. The solution is to use a different variable name, in this case `device_path` as the index variable. Other cleanups include: * Update to test to ensure exception message points to correct device * Simplify for-loop by using set to collect letters, instead of a list which is later turned into a set * Added blank lines to improve readability in particularly dense sections * Made `req_letters` and `_get_unused_letters` singular because they, in fact, are. Change-Id: Iad33f7ff0c109c6cea5df84d54c11cee750bf19d
* | | | | | | | | | | | | | | | | | Merge "VMware VC Compute Driver"Jenkins2013-02-059-111/+296
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | VMware VC Compute DriverSean Chen2013-02-049-111/+296
| | |/ / / / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | blueprint vmware-compute-driver Add VMwareVCDriver for VMware vCenter Server and Cluster support DocImpact Nova conf: compute_driver=vmwareapi.VMwareVCDriver vmwareapi_cluster_name=VMware cluster name Change-Id: Ia8bd40a912971ceb20b0f8015a5fcc42e525828d
* | | | | | | | | | | | | | | | | | Merge "Record instance actions and events"Jenkins2013-02-0515-42/+579
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | Record instance actions and eventsAndrew Laski2013-02-0115-42/+579
| | |_|_|_|_|_|_|_|_|_|_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Record when an action is initiated on an instance, and the underlying events related to completing that action. Actions will typically occur at the API level and should match what a user intended to do with an instance. Events will typically track what happens behind the scenes and may include things that would be unclear for a user if exposed, but should be beneficial to an admin/deployer. Adds a new wrapper to the compute manager. The wrapper will record when an event begins and finishes from the point of view of the compute manager. It will also record errors if they occur. Blueprint instance-actions Change-Id: I801f3e796d091e146413f84c2ccfab95ad2e1af4
* | | | | | | | | | | | | | | | | | Merge "Adds evacuate method to compute.api"Jenkins2013-02-054-0/+144
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | Adds evacuate method to compute.apiKravchenko Pavel2013-02-044-0/+144
| | |/ / / / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added decorator to check whether the vm is suitable to be evacuated. The decorator validates that vm host is down. The new evacuate method calls rebuild with the recreate flag and specified target host. This implements blueprint rebuild-for-ha DocImpact Change-Id: I877b29928c922fa366fb85deb85ddfc72d97daf8 Co-authored-by: Oshrit Feder <oshritf@il.ibm.com>
* | | | | | | | | | | | | | | | | | Merge "Merge LibvirtOpenVswitchDriver class into LibvirtGenericVIFDriver"Jenkins2013-02-042-35/+84
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | Merge LibvirtOpenVswitchDriver class into LibvirtGenericVIFDriverDaniel P. Berrange2013-01-302-35/+84
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The LibvirtGenericVIFDriver class can use the 'vif_type' mapping field to determine whether an OVS network configuration is required, thus a dedicated driver for OVS is no longer required. The LibvirtOpenVswitchDriver class functionality is merged into LibvirtGenericVIFDriver. For backwards compatibility with the Folsom release, the existing LibvirtOpenVswitchDriver class is made to inherit from LibvirtGenericVIFDriver and directly call the bridge specific setup APIs. This eases migration to the new VIF impl during the Grizzly deployment lifecycle, with an expectation that the LibvirtOpenVswitchDriver stub will be deleted after the Havana release. Blueprint: libvirt-vif-driver Change-Id: Iac79e0880b6fbcca0fd8cc26b59d23e579b947fa Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
* | | | | | | | | | | | | | | | | | | Merge "Merge QuantumLinuxBridgeVIFDriver into LibvirtGenericVIFDriver"Jenkins2013-02-042-45/+50
|\| | | | | | | | | | | | | | | | | | | |/ / / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | Merge QuantumLinuxBridgeVIFDriver into LibvirtGenericVIFDriverDaniel P. Berrange2013-01-302-45/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The only difference between the Quantum bridge driver and the new generic VIF driver bridge support is the name of the bridge. This is now provided via the VIF mapping parameter, so there is no need for the dedicated Quantum VIF driver for bridging. This This has the advantage that libvirt firewall filter parameters are correctly configured. The QuantumLinuxBridgeVIFDriver class functionality is merged into LibvirtGenericVIFDriver. For backwards compatibility with the Folsom release, the existing QuantumLinuxBridgeVIFDriver class is made to inherit from LibvirtGenericVIFDriver and directly call the bridge specific setup APIs. This eases migration to the new VIF impl during the Grizzly deployment lifecycle, with an expectation that the QuantumLinuxBridgeVIFDriver stub will be deleted after the Havana release. Blueprint: libvirt-vif-driver Change-Id: I19664f20e5137f495513999214f66279292bda58 Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
* | | | | | | | | | | | | | | | | | Merge "Optimize floating ip list to make one db query"Jenkins2013-02-044-46/+37
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | Optimize floating ip list to make one db queryVishvananda Ishaya2013-02-024-46/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently the floating ip code will make 2 db queries for every single associated floating ip when list is called. This adds a couple of joins in the db layer to avoid having to make so many calls. This dramatically improves floating-ip-list. On a simple one node test the time dropped from 2.35 seconds down to 0.5 seconds for a list with 10 associated floating ips. Part of blueprint optimize-nova-network Change-Id: I0571013393b2dbad42c15e690c7783d5ceecaeb2
* | | | | | | | | | | | | | | | | | | Merge "Update HACKING.rst per recent changes"Jenkins2013-02-043-5/+12
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | | Update HACKING.rst per recent changesZhongyue Luo2013-02-043-5/+12
| | |_|_|_|/ / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added "is not" usage with examples * https://review.openstack.org/#/c/20865/ Fixed "not in" usage description * https://review.openstack.org/#/c/20875/ Fixed some "not X in Y" corner cases. Change-Id: I7534ef73e6fd525fd8f4bee594a4b37524699c08
* | | | | | | | | | | | | | | | | | | Merge "Mark password config options with secret"Jenkins2013-02-045-6/+12
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|/ / / / / / / / / / / |/| | | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | | Mark password config options with secretMichael J Fork2013-02-015-6/+12
| | |_|_|_|/ / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Config object supports masking values when writing out if the secret flag is set on the option definition. This change flags all nova options containing a password. Change-Id: I538c93bac736c317822b5d78b53c4181b2009b49
* | | | | | | | | | | | | | | | | | Fix incorrect logs in networkWangpan2013-02-042-3/+6
| |/ / / / / / / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These are negligible typo issues, so a bug isn't filed on LP. We should log floating ip address and details of networks, rather than "|%s|" or "|[<nova.db.sqlalchemy.models.Network object at 0x5227510>]|", so moving the log of floating ip address to the correct location and converting the sqlalchemy Network object to dict for logging. Change-Id: I815bf01100d234401114bfd80fd9aa644d1a3447
* | | | | | | | | | | | | | | | | Merge "Reimplement is_valid_ipv4()."Jenkins2013-02-032-12/+10
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | Reimplement is_valid_ipv4().Russell Bryant2013-02-022-12/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch reimplements the is_valid_ipv4() function is nova.utils. Instead of open-coding the validity check, just make use of the netaddr module, which is already used elsewhere in nova.utils. Also add a unit test for this code. Part of indigoprint bored-on-an-airplane. Change-Id: Ia89cbbd94a0ac4631d794d658b53c244237e1ca2
* | | | | | | | | | | | | | | | | | Merge "Tweakify is_valid_boolstr()."Jenkins2013-02-032-5/+15
|\| | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | Tweakify is_valid_boolstr().Russell Bryant2013-02-022-5/+15
| | |_|_|_|_|_|_|/ / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This function used a few backslashes for continuation. According to HACKING, we prefer to not use this. This code gets rid of them by changing the way the checks are done. Look at those backslashes ... sitting there at the end of the line. It's like they're looking down on the code that precedes them, as if they are somehow superior to the other characters. Banish them from this utility function! Also add a unit test for this code. Part of greenprint bored-on-an-airplane. Change-Id: If88b6e63ab078916ce1b9cf2f5e99623c402996c
* | | | | | | | | | | | | | | | | Merge "Make system_metadata update in place"Jenkins2013-02-033-9/+43
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | Make system_metadata update in placeDan Smith2013-02-013-9/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes the system_metadata updated in-place like regular metadata during instance_update(). It also modifies that function to avoid actually removing the item from the list, which will cause it to be deleted from the database when the session ends. The soft_delete is sufficient for our purposes. Related to blueprint no-db-compute Change-Id: I0da28ec485dc7850d246dab0a2bb95ae10c05c3e
* | | | | | | | | | | | | | | | | | Make NFS mount hashes consistent with CinderRafi Khardalian2013-02-031-2/+2
| |_|_|_|_|_|/ / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cinder is using a different hash method for mount point uniqueness. This change makes Nova's hash method identical. Change-Id: I64e0200ae341fa90cf66b86fa3cb11ad02e773c6
* | | | | | | | | | | | | | | | | Imported Translations from TransifexOpenStack Jenkins2013-02-032-1926/+12219
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ideb135261b0f9b2c32b118e7dd033809a14acdcd
* | | | | | | | | | | | | | | | | Merge "Relax API restrictions around the use of reboot"Jenkins2013-02-021-2/+8
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|_|_|_|/ / / / |/| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | Relax API restrictions around the use of rebootRafi Khardalian2013-01-261-2/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1101082 bug 982108 Reboots are the only sledge hammer users have available to resolve issues with their instances without administrative intervention. This patch modifies the API policy to allow reboot calls to be made in many more power and task states. Change-Id: Ia8702448f6b7b863da40e4d498f2e2ee0a12882e
* | | | | | | | | | | | | | | | | Merge "Add option to control where bridges forward"Jenkins2013-02-021-16/+31
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | Add option to control where bridges forwardVishvananda Ishaya2013-01-291-16/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds a new config option to control where bridges can forward. The default behavior is unchanged, but if you specify: forward_bridge_interface=foo Then only traffic on the same bridge or going to and from the foo interface will be allowed. This is useful in vlan mode when a deployer doesn't want traffic to be routed between vlans. Note that forward_bridge_interface can be specified multiple times and should be specified once for each interface that supports floating ips. i.e.: forward_bridge_interface=eth0.10 forward_bridge_interface=eth0.20 DocImpact Change-Id: I82d6b288b60cd6deae64690b55546730b1c2da55
* | | | | | | | | | | | | | | | | | Merge "instance.update notifications don't always identify the service"Jenkins2013-02-024-8/+14
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / / / / / / / / / / |/| | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | instance.update notifications don't always identify the servicePhil Day2013-01-314-8/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | instance_update in conductor/manager.py doesn't identify the service when calling notifications.send_update(), resulting in mesages on the notification queue having a service value of None. This change makes sure that the correct service is identified even when running conductor in local mode Change-Id: I1b22c0b46040f51ba715a4d9fa19ef4c98359070
* | | | | | | | | | | | | | | | | | Merge "import tools/flakes from oslo"Jenkins2013-02-022-12/+5
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | import tools/flakes from osloJoe Gordon2013-01-312-12/+5
| | |_|_|/ / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Id6d99fe1fd39e8c061f591e9a82058aea9664674
* | | | | | | | | | | | | | | | | | Merge "Condense multiple authorizers into a single one."Jenkins2013-02-021-12/+6
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \