Update from update/networkservicemesh/sdk #118
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…k@main networkservicemesh/sdk# networkservicemesh/sdk PR link: https://github.com/networkservicemesh/sdk/pull/ networkservicemesh/sdk commit message: commit 19c7a102f2a7bb9914763ec77304a69ce7790f0e Author: Uzlov Danil <36223296+d-uzlov@users.noreply.github.com> Date: Thu Apr 22 12:47:30 2021 +0700 Add Client Monitor to the Heal Client (#810) * Fix selector handling in monitorServer.MonitorConnections Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Move event monitoring from heal server to heal client Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Fix heal not working in sandbox tests Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Fix pending issues and clean code in heal package Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Fix heal client not working reliably on first request Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Fix issues, discussed in PR comments Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Fix heal unit test panic Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Fix imports layout Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Fix heal client using wrong id when listening to connection updates Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Stop heal client from potentially sending repeating heal requests Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Use updated connection path on heal Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Move connection path restoring from heal server to heal client Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * In heal client use mutex per connection instead of one executor Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * In heal server use mutex per connection instead of one executor Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Fix linter warning Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Add comments to "heal" package Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Fix heal client not saving endpoint name on connection update events Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * Make heal client wait for confirmation in Request if connection was broken Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * extract initialization method in heal client Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * use sync.Cond instead of channel to confirm connection in heal client Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * use separate functions for "heal" and "restore connection" Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * use better error message in heal client Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * fix file formatting Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * refactor heal server code Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * use single type for both heal requests Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * combine types in healClient.listenToConnectionChanges() Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * remove event-check in heal client, remove incorrect heal test Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * remove complex state from conn info in heal client Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * use mutex instead of sync.Cond in heal client Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * fix linter warning Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * fix heal client not removing unsuccessful connection Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> * fix heal server not using clock package during healing Signed-off-by: Danil Uzlov <DanilUzlov@yandex.ru> Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
commit 870fc74 Author: NSMBot nsmbot@networkservicmesh.io Date: Thu Apr 22 05:47:57 2021 +0000 Update go.mod and go.sum to latest version from networkservicemesh/sdk@main networkservicemesh/sdk# networkservicemesh/sdk PR link: https://github.com/networkservicemesh/sdk/pull/ networkservicemesh/sdk commit message: commit 19c7a102f2a7bb9914763ec77304a69ce7790f0e Author: Uzlov Danil 36223296+d-uzlov@users.noreply.github.com Date: Thu Apr 22 12:47:30 2021 +0700 Add Client Monitor to the Heal Client (#810) README.md go.mod go.sum pkg Fix selector handling in monitorServer.MonitorConnections Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg Move event monitoring from heal server to heal client Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg Fix heal not working in sandbox tests Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg Fix pending issues and clean code in heal package Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg Fix heal client not working reliably on first request Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg Fix issues, discussed in PR comments Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg Fix heal unit test panic Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg Fix imports layout Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg Fix heal client using wrong id when listening to connection updates Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg Stop heal client from potentially sending repeating heal requests Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg Use updated connection path on heal Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg Move connection path restoring from heal server to heal client Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg In heal client use mutex per connection instead of one executor Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg In heal server use mutex per connection instead of one executor Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg Fix linter warning Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg Add comments to "heal" package Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg Fix heal client not saving endpoint name on connection update events Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg Make heal client wait for confirmation in Request if connection was broken Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg extract initialization method in heal client Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg use sync.Cond instead of channel to confirm connection in heal client Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg use separate functions for "heal" and "restore connection" Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg use better error message in heal client Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg fix file formatting Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg refactor heal server code Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg use single type for both heal requests Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg combine types in healClient.listenToConnectionChanges() Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg remove event-check in heal client, remove incorrect heal test Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg remove complex state from conn info in heal client Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg use mutex instead of sync.Cond in heal client Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg fix linter warning Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg fix heal client not removing unsuccessful connection Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru README.md go.mod go.sum pkg fix heal server not using clock package during healing Signed-off-by: Danil Uzlov DanilUzlov@yandex.ru Signed-off-by: NSMBot nsmbot@networkservicmesh.io