Skip to content

keystone: Stop syncing signing keys (SCRD-781)#1204

Merged
toabctl merged 1 commit into
crowbar:masterfrom
cmurphy:keystone-pike
Aug 30, 2017
Merged

keystone: Stop syncing signing keys (SCRD-781)#1204
toabctl merged 1 commit into
crowbar:masterfrom
cmurphy:keystone-pike

Conversation

@cmurphy

@cmurphy cmurphy commented Aug 29, 2017

Copy link
Copy Markdown
Contributor

PKI tokens have been removed from keystone, and the pki_setup command
that used to generate the initial signing key pair has been removed from
the keystone package[1]. Since there is no longer an initial signing key
pair at the point where the keystone barclamp is supposed to try to
synchronize them, the keystone barclamp fails on pike with "No such file
or directory". If a key pair is needed to HTTPS, the barclamp will
generate it, but it is no longer needed for token functionality, so we
can just remove the synchronization steps.

[1] https://build.opensuse.org/package/rdiff/Cloud:OpenStack:Master/openstack-keystone?linkrev=base&rev=1001

stefannica
stefannica previously approved these changes Aug 29, 2017
PKI tokens have been removed from keystone, and the pki_setup command
that used to generate the initial signing key pair has been removed from
the keystone package[1]. Since there is no longer an initial signing key
pair at the point where the keystone barclamp is supposed to try to
synchronize them, the keystone barclamp fails on pike with "No such file
or directory". If a key pair is needed to HTTPS, the barclamp will
generate it, but it is no longer needed for token functionality, so we
can just remove the synchronization steps.

[1] https://build.opensuse.org/package/rdiff/Cloud:OpenStack:Master/openstack-keystone?linkrev=base&rev=1001
@cmurphy

cmurphy commented Aug 29, 2017

Copy link
Copy Markdown
Contributor Author

Updated, I think the sync marks can go away too

@stefannica stefannica left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point, I was just about to suggest that you remove the crowbar sync too 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants