Skip to content

Conversation

@rnishtala-sumo
Copy link

@rnishtala-sumo rnishtala-sumo commented Dec 8, 2025

Description

Add support for Austria East, Chile Central, and Malaysia West regions

All SDK Contribution checklist:

  • The pull request does not introduce [breaking changes]
  • CHANGELOG is updated for new features, bug fixes or other significant changes.
  • I have read the contribution guidelines.

General Guidelines and Best Practices

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

Copilot AI review requested due to automatic review settings December 8, 2025 20:07
Copy link
Contributor

Copilot AI left a 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

Comment on lines +71 to +79
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);
}
Copy link

Copilot AI Dec 8, 2025

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.

Copilot uses AI. Check for mistakes.
@github-actions github-actions bot added Azure.Core azure-core Community Contribution Community members are working on the issue customer-reported Issues that are reported by GitHub users external to the Azure organization. labels Dec 8, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 8, 2025

Thank you for your contribution @rnishtala-sumo! We will review the pull request and get back to you soon.

@rnishtala-sumo
Copy link
Author

rnishtala-sumo commented Dec 8, 2025

@microsoft-github-policy-service agree company="SumoLogic"

@github-actions
Copy link
Contributor

github-actions bot commented Dec 8, 2025

API Change Check

APIView identified API level changes in this PR and created the following API reviews

com.azure:azure-core-management

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Azure.Core azure-core Community Contribution Community members are working on the issue customer-reported Issues that are reported by GitHub users external to the Azure organization.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant