diff options
| author | Chris Behrens <cbehrens@codestud.com> | 2011-05-25 20:31:05 +0000 |
|---|---|---|
| committer | Tarmac <> | 2011-05-25 20:31:05 +0000 |
| commit | 9ec9a8b39db2d65b7ce8d74f994cb32c49a56df8 (patch) | |
| tree | 9fa99a51a97288cfddd4a4cdd040d3d7e7cf7480 /nova/tests | |
| parent | a2505ee554052ee591d1a79c2329c50b31a43dc4 (diff) | |
| parent | bd0b4b87da9e960042c3d0caf00370ef526ce8b7 (diff) | |
During the API create call, the API would kick off a build and then loop in a greenthread waiting for the scheduler to pick a host for the instance. After API would see a host was picked, it would cast to the compute node's set_admin_password method.
The API server really should not have to do this. The password to set should be pushed along with the build request, instead. The compute node can then set the password after it detects the instance has booted. This removes a greenthread from the API server, a loop that constantly checks the DB for the host, and finally a cast to the compute node.
Diffstat (limited to 'nova/tests')
| -rw-r--r-- | nova/tests/api/openstack/test_servers.py | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/nova/tests/api/openstack/test_servers.py b/nova/tests/api/openstack/test_servers.py index dc8815845..fbde5c9ce 100644 --- a/nova/tests/api/openstack/test_servers.py +++ b/nova/tests/api/openstack/test_servers.py @@ -774,8 +774,7 @@ class ServersTest(test.TestCase): def server_update(context, id, params): filtered_dict = dict( - display_name='server_test', - admin_pass='bacon', + display_name='server_test' ) self.assertEqual(params, filtered_dict) return filtered_dict |