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 e2e regarding ippool.spec.multusName affinity. #2420

Merged
merged 1 commit into from
Oct 12, 2023

Conversation

ty-dc
Copy link
Collaborator

@ty-dc ty-dc commented Oct 12, 2023

What this PR does / why we need it:
Add e2e regarding ippool.spec.multusName affinity.

case1: multusName matches, IP can be assigned 
case2: multusName mismatch, unable to assign IP

Which issue(s) this PR fixes:

NONE

make sure your commit is signed off
Signed-off-by: ty-dc [email protected]

@ty-dc ty-dc added the release/none no release note label Oct 12, 2023
@ty-dc ty-dc force-pushed the e2e/new-2 branch 2 times, most recently from 994ae19 to ec195c3 Compare October 12, 2023 10:33
@codecov
Copy link

codecov bot commented Oct 12, 2023

Codecov Report

Merging #2420 (ec195c3) into main (a972b1a) will not change coverage.
Report is 7 commits behind head on main.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2420   +/-   ##
=======================================
  Coverage   80.97%   80.97%           
=======================================
  Files          49       49           
  Lines        5251     5251           
=======================================
  Hits         4252     4252           
  Misses        842      842           
  Partials      157      157           
Flag Coverage Δ
unittests 80.97% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@weizhoublue weizhoublue merged commit 1cde6cb into spidernet-io:main Oct 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release/none no release note
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants