summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | | | | Move DB thread pooling to DB API.Chris Behrens2013-02-215-42/+111
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The eventlet db_pool wrapping done in sqlalchemy is wrong. It needs to be around the whole DB API call. Fixes bug 1128605 Syncs nova with the fix in oslo. Moves nova to use the oslo version of db/api.py to import the DB API implementation. Renames configuration option sql_dbpool_enable to dbapi_use_tpool NOTE: tpool will not work without a fix to eventlet for Threads + locking. DocImpact Change-Id: I6c75b6138d38d12261d133f2cb2f5e59c667f837
* | | | | | | | | | | Merge "Make add-fixed-ip update nwfilter wth in libvirt"Jenkins2013-02-211-0/+3
|\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | |
| * | | | | | | | | | Make add-fixed-ip update nwfilter wth in libvirtChet Burgess2013-02-201-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add inject_network_info to the libvirt driver so that the add-fixed-ip call will trigger a refresh of the nwfilters. Change-Id: I00eccf34b8f92fb1e26a72e4bddb47dd39c4558f Fixes: bug #1124722
* | | | | | | | | | | Merge "Add an extension to show image size."Jenkins2013-02-2122-5/+955
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Add an extension to show image size.Andrew Laski2013-02-1922-11/+961
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds image size data that is returned from Glance into the API response. This becomes really useful for users when they take a snapshot and want to track the storage used by it. Since this information is already being pulled by Nova we should display it. DocImpact Change-Id: I8a2d4001c21bcc54cac1a2992034dfc9fbe39d7e
* | | | | | | | | | | | Merge "Retry bw_usage_update() on innodb Deadlock"Jenkins2013-02-211-0/+37
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | Retry bw_usage_update() on innodb DeadlockChris Behrens2013-02-191-0/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds a new decorator _retry_on_deadlock() to sqlalchemy api. This patch makes bw_usage_update() use it. Fixes bug 1129622 Change-Id: I0293c62d2dd5ac036445bc639cabbd05ba016e83
* | | | | | | | | | | | | Merge "Fix message envelope keys."Jenkins2013-02-211-4/+4
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|_|_|/ |/| | | | | | | | | | | |
| * | | | | | | | | | | | Fix message envelope keys.Russell Bryant2013-02-201-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | update.py from oslo-incubator broke the keys used in the rpc message envelope. This fixes it. Change-Id: Idabd18720cad8c4469c5dc3ec1ffd20a3a39e897
* | | | | | | | | | | | | Merge "Move some context checking code from sqlalchemy"Jenkins2013-02-215-68/+84
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|/ / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Move some context checking code from sqlalchemyChris Behrens2013-02-205-68/+84
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move some context related code from sqlalchemy into nova/context.py where it can be used outside of sqlalchemy. Change-Id: I6522a072132e27b42561435cb4fd671a2ece4867
* | | | | | | | | | | | | Merge "Fix pep8 issues with test_manager.py"Jenkins2013-02-211-1/+1
|\ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | Fix pep8 issues with test_manager.pyChris Yeoh2013-02-201-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes E231 (missing whitespace) problems with nova/tests/network/test_manager.py Change-Id: I99d4c3bf7ed1a137fbfb26e9fe2f31d9104d5ae4
* | | | | | | | | | | | | | Merge "More gracefully handle TimeoutException in test"Jenkins2013-02-211-2/+11
|\ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | More gracefully handle TimeoutException in testChris Yeoh2013-02-201-2/+11
| |/ / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sometimes although the test_killed_worker_recover test passes, the cleanup of a forked child takes too long and a TimeoutException occurs. This patch better handles the TimeoutException and ensures the child process is killed. Fixes bug #1128574 Change-Id: I39ea2fbb6c9cb95ad10b68bfa216686467951a93
* | | | | | | | | | | | | | Merge "ensure we run db tests in CI"Jenkins2013-02-211-2/+9
|\ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | ensure we run db tests in CISean Dague2013-02-201-2/+9
| | |_|_|_|_|_|_|_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | CI had a set of environmental variables that refused to allow the tests from failing gracefully. These seem to have gotten lost along the way in refactoring. Change-Id: Ib19b5d472c51ed72b8acf1571dd19f41474532c5
* | | | | | | | | | | | | | Merge "libvirt: fix volume walk of /dev/disk/by-path"Jenkins2013-02-211-2/+5
|\ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | libvirt: fix volume walk of /dev/disk/by-pathVishvananda Ishaya2013-02-201-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If /dev/disk/by-path does not exist, then _get_iscsi_devices() will raise an IndexError. Handle this case and return an empty list. This fixes a test on OSX. Fixes bug 1130841 Change-Id: Ibae2f1238feac9a10b8dc2b74886ea59edac0bcc
* | | | | | | | | | | | | | | Merge "Removed duplicate spawn code in PowerVM driver"Jenkins2013-02-211-78/+24
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | Removed duplicate spawn code in PowerVM driverLance Bragstad2013-02-201-78/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After introducing resize/migrate in PowerVM there was some code that was duplicated after a rebase. The commit removes the duplicate code. The functionality of resize/migrate remains the same as introducted in Change I72345626 Bug #1130680 Change-Id: I1fe3d8e1f2a19fd40a03c8c38d90c01711356b38
* | | | | | | | | | | | | | | | Merge "Remove race condition (in Networks)"Jenkins2013-02-214-9/+76
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | Remove race condition (in Networks)Boris Pavlovic2013-02-214-9/+76
| | |_|_|_|/ / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Soft delete all duplicate rows with the same vlan except one with the biggest value in `id` column. Create UC on fields (vlan, deleted) Replace Select then Insert -> Try to Insert. Fix error in network_update method. We were able to update vlan to duplicate. Fix typo in description of method drop_old_duplicate_entries_from_table() blueprint db-enforce-unique-keys Change-Id: Ia1b58ab5a5e369a1939de0b5c2f89aee6fc2c40b
* | | | | | | | | | | | | | | | Merge "Baremetal driver returns accurate list of instance"Jenkins2013-02-214-13/+88
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | Baremetal driver returns accurate list of instanceDevananda van der Veen2013-02-204-13/+88
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add 'instance_name' to bm_nodes table so that baremetal driver is able to return the names of all instances it believes are still running. Previously, baremetal.driver.list_instances was fetching all allocated instances from baremetal database, then calling VirtAPI to get the instance name. This would raise an InstanceNotFound exception for deleted instances. This prevented ComputeManager from ever detecting a running-but-deleted baremetal instance, and could leave baremetal instances in an undeletable state. Fixes bug 1096723. Change-Id: Ifae532e8e70e97e48c589608cb3c7000bb6a7609
* | | | | | | | | | | | | | | | | Merge "Identify baremetal nodes by UUID."Jenkins2013-02-2110-43/+123
|\| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | Identify baremetal nodes by UUID.Devananda van der Veen2013-02-2010-43/+123
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - add a new 'uuid' column to 'bm_nodes' table - provide a new db/api method for accessing nodes by their uuid - return this to Nova as the nodename / hypervisor_hostname In this way, a baremetal node is uniquely identified to the user, whether they use a per-compute baremetal db or global baremetal db. It will also allow for an instance to be booted on a specific baremetal node using a means akin to the current force-hosts (TBD in later patch). Also, creates two new Exceptions to disambiguate between InstanceNotFound and NodeNotFound / NodeNotFoundByUUID. Change-Id: I81105a201588fdef31cffabdae260bb43017bcd1
* | | | | | | | | | | | | | | | | Merge "Improve performance of baremetal list_instances."Jenkins2013-02-215-71/+179
|\| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | Improve performance of baremetal list_instances.Devananda van der Veen2013-02-205-71/+179
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds two new methods to baremetal/db/api: - bm_node_get_associated - bm_node_get_unassociated which return a list of nodes either associated or not associated to a Nova instance, as determined by the instance_uuid field, respectively. Now, list_instances and get_available_nodes methods only fetch assocated and unassociated nodes from the baremetal database, instead of fetching all nodes. Change-Id: Ib875d26d7b85a9dbf08c925a89802e2cd614e363
* | | | | | | | | | | | | | | | | Merge "Better error handling in baremetal spawn & destroy"Jenkins2013-02-212-80/+148
|\| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | Better error handling in baremetal spawn & destroyDevananda van der Veen2013-02-202-80/+148
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improve the exception handling in baremetal driver spawn and destroy. Adds unit tests for destroy to check that bm_node is disassociated from instance_uuid only when destroy succeeds, and is set to ERROR state (but still associated) if any exception is not handled. Change-Id: I75853210d4883b352da9876a682dbb7288689541
* | | | | | | | | | | | | | | | | Merge "Wait for baremetal deploy inside driver.spawn"Jenkins2013-02-214-15/+98
|\| | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | Wait for baremetal deploy inside driver.spawnDevananda van der Veen2013-02-204-15/+98
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, baremetal driver.spawn returned as soon as the machine power turned on, but before the user-image was deployed to the hardware node, and long before the node was available on the network. This meant the nova instance was marked as ACTIVE before provisioning had actually finished. If the deploy failed and the baremetal node was set to an ERROR state, the nova instance could still be left as ACTIVE and the user was never informed of the error. This patch introduces a LoopingCall to monitor the deployment status in the baremetal database. As the deployment is performed by nova-baremetal-deploy-helper, the database record is updated. Once the deployment is complete, driver.spawn() sets the baremetal node status and the nova instance status is also set properly. If an error occurs during the deployment, an exception is raised within driver.spawn() allowing nova to follow the normal cleanup and notify paths. This also allows the baremetal PXE driver to delete cached image files when a baremetal deployment fails. Fixes bug 1088655. Change-Id: I4feefd462fd956c9780995ec8b05b13e78278c8b
* | | | | | | | | | | | | | | | | Merge "Add Nova quantum security group proxy"Jenkins2013-02-2113-86/+1317
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | | Add Nova quantum security group proxyAaron Rosen2013-02-2013-86/+1317
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implements blueprint nova-quantum-security-group-proxy Change-Id: I21d70f1dc8e61d6412d14e30ab2aa1a83a711de2
* | | | | | | | | | | | | | | | | | Merge "Make nova security groups more pluggable"Jenkins2013-02-2110-238/+419
|\| | | | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | | | Make nova security groups more pluggableAaron Rosen2013-02-2010-238/+419
| | |_|_|_|_|_|_|_|_|_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch moves the nova security group code out of nova/compute/api.py into nova/network/security_group. It also removes any query to the database from security group api into the nova security group driver. This allows security group drivers the ability to decouple themselves from storing security group information in the nova_db. Change-Id: Ib183515a0418203c8bcc88176e3a1498d7333300
* | | | | | | | | | | | | | | | | Merge "Refactor nwfilter parameters"Jenkins2013-02-204-62/+117
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|/ / / / / / / / / |/| | | | | | | | | | / / / / / / | | |_|_|_|_|_|_|_|_|/ / / / / / | |/| | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | Refactor nwfilter parametersChet Burgess2013-02-204-62/+117
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * libvirt < 1.0.1 does not allow dynamically updating filter parameters inside an interface block of theinstance XML. To address this we move the nwfilter paramters from the instance XML to the per-instance base nwfilter. Relates to bug #1124722 Change-Id: I50ec5dcffa8ff0232ae0aba0a08fce5151962238
* | | | | | | | | | | | | | | | Merge "cfg should be imported from oslo.config"Jenkins2013-02-201-1/+2
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | | cfg should be imported from oslo.configAaron Rosen2013-02-201-1/+2
| | |/ / / / / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Idd9c3ededd2f63156eb78ed01fd38ddb3a20437f
* | | | | | | | | | | | | | | | Merge "Add a volume driver in Nova for Scality SOFS"Jenkins2013-02-204-1/+124
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / / / / / / / |/| | | | | | | | | | | | | | |
| * | | | | | | | | | | | | | | Add a volume driver in Nova for Scality SOFSJean-Marc Saffroy2013-02-204-1/+124
| | |_|_|_|/ / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Scality SOFS is a network filesystem mounted with FUSE, with most options given in a configuration file. Given a mount point and a SOFS configuration file as driver options, the Scality volume driver mounts SOFS, and then creates, accesses and deletes volumes as regular (sparse) files on SOFS. Change-Id: I84bf268e5a2c5c33b8706830e8067914fae44aed Implements: blueprint scality-volume-driver
* | | | | | | | | | | | | | | Merge "Add better status to baremetal deployments."Jenkins2013-02-2011-205/+407
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / / / / |/| | | / / / / / / / / / / / | | |_|/ / / / / / / / / / / | |/| | | | | | | | | | | |
| * | | | | | | | | | | | | Add better status to baremetal deployments.Devananda van der Veen2013-02-2011-205/+407
| | |/ / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch introduces a few new baremetal states, which are used to track the deploy process. Now, nova-baremetal-deploy-helper updates the bm_nodes record directly when it begins and finishes deploying an image to that node. The next patch will add a LoopingCall inside driver.spawn() to wait for the deploy to complete. Also, since there can not be >1 active deployment per node, there is no need to have a separate table for storing them. This patch drops the table bm_deployments and adds the important information it contained to bm_nodes. Since the previous behavior was to mark a deployment as deleted once it completed, there is no need to copy any data from bm_deployments prior to dropping the table -- assuming that no active deployments are in process when the migration is run. Since this is the first migration for the baremetal database, it also adds a new test class, TestBaremetalMigrations, and refactors the test_migrations.py file to allow for multiple test classes. partially implements fix for bug 1096723 Change-Id: Iad30b462d49c88fc19babed43a2fb8540b1fad30
* | | | | | | | | | | | | Merge "Fix handling of source_groups with no-db-compute."Jenkins2013-02-204-5/+11
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Fix handling of source_groups with no-db-compute.Vishvananda Ishaya2013-02-204-5/+11
| | |_|_|_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The moving of data around in no-db-compute broke source groups. This is a combination of a few errors. First, the instance_type data wasn't being retrieved from system_metadata. Second, the instance_type data was too deep in the heirarchy and being ignored. Finally, source groups require a nw info call which causes a db access by nova-compute. Fixes bug 1122316 Change-Id: Iccb6e5d336c0b2c8ba41c41ab2e046fc9617835a
* | | | | | | | | | | | Merge "Prevent the unexpected with nova-manage network modify."Jenkins2013-02-201-4/+21
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / |/| | | | | | | | | | |
| * | | | | | | | | | | Prevent the unexpected with nova-manage network modify.Brent Eagles2013-02-181-4/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "nova-manage network modify" can misinterpret arguments if the user provides values where they are unexpected, specifically the --disassociate-project and --disassociate-host options. If the user provides a project or hostname value for either of these values, it will be interpreted as a positional argument and modify handler may behave as if other options were provided (e.g. project or host). This change handles the presence of a project or host option along with --disassociate-project or --disassociate-host as an error, printing a message and exiting immediately with an error code. While there are other approaches (e.g. perform the disassociate modifications while ignoring the others), preventing unexpected or inconsistent results by doing nothing seems safest. Change-Id: Iff7817fe3ed19a4ff330b4029f9c558d3b405e9b
* | | | | | | | | | | | Merge "Add API Sample tests for Hypervisors extension."Jenkins2013-02-2029-1/+310
|\ \ \ \ \ \ \ \ \ \ \ \