summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | Merge "Remove network_api from Servers Controller"Jenkins2012-01-121-2/+0
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Remove network_api from Servers ControllerBrian Waldon2012-01-121-2/+0
| | |_|_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I7f2379286c6a6ccb758f14d5ec642b02173435a2
* | | | | | | | | Fix policy import in nova.compute.apiBrian Waldon2012-01-121-1/+1
| |/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | Change-Id: Id9c7cb3563e34c27ea9f956ce18d2aed403a0f82
* | | | | | | | minor fix in commentJoe Gordon2012-01-121-1/+1
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | Change-Id: I2bdb78e8dfaa504f5b9969606871c832e7936bd7
* / / / / / / Add policy checks to Compute.APIBrian Waldon2012-01-128-38/+363
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Second step of blueprint interim-nova-authz-service * Adds policy.json to define policy * Add nova.policy.wrap_enforce decorator * wrap majority of compute api functions with wrap_enforce Change-Id: If6702873db3249921f931a42e889ee7d0338e4b8
* | | | | | Merge "Unbreak start instance and fixes bug 905270"Jenkins2012-01-1217-87/+401
|\ \ \ \ \ \
| * | | | | | Unbreak start instance and fixes bug 905270Isaku Yamahata2012-01-1217-87/+401
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch fixes the bug 905270 https://bugs.launchpad.net/nova/+bug/905270 According to EC2 documentation, EBS-instances that initiated shutdown result in stopped state. And then it can be started again. (On the other hand non-EBS instance result in terminted when instance initiated shutdown) However, the current nova case, the shutdowned instance always results in terminated status. As related issues are - describe-instance-attribute instance_initiated_shutdown_behavior doesn't work correctly - instance attribute disable_api_termination isn't supported - stop instance was broken by the change set of the following. It needs unbreak. > commit eb03d47fecd3bfc24243da29ee01679b334a08fe > Author: Vishvananda Ishaya <vishvananda@gmail.com> > Date: Fri Sep 23 09:22:32 2011 -0700 > > Remove AoE, Clean up volume code > > * Removes Ata Over Ethernet > * Adds drivers to libvirt for volumes > * Adds initialize_connection and terminate_connection to volume api > * Passes connection info back through volume api > > Change-Id: I1b1626f40bebe8466ab410fb174683293c7c474f This patch - unbreak start instance - implement instance_initiated_shutdown_behavior and make it EC2 compatible - implement disable_api_termination --- Changes 5 -> 6: - fixes to catch up 26b7b9457a5899ecca93fd67d3879efcad4e4968 Changes 4 -> 5: - HACKING compilance Changes 3 -> 4: - rebased to 4c5586a28fd7a085369c49f6039876ffdc86b526 sqlalchemy migrate version Changes 2 -> 3: - rename long name to shorter one s/instance_initiated_shutdown_behavior/shutdown_terminate/g s/disable_api_termination/disable_terminate/g as suggested Kevin L. Mitchell - improved nova.api.ec2.cloud._state_description - pep8 - broken out patches are available for easy review at git://github.com/yamahata/nova.git lp905270-2 Changes 1 -> 2: - fixed an unit test failure pointed out by Mark. (I think ebtabls failure strongly suggests installation problem) - introduce vm_states.SHUTOFF and put instance state which is in power_state.{NOSTATE, SHUTOFF} into vm_states.SHUTOFF. - simplified logic a bit by vm_states.SHUTOFF as suggested by Vish. - instance_initiated_shutdown_behavior:String(255) => instance_initiated_shutdown_terminate:Boolean() as suggested by Vish. - Added Johannes Erdfelt to reviews as they written the vm_states state machine checker. I'd have liked to add David Subiros either, but he doesn't seem to be a registered user of the gerrit. Change-Id: Ibeb94f65137feadad2c343913b39195e3f96a35e
* | | | | | | Merge "Add 'os-networks' extension"Jenkins2012-01-128-2/+374
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | |
| * | | | | | Add 'os-networks' extensionBrian Waldon2011-12-268-2/+374
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Admin-only extension that supports simple network-related actions. This functionality is needed to deprecate openstackx Change-Id: Id930d823913fb9c9f6f1e476817cc12d1a64c418
* | | | | | | catch InstanceInvalidState in more placesChris Behrens2012-01-1210-44/+324
| |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Further fixes to bug 911879 500s or 400s are returned in the OS API when actions are denied due to being in an invalid state. 409 should be returned, instead. A previous review (2846) fixed the delete case and this fixes more. When writing tests, I found a number of exceptions that are not raised anymore, and they were being incorrectly used in tests still. I fixed those up. Change-Id: I0d5b1ed52e0cc9766be8e2a7de84c8601f4bdf26
* | | | | | Merge "Blueprint xenapi-security-groups"Jenkins2012-01-1215-286/+915
|\ \ \ \ \ \
| * | | | | | Blueprint xenapi-security-groupsSalvatore Orlando2012-01-1015-286/+915
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Provides two drivers for implementing security groups in xenapi: 1) domU driver that enforces security groups on the Openstack virtual appliance (use advised with FlatDHCP in HA mode) 2) dom0 driver that enforces security groups where VIFs are attached Both drivers translate security groups into iptables rules. Existing libvirt code has been refactored to reduce the amount of duplicated code to a minimum Now Addressing reviewers's comments on style. Fixing issue spotted with snapshots Change-Id: Ifa16a8f2508a709be03241bac0f942fe1a51d1e8
* | | | | | | Merge "Fix metadata mapping in s3._s3_parse_manifest."Jenkins2012-01-121-2/+2
|\ \ \ \ \ \ \
| * | | | | | | Fix metadata mapping in s3._s3_parse_manifest.Hengqing Hu2012-01-101-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug #827617 Fixes incorrect output of the following euca2ools commands: 'euca-describe-images' 'euca-describe-instances' Which occours when user published 'ami' image with 'ari&aki' or 'aki' image(s). Fixes minor typo: 'manifext' -> 'manifest' Change-Id: If32357e3ea6685476df9dbb6e74af710153457b9
* | | | | | | | Merge "Call to instance_info_cache_delete to use uuid"Jenkins2012-01-125-72/+73
|\ \ \ \ \ \ \ \
| * | | | | | | | Call to instance_info_cache_delete to use uuidAlex Meade2012-01-115-72/+73
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 903497 Also updated incorrect calls to instance_destroy that were using uuids. Change-Id: I25eead020ceb7ebf7234c268543ad77d8ecf1185
* | | | | | | | | Merge "Fix libguestfs operation with specified partitions"Jenkins2012-01-111-3/+9
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Fix libguestfs operation with specified partitionsPádraig Brady2012-01-101-3/+9
| | |/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * nova/virt/disk/guestfs.py (mnt_dev): Convert the partition parameter to int, so we now support a "1" string passed from nova/virt/libvirt/connection.py. Also diagnose any unconvertible strings. Change-Id: I744bd1fc3ea065891b5ce112dbe27f73086f302e
* | | | | | | | | Merge "Don't overwrite local context on elevated"Jenkins2012-01-111-4/+10
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|/ / |/| | | | | | | |
| * | | | | | | | Don't overwrite local context on elevatedVishvananda Ishaya2012-01-091-4/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Adds an 'overwrite' parameter to context.__init__ * Overwrite is set to false for elevated and get_admin_context * Fixes bug 899302 Change-Id: Ic9464e5813d6db38c48239c7ea8f15aa1bff5562
* | | | | | | | | Merge "Remove 'location' from GlanceImageService"Jenkins2012-01-113-7/+2
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Remove 'location' from GlanceImageServiceBrian Waldon2012-01-113-7/+2
| | |_|_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Glance no longer returns location through its public API, so we should not attempt to display it. Addresses bug 911599. Change-Id: I3400006eb6ab94095c0c2d2b5dc90cb9b7775a84
* / | | | | | | | Makes common/cfg.py raise AttributeErrorVishvananda Ishaya2012-01-112-1/+6
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * fixes bug 915039 * includes test Change-Id: I67b886be3b5af3763f52fffe54085975d61d61eb
* | | | | | | | Merge "fix reboot_instance typo"Jenkins2012-01-111-1/+1
|\ \ \ \ \ \ \ \ | |_|_|/ / / / / |/| | | | | | |
| * | | | | | | fix reboot_instance typolzyeval2012-01-101-1/+1
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixed bug #909745 Change-Id: I67fa47ae339f03290f21414e1eaae4c4e477424d
* | | | | | | Merge "Add @utils.deprecated()."Jenkins2012-01-112-1/+280
|\ \ \ \ \ \ \
| * | | | | | | Add @utils.deprecated().Kevin L. Mitchell2012-01-102-1/+280
| | |_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will allow us to mark deprecated classes and functions/methods as such. A warning is issued each time a deprecated function/method is called, or when a deprecated class is instantiated, or when any class or static method on a deprecated class is called. Change-Id: I4b5858492bc14768ac2e12c542bc343962761e34
* | | | | | | Bug #914907: register_models in db/sqlalchemy/models.py references ↵Armando Migliaccio2012-01-111-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | non-existent ExportDevice Remove ExportDevice from register_models as this no longer exists. Change-Id: I68555eb33e035fa3669e4ac4b77cb2c0598cf52c
* | | | | | | Merge "Update logging in compute manager to use uuids"Jenkins2012-01-111-38/+43
|\ \ \ \ \ \ \
| * | | | | | | Update logging in compute manager to use uuidsAlex Meade2012-01-111-38/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 914814 Change-Id: I8f6c5335f6e9e26e9e8304a54ca31d967227dfc3
* | | | | | | | Merge "nova-manage floating ip fixes"Jenkins2012-01-111-5/+7
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | |
| * | | | | | | nova-manage floating ip fixesDean Troyer2012-01-101-5/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add pool, interface to 'floating list' output These columns were added to the floating_ips table in https://review.openstack.org/2892 and are not visible elsewhere * Fix 'floating delete' to skip network/broadcast addresses Makes the delete command match the create command behaviour introduced in https://review.openstack.org/2663 * Address changes listed in bug 817281 Change host column in output to project ID, change hostname column to uuid Change-Id: Ibbfbe9ed5543927a6b2a1e676f5617901a32f897
* | | | | | | | Do not overwrite project_id from request paramsThierry Carrez2012-01-1122-200/+202
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prevent project_id overwriting from OSAPI request parameters. The patch is actually very simple (nova/api/openstack/wsgi.py) but needs significant test adjustments (nova/tests/*) to pass. Fixes bug 904072. Patch from Vish Ishaya and Mark McLoughlin. Change-Id: I66ea0f178ce6271ec1020e9f1a73bd4e8c83ddab
* | | | | | | | Add optional revision field to version numberMark McLoughlin2012-01-111-3/+3
| |_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a revision field to the version number to account for releases from the stable branch, e.g. 2011.3.1. See http://wiki.openstack.org/StableBranchRelease Change-Id: I34238a5639d34c9cf44655775a6be80fdd446233
* | | | | | | Add a modify function to the floating ip dns api.Andrew Bogott2012-01-108-27/+87
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removed the dns driver function rename_entry(). Since 'name' is the unique identifier for an entry, trying to rename things causes various confusions. Better to leave the user to delete and recreate. For blueprint public-and-private-dns. Change-Id: I766af01299b7b28ad8d9f6c072a8f3e049bd286b
* | | | | | Adding the request id to response headers.Naveed Massjouni2012-01-102-3/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bp:nova-request-response-id Change-Id: Ie2d49337010f6e7a540018f9c8d542b6c8ebbce2
* | | | | | Merge "Bug #894683: nova.service does not handle attribute specific ↵Jenkins2012-01-101-3/+3
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | exceptions and client hangs"
| * | | | | | Bug #894683: nova.service does not handle attribute specific exceptions and ↵Armando Migliaccio2012-01-061-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | client hangs As Sateesh points out nova.service.Service.__getattr__ throws an AttributeError exception when the upcall method is not available. However nova.rpc should catch all exceptions. This does not happen in this specific case because the following statement was outside the try-catch-all: node_func = getattr(self.proxy, str(method)) Change-Id: I437c88783bca037e4054078d2687ef41c8fc2b83
* | | | | | | Merge "libvirt: implements boot from ISO images"Jenkins2012-01-103-22/+55
|\ \ \ \ \ \ \
| * | | | | | | libvirt: implements boot from ISO imagesJulien Danjou2012-01-033-22/+55
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows to boot correctly ISO images. We just set the device type in the XML template to 'cdrom' if the image is in ISO format, and the bus type to IDE so it has maximum compatibility. This implements the blueprint support-kvm-boot-from-iso. Change-Id: I4d1ea52a17f6f24e944b10020d54aaec9e86c95c Signed-off-by: Julien Danjou <julien.danjou@enovance.com>
* | | | | | | | Merge "Isolate certain images on certain hosts."Jenkins2012-01-102-2/+118
|\ \ \ \ \ \ \ \
| * | | | | | | | Isolate certain images on certain hosts.Todd Willey2012-01-082-2/+118
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This implements a [hosts] <=> [images] mapping in the simple scheduler that partitions your host resources into the part that services a particular image set, and the general cloud. This is useful, for example, if you want to specify a set of hosts to run utility VMs (cloudpipe, bastion, etc) that you don't want consuming resources from your generally available pool. When specifying a host with --isolated_hosts flags (comma-separated list) those hosts will only run the images specified in --isolated_images, and will not run any other images. The isolated images will not run on any other hosts. You can specify --skip_isolated_core_check to allow overcommitting of the isolated hosts. This allows utility vms that are not cpu bound to avoid the resource cheks the scheduler usually performs (based off of --max_cores). Change-Id: Ib2db5a605cb7560a169af9ff2a6dadb649da9c1d
* | | | | | | | | Merge "Fix call to compute_api.resize from _migrate"Jenkins2012-01-101-1/+3
|\ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / |/| | | | | | | |
| * | | | | | | | Fix call to compute_api.resize from _migrateFrançois Charlier2012-01-101-1/+3
| | |_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | _migrate called compute_api.resize with an instance id, compute_api.resize expects an instance object. Change-Id: I0ab9ac48f8d7b4d5d92e0212969cc8febc5cf82b
* | | | | | | | Merge "Use instance_properties in resize."Jenkins2012-01-106-15/+21
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | |
| * | | | | | | Use instance_properties in resize.Mark Washenberger2012-01-056-15/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This just makes resize behave a bit more like create, which is useful for custom filtering. Change-Id: I10ba726712d8f601f5ee97e0bb033bd7339a7a88
* | | | | | | | Merge "Adds support for floating ip pools"Jenkins2012-01-1016-45/+371
|\ \ \ \ \ \ \ \
| * | | | | | | | Adds support for floating ip poolsVishvananda Ishaya2012-01-0916-45/+371
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Implements blueprint multiple-floating-ip-ranges * Adds pool and interface fields to floating ip tables * Adds extension to get a list of available pools * Optionally allows a pool to be specified when allocating * Changes nova-manage command to allow pool and interface * Ip binding uses the interface from table instead of flag * Adds default pool flag to use when pool is not specified * updates test to work with new fields * adds tests for extension Change-Id: Ieb4cbbf07b211697d08178b1cf2252caf75049a2
* | | | | | | | | Merge "fix bug 914049: private key in log"Jenkins2012-01-101-1/+7
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | fix bug 914049: private key in logJoe Gordon2012-01-091-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Id004257ae425291bb18427c5d9b03602a62da043