summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Add an rpc API common to all services.Russell Bryant2013-04-2310-9/+136
| | | | | | | | | | | | | | | | | | | | This patch adds an rpc API that is exposed by all services. The methods in this API exist in their own namespace and are versioned independently of the main API for the service. The first method for this API is a simple ping() method. This method exists in the conductor rpc API already, and could be more generally useful. Other methods will be added in later patches. The base rpc API will be exposed from all services automatically unless they override the create_rpc_dispatcher method in the base manager class. All services need to pass a service_name into the base manager constructor. Some services already did this, but now it's needed for all of them. Implements blueprint base-rpc-api. Change-Id: I02ab1970578bc53ba26461b533d06d1055c2d88e
* Merge "Translate cinder NotFound exception"Jenkins2013-04-232-1/+24
|\
| * Translate cinder NotFound exceptionChris Yeoh2013-04-172-1/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When we attempt to get information about a snapshot which doesn't exist we get a cinder NotFound exception. However, this is not caught and causes a stacktrace in the logs. This occurs on every tempest run in the test EC2VolumesTest.test_create_volume_from_snapshot. The fix is to translate the exception to SnapshotNotFound which is handled correctly further up the call chain. Fixes bug 1169777 Part of blueprint no-stacktraces-in-logs Change-Id: If893b5083b93894e671c978bd23bd89d3d017531
* | Merge "hypervisor tests more accurate db"Jenkins2013-04-231-2/+6
|\ \
| * | hypervisor tests more accurate dbBrant Knudson2013-04-161-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The hypervisor tests don't accurately model the db because the sqlalchemy DB API requires admin for the compute_node_get_all function. This change makes it so that the fake compute_node_get_all function in the hypervisor test module requires admin just like the sqlalchemy DB function. Change-Id: I5acd45cfbd10ec06d8c5a9425e9c6ed18cff669e
* | | Merge "Deprecate old vif drivers."Jenkins2013-04-231-0/+25
|\ \ \
| * | | Deprecate old vif drivers.Joe Gordon2013-04-041-0/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As per commit 4c0cecd3 (Change-Id I292c2fd7aa0c32b1c91d737152ed8e54ea2cf6ac) Deprecate: * LibvirtBridgeDriver * LibvirtOpenVswitchDriver * LibvirtHybridOVSBridgeDriver * LibvirtOpenVswitchVirtualPortDriver * QuantumLinuxBridgeVIFDriver Change-Id: Iad854dfe91aaa18622f78aa92746d3ea2f986960
* | | | Merge "Update the log module from oslo-incubator"Jenkins2013-04-231-6/+3
|\ \ \ \
| * | | | Update the log module from oslo-incubatorMichael Kerrin2013-04-181-6/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Contains fix to stop the logger system from logging events twice. Also includes PID in default logging_context_format_string Fixes bug: 1170049 Change-Id: I007786604138c41fbcf1127c01339132c56a6aab
* | | | | Merge "Add option to exclude joins from instance_get_by_uuid"Jenkins2013-04-238-23/+83
|\ \ \ \ \
| * | | | | Add option to exclude joins from instance_get_by_uuidAndrew Laski2013-04-228-23/+83
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some callers of instance_get_by_uuid do not need the data that it joins by default. This allows the caller the choice of getting it or not. Specifically in this case the _sync_power_state periodic task is pulling it unnecessarily. Change-Id: I48cc2ea1de993b87ce6f96f4877c8d8513f507e2
* | | | | | Merge "Fix hyperv copy file error logged incorrect"Jenkins2013-04-221-1/+2
|\ \ \ \ \ \
| * | | | | | Fix hyperv copy file error logged incorrectguohliu2013-04-191-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch just fix hyperv copy file error message incorrect, missing src and dest value. Fixes bug #1170580 Change-Id: I309fd67b3f491c9be872ff9cdd58f6364a26a96b
* | | | | | | Merge "Can now reboot rescued instances in xenapi"Jenkins2013-04-222-4/+21
|\ \ \ \ \ \ \
| * | | | | | | Can now reboot rescued instances in xenapiMatthew Sherborne2013-04-202-4/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch makes it possible to reboot an instance that is in rescue mode. It causes the reboot action to search first for rescue mode VM (the normal VMs name plus '-rescue') .. if it can't find it, it does a reboot of the normal VM. If it finds the rescue one first, it'll reboot the rescue instance. If there is more than one VM named 'myvm-rescue', it'll raise an exception just like it did if there were more than one VM named 'myvm'. Helps with bug: 1170237 Change-Id: I1858d86bf93546618eeaaec92d856f5236f3f090
* | | | | | | | Merge "Allows xenapi 'lookup' to look for rescue mode VMs"Jenkins2013-04-222-2/+36
|\ \ \ \ \ \ \ \ | |/ / / / / / /
| * | | | | | | Allows xenapi 'lookup' to look for rescue mode VMsMatthew Sherborne2013-04-202-2/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When you lookup a VM by name, you can specify if you want it to check if it has a rescue mode instance running. If it finds the rescue mode instance, it'll return that one instead of the original name. This is useful for operations that should work on the rescue mode vm if it's there, or the normal one, if it's not. Work towards bug: 1170237 Change-Id: I41b31e59631b78a62faadb9970b6b824929e7061
* | | | | | | | Merge "Adds tests to xenapi.vm_utils's 'lookup' method"Jenkins2013-04-221-0/+30
|\ \ \ \ \ \ \ \ | |/ / / / / / /
| * | | | | | | Adds tests to xenapi.vm_utils's 'lookup' methodMatthew Sherborne2013-04-201-0/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I140f091b50c3e4076d3f4812cae00e37f120a277
* | | | | | | | Merge "Stop vm_state reset on reboot of rescued vm"Jenkins2013-04-222-4/+27
|\ \ \ \ \ \ \ \
| * | | | | | | | Stop vm_state reset on reboot of rescued vmMatthew Sherborne2013-04-192-4/+27
| | |/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a VM is in rescue mode, giving it a reboot will reset its state in the DB back to ACTIVE. This patch stops that happening. Work towards bug: 1170237 Change-Id: Ie442bcc85ed7ad5d897b88ffd0dbcf6eeee7b990
* | | | | | | | Merge "Cleanup and test volume usage on volume detach"Jenkins2013-04-223-28/+83
|\ \ \ \ \ \ \ \
| * | | | | | | | Cleanup and test volume usage on volume detachMichael Kerrin2013-04-163-28/+83
| | |_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Refactor _poll_volume_usage to be more readable * Test that detaching a volume updates the totals on the volume usage cache correctly. This works but adding test to make sure that it always works. Change-Id: I82da44137bc65d5f04bca0820eb327438d6e1f35
* | | | | | | | Merge "Delete extra space to api/volumes message"Jenkins2013-04-221-1/+1
|\ \ \ \ \ \ \ \ | |_|_|_|/ / / / |/| | | | | | |
| * | | | | | | Delete extra space to api/volumes messageYou Ji2013-04-171-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I04efa9c8ddb5790f99d66c302198bf92000f6e66 Signed-off-by: You Ji <jiyou09@gmail.com>
* | | | | | | | Merge "baremetal: Change input for sfdisk"Jenkins2013-04-221-1/+1
|\ \ \ \ \ \ \ \
| * | | | | | | | baremetal: Change input for sfdiskArata Notsu2013-04-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To prevent sfdisk stopping with "bad input", use a empty string instead of "-" (and doing so requires using "," as a separator). Fixes bug 1166112 Change-Id: I5e4cffaa326fbafd1583d55744f921a40b3ae76f
* | | | | | | | | Merge "Raise correct exception for duplicate networks"Jenkins2013-04-222-9/+23
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Raise correct exception for duplicate networksBen Nemec2013-04-082-9/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When using Quantum, if duplicate network ids are passed in on a nova boot, nova-api raises a NetworkNotFound exception, which is not correct. This change causes it to raise a NetworkDuplicated exception instead. Fixes bug 1165088 Change-Id: I4f1f9aa5e1fa0e740b78e67760d924e2dd1d4d98
* | | | | | | | | | Merge "baremetal: VirtualPowerDriver uses mac addresses in bm_interfaces"Jenkins2013-04-222-9/+16
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | baremetal: VirtualPowerDriver uses mac addresses in bm_interfacesArata Notsu2013-04-152-9/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After the change I6653829364b0a641442d45e766493180d6f2a880, bm_nodes.prov_mac_address is no longer set. However VPD still uses it. This patch changes VPD to use bm_interfaces.address instead of bm_nodes.prov_mac_address. Fixes bug 1168574. Change-Id: I2bab869b282958ee9fd0e4776928e56351018324
* | | | | | | | | | | Merge "Remove unnecessary method argument."Jenkins2013-04-221-1/+1
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Remove unnecessary method argument.Yaguang Tang2013-04-221-1/+1
| | |_|_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The 'self' argument is unnecessary and when the the method is invoked, cause parameters error. Fix bug #1171376 Change-Id: I612b466a955cb8418855849119599222ba6479d3
* | | | | | | | | | | Improve Python 3.x compatibilityDirk Mueller2013-04-2234-95/+95
|/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A bit of mechanical translation to clean out the deprecated except x,y: construct Change-Id: I80883b6b0c014bdf4799e8b7b2c4a4a07d951a0b
* | | | | | | | | | Merge "Sync rpc from oslo-incubator."Jenkins2013-04-2013-16/+257
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Sync rpc from oslo-incubator.Russell Bryant2013-04-1413-16/+257
| |/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This includes rpc api namespace support, some minor configuration related fixes, and the redis based matchmaker for use with impl_zmq. Change-Id: I9fc826a498461417afe8b2a923edac8d68c2c18f
* | | | | | | | | | Imported Translations from TransifexOpenStack Jenkins2013-04-2022-997/+730
| |_|_|_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ib7dd5ace2eaf2060ea0a65ff9100a1fdbc10ccdf
* | | | | | | | | Merge "Don't swallow PolicyNotAuthorized for resize/reboot actions"Jenkins2013-04-193-21/+40
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Don't swallow PolicyNotAuthorized for resize/reboot actionsAndrew Laski2013-04-183-21/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Removes 'except Exception' lines in the api which were preventing PolicyNotAuthorized exceptions from bubbling up and returning 403 to the caller. This addresses policy exceptions in confirmResize, revertResize, and reboot. The try block calls into the compute api which sends a cast to perform to bulk of the work, so there is little chance of an exception bubbling up which is not already being caught. Removes a unit test for a condition that should not exist. Bug 1170453 Change-Id: I516a19894ab3d183057c774e84c4faa7053a6463
* | | | | | | | | | Imported Translations from TransifexOpenStack Jenkins2013-04-1922-2091/+1997
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ie208685045a154bc4f27706ae0c27d5de155a108
* | | | | | | | | | Remove unused exception and variable from scheduler.Alvaro Lopez Garcia2013-04-182-10/+0
| |_|_|_|_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are no cost functions, so there is no cache and no Exception to be raised. Change-Id: I4a67bd45e3f0c91e5507ed3794dc20b13292596b
* | | | | | | | | Imported Translations from TransifexOpenStack Jenkins2013-04-1822-529/+529
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ib960a546f1d7acff887f89410558e49d67b9ae26
* | | | | | | | Merge "removing unused variable from a test"Jenkins2013-04-171-1/+0
|\ \ \ \ \ \ \ \
| * | | | | | | | removing unused variable from a testtilottama gaat2013-04-171-1/+0
| | |_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | Change-Id: I4b5613ae756bac794f82dec4abfa1361efac4054
* | | | | | | | Merge "Remove orphaned db method instance_test_and_set"Jenkins2013-04-171-9/+0
|\ \ \ \ \ \ \ \ | |_|_|_|_|/ / / |/| | | | | | |
| * | | | | | | Remove orphaned db method instance_test_and_setHans Lindgren2013-04-151-9/+0
| | |_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit 86544fad81f95b74407a76bea8b081f490e2832f removed the only use of this method together with the sqlalchemy implementation of it but forgot to remove it from db.api. Change-Id: I751f99ee64cdeab56f1f3fd9f846157f241804a2
* | | | | | | Merge "Added comments to quantum api client"Jenkins2013-04-171-4/+32
|\ \ \ \ \ \ \
| * | | | | | | Added comments to quantum api clientVic Howard2013-04-161-4/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added a few comments for bug 1020184 Change-Id: I6bc59b5b8b57685aae461e5b1111d8dbd168123d
* | | | | | | | Merge "Fix bug in os-availability-zone extension."Jenkins2013-04-172-12/+31
|\ \ \ \ \ \ \ \ | |_|_|/ / / / / |/| | | | | | |
| * | | | | | | Fix bug in os-availability-zone extension.Andrea Rosa2013-04-112-12/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1167956. The call to get the list of availability zones (GET V2/os-availability-zone) doesn't hide internal availability zone if all services in that zone are disabled. Change-Id: I2c49ad68c2ee5ddea0536dfa770f649fbb392b63
View Lorry Controller Status