Skip to content

Default identity-endpoint to v3 if application credentials will be used#93

Open
ProbstDJakob wants to merge 1 commit intosyseleven:masterfrom
ProbstDJakob:default-to-v3-with-app-credentials
Open

Default identity-endpoint to v3 if application credentials will be used#93
ProbstDJakob wants to merge 1 commit intosyseleven:masterfrom
ProbstDJakob:default-to-v3-with-app-credentials

Conversation

@ProbstDJakob
Copy link
Copy Markdown
Contributor

Application credentials need the api version 3 to be supported, thus default to v3/ if no version has been specified in the url and application credentials have been passed.

Copy link
Copy Markdown
Collaborator

@gtema gtema left a comment

Choose a reason for hiding this comment

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

there is absolutely no reason to keep v2.0 at all, for Keystone take v3/ as granted (actually perhaps even v4 will come up soon). So I would rather simply replace v2.0/ with v3 without any conditions

@ProbstDJakob ProbstDJakob force-pushed the default-to-v3-with-app-credentials branch from 432641f to ba279d5 Compare April 30, 2026 13:52
@ProbstDJakob
Copy link
Copy Markdown
Contributor Author

Done :)

@ProbstDJakob
Copy link
Copy Markdown
Contributor Author

By the way, your helm charts reference images without a registry (alpine and syseleven/designate-certmanager-webhook) and this is not allowed in OpenShift/OKD >=4.21.0 any more. I've opened #134 to fix this. Would be nice if this could get also merged.

@ProbstDJakob ProbstDJakob force-pushed the default-to-v3-with-app-credentials branch from ba279d5 to 957d904 Compare April 30, 2026 23:44
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