diff options
| author | Jenkins <jenkins@review.openstack.org> | 2011-12-21 18:53:55 +0000 |
|---|---|---|
| committer | Gerrit Code Review <review@openstack.org> | 2011-12-21 18:53:55 +0000 |
| commit | 8753ddee7fc4b1de569d82d4a3f1ed90afa754ee (patch) | |
| tree | 3a0cb06c9e16c4e443989ab67b464a3c55fe20a2 | |
| parent | ab2b1e15b60f37e7b785052a53d8526309809c59 (diff) | |
| parent | 397933a5464c47c40edde39010e58076045e5545 (diff) | |
| download | nova-8753ddee7fc4b1de569d82d4a3f1ed90afa754ee.tar.gz nova-8753ddee7fc4b1de569d82d4a3f1ed90afa754ee.tar.xz nova-8753ddee7fc4b1de569d82d4a3f1ed90afa754ee.zip | |
Merge "Removing extra code from servers controller"
| -rw-r--r-- | nova/api/openstack/v2/servers.py | 39 |
1 files changed, 8 insertions, 31 deletions
diff --git a/nova/api/openstack/v2/servers.py b/nova/api/openstack/v2/servers.py index e2d84b688..5c49dc725 100644 --- a/nova/api/openstack/v2/servers.py +++ b/nova/api/openstack/v2/servers.py @@ -184,18 +184,6 @@ class Controller(wsgi.Controller): # if the original error is okay, just reraise it raise error - def _deserialize_create(self, request): - """ - Deserialize a create request - - Overrides normal behavior in the case of xml content - """ - if request.content_type == "application/xml": - deserializer = ServerXMLDeserializer() - return deserializer.deserialize(request.body) - else: - return self._deserialize(request.body, request.get_content_type()) - def _validate_server_name(self, value): if not isinstance(value, basestring): msg = _("Server name is not a string or unicode") @@ -233,21 +221,6 @@ class Controller(wsgi.Controller): injected_files.append((path, contents)) return injected_files - def _get_server_admin_password_old_style(self, server): - """ Determine the admin password for a server on creation """ - return utils.generate_password(FLAGS.password_length) - - def _get_server_admin_password_new_style(self, server): - """ Determine the admin password for a server on creation """ - password = server.get('adminPass') - - if password is None: - return utils.generate_password(FLAGS.password_length) - if not isinstance(password, basestring) or password == '': - msg = _("Invalid adminPass") - raise exc.HTTPBadRequest(explanation=msg) - return password - def _get_requested_networks(self, requested_networks): """ Create a list of requested networks from the networks attribute @@ -861,12 +834,16 @@ class Controller(wsgi.Controller): resp.headers['Location'] = image_ref return resp - def get_default_xmlns(self, req): - return common.XML_NS_V11 - def _get_server_admin_password(self, server): """ Determine the admin password for a server on creation """ - return self._get_server_admin_password_new_style(server) + password = server.get('adminPass') + + if password is None: + return utils.generate_password(FLAGS.password_length) + if not isinstance(password, basestring) or password == '': + msg = _("Invalid adminPass") + raise exc.HTTPBadRequest(explanation=msg) + return password def _get_server_search_options(self): """Return server search options allowed by non-admin""" |
