magnum wrong keystone section used

[Cyril Lopez]

Section of keystone information is set in keystone_authtoken instead of keystone_auth and create this issue following.

Oct 30 09:25:53 weasel.isim.intra magnum-conductor[1871601]: 2020-10-30 09:25:53.483 1871643 WARNING magnum.common.cert_manager.local_cert_manager [req-f6ad9d42-d4b4-4f04-b59a-17f24283f1d0 -
 - - - -] Storing certificate data on the local filesystem. CertManager type 'local' should be used for testing purpose.
Oct 30 09:25:53 weasel.isim.intra magnum-conductor[1871601]: 2020-10-30 09:25:53.485 1871643 ERROR magnum.conductor.handlers.common.cert_manager [req-f6ad9d42-d4b4-4f04-b59a-17f24283f1d0 - $
 - - -] Failed to generate certificates for Cluster: 31765bbb-660a-4844-9516-22b90ff42f3a: TypeError: write() argument must be str, not bytes
Oct 30 09:25:53 weasel.isim.intra magnum-conductor[1871601]: 2020-10-30 09:25:53.485 1871643 ERROR magnum.conductor.handlers.common.cert_manager Traceback (most recent call last):
Oct 30 09:25:53 weasel.isim.intra magnum-conductor[1871601]: 2020-10-30 09:25:53.485 1871643 ERROR magnum.conductor.handlers.common.cert_manager File "/usr/lib/python3.6/site-packages/mag$um/conductor/handlers/common/cert_manager.py", line 112, in generate_certificates_to_cluster
Oct 30 09:25:53 weasel.isim.intra magnum-conductor[1871601]: 2020-10-30 09:25:53.485 1871643 ERROR magnum.conductor.handlers.common.cert_manager context=context)
Oct 30 09:25:53 weasel.isim.intra magnum-conductor[1871601]: 2020-10-30 09:25:53.485 1871643 ERROR magnum.conductor.handlers.common.cert_manager File "/usr/lib/python3.6/site-packages/mag$
um/conductor/handlers/common/cert_manager.py", line 49, in _generate_ca_cert
Oct 30 09:25:53 weasel.isim.intra magnum-conductor[1871601]: 2020-10-30 09:25:53.485 1871643 ERROR magnum.conductor.handlers.common.cert_manager context=context,
Oct 30 09:25:53 weasel.isim.intra magnum-conductor[1871601]: 2020-10-30 09:25:53.485 1871643 ERROR magnum.conductor.handlers.common.cert_manager File "/usr/lib/python3.6/site-packages/mag$
um/common/cert_manager/local_cert_manager.py", line 85, in store_cert
Oct 30 09:25:53 weasel.isim.intra magnum-conductor[1871601]: 2020-10-30 09:25:53.485 1871643 ERROR magnum.conductor.handlers.common.cert_manager cert_file.write(certificate)
Oct 30 09:25:53 weasel.isim.intra magnum-conductor[1871601]: 2020-10-30 09:25:53.485 1871643 ERROR magnum.conductor.handlers.common.cert_manager TypeError: write() argument must be str, not
bytes
Oct 30 09:25:53 weasel.isim.intra magnum-conductor[1871601]: 2020-10-30 09:25:53.485 1871643 ERROR magnum.conductor.handlers.common.cert_manager

openstack-packstack-puppet-16.0.2-1.el8.noarch
openstack-packstack-16.0.2-1.el8.noarch
openstack-magnum-ui-6.0.0-2.el8.noarch
openstack-magnum-common-10.0.0-1.el8.noarch
python3-magnum-10.0.0-1.el8.noarch
python3-magnumclient-3.0.1-1.el8.noarch
openstack-magnum-conductor-10.0.0-1.el8.noarch
openstack-magnum-api-10.0.0-1.el8.noarch
puppet-magnum-16.3.0-1.el8.noarch

[Javier Peña (jpena-c)]

Thanks for the bug report!

I think the issue comes from https://review.opendev.org/659774 and https://review.opendev.org/758981, since we had not adapted to that change. I have proposed https://review.opendev.org/760644 with a fix.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/760644
Committed: https://git.openstack.org/cgit/x/packstack/commit/?id=745214d0a7941038ff04775e63c339305464246b
Submitter: Zuul
Branch: master

commit 745214d0a7941038ff04775e63c339305464246b
Author: Javier Pena <email address hidden>
Date: Fri Oct 30 18:42:30 2020 +0100

    Use magnum::keystone::keystone_auth for magnum config

    Puppet-magnum removed the old fallback method of configuring
    the service user with [1], and added a new one using keystone_auth
    with [2]. This commit uses the new magnum::keystone::keystone_auth
    class to fix the configuration.

    Note that the change started in stable/ussuri, so this patch should
    be backported all the way back to that release.

    Closes-Bug: #1902252

    [1] - https://review.opendev.org/659774
    [2] - https://review.opendev.org/758981

    Change-Id: I81464c3267190b3d21b99796ac42f30b6bc86cb7