-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
{Packaging} Bump azure-core from 1.21.1 to 1.24.0 #22676
Conversation
Packaging |
…-azure-core � Conflicts: � src/azure-cli/azure/cli/command_modules/network/tests/latest/recordings/test_network_public_ip_prefix.yaml
…re-cli into upgrade-azure-core
/azp run |
Azure Pipelines successfully started running 2 pipeline(s). |
@@ -1406,12 +1406,13 @@ def test_sql_pool(self): | |||
self.cmd('az synapse sql pool show --name {sql-pool} --workspace {workspace} --resource-group {rg}', | |||
expect_failure=True) | |||
|
|||
@record_only() | |||
@unittest.skip('Resource group \'rgtesting\' could not be found') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Skip it temporarily as we cannot pass test lively as the test used fixed resources, will contact service team to fix it later.
@@ -2629,6 +2630,7 @@ def test_notebook(self): | |||
'az synapse notebook show --workspace-name {workspace} --name {name}', | |||
expect_failure=True) | |||
|
|||
@unittest.skip('UnicodeDecodeError') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Skip it temporarily as we cannot pass test lively as the test used corrupted file that cannot be decoded, will contact service team to fix it later.
kubernetes_version = "1.20.7" | ||
kubernetes_version = "1.23.5" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just of curiosity, may I ask why do we need to upgrade this kubernetes_version
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because version 1.20.7 is not currently supported in this region.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you add the PR description to explain the background and context (such as the related issue/PR link) of bumping azure-core
version?
This is convenient for us to track the cause and impact of this PR in the future
…-azure-core � Conflicts: � src/azure-cli/azure/cli/command_modules/servicefabric/tests/latest/recordings/test_basic_cluster.yaml � src/azure-cli/azure/cli/command_modules/servicefabric/tests/latest/test_sf_managed_cluster.py
Related command
Description
Some modules (#22486) and extensions (AML) which use new SDK need newer version of azure-core, so we need upgrade it to the latest version (1.24.0).
Testing Guide
History Notes
[Component Name 1] BREAKING CHANGE:
az command a
: Make some customer-facing breaking change[Component Name 2]
az command b
: Add some customer-facing featureThis 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.