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

ca-west-1 region is not in the service metadata #4805

Closed
daeho-ro opened this issue Jan 2, 2024 · 5 comments
Closed

ca-west-1 region is not in the service metadata #4805

daeho-ro opened this issue Jan 2, 2024 · 5 comments
Labels
bug This issue is a bug. p2 This is a standard priority issue service:ec2 service-api This issue is due to a problem in a service API, not the SDK implementation.

Comments

@daeho-ro
Copy link

daeho-ro commented Jan 2, 2024

Describe the bug

Simple test for new region ca-west-1 is failed.

        String region = "ca-west-1";
        Assert.assertTrue(Ec2Client.serviceMetadata().regions().contains(Region.of(region)));

I suspect that the service metadata for the ec2 is not updated well.
Maybe the new region should be added into the file, endpoints.json right?

Expected Behavior

Since the new region, ca-west-1 is announced, the region can be found from the service metadata.

Current Behavior

I cannot find the new region ca-west-1 from the service metadata.

Reproduction Steps

        String region = "ca-west-1";
        Assert.assertTrue(Ec2Client.serviceMetadata().regions().contains(Region.of(region)));

FAILED.

Possible Solution

No response

Additional Information/Context

No response

AWS Java SDK version used

2.22.9

JDK version used

17

Operating System and version

M1

@daeho-ro daeho-ro added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Jan 2, 2024
@debora-ito
Copy link
Member

Maybe the new region should be added into the file, endpoints.json right?

Yup, that's right. I'll raise this issue to the EC2 team, they need to add ca-west-1 to their list of supported regions. Thank you for reaching out.

@debora-ito debora-ito added p2 This is a standard priority issue service-api This issue is due to a problem in a service API, not the SDK implementation. service:ec2 and removed needs-triage This issue or PR still needs to be triaged. labels Jan 2, 2024
@daeho-ro
Copy link
Author

daeho-ro commented Jan 3, 2024

I have checked the service metadata for ec2 only, but it is better to check the metadata for other services, too.

@debora-ito
Copy link
Member

@daeho-ro starting in SDK version 2.24.5, ec2 metadata now lists the ca-west-1 region. Thank you for bringing this up.

@debora-ito debora-ito added the closing-soon This issue will close in 4 days unless further comments are made. label Feb 19, 2024
@daeho-ro
Copy link
Author

It is working in my test code. Thanks!

@github-actions github-actions bot removed the closing-soon This issue will close in 4 days unless further comments are made. label Feb 20, 2024
Copy link

This issue is now closed. Comments on closed issues are hard for our team to see.
If you need more assistance, please open a new issue that references this one.

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. p2 This is a standard priority issue service:ec2 service-api This issue is due to a problem in a service API, not the SDK implementation.
Projects
None yet
Development

No branches or pull requests

2 participants