cinder.tests.keymgr.test_barbican fails with barbicanclient 3.0.0

[Alan Pevec]

  File "cinder/tests/keymgr/test_barbican.py", line 25, in <module>
    from barbicanclient.common import auth
ImportError: cannot import name auth

auth was removed in "Refactored Client to use Keystone Sessions"
http://git.openstack.org/cgit/openstack/python-barbicanclient/commit/?id=ddec180c419132589fac4d689ec1904b1b698f24

[Alan Pevec (apevec)]

Detected in periodic job
http://logs.openstack.org/periodic-stable/periodic-cinder-python27-juno/d9bf4cb/testr_results.html.gz

[Brianna Poulos (brianna-poulos)]

Thank you for assigning the bug to me. I will update test_barbican.py and barbican.py in cinder to work with the new barbican client (3.0.0) this week.

[Juan Antonio Osorio Robles (juan-osorio-robles)]

python-barbicanclient has been fixed to support the global requirements, and the fix is available as of python-barbicanclient 3.0.1

[Ihar Hrachyshka (ihar-hrachyshka)]

I see the failure with python-barbicanclient 3.0.1 here: http://logs.openstack.org/periodic-stable/periodic-cinder-python26-juno/37e6324/console.html

Is there more work to do/patches to merge to fix it?

[Brianna Poulos (brianna-poulos)]

I am working on a patch to fix this, which I will submit by the end of this week.

[Jay Bryant (jsbryant)]

Cinder needed this fixed ASAP so we merged patches for master and Juno under this bug: https://bugs.launchpad.net/cinder/+bug/1389419

[Brianna Poulos (brianna-poulos)]

Thank you for pointing to the other bug. I apologize for the problems with the unit test. I will use this bug to update the unit test to not rely on barbican client, and update barbican.py so that it works with version 3.0.0. (Unless it would be better to separate the two).

[John Griffith (john-griffith)]

No problem at all. Yeah, using that bug and combining the work should be
fine.

Thanks,
John
On Nov 5, 2014 5:55 PM, "Brianna Poulos" <email address hidden> wrote:

> Thank you for pointing to the other bug. I apologize for the problems
> with the unit test. I will use this bug to update the unit test to not
> rely on barbican client, and update barbican.py so that it works with
> version 3.0.0. (Unless it would be better to separate the two).
>
> --
> You received this bug notification because you are a member of Cinder
> Bug Team, which is subscribed to Cinder.
> https://bugs.launchpad.net/bugs/1388461
>
> Title:
> cinder.tests.keymgr.test_barbican fails with barbicanclient 3.0.0
>
> Status in Cinder:
> Confirmed
> Status in Cinder juno series:
> Confirmed
>
> Bug description:
> File "cinder/tests/keymgr/test_barbican.py", line 25, in <module>
> from barbicanclient.common import auth
> ImportError: cannot import name auth
>
> auth was removed in "Refactored Client to use Keystone Sessions"
>
> http://git.openstack.org/cgit/openstack/python-barbicanclient/commit/?id=ddec180c419132589fac4d689ec1904b1b698f24
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/cinder/+bug/1388461/+subscriptions
>

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/133113

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/133113
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=2d52d4177ca7cf41ee8162f07e444d9b75c377dd
Submitter: Jenkins
Branch: master

commit 2d52d4177ca7cf41ee8162f07e444d9b75c377dd
Author: Brianna Poulos <email address hidden>
Date: Thu Nov 6 13:07:16 2014 -0800

    Fix wrapper to work with barbicanclient 3.0.1

    Due to the updates to the python-barbicanclient in version 3.0.1, the cinder
    barbican wrapper no longer functions. This patch updates the cinder barbican
    wrapper and test cases to work with barbicanclient 3.0.1.

    Change-Id: Ibef4abd5fd9b12962420eaa5d7ebf62700171861
    Closes-Bug: #1388461

[Brianna Poulos (brianna-poulos)]

Should this same fix be added to Juno as well?

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/juno
Review: https://review.openstack.org/135336

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/135336
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=fd323fed3562b3f7d2a4b044c0854fe007c1a3d0
Submitter: Jenkins
Branch: stable/juno

commit fd323fed3562b3f7d2a4b044c0854fe007c1a3d0
Author: Brianna Poulos <email address hidden>
Date: Thu Nov 6 13:07:16 2014 -0800

    Fix wrapper to work with barbicanclient 3.0.1

    Due to the updates to the python-barbicanclient in version 3.0.1, the cinder
    barbican wrapper no longer functions. This patch updates the cinder barbican
    wrapper and test cases to work with barbicanclient 3.0.1.

    Change-Id: Ibef4abd5fd9b12962420eaa5d7ebf62700171861
    Closes-Bug: #1388461
    (cherry picked from commit 2d52d4177ca7cf41ee8162f07e444d9b75c377dd)