-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Add support for Austria East, Chile Central, and Malaysia West regions #47490
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds support for three new Azure regions: Austria East, Chile Central, and Malaysia West. The changes include adding the new region constants to the Region class and updating the changelog to document these additions.
Key Changes
- Added three new region constants (AUSTRIA_EAST, CHILE_CENTRAL, MALAYSIA_WEST) to the Region class
- Updated CHANGELOG.md to document the new regions
- Added test coverage for the new Austria East region
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| sdk/core/azure-core-management/src/main/java/com/azure/core/management/Region.java | Added three new region constants with appropriate JavaDoc comments |
| sdk/core/azure-core-management/CHANGELOG.md | Documented the new regions in the Features Added section |
| sdk/core/azure-core-management/src/test/java/com/azure/core/management/RegionTests.java | Added test coverage for Austria East region |
| public void testNewRegions() { | ||
| // Test Austria East region | ||
| Assertions.assertNotNull(Region.AUSTRIA_EAST); | ||
| Assertions.assertEquals("austriaeast", Region.AUSTRIA_EAST.name()); | ||
| Assertions.assertEquals("Austria East", Region.AUSTRIA_EAST.label()); | ||
|
|
||
| Region austriaEastFromName = Region.fromName("austriaeast"); | ||
| Assertions.assertEquals(Region.AUSTRIA_EAST, austriaEastFromName); | ||
| } |
Copilot
AI
Dec 8, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The test only covers Austria East but the PR adds three new regions (Austria East, Chile Central, and Malaysia West). Add similar test assertions for CHILE_CENTRAL and MALAYSIA_WEST to ensure all new regions are properly tested.
|
Thank you for your contribution @rnishtala-sumo! We will review the pull request and get back to you soon. |
|
@microsoft-github-policy-service agree company="SumoLogic" |
API Change CheckAPIView identified API level changes in this PR and created the following API reviews |
Description
Add support for Austria East, Chile Central, and Malaysia West regions
All SDK Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines