keystone::client should install python-openstackclient

[Matthew Mosesohn]

If I'm deploying Cinder on a host without Keystone service, I should include keystone::client class. However, since Kilo, the cinder::keystone::auth calls keystone::resource::service_identity, which needs python-openstackclient.
Refs:
https://github.com/openstack/puppet-cinder/blob/master/manifests/keystone/auth.pp#L282
https://github.com/openstack/puppet-keystone/blob/master/manifests/client.pp

[Emilien Macchi (emilienm)]

In fact, openstackclient is installed with https://github.com/openstack/puppet-keystone/blob/master/manifests/init.pp#L541-L547

We might need to:

1/ add the code to keystone::client
2/ use ensure_resource to avoid resource duplication.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/209965
Committed: https://git.openstack.org/cgit/openstack/puppet-keystone/commit/?id=ff89c950a059b51419fd7c015416987a5d86fe7c
Submitter: Jenkins
Branch: master

commit ff89c950a059b51419fd7c015416987a5d86fe7c
Author: Matthew Mosesohn <email address hidden>
Date: Thu Aug 6 17:41:01 2015 +0300

    Move openstackclient install to keystone::client

    Openstackclient should be installed in the client
    class, not the main class. It should be possible
    to use keystone as a client without the main class.
    For example, it may be necessary to create cinder
    keystone endpoints on a Cinder node, but not
    require all of Keystone.

    Change-Id: Ic9e688d698adffb24f99e9d75a8754391940a7b4
    Partial-Bug: #1479317

[Matthew Mosesohn (raytrac3r)]

8.0 task is just backport https://review.openstack.org/209965

[Ivan Berezovskiy (iberezovskiy)]

This patch is already here (after switching on librarian https://review.openstack.org/#/c/226290/)

[Timur Nurlygayanov (tnurlygayanov)]

Now we have OpenStack client in 8.0 environments.
Marked as Fix Released for 8.0