[library] neutron.rb uses cached call to keystone provider and can end up raising nil exception or otherwise fail to find resources created during this run of puppet

[Andrew Woodward]

https://github.com/stackforge/puppet-neutron/blob/master/lib/puppet/provider/neutron.rb#L157-170 can end up raising a nil exception in https://github.com/stackforge/puppet-keystone/blob/master/lib/puppet/provider/keystone_tenant/keystone.rb

changing https://github.com/stackforge/puppet-keystone/blob/master/lib/puppet/provider/keystone_tenant/keystone.rb#L29

from
  @tenant_hash ||= build_tenant_hash
to
 @tenant_hash = build_tenant_hash

was enough to stop the error.

Note this was noted with an much older version of keystone and could possibly not apply any longer, however it will be re-evaluated when we pull the new kesystone module in the next couple of weeks

[Dmitry Ilyin (idv1985)]

It's easy to fix but it will require modificaton of the upstream keystone module. I don't think its woth it.
Or we can send patch to the upstream...

[Andrew Woodward (xarses)]

This is a confirmed issue and must be addressed upstream or in fuel before closing

[Matthew Mosesohn (raytrac3r)]

Status update?

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/134566
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=0ef4afd9f5a78b82e50bb85affcce8afa5421544
Submitter: Jenkins
Branch: master

commit 0ef4afd9f5a78b82e50bb85affcce8afa5421544
Author: Bogdan Dobrelya <email address hidden>
Date: Fri Nov 14 16:25:10 2014 +0100

    Sync fix keystone entities evaluation for cached catalog usage

    Source upstream patch-set: https://review.openstack.org/#/c/134305/

    Closes-bug: #1344293

    Change-Id: I2fc529155c7e6f615fcf1b32247f890266f7d6c9
    Signed-off-by: Bogdan Dobrelya <email address hidden>

[Sergey Vasilenko (xenolog)]

Can't reproduce on #114 and later

[OpenStack Infra (hudson-openstack)]

Change abandoned by Bogdan Dobrelya (<email address hidden>) on branch: master
Review: https://review.openstack.org/134305

[Emilien Macchi (emilienm)]

is this bug still in progress upstream side?

[Takashi Kajinami (kajinamit)]

It seems at some point we stopped using tenant_hash. Also, the latest version uses openstack CLI and doesn't require conversion from project name to project id.