summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Optimize db.instance_floating_address_get_all methodBoris Pavlovic2013-05-083-7/+44
| | | | | | | | | | | | | | | | Optimize db.instance_floating_address_get_all Was: Get FixedIps with corresponding instance_uuid For each FixedIp get associated FloatIp (N request) Now: Get only list of FixedIps ids Get all corresponding FloatingIp with one query (select in) Get only addresses from FloatingIps (because we are using only it) Add temporary test for db.instance_floating_address_get_all to ensure that it works as expected. Change-Id: Id3c515de0ca35707bbfe46b991c2d7cf591e431b
* Session cleanup for db.floating_ip_* methodsBoris Pavlovic2013-05-082-12/+7
| | | | | | | | | | | | | | | | | | | | Session objects shouldn't be used in public db.api methods because db.api should be independent of db implementation. Remove session parameter from: floating_ip_create floating_ip_get_by_fixed_ip_id I didn't remove session parameter from floating_ip_count_by_project because it causes changes in nova.quota. So it will be done in separated patch. Cleanup test for floating_ip_count_by_project removed 3rd parameter (it should be session, but it is not required). blueprint db-session-cleanup Change-Id: Ifdf9edb17746a8e5b486317c917ea69016a2b99d
* Merge "Transition from openstack.common.setup to pbr."Jenkins2013-05-087-445/+77
|\
| * Transition from openstack.common.setup to pbr.Monty Taylor2013-05-077-445/+77
| | | | | | | | | | | | | | | | | | | | | | Declarative approach to packaging, obviates the need for copying files. Also, removes the need for setup.py to import files from the nova module. ./run_tests.sh now ensure's that nova.egg-info/entry_points.txt is up to date before running tests. Change-Id: I7b7a18e065a62491ade54fbed5d8400db1d4e6c7
* | Merge "libvirt: ignore NOSTATE in resume_state_on_host_boot() method."Jenkins2013-05-081-0/+1
|\ \
| * | libvirt: ignore NOSTATE in resume_state_on_host_boot() method.Yufang Zhang2013-05-061-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Occasionally, libvirt would get NOSTATE from hypervisor as instance state. Thus if we don't ignore instances in NOSTATE in resume_state_on_host_boot() method, we would mistakenly reboot such instances when we restart nova-compute service or on host boot. This patch fixes this by adding NOSTATE into ignored_states. Bug 1176897 Change-Id: I651cb043d7f97fde29b2a62e6ead8222062f01f7
* | | Merge "Add an index to compute_node_stats"Jenkins2013-05-081-0/+37
|\ \ \
| * | | Add an index to compute_node_statsBrian Elliott2013-05-041-0/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will improve the performance of scheduler lookups of compute nodes and their associated stats. bug 1177487 Change-Id: I0e04849543916e874ea0ddfc76c3d70ff71c09d0
* | | | Imported Translations from TransifexOpenStack Jenkins2013-05-0723-3396/+2025
|/ / / | | | | | | | | | Change-Id: Ic42793b5844219cdc2dfed82a5e3b8aad0d88136
* | | Merge "Nova evacuate failed when VM is in SHUTOFF status"Jenkins2013-05-072-1/+19
|\ \ \
| * | | Nova evacuate failed when VM is in SHUTOFF statusYang Yu2013-05-072-1/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When the VM is in SHUTOFF status and the original compute host is down, issue the "nova evacuate" command to rebuild the VM on another host, it is failed due to a type error: string indices must be integers. Fix bug 1176773 Change-Id: I76ebce6882198365717167f29a484e4ec994db0c
* | | | Merge "Remove security_group_handler"Jenkins2013-05-079-263/+5
|\ \ \ \ | |/ / / |/| | |
| * | | Remove security_group_handlerAaron Rosen2013-05-069-263/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now that nova's security groups are more plugable and decoupled from the database the security_group_handler code is not very useful so this patch removes it. A little info about the security_group_handler: The security_group_handle code was added to provide a hook into nova security groups so that one could get security group add/delete/update notification and proxy them somewhere else (i.e quantum). Trying to actually using this method opens one up to several transactional issues because in the current implemenation the security group is commited to the nova database before security_group_handler is called. Implements blueprint: remove-security-group-handler Change-Id: I45047fe8dbb81555505e03309838910113080c37
* | | | Merge "Remove random print statements"Jenkins2013-05-074-5/+0
|\ \ \ \
| * | | | Remove random print statementsAaron Rosen2013-05-064-5/+0
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | Noticed a random print statement in the code and then grep around for the rest. Might as well remove these. Change-Id: Ieb2d72cf55cbf077c2e824d430707211d1f87b8e
* | | | Merge "Imported Translations from Transifex"OpenStack Jenkins2013-05-0723-26644/+38949
|\ \ \ \ | |_|_|/ |/| | |
| * | | Imported Translations from TransifexOpenStack Jenkins2013-05-0623-26644/+38949
| | | | | | | | | | | | | | | | Change-Id: I870caf5258a16e00134d22d1e6757b1e3deab41d
* | | | Merge "Sync oslo-incubator print statement changes"Jenkins2013-05-073-17/+23
|\ \ \ \
| * | | | Sync oslo-incubator print statement changesMatthew Sherborne2013-05-063-17/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Just changes openstack.common to use the new print statements. This is in preparation for another sync coming shortly for the scheduler library. Change-Id: I17a86ef87f3253d6bb756cbd40a3ff5df9be23ff
* | | | | Merge "Delete InstanceSystemMetadata on instance deletion"Jenkins2013-05-073-25/+13
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Delete InstanceSystemMetadata on instance deletionJoe Gordon2013-05-023-25/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Thanks to no-db-compute only pre-delete instance data is used in notifications. So now we can go back to deleting InstanceSystemMetadata when an instance is deleted. Fixes bug 1153827 Change-Id: Ic66b998cde2a15a24f302f08c34753a8b57da73d
* | | | | Remove referances to LegacyFormatter in example logging.conf.Tim Miller2013-05-061-7/+7
| |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A recent change (Ibe41afc6e9d7b432453785819821b3c8f0078613), removed nova.openstack.common.log.LegacyFormatter, and replaced it with ContextFormatter. This change updates the example logging config file to reflect this change, as any reference to the now-removed LegacyFormatter causes nova services to fail to start. Change-Id: I55a984ed7ea12c1dde49c4eb13d5b4f47471bf95
* | | | Merge "xenapi: Always set other_config for VDIs"Jenkins2013-05-063-40/+251
|\ \ \ \
| * | | | xenapi: Always set other_config for VDIsRick Harris2013-05-023-40/+251
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The existing code only set `other_config` on a VDI when it was created using `create_vdi`. This patch updates the code so that `other_config` is also set when a VDI is created from a dom0 plugin or during a migration. Also included is a one-time script to set the other_config for existing VDIs that were affected by this bug. Fixes bug 1162029 Change-Id: I4fa856754487f77ce9c8e39d45eee7ea5187123e
* | | | | Merge "Convert to using oslo's execute() method."Jenkins2013-05-0628-268/+97
|\ \ \ \ \
| * | | | | Convert to using oslo's execute() method.Michael Still2013-05-0428-268/+97
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are some small warts (mostly around other execute() like methods which reuse the exception). I will fix these warts in later reviews. Change-Id: Ice9cdbdc5f3e5a9f8365f5d99acf1863a9fe3e7a
* | | | | | Merge "Fix stub_instance() to include missing attributes"Jenkins2013-05-061-1/+8
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | |
| * | | | | Fix stub_instance() to include missing attributesDan Smith2013-05-041-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The stub_instance() function aims to create a fake instance dict, but omits several attributes present on the real database object. This patch adds them. While they're not used anywhere (obviously) at the moment, some WIP code I'm working on expects them to look like the DB object, and thus fails. Change-Id: I71fb596f8b0a80de21ee58ef378440da8eea1777
* | | | | | Merge "Import latest log module from oslo"Jenkins2013-05-052-7/+12
|\ \ \ \ \ \
| * | | | | | Import latest log module from osloAndrew Laski2013-05-032-21/+47
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Deprecates log_format in favor of the formatting options used by the context aware formatter, such as logging_default_format_string. Changes the default logging formatter back to a colorful and context aware formatter. The default was inadvertently changed in the great oslo sync at the start of Havana, this returns it to what was in at the end of Grizzly. Adds support for lazy instantiation of the logger. Bug 1167081 Bug 1165211 Change-Id: Ibe41afc6e9d7b432453785819821b3c8f0078613
* | | | | | | Merge "Make nova-network support requested nic ordering"Jenkins2013-05-052-27/+87
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | |
| * | | | | | Make nova-network support requested nic orderingZhi Yan Liu2013-05-032-27/+87
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The nova-network did not honor nic ordering, so nics were attached in db natural order. This patch sorts the networks honor requested nic ordering when Nova calls the nova-network API. Fixes bug: #1170725 Change-Id: I7dd04ac7bdda95c626cbb0fe35be0aadacb4bd2a Signed-off-by: zhiyanl-cn <zhiyanl@cn.ibm.com>
* | | | | | Merge "vmwareapi: Add supported_instances to host state"Jenkins2013-05-052-0/+5
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | vmwareapi: Add supported_instances to host stateJim Fehlig2013-05-022-0/+5
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The VMWareVCDriver uses the VCState class to report host state, but VCState.update_status() does not include supported_instances. Add supported_instances to the data returned by update_status(), similar to HostState.update_status(). Change-Id: I15c0f930dd9e540f6fdb0d7246661a0c98e35ed4
* | | | | Merge "Update hypervisor_hostname after live migration"Jenkins2013-05-042-28/+59
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Update hypervisor_hostname after live migrationBen Nemec2013-05-032-28/+59
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After a live migration, the OS-EXT-SRV-ATTR:hypervisor_hostname attribute of the instance is not updated correctly. This change adds a parameter to the call to _instance_update that fixes the problem. Fixes bug 1173376 Change-Id: I40f1ead2d93c7a9979a914e0fb96e3e5bb287b48
* | | | | Merge "Being more defensive around the use_ipv6 config option"Jenkins2013-05-041-2/+3
|\ \ \ \ \
| * | | | | Being more defensive around the use_ipv6 config optionDavid Peraza2013-05-031-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fixes bug 1175725 Will allow for the flag to be set to filter out ipv6 if configured but will not add ipv6 to template if ipv6 is not configured in quantum. This will allow spwan to continue working in the case where ipv6 is not configured but flag use_ipv6 is set in nova. Change-Id: I26ebdff81efcdeff827521cec247d2c443a5e0ad
* | | | | | Merge "Fix EC2 instance bdm response"Jenkins2013-05-034-16/+17
|\ \ \ \ \ \
| * | | | | | Fix EC2 instance bdm responseMotoKen2013-05-014-16/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make the bdm response respect Amazon EC2 spec: * use ec2 volume id instead of uuid * use volume attach_status instead of status References: http://docs.aws.amazon.com/AWSEC2/latest/APIReference/ApiReference-ItemType-EbsInstanceBlockDeviceMappingResponseType.html Resolves bug 1165973. Change-Id: Ib68397b79f63e01e92ed4bebbc245ba5e0a3ad11
* | | | | | | Merge "Adds --addn-hosts to the dnsmasq arg list"Jenkins2013-05-031-1/+1
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | |
| * | | | | | Adds --addn-hosts to the dnsmasq arg listdavid martin2013-05-021-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When using multi_host=True and update_dns_entries=True, dns records from other compute hosts were not able to be resolved. The --addn-host arg was constructed but not appended to the cmd list. Change-Id: I9cafb59256e92bdad0df3e9166e0afd079ae0c09 Fixes: bug #1170529
* | | | | | | Merge "Update our import of oslo's processutils."Jenkins2013-05-031-12/+52
|\ \ \ \ \ \ \
| * | | | | | | Update our import of oslo's processutils.Michael Still2013-05-031-12/+52
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I've just synced nova's processutils back to oslo and this is now the sync back in the other direction. Change-Id: I8389bdd85ed3dacbb02aec043dfd0f84ef07b18b
* | | | | | | Merge "Update oslo-incubator import."Jenkins2013-05-035-29/+100
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | |
| * | | | | | Update oslo-incubator import.Michael Still2013-05-035-29/+100
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These are all minor changes bringing nova up to commit 90e83530d4dc49d570fa05ea63a93805717dcfa0 in oslo-incubator. Change-Id: I0291eed31b1e650da211fe2a8b65fad0c35c9053
* | | | | | Merge "fix broken WSDL logic"Jenkins2013-05-033-16/+85
|\ \ \ \ \ \
| * | | | | | fix broken WSDL logichartsocks2013-05-023-16/+85
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | vSphere driver incorrectly assumes that a WSDL is locally required. A local WSDL is only needed as a work-around for certain situations. Fixes bug 1171215 Change-Id: Id8c5fc104c244ba768c58b97afdc26703c5c42ac
* | | | | | | Merge "nova coverage creates lots of empty folders"Jenkins2013-05-031-1/+3
|\ \ \ \ \ \ \
| * | | | | | | nova coverage creates lots of empty foldersYang Yu2013-05-031-1/+3
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With nova coverage extension disabled as a default behaviour, nova will create lots of empty folders under /tmp folder. And also with each time the nova service reboot, it will create a new directory with name like nova_coverage_*. There will be number of the empty folders accumulated. Patch-Set 3: Rebase for merging code Fixes: bug 1169005 Change-Id: I26d1037eb66d781ac6ef6b600cca04a709bff3f7