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

Stabilize heal tests #642

Merged
merged 4 commits into from
Jan 15, 2021

Conversation

artem-belov
Copy link

Fixes #375

TestHealClient_Request - removed first MonitorConnections with the calls context to avoid Server to send commands to almost canceled context.
TestNewClient_MissingConnectionsInInit - test removed. Heal client has a check that the expected connection was received and return an error on Request otherwise.

Artem Belov added 4 commits January 12, 2021 14:40
Signed-off-by: Artem Belov <artem.belov@xored.com>
Signed-off-by: Artem Belov <artem.belov@xored.com>
Signed-off-by: Artem Belov <artem.belov@xored.com>
Signed-off-by: Artem Belov <artem.belov@xored.com>
@haiodo haiodo merged commit 35af2e4 into networkservicemesh:master Jan 15, 2021
nsmbot pushed a commit to networkservicemesh/cmd-nse-icmp-responder that referenced this pull request Jan 15, 2021
…k@master networkservicemesh/sdk#642

networkservicemesh/sdk PR link: networkservicemesh/sdk#642

networkservicemesh/sdk commit message:
commit 35af2e4e173cceccf108c1b32c6cb26402b44bb3
Author: Artem Belov <artem.belov@xored.com>
Date:   Fri Jan 15 14:05:45 2021 +0700

    Stabilize heal tests (#642)

    * Fix TestHealClient_Request

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Remove TestNewClient_MissingConnectionsInInit test

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Add more comments

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Fix go lint

    Signed-off-by: Artem Belov <artem.belov@xored.com>

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-registry-memory that referenced this pull request Jan 15, 2021
…k@master networkservicemesh/sdk#642

networkservicemesh/sdk PR link: networkservicemesh/sdk#642

networkservicemesh/sdk commit message:
commit 35af2e4e173cceccf108c1b32c6cb26402b44bb3
Author: Artem Belov <artem.belov@xored.com>
Date:   Fri Jan 15 14:05:45 2021 +0700

    Stabilize heal tests (#642)

    * Fix TestHealClient_Request

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Remove TestNewClient_MissingConnectionsInInit test

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Add more comments

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Fix go lint

    Signed-off-by: Artem Belov <artem.belov@xored.com>

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-registry-proxy-dns that referenced this pull request Jan 15, 2021
…k@master networkservicemesh/sdk#642

networkservicemesh/sdk PR link: networkservicemesh/sdk#642

networkservicemesh/sdk commit message:
commit 35af2e4e173cceccf108c1b32c6cb26402b44bb3
Author: Artem Belov <artem.belov@xored.com>
Date:   Fri Jan 15 14:05:45 2021 +0700

    Stabilize heal tests (#642)

    * Fix TestHealClient_Request

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Remove TestNewClient_MissingConnectionsInInit test

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Add more comments

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Fix go lint

    Signed-off-by: Artem Belov <artem.belov@xored.com>

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nsmgr that referenced this pull request Jan 15, 2021
…k@master networkservicemesh/sdk#642

networkservicemesh/sdk PR link: networkservicemesh/sdk#642

networkservicemesh/sdk commit message:
commit 35af2e4e173cceccf108c1b32c6cb26402b44bb3
Author: Artem Belov <artem.belov@xored.com>
Date:   Fri Jan 15 14:05:45 2021 +0700

    Stabilize heal tests (#642)

    * Fix TestHealClient_Request

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Remove TestNewClient_MissingConnectionsInInit test

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Add more comments

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Fix go lint

    Signed-off-by: Artem Belov <artem.belov@xored.com>

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/sdk-k8s that referenced this pull request Jan 15, 2021
…k@master networkservicemesh/sdk#642

networkservicemesh/sdk PR link: networkservicemesh/sdk#642

networkservicemesh/sdk commit message:
commit 35af2e4e173cceccf108c1b32c6cb26402b44bb3
Author: Artem Belov <artem.belov@xored.com>
Date:   Fri Jan 15 14:05:45 2021 +0700

    Stabilize heal tests (#642)

    * Fix TestHealClient_Request

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Remove TestNewClient_MissingConnectionsInInit test

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Add more comments

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Fix go lint

    Signed-off-by: Artem Belov <artem.belov@xored.com>

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nse-vfio that referenced this pull request Jan 15, 2021
…k@master networkservicemesh/sdk#642

networkservicemesh/sdk PR link: networkservicemesh/sdk#642

networkservicemesh/sdk commit message:
commit 35af2e4e173cceccf108c1b32c6cb26402b44bb3
Author: Artem Belov <artem.belov@xored.com>
Date:   Fri Jan 15 14:05:45 2021 +0700

    Stabilize heal tests (#642)

    * Fix TestHealClient_Request

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Remove TestNewClient_MissingConnectionsInInit test

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Add more comments

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Fix go lint

    Signed-off-by: Artem Belov <artem.belov@xored.com>

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nsmgr-proxy that referenced this pull request Jan 15, 2021
…k@master networkservicemesh/sdk#642

networkservicemesh/sdk PR link: networkservicemesh/sdk#642

networkservicemesh/sdk commit message:
commit 35af2e4e173cceccf108c1b32c6cb26402b44bb3
Author: Artem Belov <artem.belov@xored.com>
Date:   Fri Jan 15 14:05:45 2021 +0700

    Stabilize heal tests (#642)

    * Fix TestHealClient_Request

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Remove TestNewClient_MissingConnectionsInInit test

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Add more comments

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Fix go lint

    Signed-off-by: Artem Belov <artem.belov@xored.com>

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/sdk-vppagent that referenced this pull request Jan 15, 2021
…k@master networkservicemesh/sdk#642

networkservicemesh/sdk PR link: networkservicemesh/sdk#642

networkservicemesh/sdk commit message:
commit 35af2e4e173cceccf108c1b32c6cb26402b44bb3
Author: Artem Belov <artem.belov@xored.com>
Date:   Fri Jan 15 14:05:45 2021 +0700

    Stabilize heal tests (#642)

    * Fix TestHealClient_Request

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Remove TestNewClient_MissingConnectionsInInit test

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Add more comments

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Fix go lint

    Signed-off-by: Artem Belov <artem.belov@xored.com>

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/sdk-kernel that referenced this pull request Jan 15, 2021
…k@master networkservicemesh/sdk#642

networkservicemesh/sdk PR link: networkservicemesh/sdk#642

networkservicemesh/sdk commit message:
commit 35af2e4e173cceccf108c1b32c6cb26402b44bb3
Author: Artem Belov <artem.belov@xored.com>
Date:   Fri Jan 15 14:05:45 2021 +0700

    Stabilize heal tests (#642)

    * Fix TestHealClient_Request

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Remove TestNewClient_MissingConnectionsInInit test

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Add more comments

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Fix go lint

    Signed-off-by: Artem Belov <artem.belov@xored.com>

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/sdk-vpp that referenced this pull request Jan 15, 2021
…k@master networkservicemesh/sdk#642

networkservicemesh/sdk PR link: networkservicemesh/sdk#642

networkservicemesh/sdk commit message:
commit 35af2e4e173cceccf108c1b32c6cb26402b44bb3
Author: Artem Belov <artem.belov@xored.com>
Date:   Fri Jan 15 14:05:45 2021 +0700

    Stabilize heal tests (#642)

    * Fix TestHealClient_Request

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Remove TestNewClient_MissingConnectionsInInit test

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Add more comments

    Signed-off-by: Artem Belov <artem.belov@xored.com>

    * Fix go lint

    Signed-off-by: Artem Belov <artem.belov@xored.com>

Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
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.

Stabilize heal tests
2 participants