Conversation
️✔️AzureCLI-FullTest
|
|
Hi @nasc17, |
|
| rule | cmd_name | rule_message | suggest_message |
|---|---|---|---|
| postgres db | sub group postgres db added property deprecate_info_target |
||
| postgres flexible-server create | cmd postgres flexible-server create added property deprecate_info_target |
||
| postgres server | sub group postgres server added property deprecate_info_target |
||
| postgres server-logs | sub group postgres server-logs added property deprecate_info_target |
|
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 adds breaking change announcements for PostgreSQL commands to inform users about deprecations and upcoming changes. The changes prepare users for the transition from Single Server to Flexible Server and modifications to default behavior.
Key Changes:
- Updates deprecation messaging in help text for PostgreSQL Single Server commands
- Registers breaking change announcements for version default value updates and argument deprecations
- Adds deprecation notices for Single Server command groups
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| src/azure-cli/azure/cli/command_modules/rdbms/_help.py | Updates short-summary deprecation messages for postgres db, postgres server, and postgres server-logs command groups |
| src/azure-cli/azure/cli/command_modules/rdbms/_breaking_change.py | New file that registers breaking changes for flexible-server version defaults and database-related arguments, plus deprecation notices for Single Server commands |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Starting over from #32185 |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
Related command
az postgres flexible-server create
az postgres server
az postgres db
az postgres server-logs
Description
All commands under the Postgres server removed.
PG default version Currently 'PG17'. Remove hardcoded value being set for version, instead utilize capabilities API.
Last breaking change we removed having a database be created as default for every server. Can we now remove the argument '--create-default-database' and stick to having customers use the 'az postgres flexible-server database' group, if they want to create any databases.
Testing Guide
History Notes
{RDBMS}
az postgres flexible-server create, az postgres server, az postgres db, az postgres server-logs: Add upcoming breaking change announcements