Skip to content

Conversation

@jadonn
Copy link
Contributor

@jadonn jadonn commented Jul 3, 2024

This change makes z-o-t pass in the value of idp_id when configuring the Keystone OpenIDC charm during testing.

(cherry picked from commit 3a6a3f8)

This change makes z-o-t pass in the value of idp_id when configuring the Keystone OpenIDC charm during testing.

(cherry picked from commit 3a6a3f8)
@jadonn
Copy link
Contributor Author

jadonn commented Jul 3, 2024

@ajkavanagh ajkavanagh changed the title Added required config option idp_id to Keystone OpenIDC charm [bobcat] Added required config option idp_id to Keystone OpenIDC charm Jul 3, 2024
Copy link
Member

@freyes freyes left a comment

Choose a reason for hiding this comment

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

LGTM.

@freyes freyes requested a review from gboutry October 29, 2024 10:43
openstack-mirroring pushed a commit to openstack/charm-keystone-openidc that referenced this pull request Apr 2, 2025
The values of the protocol ID and identity provider ID are
reversed in the charm's code. This causes the mod_auth_openidc
config to render incorrectly in the Apache template. This change
fixes the reversed values and adds a config option for
defining the identity provider ID. This change also adds new
config options for enabling using mod_auth_openidc with
multiple Keystone units and with proxies.

Closes-Bug: #2065590
Func-Test-Pr: openstack-charmers/zaza-openstack-tests#1240
Change-Id: Ie7da5f9a85027a287aad9958c54848a948f8495c
(cherry picked from commit da043cb)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants