Skip to content

Don't Error on Multiple Endpoints#1766

Merged
jtopjian merged 1 commit intogophercloud:masterfrom
jtopjian:multiple-endpoints-workaround
Nov 11, 2019
Merged

Don't Error on Multiple Endpoints#1766
jtopjian merged 1 commit intogophercloud:masterfrom
jtopjian:multiple-endpoints-workaround

Conversation

@jtopjian
Copy link
Copy Markdown
Contributor

This commit changes the behvior when multiple endpoints are
discovered. Instead of returning an error, the first endpoint will
be used, discarding the other endpoints.

This change helps keep behavior in line with the Python SDK.

For #1764

This commit changes the behvior when multiple endpoints are
discovered. Instead of returning an error, the first endpoint will
be used, discarding the other endpoints.

This change helps keep behavior in line with the Python SDK.
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.008%) to 76.965% when pulling 0578e0c on jtopjian:multiple-endpoints-workaround into d7ebd70 on gophercloud:master.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci bot commented Nov 11, 2019

Build succeeded.

@jtopjian jtopjian merged commit dd0bb13 into gophercloud:master Nov 11, 2019
outcatcher added a commit to opentelekomcloud/gophertelekomcloud that referenced this pull request Dec 4, 2020
Replace OTC-specific endpoint location with general openstack approach

If opts.Region is empty, first matching endpoint will be selected
otc-zuul bot pushed a commit to opentelekomcloud/gophertelekomcloud that referenced this pull request Dec 7, 2020
Backport gophercloud/gophercloud#1766

Replace OTC-specific endpoint location with general OpenStack and gophercloud approach
If opts.Region is empty, the first matching endpoint will be selected

Reviewed-by: None <None>
Reviewed-by: Anton Sidelnikov <None>
Reviewed-by: Rodion Gyrbu <fpsoff@outlook.com>
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.

3 participants