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.

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

Location service doesn't load the aws region in request #1325

Closed
3 tasks done
hsequeda opened this issue Jun 30, 2021 · 10 comments · Fixed by #1349
Closed
3 tasks done

Location service doesn't load the aws region in request #1325

hsequeda opened this issue Jun 30, 2021 · 10 comments · Fixed by #1349
Assignees
Labels
bug This issue is a bug.

Comments

@hsequeda
Copy link

Confirm by changing [ ] to [x] below to ensure that it's a bug:

Describe the bug
https://gist.github.com/hsequeda/4c9ace9c0e76cbabd9b9e8312624662f
When I try to run this test the location service throw me this:
--- FAIL: TestCreateRoute (1.89s) aws_location_map_service_test.go:32: Error Trace: aws_location_map_service_test.go:32 Error: Received unexpected error: operation error Location: SearchPlaceIndexForText, failed to resolve service endpoint, an AWS region is required, but was not found Test: TestCreateRoute FAIL

To verify if the credentials are fine, the code contains an invocation of the list buckets method of s3 and works fine with the same 'configs'.

Version of AWS SDK for Go?
v1.3.0

Version of Go (1.16)?

@hsequeda hsequeda added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Jun 30, 2021
@KaibaLopez
Copy link
Contributor

Hi @hsequeda ,
thanks for bringing this up to us, we'll start investigating on the issue.

@KaibaLopez KaibaLopez added needs-reproduction This issue needs reproduction. and removed needs-triage This issue or PR still needs to be triaged. labels Jul 15, 2021
@skmcgrail skmcgrail self-assigned this Jul 16, 2021
@skmcgrail skmcgrail removed the needs-reproduction This issue needs reproduction. label Jul 16, 2021
@skmcgrail
Copy link
Member

Was able to confirm this behavior. From some brief debugging it appears the client region is not getting set on the context within the stack. Still investigating the root cause.

@skmcgrail
Copy link
Member

It appears that there is a bug in code generation which result in the RegisterServiceMetadata middleware not being added to the stack for any of the location service operations.

@skmcgrail
Copy link
Member

skmcgrail commented Jul 16, 2021

Looks like this bug affects the following API clients:

accessanalyzer
amp
appmesh
braket
codeguruprofiler
groundstation
location
mgn
proton

@skmcgrail
Copy link
Member

skmcgrail commented Jul 19, 2021

Related: #1260

@github-actions
Copy link

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

@skmcgrail skmcgrail reopened this Jul 19, 2021
@skmcgrail
Copy link
Member

Re-opning this until we cut a new release with the merged fix.

@Skarlso
Copy link

Skarlso commented Aug 25, 2021

Hi! Same observation can be made for the marketplacemetering service.

@skmcgrail
Copy link
Member

This particular issue was resolved in https://github.com/aws/aws-sdk-go-v2/releases/tag/release-2021-08-04 but was not closed in a timely manner.

@Skarlso if you are still having issues with the AWS Marketplace Metering service please cut us a new issue describing the issue, and provide a code sample for reproducing the error you are seeing. From a quick glace this service does not have the root cause issue here.

@github-actions
Copy link

⚠️COMMENT VISIBILITY WARNING⚠️

Comments on closed issues are hard for our team to see.
If you need more assistance, please either tag a team member or open a new issue that references this one.
If you wish to keep having a conversation with other community members under this issue feel free to do so.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug.
Projects
None yet
4 participants