Skip to content
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

Update Branch v3.x with Branch dev #2866

Closed
wants to merge 17 commits into from
Closed

Conversation

nytian
Copy link
Collaborator

@nytian nytian commented Jun 27, 2024

Issue describing the changes in this PR

As titled. Routine PR to keep v3.x updated with dev.

Pull request checklist

  • My changes do not require documentation changes
    • Otherwise: Documentation PR is ready to merge and referenced in pending_docs.md
  • My changes should not be added to the release notes for the next release
    • Otherwise: I've added my notes to release_notes.md
  • My changes do not need to be backported to a previous version
    • Otherwise: Backport tracked by issue/PR #issue_or_pr
  • I have added all required tests (Unit tests, E2E tests)
  • My changes do not require any extra work to be leveraged by OutOfProc SDKs
    • Otherwise: That work is being tracked here: #issue_or_pr_in_each_sdk
  • My changes do not change the version of the WebJobs.Extensions.DurableTask package
    • Otherwise: major or minor version updates are reflected in /src/Worker.Extensions.DurableTask/AssemblyInfo.cs
  • My changes do not add EventIds to our EventSource logs
    • Otherwise: Ensure the EventIds are within the supported range in our existing Windows infrastructure. You may validate this with a deployed app's telemetry. You may also extend the range by completing a PR such as this one.
  • My changes should be added to v3.x branch.
    • Otherwise: This change only applies to Durable Functions v2.x and will not be merged to branch v3.x.

davidmrdavid and others added 15 commits May 30, 2024 16:42
This PR caches TokenCredential to resolve a memory issue that has been reported when managed identity is enabled. With these changes, we only create the TokenCredential object once instead of every time that GetStorageAccountDetails() is called. This also means that TokenRenewalState is also created once.
Increase Azure.Identity Dependency to v1.11.4
This PR migrates the pipelines to 1ES pipelines. It adds 2 files - official-build.yml and build.yml. official-build.yml references build.yml and runs on any changes pushed to the main branch.
…2861)

* initial commit

* version update

* reword description

* udpate by comment

* reword

* remove typo
* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* add \
* Update StandardConnectionInfoProvider.cs

* Update StandardConnectionInfoProvider.cs

* Update StandardConnectionInfoProvider.cs

* Update StandardConnectionInfoProvider.cs

* add description about the resolve method implementation

* remove whitespace

* update by comment
@nytian nytian changed the title Nytian/update v3 with dev Update Branch v3.x with Branch dev Jun 27, 2024
@nytian nytian requested review from davidmrdavid and bachuv June 27, 2024 03:31
@nytian
Copy link
Collaborator Author

nytian commented Jul 24, 2024

Close this PR as #2878 (review) covered this one.

@nytian nytian closed this Jul 24, 2024
@nytian nytian deleted the nytian/update-v3-with-dev branch October 17, 2024 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants