{RDBMS} az postgres flexible-server create/restore: Enable CMK, PG17, geo-restore for elastic cluster#32379
Conversation
…7, geo-restore for elastic cluster
️✔️AzureCLI-FullTest
|
|
Hi @mattboentoro, |
️✔️AzureCLI-BreakingChangeTest
|
|
Thank you for your contribution! We will review the pull request and get back to you soon. |
|
The git hooks are available for azure-cli and azure-cli-extensions repos. They could help you run required checks before creating the PR. Please sync the latest code with latest dev branch (for azure-cli) or main branch (for azure-cli-extensions). pip install azdev --upgrade
azdev setup -c <your azure-cli repo path> -r <your azure-cli-extensions repo path>
|
There was a problem hiding this comment.
Pull Request Overview
This PR updates PostgreSQL elastic cluster support to make version 17 the only supported version (reversing previous restrictions) and removes several feature restrictions for elastic clusters. The changes simplify validators by removing encryption and geo-redundancy checks for elastic clusters, which are now supported features.
- Updates elastic cluster version validation to require PostgreSQL 17 (previously blocked version 17)
- Removes validation restrictions for geo-redundancy backup and data encryption (BYOK) on elastic clusters
- Updates test to use version 17 for elastic cluster testing
Reviewed Changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/azure-cli/azure/cli/command_modules/rdbms/validators.py | Modified _cluster_validator to require version 17 for elastic clusters, removed geo-redundancy and BYOK validations, and deleted the now-unused cluster_byok_validator function |
| src/azure-cli/azure/cli/command_modules/rdbms/flexible_server_custom_postgres.py | Removed import and call to deleted cluster_byok_validator function |
| src/azure-cli/azure/cli/command_modules/rdbms/tests/latest/test_rdbms_flexible_commands.py | Updated test to use PostgreSQL version 17 for elastic cluster testing |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@evelyn-ys @yonzhan do you mind reviewing this PR? |
az postgres flexible-server create / restore: Enable CMK, PG17, geo-restore for elastic clusteraz postgres flexible-server create/restore: Enable CMK, PG17, geo-restore for elastic cluster
Related command
az postgres flexible-server create
az postgres flexible-server restore
Description
Enable CMK, PG17, geo-restore for elastic cluster
Testing Guide
Manual Testing
History Notes
{RDBMS}
az postgres flexible-server create / restore: Enable CMK, PG17, geo-restore for elastic clusterThis checklist is used to make sure that common guidelines for a pull request are followed.
The PR title and description has followed the guideline in Submitting Pull Requests.
I adhere to the Command Guidelines.
I adhere to the Error Handling Guidelines.