Update Azure.Identity to 1.16.0 & Azure.Core to 1.49.0
Issue describing the changes in this PR
resolves #11402
Pull request checklist
IMPORTANT: Currently, changes must be backported to the in-proc branch to be included in Core Tools and non-Flex deployments.
- [x] Backporting to the
in-procbranch is not required- Otherwise: Link to backporting PR
- [x] My changes do not require documentation changes
- [ ] Otherwise: Documentation issue linked to PR
- [ ] My changes should not be added to the release notes for the next release
- [x] Otherwise: I've added my notes to
release_notes.md
- [x] Otherwise: I've added my notes to
- [x] My changes do not need to be backported to a previous version
- [ ] Otherwise: Backport tracked by issue/PR #issue_or_pr
- [x] My changes do not require diagnostic events changes
- Otherwise: I have added/updated all related diagnostic events and their documentation (Documentation issue linked to PR)
- [x] I have added all required tests (Unit tests, E2E tests)
Additional information
Additional PR information
/azp run host.public
Azure Pipelines successfully started running 1 pipeline(s).
@cataggar the build failed due to a package downgrade error. You should be able to get the same error with a local dotnet build.
/azp run host.public
Commenter does not have sufficient privileges for PR 11404 in repo Azure/azure-functions-host
@jviau It restores and builds locally now. I don't have permissions to start CI. Can you kick off another run or can I get permissions?
/azp run host.public
Azure Pipelines successfully started running 1 pipeline(s).
@cataggar another change will be needed. We have a unit test which verifies our transitive dependency closure to identify changes like this. We need to update a file to signal to the test this change is intentional. Please replace the contents of test/WebJobs.Script.Tests/Microsoft.Azure.WebJobs.Script.WebHost.deps.json with this artifact
/azp run host.public
Azure Pipelines successfully started running 1 pipeline(s).
@cataggar another change will be needed. We have a unit test which verifies our transitive dependency closure to identify changes like this. We need to update a file to signal to the test this change is intentional. Please replace the contents of test/WebJobs.Script.Tests/Microsoft.Azure.WebJobs.Script.WebHost.deps.json with this artifact
Thanks @jviau . This is done.
Thanks @cataggar. I am going to wait on review/sign off from @mattchenderson. He keeps an eye on Azure.Identity changes as it has been a source of many regressions for customers.
Will take a look at this today and validate the dependency updates.
Closing this PR as we will need to update to an even higher version. #11531