summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | | | XCP-XAPI version fixMate Lakat2012-08-213-14/+97
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1033933. On an Kronos installation (Ubuntu + XCP-XAPI), the reported software_version dictionary did not contain both product_version and product_brand. This caused a KeyError. With this fix, the session's product_version and product_brand will default to None in such cases. The resize operation's name in these cases will default to VDI.resize. Change-Id: Ic0db4128a7b27c08c24461562423bcecd1373e58
* | | | | | | | | | Merge "Implement network creation in compute API"Jenkins2012-08-248-90/+195
|\ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / |/| | | | | | | | |
| * | | | | | | | | Implement network creation in compute APIAlessio Ababilov2012-08-218-90/+195
| | |_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implements blueprint os-api-network-create The ability to create new networks is currently only exposed by the nova-manage CLI. Here we add support for network creation in the os-networks API extension. With the exception of num_networks and network_size, all the parameters supported by 'nova-manage network create' are supported. Only a single network may be created by each API call. To avoid code duplication, the nova-manage code is refactored and moved into NetworkManager so that it can be re-used by the API. DocImpact Change-Id: I682d498ab35ea43b553b64e13e677fe9eeb8e08b
* | | | | | | | | Merge "Remove unused instance id-to-uuid function"Jenkins2012-08-234-27/+0
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / |/| | | | | | | |
| * | | | | | | | Remove unused instance id-to-uuid functionMark McLoughlin2012-08-234-27/+0
| | |_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This has been unused since commit 0d9d2487e2. Change-Id: I5a3171d566da60e1d4aaadc8a442e88103501ad6
* | | | | | | | Merge "Allow generate_sample.sh to be run from toplevel dir"Jenkins2012-08-233-8/+17
|\ \ \ \ \ \ \ \
| * | | | | | | | Allow generate_sample.sh to be run from toplevel dirMark McLoughlin2012-08-233-8/+17
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I hit an issue where extract_opts.py couldn't import importutils since it's not in my sys.path, so it's clear we need to set PYTHONPATH before running. The whole business of running the tool from the tools/conf dir is fairly awkward, so change it to run from the top-level dir instead. Change-Id: I89b4d874a6ee3732234a9704a8ee6001a293ed71
* | | | | | | | Merge "Let admin list instances in vm_states.DELETED"Jenkins2012-08-232-0/+32
|\ \ \ \ \ \ \ \
| * | | | | | | | Let admin list instances in vm_states.DELETEDJian Wen2012-08-232-0/+32
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Command 'nova list --status DELETED' returns nothing at the moment. Make 'nova list --status DELETED' work and work for admin only. Fixes LP Bug #1037885 Change-Id: I0bace95df713a767bb17e470e66361e1891e628c
* | | | | | | | Merge "Re-work the handling of firewall_driver default"Jenkins2012-08-238-30/+37
|\ \ \ \ \ \ \ \ | |_|_|_|_|/ / / |/| | | | | | |
| * | | | | | | Re-work the handling of firewall_driver defaultMark McLoughlin2012-08-238-30/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug #1040430 We have a different default firewall driver for libvirt and xenapi, yet the sample config file currently contains: firewall_driver=nova.virt.firewall.IptablesFirewallDriver In the case of libvirt, it should actually be: firewall_driver=nova.virt.firewall.libvirt.IptablesFirewallDriver This is really easy for users to get confused about. Since we don't have a different sample config file for each hypervisor, the best we can do is to just not include the default in the sample config and have each hypervisor supply its default at runtime. DocImpact: update nova.conf docs Change-Id: Ie78371bcceac5a65978d695934e0246022f748a3
* | | | | | | | Merge "Provide a hint for missing EC2 image ids"Jenkins2012-08-232-1/+8
|\ \ \ \ \ \ \ \ | |_|_|_|_|_|_|/ |/| | | | | | |
| * | | | | | | Provide a hint for missing EC2 image idsMichael Still2012-08-232-1/+8
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | EC2 image ids are allocated when the images are listed for the first time. Remind users of this if they specify an image id which does not exist. Resolves bug 961451. Change-Id: Id7326ba32e58051dbbf25c3c940fd0c9cc611ff3
* | | | | | | Merge "Allows libvirt to set a serial number for a volume"Jenkins2012-08-233-1/+33
|\ \ \ \ \ \ \
| * | | | | | | Allows libvirt to set a serial number for a volumeVishvananda Ishaya2012-08-163-1/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The serial number defaults to the volume_id of the volume being attached. We may expose a method in the future to set a different serial number when creating or attaching a volume. The purpose of this change is to give users a consistent place they can find their volume. It should show up now in most flavors of linux under /disk/by-id/virtio-<volume_uuid> Fixes bug 1004328 Change-Id: Id1c56b5b23d799deb7da2d39ae57ecb48965c55f
* | | | | | | | Merge "Include CommonConfigOpts options in sample config"Jenkins2012-08-232-8/+51
|\ \ \ \ \ \ \ \
| * | | | | | | | Include CommonConfigOpts options in sample configMark McLoughlin2012-08-232-8/+51
| | |_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Options defined by CommonConfigOpts are declared within the class rather than at module level, so they weren't being included in the sample conf. Also, in essex (and up until commit 991614add8), the sample conf file didn't contain these options but they included the hyphen in the name. The hyphen is only used on the command line, it is converted to an underscore automatically in the config file. Use opt.dest rather than opt.name as the config file key. Fixes bug #1034970. DocImpact: update nova.conf docs Change-Id: Ia7f3dded9148deedeb198c19a8d343db6dd93f99
* | | | | | | | Merge "Re-generate nova.conf.sample"Jenkins2012-08-231-226/+346
|\| | | | | | |
| * | | | | | | Re-generate nova.conf.sampleMark McLoughlin2012-08-231-226/+346
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It looks like this hasn't been regenerated in a looong time. DocImpact: update nova.conf docs Change-Id: I64832ca79b3456e64237bfa0b1b130201a3f678b
* | | | | | | Merge "Ignoring *.sw[op] files"Jenkins2012-08-231-0/+1
|\ \ \ \ \ \ \
| * | | | | | | Ignoring *.sw[op] filesFlavia Missi2012-08-161-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I3a059106a32f36374da787bc36a3721463189e6d
* | | | | | | | Merge "continue deleting instance even if quantum port delete fails"Jenkins2012-08-231-5/+2
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | |
| * | | | | | | continue deleting instance even if quantum port delete failsDan Wendlandt2012-08-221-5/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bug 1040385 Change-Id: Idb289437e40c7a1522b3a3f34f6c14ffe420c077
* | | | | | | | Merge "Update sqlite to use PoolEvents for regexp."Jenkins2012-08-231-11/+10
|\ \ \ \ \ \ \ \
| * | | | | | | | Update sqlite to use PoolEvents for regexp.Alex Meade2012-08-151-11/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use PoolEvents instead of PoolListener for adding regexp function. Fixes bug 1037219 Change-Id: I205ecce133f628e995f38dea5ce89ddf36b01a9b
* | | | | | | | | Merge "add availability_zone to openstack metadata"Jenkins2012-08-231-0/+1
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | add availability_zone to openstack metadataScott Moser2012-08-211-0/+1
| | |_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds the key 'availability_zone' to the metadata for openstack. 'availability-zone' in EC2 is very useful to the instance to figure out where it is. For example, Ubuntu uses this information to determine mirror location on EC2. Fixes bug 1039579. Change-Id: I212d3854b75e08a67138b5cd0bea178c47840e11
* | | | | | | | | Merge "Adds support for serial to libvirt config disks."Jenkins2012-08-232-0/+20
|\ \ \ \ \ \ \ \ \ | | |_|_|_|/ / / / | |/| | | | | | |
| * | | | | | | | Adds support for serial to libvirt config disks.Vishvananda Ishaya2012-08-162-0/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In order for users to find a volume that they have attached to a vm, it is valuable to be able to find it in a consistent location. A following patch wil accomplish this by setting the serial number of the device to the uuid of the volume. This patch prepares for that change by allowing serial numbers to be set in the libvirt config disk object. Prepares to fix bug 1004328 Change-Id: Iecdfc17b45e1c38df50f844f127c0e95558ab22c
* | | | | | | | | Merge "Typo fix: existant => existent"Jenkins2012-08-237-22/+22
|\ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / |/| | | | | | | |
| * | | | | | | | Typo fix: existant => existentJoe Gordon2012-08-227-22/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Impact: * debug log message in nova/compute/manager * Test function names * Test strings and comments Change-Id: I9be1b0ced2dfe257eef9785659f2cbe27b07ad3f
* | | | | | | | | Re-define libvirt domain on "not found" exceptionRafi Khardalian2012-08-221-21/+21
| |_|_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1039891 If for any reason a libvirt-based VM becomes unregistered, there is currently no mechanism for recovering without administrative intervention. Discussing this with Vish, we agreed it was prudent to validate and accordingly define domains with libvirt on hard_reboot or hypervisor reboot. This patch will allow hard_reboot to re-define a domain in the event that a "domain not found" exception is returned from libvirt. The same approach is also applied to resume_state_on_host_boot. Change-Id: Ic0cd59087d3d84787173fdd9eefdb77b67c33995
* | | | | | | | Merge "Fix hyperv Cfgs: StrOpt to IntOpt"Jenkins2012-08-221-2/+2
|\ \ \ \ \ \ \ \
| * | | | | | | | Fix hyperv Cfgs: StrOpt to IntOptJoe Gordon2012-08-221-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change hyperv_attaching_volume_retry_count and hyperv_wait_between_attach_retry from StrOpt to IntOpt Fix bug 1040246 Change-Id: I8a70212cbc6852bdab4190ac8eb314d10e1a1ed5
* | | | | | | | | Merge "Remove unused _create_network_filters()"Jenkins2012-08-221-23/+0
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Remove unused _create_network_filters()Mark McLoughlin2012-08-221-23/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit b4fae4821f removed unused code which, in turn, made _create_network_filters() unused. Change-Id: I6e1e3aa1c1c977c5ab06d51aef1b43a06d1081b8
* | | | | | | | | | Merge "Clarify nwfilter not found error message"Jenkins2012-08-222-25/+12
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Clarify nwfilter not found error messageMark McLoughlin2012-08-222-25/+12
| |/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In bug #1039398 the user got an error message about an instance's nwfilter not being found, but it gave the impression that live migration was involved. The ensure_filtering_rules_for_instance() method was originally just used for live migration, but since: https://code.launchpad.net/~chemikadze/nova/driver-agnostic-restart-instances/+merge/69069 it has also been used when re-starting the compute service when there are running VMs. Clarify the error message and the comments in the code to reflect this. Waiting up to 30 seconds before reporting an error here seems pretty bad. It appears this is to handle the case where the nwfilters get defined in a separate thread. For reference, the code was added by this merge: https://code.launchpad.net/~nttdata/nova/live-migration/+merge/44940 For now, just add a comment explaining why we're polling for the existence of the nwfilter. Change-Id: Ieb085a6753ac72116d5f5a706fd1a908703286ad
* | | | | | | | | | Merge "Restore imagebackend in test_virt_drivers.py"Jenkins2012-08-221-0/+2
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Restore imagebackend in test_virt_drivers.pyArata Notsu2012-08-221-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the test, nova.virt.libvirt.driver.imagebackend is replaced with fake_imagebackend at setUp(). But is not restored at tearDown(). It potentially causes failure in other tests expecting imagebackend is unchanged. Example: test_libvirt.py:test_pre_block_migration_works_correctly. However it runs without failure since luckily it comes before test_virt_drivers.py. Change-Id: I9b61c6c065567472b64bfa363af8f3ba9a502dd2
* | | | | | | | | | | Merge "Don't show user-data when its not sent."Jenkins2012-08-221-1/+1
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Don't show user-data when its not sent.Chuck Short2012-08-221-1/+1
| | |_|_|/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't show user-data in the instance metadata service if the instance was not started with any user-data. LP: #1035317 Change-Id: Idf9b263a39f3b8d75aaf3e6d1debbe283cf9ff64 Signed-off-by: Chuck Short <chuck.short@canonical.com>
* | | | | | | | | | | Merge "xenapi: Tag nova volumes during attach_volume"Jenkins2012-08-225-18/+84
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | xenapi: Tag nova volumes during attach_volumeRenuka Apte2012-08-205-18/+84
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1. Set other_config:'osvol':'True' on VBD during attach. 2. Use this (instead of nodestroys list) to determine which volumes should not be destroyed on instance terminate. 3. Ensure that the SRs for the attached volumes are forgotten when instance is terminated. In the virt/xenapi layer, there is no way to determine which of the VDIs are nova volumes. This information is required when terminating and instance, to ensure the volumes are kept intact, as well as other operations like migrate. Fixes bug 1030143. Change-Id: Id1717e64bc29092ce9ffe13b7c254a3867fe8342
* | | | | | | | | | | | Merge "Fix hacking.py git checks to propagate errors"Jenkins2012-08-221-2/+7
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / / / |/| | | | | | | | | | |
| * | | | | | | | | | | Fix hacking.py git checks to propagate errorsJoe Gordon2012-08-221-2/+7
| | |/ / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now when N801 or N802 is hit tools/hacking.py will return -1. Fix bug 1039697 Change-Id: I9f6950a4396ae82861e5de0e4f3ba8b482c4fdf8
* | | | | | | | | | | Merge "Code clean up"Jenkins2012-08-221-4/+6
|\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | |
| * | | | | | | | | | Code clean upRay Chen2012-08-211-4/+6
| | |_|_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some code clean up in the file doc/source/conf.py to make the code more pretty. Change-Id: Icb25428739725c530977a011bf28f17fda1c29fb
* | | | | | | | | | Adds missing assertion to FloatingIP testsSulochan Acharya2012-08-221-3/+12
| |_|_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adding assertions to some tests to ensure that we catch the return status_int. Found that test_floating_ip_disassociate would actually fail the test if status_int was caught. Adding the get_instance_by_floating_ip_addr stub to to the disassociate test to return the test instance on IP match. This is required because get_instance_by_floating_ip_addr is stubbed in setup to return None. Fixes bug 1039432. Also making a minor change in test_bad_param_in_remove_floating_ip which should call _remove_floating_ip rather than _add_floating_ip. Change-Id: Id43f785bb0e6b7405e2d292292c4126d7ed656cb
* | | | | | | | | Merge "Introduce ImagePropertiesFilter scheduler filter"Jenkins2012-08-227-224/+146
|\ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / |/| | | | | | | |
generated by cgit (git 2.34.1) at 2026-03-31 04:15:16 +0000