cinder-api in master role test fails

Bug #1721037 reported by Jean-Philippe Evrard
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack-Ansible
In Progress
Critical
Jean-Philippe Evrard

Bug Description

running a ./run_tests.sh from openstack-ansible-os_cinder fails on cinder-api online verification.

Indeed haproxy sees the cinder-api as DOWN, and the logs of cinder-api is the following:
http://paste.openstack.org/show/622470/

Revision history for this message
Jean-Philippe Evrard (jean-philippe-evrard) wrote :

This also breaks magnum test.

Changed in openstack-ansible:
status: New → Confirmed
importance: Undecided → Critical
Revision history for this message
Jimmy McCrory (jimmy-mccrory) wrote :

castellan is providing a default for key_manager/backend, 'barbican'
https://github.com/openstack/castellan/blob/0.14.0/castellan/key_manager/__init__.py#L24-L30

cinder is expecting that value to be an importable class
https://github.com/openstack/cinder/blob/master/cinder/keymgr/__init__.py#L29

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-ansible-os_cinder (master)

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

Changed in openstack-ansible:
assignee: nobody → Jean-Philippe Evrard (jean-philippe-evrard)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-ansible-os_cinder (master)

Reviewed: https://review.openstack.org/509768
Committed: https://git.openstack.org/cgit/openstack/openstack-ansible-os_cinder/commit/?id=9ad69a9e9d86abdd192f9d418f189d71c170bc59
Submitter: Jenkins
Branch: master

commit 9ad69a9e9d86abdd192f9d418f189d71c170bc59
Author: Jean-Philippe Evrard <email address hidden>
Date: Thu Oct 5 12:48:56 2017 +0000

    Workaround bad cinder defaults

    key_manager/backend default value is 'barbican'.
    While castellan resolves this to a class using stevedore,
    cinder loads it tries to load it directly.
    So it fails if the value of backend is not set to a full class.

    Change-Id: Ia8417bbdac3f515e42d1ed760110a63ae14f8f00
    Partial-Bug: #1721037

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.