lp:~blamar/nova/pylint-deprecations

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:~hudson-openstack/nova/trunk at revision 832

Devin Carlen (community): Approve on 2011-03-18

justinsb (community): Approve on 2011-03-18

Christian Berendt (community): Approve on 2011-03-18

Diff: 500 lines (+49/-49)

24 files modified

bin/nova-ajax-console-proxy (+1/-1)
bin/nova-api (+1/-1)
bin/nova-direct-api (+1/-1)
bin/nova-instancemonitor (+1/-1)
bin/nova-objectstore (+1/-1)
nova/auth/fakeldap.py (+5/-5)
nova/auth/ldapdriver.py (+1/-1)
nova/auth/manager.py (+1/-1)
nova/compute/manager.py (+2/-2)
nova/db/api.py (+1/-1)
nova/db/base.py (+1/-1)
nova/db/sqlalchemy/api.py (+1/-1)
nova/network/linux_net.py (+3/-3)
nova/network/manager.py (+2/-2)
nova/objectstore/handler.py (+7/-7)
nova/rpc.py (+3/-3)
nova/service.py (+1/-1)
nova/tests/api/test_wsgi.py (+1/-1)
nova/tests/hyperv_unittest.py (+1/-1)
nova/tests/objectstore_unittest.py (+5/-5)
nova/tests/test_api.py (+1/-1)
nova/tests/test_middleware.py (+2/-2)
po/nova.pot (+5/-5)
tools/euca-get-ajax-console (+1/-1)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

830. By Brian Lamar on 2011-03-18

enable-msg -> enable

829. By Brian Lamar on 2011-03-18

disable-msg -> disable

828. By Anthony Young on 2011-03-18

Make "ApiError" the default error code for ApiError instances, rather than "Unknown."

Note Dashoard's error handling code:

        except boto.exception.BotoServerError, e:
            if e.status == 400 and e.error_code == 'ApiError':
                raise NovaApiError(e)
            elif e.status == 401:
                raise NovaUnauthorizedError()
            elif e.status == 503:
                raise NovaUnavailableError(e)
            raise NovaServerError(e)

So an error_code of 'Unknown' will raise an ugly exception.

This change also makes the euca errors a bit prettier:

> euca-attach-volume -i i-1 vol-2 -d /dev/vdb
> ApiError: ApiError: Volume status must be available

Rather than:

> euca-attach-volume -i i-1 vol-2 -d /dev/vdb
> Unknown: Unknown: Volume status must be available

Though the reiteration of the error code is still a bit ugly.

827. By Todd Willey on 2011-03-18

When changing the project manager, if the new manager is not yet a project member, be sure to make them be a project member.

826. By Todd Willey on 2011-03-18

Make the rpc cast/call debug calls show what topic they are sending to. This aides in debuugging.

825. By Dan Prince on 2011-03-17

Update the Openstack API to handle case where personality is set but null in the request to create a server.

824. By Soren Hansen on 2011-03-17

Fix a couple of things that assume that libvirt == kvm/qemu.

823. By Soren Hansen on 2011-03-17

Fix a number of place in the volume driver where the argv hadn't been fully split

822. By Soren Hansen on 2011-03-17

Make utils.execute not overwrite std{in,out,err} args to Popen on retries.
Make utils.execute reject unknown kwargs.

Add a couple of unit tests for utils.execute.

821. By Naveed Massjouni on 2011-03-17

- general approach for openstack api versioning
- openstack api version now preserved in request context
- added view builder classes to handle os api responses
- added imageRef and flavorRef to os api v1.1 servers
- modified addresses container structure in os api v1.1 servers