azure-functions-host icon indicating copy to clipboard operation
azure-functions-host copied to clipboard

Update Azure.Identity to 1.16.0 & Azure.Core to 1.49.0

Open cataggar opened this issue 3 months ago • 14 comments

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-proc branch 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] 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

cataggar avatar Oct 02 '25 20:10 cataggar

/azp run host.public

jviau avatar Oct 02 '25 21:10 jviau

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Oct 02 '25 21:10 azure-pipelines[bot]

@cataggar the build failed due to a package downgrade error. You should be able to get the same error with a local dotnet build.

jviau avatar Oct 02 '25 22:10 jviau

/azp run host.public

cataggar avatar Oct 03 '25 01:10 cataggar

Commenter does not have sufficient privileges for PR 11404 in repo Azure/azure-functions-host

azure-pipelines[bot] avatar Oct 03 '25 01:10 azure-pipelines[bot]

@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?

cataggar avatar Oct 03 '25 01:10 cataggar

/azp run host.public

jviau avatar Oct 03 '25 16:10 jviau

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Oct 03 '25 16:10 azure-pipelines[bot]

@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

jviau avatar Oct 03 '25 16:10 jviau

/azp run host.public

jviau avatar Oct 03 '25 22:10 jviau

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Oct 03 '25 22:10 azure-pipelines[bot]

@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.

cataggar avatar Oct 03 '25 22:10 cataggar

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.

jviau avatar Oct 06 '25 22:10 jviau

Will take a look at this today and validate the dependency updates.

fabiocav avatar Oct 15 '25 18:10 fabiocav

Closing this PR as we will need to update to an even higher version. #11531

jviau avatar Dec 16 '25 20:12 jviau