Skip to content

auth: support more scope cases in clouds.yaml parser - #3739

Merged
stephenfin merged 1 commit into
gophercloud:mainfrom
kayrus:cloud-scope
Sep 7, 2026
Merged

stephenfin merged 1 commit into
gophercloud:mainfrom
kayrus:cloud-scope

Conversation

@kayrus

@kayrus kayrus commented May 7, 2026 •

Copy link
Copy Markdown
Contributor

Fixes #3738

This PR adds support for custom project scope, domain only scope and system scope in clouds.yaml parser

@github-actions github-actions Bot added edit:config This PR updates config code semver:patch No API change backport-v2 This PR will be backported to v2 labels May 7, 2026
@kayrus
kayrus requested a review from pierreprinetti May 7, 2026 11:25
@coveralls

coveralls commented May 7, 2026 •

Copy link
Copy Markdown

Coverage Status

coverage: 63.588% (+0.04%) from 63.552% — kayrus:cloud-scope into gophercloud:main

@kayrus

kayrus commented May 12, 2026

Copy link
Copy Markdown
Contributor Author

cc @mandre @stephenfin

@stephenfin stephenfin 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.

I have made a few attempts at reworking the auth_options.go and this file to address this issue and also separate Identity v2/v3 auth, but have never settled on something I'm happy with. This is a good middle ground.

Couple of comments inline but nothing huge.

Comment thread openstack/config/clouds/clouds.go Outdated
Comment thread openstack/config/clouds/clouds.go Outdated
Comment thread openstack/config/clouds/clouds.go
Comment thread openstack/config/clouds/clouds_test.go
Comment thread openstack/config/clouds/clouds_test.go
@stephenfin
stephenfin added this pull request to the merge queue Sep 7, 2026
Merged via the queue into gophercloud:main with commit 7e3268d Sep 7, 2026
74 of 75 checks passed
@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown

Failed to backport PR to v2 branch. See logs for details.

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

Labels

backport-v2 This PR will be backported to v2 edit:config This PR updates config code semver:patch No API change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Auth] - user_domain_name and project_domain_name are silently collapsed into a single field

3 participants