create_user throws RGWAdminException instead of UserExists

[vergeev]

Steps to reproduce:

from rgwadmin import RGWAdmin

rgw = RGWAdmin(access_key='XXX', secret_key='XXX', server='obj.example.com')
rgw.create_user(
    uid='liam',
    display_name='Liam Monahan',
    email='[email protected]',
    user_caps='usage=read, write; users=read',
    max_buckets=1000)
rgw.create_user(
    uid='liam',
    display_name='Liam Monahan',
    email='[email protected]',
    user_caps='usage=read, write; users=read',
    max_buckets=1000)

Expected result: the last line throws rgwadmin.exceptions.UserExists.

Actual result: the last line throws rgwadmin.exceptions.RGWAdminException with code="UserAlreadyExists".

Why I think this is happening: the docs say the error response is UserExists, but the rgw code actually produces UserAlreadyExists.

Solution I propose:

1. Add UserAlreadyExists exception, replace with it UserExists here:

   rgwadmin/rgwadmin/rgw.py

   Line 131 in 4f90b6a

   for e in [AccessDenied, UserExists, InvalidAccessKey,

2. Make UserExists an alias of UserAlreadyExists in order to maintain backwards compatibility.

Can I go ahead and open a pull request with a fix?

[dyarnell]

Hi Pavel,

I think this is reasonable. The UserAlreadyExists goes back at least to the 12.x.x version of Ceph as the exception in the code. Please provide a pull request and I will be happy to merge.

[vergeev]

Here's the PR: #67. Please let me know if you want me to add anything.

[dyarnell]

This was released as part of https://github.com/UMIACS/rgwadmin/releases/tag/2.4.4 and is available on PyPI now.