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

qfix: Use correct VxLAN port for the client/server cases #310

Merged
merged 1 commit into from
Mar 27, 2024

Conversation

denis-tingaikin
Copy link
Member

@denis-tingaikin denis-tingaikin commented Mar 27, 2024

Description

Fixes ovs feature suite on the CI.
Root cause #303

Issue link

Related to networkservicemesh/integration-k8s-kind#995

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 functionality
  • Documentation
  • Refactoring
  • CI

Signed-off-by: Denis Tingaikin <denis.tingajkin@xored.com>
@denis-tingaikin denis-tingaikin marked this pull request as draft March 27, 2024 00:22
@denis-tingaikin denis-tingaikin changed the title qfix: Use correct VxLAN port for the client/server cases WIP: Use correct VxLAN port for the client/server cases Mar 27, 2024
@denis-tingaikin denis-tingaikin marked this pull request as ready for review March 27, 2024 08:26
@denis-tingaikin
Copy link
Member Author

@denis-tingaikin denis-tingaikin changed the title WIP: Use correct VxLAN port for the client/server cases qfix: Use correct VxLAN port for the client/server cases Mar 27, 2024
@glazychev-art glazychev-art merged commit b52aef4 into main Mar 27, 2024
13 checks passed
nsmbot pushed a commit to networkservicemesh/cmd-forwarder-ovs that referenced this pull request Mar 27, 2024
…k-ovs@main

PR link: networkservicemesh/sdk-ovs#310

Commit: b52aef4
Author: Artem Glazychev
Date: 2024-03-27 15:33:04 +0700
Message:
  - Merge pull request #310 from networkservicemesh/fix-feature-ovs-tests-on-kind
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
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants