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.

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

Issue when register endpoints with same name #250

Closed
Mixaster995 opened this issue Dec 15, 2021 · 1 comment · Fixed by networkservicemesh/sdk#1221
Closed

Issue when register endpoints with same name #250

Mixaster995 opened this issue Dec 15, 2021 · 1 comment · Fixed by networkservicemesh/sdk#1221
Assignees
@denis-tingaikin
Labels
bug Something isn't working

Comments

@Mixaster995
Copy link
Contributor

Description

In interdomain testing for floating scenario, when one example executes after another and using identical name for endpoint they register, tests failing (TestFloatingKernel2Wireguard2Kernel fails after TestFloatingKernel2Vxlan2Kernel successful, to be precise)
There is workaround for now, that makes everything working - networkservicemesh/deployments-k8s#4070
But this is not desirable behaviour and requires further investigation.

Steps to recreate

Example of failed tests

https://github.com/networkservicemesh/integration-k8s-kind/runs/4530518091?check_suite_focus=true
@denis-tingaikin
Copy link
Member

Also, I feel we need basically test that the refresh scenario is fine for cmd-registry-k8s.

@denis-tingaikin denis-tingaikin added the bug Something isn't working label Dec 15, 2021
@denis-tingaikin denis-tingaikin moved this from Todo to In Progress in Release 1.2.0 Jan 30, 2022
denis-tingaikin added a commit to denis-tingaikin/sdk that referenced this issue Jan 30, 2022
@denis-tingaikin
apply fixes for networkservicemesh/cmd-registry-k8s#250
aa3b2c8 
Signed-off-by: Denis Tingaikin <denis.tingajkin@xored.com>
@denis-tingaikin denis-tingaikin mentioned this issue Jan 30, 2022
Merged
9 tasks
denis-tingaikin added a commit to denis-tingaikin/sdk that referenced this issue Jan 30, 2022
@denis-tingaikin
apply fixes for networkservicemesh/cmd-registry-k8s#250
75e2524 
Signed-off-by: Denis Tingaikin <denis.tingajkin@xored.com>
@denis-tingaikin denis-tingaikin moved this from In Progress to Review in progress in Release 1.2.0 Jan 30, 2022
edwarnicke pushed a commit to networkservicemesh/sdk that referenced this issue Jan 30, 2022
@denis-tingaikin
apply fixes for networkservicemesh/cmd-registry-k8s#250 (#1221)
953f555 
Signed-off-by: Denis Tingaikin <denis.tingajkin@xored.com>
Repository owner moved this from Review in progress to Done in Release 1.2.0 Jan 30, 2022
nsmbot pushed a commit to networkservicemesh/cmd-admission-webhook-k8s that referenced this issue Jan 30, 2022
Update go.mod and go.sum to latest version from networkservicemesh/sd…
c95d78e 
…k@main

PR link: networkservicemesh/sdk#1221

Commit: 953f555
Author: Denis Tingaikin
Date: 2022-01-30 21:47:45 +0300
Message:
  -   apply fixes for networkservicemesh/cmd-registry-k8s#250 (#1221)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nsc-init that referenced this issue Jan 30, 2022
Update go.mod and go.sum to latest version from networkservicemesh/sd…
1715eea 
…k@main

PR link: networkservicemesh/sdk#1221

Commit: 953f555
Author: Denis Tingaikin
Date: 2022-01-30 21:47:45 +0300
Message:
  -   apply fixes for networkservicemesh/cmd-registry-k8s#250 (#1221)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nsmgr that referenced this issue Jan 30, 2022
Update go.mod and go.sum to latest version from networkservicemesh/sd…
fd5a906 
…k@main

PR link: networkservicemesh/sdk#1221

Commit: 953f555
Author: Denis Tingaikin
Date: 2022-01-30 21:47:45 +0300
Message:
  -   apply fixes for networkservicemesh/cmd-registry-k8s#250 (#1221)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-registry-memory that referenced this issue Jan 30, 2022
Update go.mod and go.sum to latest version from networkservicemesh/sd…
29a0754 
…k@main

PR link: networkservicemesh/sdk#1221

Commit: 953f555
Author: Denis Tingaikin
Date: 2022-01-30 21:47:45 +0300
Message:
  -   apply fixes for networkservicemesh/cmd-registry-k8s#250 (#1221)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/sdk-kernel that referenced this issue Jan 30, 2022
Update go.mod and go.sum to latest version from networkservicemesh/sd…
39a2038 
…k@main

PR link: networkservicemesh/sdk#1221

Commit: 953f555
Author: Denis Tingaikin
Date: 2022-01-30 21:47:45 +0300
Message:
  -   apply fixes for networkservicemesh/cmd-registry-k8s#250 (#1221)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nse-vfio that referenced this issue Jan 30, 2022
Update go.mod and go.sum to latest version from networkservicemesh/sd…
5f62f89 
…k@main

PR link: networkservicemesh/sdk#1221

Commit: 953f555
Author: Denis Tingaikin
Date: 2022-01-30 21:47:45 +0300
Message:
  -   apply fixes for networkservicemesh/cmd-registry-k8s#250 (#1221)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nse-remote-vlan that referenced this issue Jan 30, 2022
Update go.mod and go.sum to latest version from networkservicemesh/sd…
e94ec56 
…k@main

PR link: networkservicemesh/sdk#1221

Commit: 953f555
Author: Denis Tingaikin
Date: 2022-01-30 21:47:45 +0300
Message:
  -   apply fixes for networkservicemesh/cmd-registry-k8s#250 (#1221)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/sdk-k8s that referenced this issue Jan 30, 2022
Update go.mod and go.sum to latest version from networkservicemesh/sd…
679a024 
…k@main

PR link: networkservicemesh/sdk#1221

Commit: 953f555
Author: Denis Tingaikin
Date: 2022-01-30 21:47:45 +0300
Message:
  -   apply fixes for networkservicemesh/cmd-registry-k8s#250 (#1221)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nsmgr-proxy that referenced this issue Jan 30, 2022
Update go.mod and go.sum to latest version from networkservicemesh/sd…
2e24b14 
…k@main

PR link: networkservicemesh/sdk#1221

Commit: 953f555
Author: Denis Tingaikin
Date: 2022-01-30 21:47:45 +0300
Message:
  -   apply fixes for networkservicemesh/cmd-registry-k8s#250 (#1221)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-registry-proxy-dns that referenced this issue Jan 30, 2022
Update go.mod and go.sum to latest version from networkservicemesh/sd…
8f8fc67 
…k@main

PR link: networkservicemesh/sdk#1221

Commit: 953f555
Author: Denis Tingaikin
Date: 2022-01-30 21:47:45 +0300
Message:
  -   apply fixes for networkservicemesh/cmd-registry-k8s#250 (#1221)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
This was referenced Jan 30, 2022
Merged
Merged
Merged
Merged
Merged
This was referenced Jan 30, 2022
Merged
Merged
Merged
nsmbot pushed a commit to networkservicemesh/cmd-nse-vfio that referenced this issue Feb 1, 2022
Update go.mod and go.sum to latest version from networkservicemesh/sd…
648b5d3 
…k@main

PR link: networkservicemesh/sdk#1221

Commit: 953f555
Author: Denis Tingaikin
Date: 2022-01-30 21:47:45 +0300
Message:
  -   apply fixes for networkservicemesh/cmd-registry-k8s#250 (#1221)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nse-remote-vlan that referenced this issue Feb 1, 2022
Update go.mod and go.sum to latest version from networkservicemesh/sd…
2d63cbb 
…k@main

PR link: networkservicemesh/sdk#1221

Commit: 953f555
Author: Denis Tingaikin
Date: 2022-01-30 21:47:45 +0300
Message:
  -   apply fixes for networkservicemesh/cmd-registry-k8s#250 (#1221)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
@nsmbot nsmbot mentioned this issue Feb 1, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@denis-tingaikin denis-tingaikin

Labels
bug Something isn't working
Projects
Release 1.2.0
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: Issue when register endpoints with same name denis-tingaikin/sdk
2 participants
@Mixaster995 @denis-tingaikin