diff --git a/src/azure-cli/azure/cli/command_modules/rdbms/_breaking_change.py b/src/azure-cli/azure/cli/command_modules/rdbms/_breaking_change.py index 1fe90d89ba7..e1168c7bec8 100644 --- a/src/azure-cli/azure/cli/command_modules/rdbms/_breaking_change.py +++ b/src/azure-cli/azure/cli/command_modules/rdbms/_breaking_change.py @@ -3,16 +3,12 @@ # Licensed under the MIT License. See License.txt in the project root for license information. # -------------------------------------------------------------------------------------------- -from azure.cli.core.breaking_change import register_default_value_breaking_change, \ - register_other_breaking_change, register_logic_breaking_change +from azure.cli.core.breaking_change import register_other_breaking_change, register_logic_breaking_change register_logic_breaking_change('postgres flexible-server create', 'Update default value of "--sku-name"', detail='The default value will be changed from "Standard_D2s_v3" to a ' 'supported sku based on regional capabilities.') -register_default_value_breaking_change('postgres flexible-server create', '--version', '16', '17') -register_default_value_breaking_change('postgres flexible-server create', '--create-default-database', 'Enabled', - 'Disabled') register_other_breaking_change('postgres flexible-server update', message='User confirmation will be needed for compute and storage updates ' 'that trigger a restart of the server.') diff --git a/src/azure-cli/azure/cli/command_modules/rdbms/validators.py b/src/azure-cli/azure/cli/command_modules/rdbms/validators.py index bbe6e1e08bb..0c36e614bcb 100644 --- a/src/azure-cli/azure/cli/command_modules/rdbms/validators.py +++ b/src/azure-cli/azure/cli/command_modules/rdbms/validators.py @@ -351,7 +351,7 @@ def pg_arguments_validator(db_context, location, tier, sku_name, storage_gb, ser _pg_storage_validator(storage_gb, sku_info, tier, storage_type, iops, throughput, instance) _pg_sku_name_validator(sku_name, sku_info, tier, instance) _pg_high_availability_validator(high_availability, standby_availability_zone, zone, tier, single_az, instance) - _pg_version_validator(version, list_location_capability_info['server_versions'], is_create) + _pg_version_validator(version, list_location_capability_info['server_versions']) pg_byok_validator(byok_identity, byok_key, backup_byok_identity, backup_byok_key, geo_redundant_backup, instance) is_microsoft_entra_auth = bool(microsoft_entra_auth is not None and microsoft_entra_auth.lower() == 'enabled') _pg_authentication_validator(password_auth, is_microsoft_entra_auth, @@ -510,7 +510,7 @@ def _pg_storage_performance_tier_validator(performance_tier, sku_info, tier=None ' Allowed values : {}'.format(storage_size, performance_tiers)) -def _pg_version_validator(version, versions, is_create): +def _pg_version_validator(version, versions): if version: if version not in versions: raise CLIError('Incorrect value for --version. Allowed values : {}'.format(sorted(versions))) @@ -519,11 +519,6 @@ def _pg_version_validator(version, versions, is_create): "We recommend selecting PostgreSQL 13 or a later version for " "all future operations.") - if is_create: - # Warning for upcoming breaking change to default value of pg version - logger.warning("The default value for the PostgreSQL server major version " - "will be updating to 17 in the near future.") - def _pg_high_availability_validator(high_availability, standby_availability_zone, zone, tier, single_az, instance): if instance: