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

Add remote vlan mechanism test #4293

Merged
merged 1 commit into from
Jan 25, 2022
Merged

Conversation

ljkiraly
Copy link
Contributor

Signed-off-by: Laszlo Kiraly [email protected]

Signed-off-by: Laszlo Kiraly <[email protected]>
@denis-tingaikin
Copy link
Member

Hello @ljkiraly!

Did you run this example locally?

@ljkiraly
Copy link
Contributor Author

Hello @denis-tingaikin ,
I ran this test locally and found some bugs in the code (fixed, merged).
sdk-vpp/494
sdk-kernel/411

Now sometimes fails with timeout since I did not added the required images to prefetch - some element startup times out. Please wait with merging this script upstream, I will break out into two parts to shorten the runtime and to make place to other checks like IPv6, udp, etc. I created this PR just to show the concept and as reference. I will ask for a review as soon as I am done with the remainders.
Thanks,
Laszlo

@denis-tingaikin
Copy link
Member

@ljkiraly OK, got it, thanks for the clarification.

@edwarnicke edwarnicke self-requested a review January 25, 2022 15:55
@edwarnicke edwarnicke merged commit b9dca41 into networkservicemesh:main Jan 25, 2022
nsmbot pushed a commit to networkservicemesh/integration-tests that referenced this pull request Jan 25, 2022
…ployments-k8s@main

PR link: networkservicemesh/deployments-k8s#4293

Commit: d3e8e6d
Author: Network Service Mesh Bot
Date: 2022-01-25 13:12:09 -0600
Message:
  - Update go.mod and go.sum to latest version from networkservicemesh/de…
Signed-off-by: NSMBot <[email protected]>
@ljkiraly ljkiraly deleted the vlansup branch February 8, 2022 12:10
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.

3 participants