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

[WIP] Vl3 next-hop fix #1309

Merged
merged 1 commit into from
Jun 3, 2022
Merged

Conversation

glazychev-art
Copy link
Contributor

Signed-off-by: Artem Glazychev artem.glazychev@xored.com

Description

Tested only with this test on GKE - https://github.com/networkservicemesh/deployments-k8s/tree/main/examples/features/vl3-basic

Would like to find out, why this functionality is not working - https://github.com/networkservicemesh/sdk-kernel/blob/main/pkg/kernel/networkservice/connectioncontextkernel/ipcontext/routes/common.go#L64

Issue link

How Has This Been Tested?

  • Added unit testing to cover
  • Tested manually
  • Tested by integration testing
  • Have not tested

Types of changes

  • Bug fix
  • New functionallity
  • Documentation
  • Refactoring
  • CI

Signed-off-by: Artem Glazychev <artem.glazychev@xored.com>
@denis-tingaikin
Copy link
Member

denis-tingaikin commented Jun 3, 2022

@edwarnicke I think this can be merged right now. Note: we can create release base on rc-2 branches for case if these changes are unstable OR revert this

@denis-tingaikin denis-tingaikin marked this pull request as ready for review June 3, 2022 18:50
@edwarnicke edwarnicke merged commit 6abdf1e into networkservicemesh:main Jun 3, 2022
nsmbot pushed a commit to networkservicemesh/cmd-nsmgr that referenced this pull request Jun 3, 2022
…k@main

PR link: networkservicemesh/sdk#1309

Commit: 6abdf1e
Author: Artem Glazychev
Date: 2022-06-04 02:10:23 +0700
Message:
  - Vl3 next-hop fix (#1309)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-admission-webhook-k8s that referenced this pull request Jun 3, 2022
…k@main

PR link: networkservicemesh/sdk#1309

Commit: 6abdf1e
Author: Artem Glazychev
Date: 2022-06-04 02:10:23 +0700
Message:
  - Vl3 next-hop fix (#1309)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-map-ip-k8s that referenced this pull request Jun 3, 2022
…k@main

PR link: networkservicemesh/sdk#1309

Commit: 6abdf1e
Author: Artem Glazychev
Date: 2022-06-04 02:10:23 +0700
Message:
  - Vl3 next-hop fix (#1309)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-ipam-vl3 that referenced this pull request Jun 3, 2022
…k@main

PR link: networkservicemesh/sdk#1309

Commit: 6abdf1e
Author: Artem Glazychev
Date: 2022-06-04 02:10:23 +0700
Message:
  - Vl3 next-hop fix (#1309)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/sdk-k8s that referenced this pull request Jun 3, 2022
…k@main

PR link: networkservicemesh/sdk#1309

Commit: 6abdf1e
Author: Artem Glazychev
Date: 2022-06-04 02:10:23 +0700
Message:
  - Vl3 next-hop fix (#1309)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nse-vfio that referenced this pull request Jun 3, 2022
…k@main

PR link: networkservicemesh/sdk#1309

Commit: 6abdf1e
Author: Artem Glazychev
Date: 2022-06-04 02:10:23 +0700
Message:
  - Vl3 next-hop fix (#1309)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-registry-memory that referenced this pull request Jun 3, 2022
…k@main

PR link: networkservicemesh/sdk#1309

Commit: 6abdf1e
Author: Artem Glazychev
Date: 2022-06-04 02:10:23 +0700
Message:
  - Vl3 next-hop fix (#1309)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nsc-init that referenced this pull request Jun 3, 2022
…k@main

PR link: networkservicemesh/sdk#1309

Commit: 6abdf1e
Author: Artem Glazychev
Date: 2022-06-04 02:10:23 +0700
Message:
  - Vl3 next-hop fix (#1309)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nse-remote-vlan that referenced this pull request Jun 3, 2022
…k@main

PR link: networkservicemesh/sdk#1309

Commit: 6abdf1e
Author: Artem Glazychev
Date: 2022-06-04 02:10:23 +0700
Message:
  - Vl3 next-hop fix (#1309)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nsmgr-proxy that referenced this pull request Jun 3, 2022
…k@main

PR link: networkservicemesh/sdk#1309

Commit: 6abdf1e
Author: Artem Glazychev
Date: 2022-06-04 02:10:23 +0700
Message:
  - Vl3 next-hop fix (#1309)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-registry-proxy-dns that referenced this pull request Jun 3, 2022
…k@main

PR link: networkservicemesh/sdk#1309

Commit: 6abdf1e
Author: Artem Glazychev
Date: 2022-06-04 02:10:23 +0700
Message:
  - Vl3 next-hop fix (#1309)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/sdk-kernel that referenced this pull request Jun 3, 2022
…k@main

PR link: networkservicemesh/sdk#1309

Commit: 6abdf1e
Author: Artem Glazychev
Date: 2022-06-04 02:10:23 +0700
Message:
  - Vl3 next-hop fix (#1309)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
@edwarnicke
Copy link
Member

edwarnicke commented Jun 3, 2022

Would like to find out, why this functionality is not working - https://github.com/networkservicemesh/sdk-kernel/blob/main/pkg/kernel/networkservice/connectioncontextkernel/ipcontext/routes/common.go#L64

@glazychev-art Could we open an issue for this for v1.5?

nsmbot pushed a commit to networkservicemesh/cmd-nse-remote-vlan that referenced this pull request Jun 6, 2022
…k@main

PR link: networkservicemesh/sdk#1309

Commit: 6abdf1e
Author: Artem Glazychev
Date: 2022-06-04 02:10:23 +0700
Message:
  - Vl3 next-hop fix (#1309)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/sdk-kernel that referenced this pull request Jun 6, 2022
…k@main

PR link: networkservicemesh/sdk#1309

Commit: 6abdf1e
Author: Artem Glazychev
Date: 2022-06-04 02:10:23 +0700
Message:
  - Vl3 next-hop fix (#1309)
Signed-off-by: NSMBot <nsmbot@networkservicmesh.io>
nsmbot pushed a commit to networkservicemesh/cmd-nsc-init that referenced this pull request Jun 6, 2022
…k@main

PR link: networkservicemesh/sdk#1309

Commit: 6abdf1e
Author: Artem Glazychev
Date: 2022-06-04 02:10:23 +0700
Message:
  - Vl3 next-hop fix (#1309)
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.

3 participants