Skip to content

clouds: Parse trust_id from clouds.yaml#3106

Merged
EmilienM merged 1 commit intogophercloud:masterfrom
shiftstack:parse_trustid
Jul 17, 2024
Merged

clouds: Parse trust_id from clouds.yaml#3106
EmilienM merged 1 commit intogophercloud:masterfrom
shiftstack:parse_trustid

Conversation

@pierreprinetti
Copy link
Copy Markdown
Member

Fixes #3105

@pierreprinetti pierreprinetti requested a review from a team July 16, 2024 13:40
@github-actions github-actions bot added the semver:minor Backwards-compatible change label Jul 16, 2024
@github-actions github-actions bot added semver:minor Backwards-compatible change and removed semver:minor Backwards-compatible change labels Jul 16, 2024
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 78.738% (-0.009%) from 78.747%
when pulling 2a17e64 on shiftstack:parse_trustid
into f3dc557 on gophercloud:master.

Copy link
Copy Markdown

@pgaxatte pgaxatte left a comment

Choose a reason for hiding this comment

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

Tested on my end and it works as expected with a clouds.yaml file in this form:

clouds:
  mycloud:
    auth:
      auth_url: https://...
      user_id: "..."
      password: "..."
      user_domain_name: "Default"
      trust_id: "XXXX"
    region_name: "RegionOne"

@pierreprinetti pierreprinetti marked this pull request as ready for review July 17, 2024 08:00
@pierreprinetti pierreprinetti added the backport-v2 This PR will be backported to v2 label Jul 17, 2024
@EmilienM EmilienM merged commit 1b170fc into gophercloud:master Jul 17, 2024
@EmilienM EmilienM deleted the parse_trustid branch July 17, 2024 12:12
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 semver:minor Backwards-compatible change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

v2: Support TrustID in AuthOptions (clouds.yaml)

4 participants