Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use resource /instance/canonicalRegionName to get region #3868

Merged
merged 1 commit into from
Feb 4, 2022
Merged

Use resource /instance/canonicalRegionName to get region #3868

merged 1 commit into from
Feb 4, 2022

Conversation

spericas
Copy link
Member

@spericas spericas commented Feb 4, 2022

Use resource /instance/canonicalRegionName to get region and construct auth URL. Using /instance/region is only compatible in certain regions and does not yield a canonical name. See https://docs.oracle.com/en-us/iaas/Content/Compute/Tasks/gettingmetadata.htm

Signed-off-by: Santiago Pericasgeertsen [email protected]

@spericas spericas self-assigned this Feb 4, 2022
@spericas spericas added the OCI label Feb 4, 2022
@spericas spericas added this to the 2.4.2 milestone Feb 4, 2022
@spericas spericas changed the title WIP: Use resource /instance/canonicalRegionName to get region Use resource /instance/canonicalRegionName to get region Feb 4, 2022
@spericas
Copy link
Member Author

spericas commented Feb 4, 2022

Sample session running on Phoenix region where problem was reported:

(helidon-client:~) >> curl H "Accept: test/plain" -H "Authorization: Bearer Oracle" --noproxy 169.254.169.254 http://169.254.169.254/opc/v2/instance/region
phx

(helidon-client:~) >> curl -H "Accept: test/plain" -H "Authorization: Bearer Oracle" --noproxy 169.254.169.254 http://169.254.169.254/opc/v2/instance/canonicalRegionName
us-phoenix-1

@spericas spericas requested a review from barchetta February 4, 2022 16:41
@spericas spericas merged commit 7703523 into helidon-io:helidon-2.x Feb 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@barchetta barchetta barchetta approved these changes

@tomas-langer tomas-langer Awaiting requested review from tomas-langer

@arjav-desai arjav-desai Awaiting requested review from arjav-desai

Assignees

@spericas spericas

Labels
OCI
Projects
Backlog
Archived in project
Milestone
2.4.2
Development

Successfully merging this pull request may close these issues.
2 participants
@spericas @barchetta