backport: ci: run some cni windows uts in the pipeline (#2925) #2962
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* make windows uts pass * separate multitenancy uts for windows into platform specific file * move ut testing to separate template * address linter issue * run windows platform tests * revert removing test create bridge * remove parameter from ut template
QxBytes
added
cni
- updates mock network manager to use the network id passed in - updates mock network manager to get all endpoint infos based on a network id - updates mock network manager find network id from net ns to reflect dualnic get network id behavior (gets multiple networks because on deletion, you have one less endpoint that matches the netns/netnspath, and so use the network id of the other endpoint for deletion) - updates windows tests to use eth1 to avoid mock endpoint client from throwing a duplicate endpoint id error - separates windows and linux multitenancy tests because their behavior is different - updates windows unit tests to allow passing in an expected num of endpoints for EACH network id after EACH command is run during delete test
QxBytes
force-pushed
the
alew/backport-windows-uts
branch
from
e3bc971 to
d55388b
Compare
QxBytes
force-pushed
the
alew/backport-windows-uts
branch
from
5bf6d4f to
b94e54b
Compare
|
/azp run Azure Container Networking PR |
|
Azure Pipelines successfully started running 1 pipeline(s). |
jpayne3506
approved these changes
Aug 28, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Reason for Change:
Backports #2925
cherry pick files to mock hns v2
modify multitenancy cni tests to reflect 1.5.x behavior (changes to be applied only to 1.5.x as behavior differs in 1.6.x):
Issue Fixed:
Requirements:
Notes: