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.

Sign up for GitHub

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

Jump to bottom

NET-1269:add mutex for upgrade #802

Merged

abhishek9686 merged 2 commits into develop from NET-1269

Jun 5, 2024

Collaborator

yabinma commented May 27, 2024 •

edited

Loading

Describe your changes

Provide Issue ticket number if applicable/not in title

Provide link to Netmaker PR if required

Provide testing steps

Checklist before requesting a review

My changes affect only 10 files or less.
I have performed a self-review of my code and tested it.
If it is a new feature, I have added thorough tests, my code is <= 1450 lines.
If it is a bugfix, my code is <= 200 lines.
My functions are <= 80 lines.
I have had my code reviewed by a peer.
My unit tests pass locally.
Netclient & Netmaker are awesome.


          add mutex for upgrade

e7f35d8

yabinma requested a review from abhishek9686

May 27, 2024 17:21

abhishek9686 approved these changes

View reviewed changes

abhishek9686 requested changes

View reviewed changes

functions/mqhandlers.go Show resolved Hide resolved


          update logic to avoid deadlock

c671767

abhishek9686 approved these changes

View reviewed changes

abhishek9686 merged commit 4868b73 into develop

15 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet