-
Notifications
You must be signed in to change notification settings - Fork 3.3k
[SQL] Update azure-mgmt-sql Package from 4.0.0b20 to 4.0.0b21 #31214
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
Conversation
️✔️AzureCLI-FullTest
|
|
| rule | cmd_name | rule_message | suggest_message |
|---|---|---|---|
| sql db export | cmd sql db export update parameter administrator_login_password: removed property required=True |
||
| sql db export | cmd sql db export update parameter authentication_type: updated property choices from ['ADPassword', 'SQL'] to ['ADPassword', 'ManagedIdentity', 'SQL'] |
||
| sql db export | cmd sql db export update parameter storage_key_type: updated property choices from ['SharedAccessKey', 'StorageAccessKey'] to ['ManagedIdentity', 'SharedAccessKey', 'StorageAccessKey'] |
||
| sql db import | cmd sql db import update parameter administrator_login_password: removed property required=True |
||
| sql db import | cmd sql db import update parameter authentication_type: updated property choices from ['ADPassword', 'SQL'] to ['ADPassword', 'ManagedIdentity', 'SQL'] |
||
| sql db import | cmd sql db import update parameter storage_key_type: updated property choices from ['SharedAccessKey', 'StorageAccessKey'] to ['ManagedIdentity', 'SharedAccessKey', 'StorageAccessKey'] |
|
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>
|
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
Related command
Description
Updating the SQL package from 4.0.0b20 to 4.0.0b21. Here is the release link https://pypi.org/project/azure-mgmt-sql/4.0.0b21/
Testing Guide
Using existing tests
Note for Reviewers
The were few changes in backend because of which few commands were removed in the tests. These tests will break even without api upgrade. Here are the PR/workitem links for reference:
Managed Instance block subnet: Pull Request 1415879: Adding check that prevents managed instances and instance pools from targeting private subnets
Min TLS lower than 1.2 is deprecated for MI: Pull Request 1481256: [SQL MI][Min TLS] Deprecate update of minimum TLS version on SQL MI for version lower than 1.2
Min TLS lower than 1.2 is deprecated for IE:
Task 3827762: Validate TLS1.3 works fine after T76
This 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.