summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | | | | | | | | Allow multiple dns servers when starting dnsmasqVishvananda Ishaya2013-02-072-7/+89
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously we only allowed one upstream dns server even though we can store two in the network object. This changes the dns_server flag to a MultiStrOpt so it can be specified multiple times. It also adds a new config 'use_network_dns_servers' which specifies to uses the dns servers from the network if they are set. This defaults to false to ensure configs do not need to be changed. DocImpact Change-Id: I4c96259bce25036392afe6e027ec47b92679ecbc
* | | | | | | | | | | | | | | Merge "Add "is not" test to hacking.py"Jenkins2013-02-141-3/+17
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | Add "is not" test to hacking.pyJoe Gordon2013-02-121-3/+17
| | |_|_|_|_|_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | HACKING.rst was updated in I7534ef73e6fd525fd8f4bee594a4b37524699c08. This adds hacking.py enforcement. Change-Id: I5d75d536762449d0a7ae38213addd02aeceede0b
* | | | | | | | | | | | | | | Merge "Return dest_check_data as expected by the Scheduler"Jenkins2013-02-143-5/+7
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | Return dest_check_data as expected by the SchedulerArmando Migliaccio2013-02-093-5/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the scheduler driver expects data in return when calling compute_rcp.check_can_live_migrate_destination in method 'schedule_live_migration'. In turn check_can_live_migrate_destination calls check_can_live_migrate_source that also expects data returned from the driver. Fixes bug #1118491 Change-Id: I3dbfa716d5d9bc849c1f67005e7bb1b8ba1ccd30
* | | | | | | | | | | | | | | | Merge "maint: remove unused imports from bin/nova-*"Jenkins2013-02-145-19/+0
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | maint: remove unused imports from bin/nova-*Pádraig Brady2013-02-075-19/+0
| | |_|_|_|_|_|_|_|_|_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I80b58ac41bbb0517221d5fd5ba9938f2b7836779
* | | | | | | | | | | | | | | | Merge "Fix regression in non-admin simple_usage:show."Jenkins2013-02-141-1/+1
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | Fix regression in non-admin simple_usage:show.Vishvananda Ishaya2013-02-111-1/+1
| | |_|_|_|_|_|_|_|_|_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit 6ff32210772a67a1b526d9d784030afc90f3ce99 optimized the db query for usage but replaced a call that had require_context with one that had require_admin_context. This causes usage show to throw a 403. We check for admin_or_owner in policy, so there is no need to check for admin context at the db layer. Fixes bug 1122267 Change-Id: Iff362f46d83a710f3883538bcb646e3457e0ba64
* | | | | | | | | | | | | | | | Merge "Fix passing conductor to get_instance_nw_info()"Jenkins2013-02-142-2/+3
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | Fix passing conductor to get_instance_nw_info()Dan Smith2013-02-122-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The recent metadata patch missed this one spot, which wasn't caught because the network stubs weren't updated for the signature change. Fixes bug 1123435 Change-Id: Iebeb98bba62189476af2c0b85615ea407a8b2ecd
* | | | | | | | | | | | | | | | | Merge "Use a more specific error reporting invalid disk hardware"Jenkins2013-02-142-6/+4
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | Use a more specific error reporting invalid disk hardwareDaniel P. Berrange2013-02-132-6/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead of using NovaException for reporting invalid user specified disk hardware, use UnsupportedHardware and UnsupportedVirtType exceptions. Change-Id: I68ed1c0569bd0c0679e9fff3ffd436197f78a9f0 Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
* | | | | | | | | | | | | | | | | | Merge "API extension for accessing instance_actions"Jenkins2013-02-1422-13/+639
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | API extension for accessing instance_actionsAndrew Laski2013-02-1322-13/+639
| | |_|_|_|_|_|_|_|_|_|_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds a new API extension for accessing the recorded actions and events on an instance. Usage is documented with api samples. Additionally it modified the db api to retrieve actions by request_id since the api does not return the db id. This extension is the first consumer of that method so there's no issue of changing behaviour elsewhere. Blueprint instance-actions DocImpact Change-Id: I74109586cc762a7f51d2b114896cf071ee0671cb
* | | | | | | | | | | | | | | | | | Merge "Resize/Migrate functions for PowerVM driver"Jenkins2013-02-146-10/+418
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | Resize/Migrate functions for PowerVM driverLance Bragstad2013-02-136-10/+418
| | |/ / / / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added resize and migrate functionality to PowerVM driver. Allows resizing of instance and cold migration of an instance to different host. bp powervm-compute-resize-migration Change-Id: I723456266d3ceb4bd4a7ce4273008201b450f533
* | | | | | | | | | | | | | | | | | Merge "Module import style checking changes"Jenkins2013-02-144-90/+111
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | Module import style checking changesAttila Fazekas2013-02-134-90/+111
| | |_|_|_|_|_|_|_|_|_|_|_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Implementing the * import detection (it is disabled for now) * New style relative import testing based on syntax rules * Old style relative import testing based on module search * Inspection based solution replaced by PYTHONPATH search in order to avoid module compile and initialization steps (code execution) in a syntax checking phase. This solution is faster and safer, but does not able to recognize modules added dynamically to the module scope. Change-Id: Ifc871f4fdbcd4a9a736170ceb4475f4f2cbe66bc
* | | | | | | | | | | | | | | | | | Merge "Correct misspelling in PowerVM comment"Jenkins2013-02-141-1/+1
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | | Correct misspelling in PowerVM commentMichael J Fork2013-02-131-1/+1
| | |_|_|_|_|_|_|_|_|_|_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Correct a misspelling in a PowerVM comment Change-Id: I12ab59826e8448a120ff3ef2b26034ea4aa2f1a0
* | | | | | | | | | | | | | | | | | Merge "Add GlusterFS libvirt volume connector"Jenkins2013-02-143-2/+92
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|_|_|_|/ / / / / |/| | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | Add GlusterFS libvirt volume connectorEric Harney2013-02-133-2/+92
| | |/ / / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is to support GlusterFS as a storage backend for Cinder, which works similarly to the NFS backend. Change-Id: I047d58ba74d27273ba8f2639bcc77999f8be0f30
* | | | | | | | | | | | | | | | | Merge "VMWare driver to use current nova.network.model"Jenkins2013-02-136-77/+169
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / / / / / / / / |/| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | VMWare driver to use current nova.network.modelAmir Sadoughi2013-02-126-77/+169
| | |_|_|_|_|_|_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Updated tests to use classes from nova.network.model instead of legacy "network,mapping" tuple. * Modified vmwareapi/{driver,vif,vmops}.py to use classes from nova.network.model instead of legacy "network,mapping" tuple. Fixes: bug 1112655 Change-Id: I7f4b39cd5add7d3cf7dcf485eb9855d0354f332c
* | | | | | | | | | | | | | | | Merge "Check the length of flavor name in "flavor-create""Jenkins2013-02-135-12/+56
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / / / / / / |/| | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | Check the length of flavor name in "flavor-create"Haiwei Xu2013-02-135-12/+56
| | |_|_|_|_|_|_|_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1102280 The length of flavor name is defined 255 charaters in database. But flavor name can be more than 255 characters when a flavor is created. This patch adds the length check of flavor name. Change-Id: If9db879e5f6340594b215b057a29d03c6fef1503
* | | | | | | | | | | | | | | Merge "Add API sample tests to Services extension"Jenkins2013-02-1311-1/+164
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | Add API sample tests to Services extensionivan-zhu2013-02-1311-1/+164
| |/ / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Partially implements blueprint nova-api-samples Bug 1091755 Change-Id: I7a0e124871c248bbb7613bfcdf335fee29a887f2
* | | | | | | | | | | | | | | Merge "Make compute manager use conductor for stopping instances"Jenkins2013-02-135-4/+33
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / / / / / |/| | | | / / / / / / / / / / | | |_|_|/ / / / / / / / / / | |/| | | | | | | | | | | |
| * | | | | | | | | | | | | Make compute manager use conductor for stopping instancesDan Smith2013-02-125-4/+33
| | |/ / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | During _sync_power_states, the manager will call to compute_api to stop instances as necessary. This takes a database hit, which isn't allowed. This directs that call through conductor, following the pattern of late. Fixes bug 1123219 Change-Id: I5d789dd73390cb1c41a245f6594b1c3aecf7efde
* | | | | | | | | | | | | Merge "Imported Translations from Transifex"OpenStack Jenkins2013-02-131-510/+579
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Imported Translations from TransifexOpenStack Jenkins2013-02-131-510/+579
| |/ / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I7c601a4dafaa6672d1e9caf4bc9fbdda535ff0ea
* | / / / / / / / / / / Allow VIF model to be chosen per imageDaniel P. Berrange2013-02-135-41/+151
| |/ / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows for an image in glance to be annotated with a property describing the required VIF model eg # glance image-update \ --property vif_model=e1000 \ f16-x86_64-openstack-sda Valid model values vary per the libvirt_type setting: qemu/kvm: 'virtio', 'ne2k_pci', 'pcnet', 'rtl8139', 'e1000' xen: 'netfront', 'ne2k_pci', 'pcnet', 'rtl8139', 'e1000' Requesting an unsupported VIF model will cause the guest instance to fail to launch. DocImpact Blueprint: libvirt-custom-hardware Change-Id: Idbee0c61cffdb43db5668fb88869a5d30278593f Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
* | | | | | | | | | | Merge "Handle compute node not available for live migration"Jenkins2013-02-136-12/+121
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Handle compute node not available for live migrationJian Wen2013-01-316-12/+121
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch handles exception.ComputeServiceUnavailable by restoring instance's vm_state and instance's task_state after live migration failure caused by unavailable source/dest compute node. Raises detailed HTTPBadRequest explanation for this exception. Fixes bug 973393 and bug 1051881 Change-Id: If825b61fad9c4e3030f2e6c5002907255eaf3661
* | | | | | | | | | | | Merge "Fix update quota with invalid value"Jenkins2013-02-132-1/+49
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | Fix update quota with invalid valueZhou ShaoYu2013-02-022-1/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If update quota with a value which is not integer, it will response with http status code 500. Catch the exception and continue for updating other quotas. Fix LP# 1111327 Change-Id: I0ab3b10ff106a2e6c7677d1acdb55c2f35c492db
* | | | | | | | | | | | | Merge "Forces flavorRef to be string in servers resize api"Jenkins2013-02-131-1/+1
|\ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | Forces flavorRef to be string in servers resize apiTiago Mello2013-02-121-1/+1
| | |_|/ / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | flavorid column is a string in the database model and MySQL silently accepts either integer or string as data type. However PostgreSQL expects the right data type. So we force it to be string as early as possible in the api. Fixes Bug 1123525 Change-Id: Id117868b3a2f840f5d7ab463f3b79b6631b3b514
* | | | | | | | | | | | | Merge "Move allowvssprovider=false to vm-data field"Jenkins2013-02-131-3/+2
|\ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | Move allowvssprovider=false to vm-data fieldNicholas Kuechler2013-02-121-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently we're setting allowvssprovider=false in the other-config field. However, this setting is not honored in the other-config field, allowing vss-based snapshots to be created. The allowvssprovider=false should be moved to the vm-data field in xenstore as that field is used to decide whether vss snapshotting is allowed or not. Amended to change: 'str(False)' to 'false'. Relevant code: https://github.com/openstack/nova/blob/master/nova/virt/xenapi/vm_utils.py#L233 Manually set allowvssprovider=false in the vm-data field: xe vm-param-set uuid=<VM-UUID> xenstore-data:vm-data/allowvssprovider=false References: https://github.com/mcclurmc/xcp-storage-managers/blob/master/drivers/vss_control#L38 https://github.com/mcclurmc/xcp-storage-managers/blob/master/snapwatchd/snapwatchd#L298 Change-Id: I96efbc18cb7eb0ee2d0e6526b3c4f3502a68e1dd Fixes: bug #1112673
* | | | | | | | | | | | | | Merge "Libvirt: Implement snapshots for LVM-backed roots"Jenkins2013-02-131-0/+24
|\ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | Libvirt: Implement snapshots for LVM-backed rootsRafi Khardalian2013-02-101-0/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Uses the built-in LVM snapshotting mechanism to create a new snapshot, use qemu-img to extract the full root into an image file, then delete the snapshot volume. Although we typically zero deleted LVM volumes manually, this is not necessary for snapshots, as LVM handles it for us (per the documentation). Change-Id: I82f21488671ffa3be3d0feba73ff631fa2a51ec0 Fixes: Bug 1120809 Flags: DocImpact
* | | | | | | | | | | | | | | Merge "Properly write non-raw LVM images on creation"Jenkins2013-02-133-8/+7
|\| | | | | | | | | | | | | |
| * | | | | | | | | | | | | | Properly write non-raw LVM images on creationRafi Khardalian2013-02-103-8/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1075333 When force_raw_images=False and libvirt_images_type=lvm were set together, selecting anything other than a RAW image would result in an unbootable instance. This is because the downloaded image was being directly dd'd onto the new LVM root, which will obviously not work for any format other than RAW. Simple change to use qemu-img to read from the image (letting it decide the format) and write to the LVM root as raw. Change-Id: I36494e6d0671c07f7511c10ef886c8cf84c18df7
* | | | | | | | | | | | | | | Merge "Remove locking declarator in ServiceGroup __new__()"Jenkins2013-02-131-2/+0
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | Remove locking declarator in ServiceGroup __new__()Yun Mao2013-02-111-2/+0
| | |_|_|_|_|_|_|_|_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The lock-based synchronization is not re-entrant, which would introduce a deadlock with the new conductor patch. Because race condition is not be possible in the eventlet threading model with GIL, we simply remove the synchronized declarator. Change-Id: I03e52227385dafd2b2b66bca18cab8445c91f3be
* | | | | | | | | | | | | | | Merge "Refine PowerVM MAC address generation algorithm"Jenkins2013-02-131-8/+19
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | Refine PowerVM MAC address generation algorithmMichael J Fork2013-02-111-8/+19
| |/ / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change MAC address generation algorithm on PowerVM to generate an additional 4 bits of data and improve comments around it. Change-Id: Icdfba54161bcb043c17127c26c57503af497d001