Skip to content

[Update] Leader Election#536

Open
v-federicoar wants to merge 2 commits into
mspnp:mainfrom
v-federicoar:refresh/leader-election
Open

[Update] Leader Election#536
v-federicoar wants to merge 2 commits into
mspnp:mainfrom
v-federicoar:refresh/leader-election

Conversation

@v-federicoar

Copy link
Copy Markdown
Contributor

Summary

This PR updates the leader-election sample so it runs reliably with the current toolchain and provides clear local emulator guidance.

What Changed

  • Updated project dependency references in the leader-election projects to a compatible modern set.
  • Improved the local Azurite setup documentation for HTTPS-based authentication flow.
  • Documented emulator startup options to avoid local API-version mismatches.

Validation

End-to-end validation was executed for the leader-election flow:

  • Build the solution successfully.
  • Start local Azurite with HTTPS configuration.
  • Run the worker process from WSL.
  • Verify leader election behavior by running multiple worker instances and observing lease acquisition/retry output.

@v-federicoar v-federicoar marked this pull request as ready for review June 18, 2026 11:48

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the leader-election sample to work reliably with the current Azure SDK toolchain and provides clearer local Azurite emulator setup guidance. The changes are straightforward: bumping Azure.Identity from 1.17.1 to 1.21.0 in both projects, and replacing the terse Azurite instructions with step-by-step HTTPS certificate setup and startup commands.

Changes:

  • Bumped Azure.Identity to version 1.21.0 in both DistributedMutex and LeaderElectionConsoleWorker project files.
  • Rewrote the Azurite local emulator setup section in Readme.md with detailed HTTPS certificate generation, trust, and startup instructions.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
leader-election/Readme.md Expanded Azurite setup documentation with step-by-step certificate generation, trust, and HTTPS/OAuth startup commands
leader-election/LeaderElectionConsoleWorker/LeaderElectionConsoleWorker.csproj Bumped Azure.Identity from 1.17.1 to 1.21.0
leader-election/DistributedMutex/DistributedMutex.csproj Bumped Azure.Identity from 1.17.1 to 1.21.0

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread leader-election/Readme.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
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.

2 participants