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

Management - add serial to Network reflecting network updates #179

Merged
merged 10 commits into from
Jan 14, 2022

Conversation

braginini
Copy link
Contributor

@braginini braginini commented Jan 1, 2022

Just adds ModificationId property to the Network struct.
it is incremented on:

  • AddPeer
  • DeletePeer

@braginini braginini linked an issue Jan 13, 2022 that may be closed by this pull request
@braginini braginini marked this pull request as ready for review January 14, 2022 12:41
@braginini braginini requested a review from mlsmaycon January 14, 2022 12:41
mlsmaycon
mlsmaycon previously approved these changes Jan 14, 2022
Copy link
Collaborator

@mlsmaycon mlsmaycon left a comment

Choose a reason for hiding this comment

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

LGTM

@mlsmaycon mlsmaycon self-requested a review January 14, 2022 13:21
@braginini braginini changed the title Management - add account modification ID Management - add serial to Network reflecting network updates Jan 14, 2022
@braginini braginini merged commit 9d1ecbb into main Jan 14, 2022
@braginini braginini deleted the add-modification-id branch January 14, 2022 13:34
pulsastrix pushed a commit to pulsastrix/netbird that referenced this pull request Dec 24, 2023
…dio#179)

* chore: [management] - add account serial ID

* Fix concurrency on the client (netbirdio#183)

* reworked peer connection establishment logic eliminating race conditions and deadlocks while running many peers

* chore: move serial to Network from Account

* feature: increment Network serial ID when adding/removing peers

* chore: extract network struct init to network.go

* chore: add serial test when adding peer to the account

* test: add ModificationID test on AddPeer and DeletePeer
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.

Tracking Network modifications (serial ID)
2 participants