Update azure-arm-rest to 3.267.1 in AzureRmWebAppDeployment V3 V4 and V5 #21731
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
rm-dmiri
requested review from
a team,
dannysongg,
jvano and
manolerazvan
as code owners
Contributor
Author
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
…7.1-in-heavy-rm-tasks
Contributor
Author
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
…7.1-in-heavy-rm-tasks
Contributor
Author
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Context
We updated common library with regression fix for MSAL library issues.
See for reference: #21585
Task Name
AzureRmWebAppDeploymentV3
AzureRmWebAppDeploymentV4
AzureRmWebAppDeploymentV5
Description
Upgraded azure-pipelines-tasks-azure-arm-rest dependency from various versions (3.254.0, 3.263.1, 3.267.0) to version 3.267.1 across all tasks that use this package to fix MSAL library dependency issues.
This PR affects part 2 of tasks (owned by RM). They have heavy CI builds which tent to timeout, there fore they are separated.
Risk Assessment (Low / Medium / High)
Low - This is a patch version update of a common dependency package with backward compatible changes and no breaking API modifications.
Change Behind Feature Flag (Yes / No)
No - This is a dependency version update that applies globally and doesn't require feature flagging as it maintains backward compatibility.
Tech Design / Approach
Documentation Changes Required (Yes/No)
No - Dependency version updates don't require user-facing documentation changes.
Unit Tests Added or Updated (Yes / No)
No - Existing unit tests cover the functionality; no behavioral changes were made.
Additional Testing Performed
Built and verified successful compilation of sample tasks including AzurePowerShellV5, JavaToolInstallerV1, and AzureRmWebAppDeploymentV5 using the updated dependency.
Logging Added/Updated (Yes/No)
No log updates except one from new package dependency. They are safe and don't log sensitive data.
Telemetry Added/Updated (Yes/No)
No.
Rollback Scenario and Process (Yes/No)
Task version downgrade, or changes revert and published with new task version.
Dependency Impact Assessed and Regression Tested (Yes/No)
Yes. Tested with CI.
Checklist