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 from update/networkservicemesh/sdk #284

Merged
merged 1 commit into from
Dec 7, 2021

Conversation

nsmbot
Copy link
Collaborator

@nsmbot nsmbot commented Dec 7, 2021

Update go.mod and go.sum to latest version from networkservicemesh/sdk@main
PR link: networkservicemesh/sdk#1173
Commit: 23e4c36
Author: Sol-0
Date: 2021-12-07 21:10:50 +0700
Message:

  • Fixed recvfd leaks (# 1173)

…k@main

PR link: networkservicemesh/sdk#1173

Commit: 23e4c36
Author: Sol-0
Date: 2021-12-07 21:10:50 +0700
Message:
  - Fixed recvfd leaks (#1173)
* Fixed recvfd leaks

Signed-off-by: Oleg Solodkov <oleg.solodkov@xored.com>

* Added const for OS name

Signed-off-by: Oleg Solodkov <oleg.solodkov@xored.com>

* Fixed formatting

Signed-off-by: Oleg Solodkov <oleg.solodkov@xored.com>

* Addressed review comments

Signed-off-by: Oleg Solodkov <oleg.solodkov@xored.com>

* Added tests

Signed-off-by: Oleg Solodkov <oleg.solodkov@xored.com>

* Addressed review comments

Signed-off-by: Oleg Solodkov <oleg.solodkov@xored.com>

* Added registry recvfd server test

Signed-off-by: Oleg Solodkov <oleg.solodkov@xored.com>

* Addressed review comments

Signed-off-by: Oleg Solodkov <oleg.solodkov@xored.com>

* Addressed review comments

Signed-off-by: Oleg Solodkov <oleg.solodkov@xored.com>

* Addressed review comments

Signed-off-by: Oleg Solodkov <oleg.solodkov@xored.com>

* Addressed review comments

Signed-off-by: Oleg Solodkov <oleg.solodkov@xored.com>

* Addressed review comments, added test suite

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
@github-actions github-actions bot merged commit 2500951 into main Dec 7, 2021
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.

1 participant