-
Notifications
You must be signed in to change notification settings - Fork 453
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
pod should use mac and ips provider by multus firstly #4800
base: master
Are you sure you want to change the base?
Conversation
Pull Request Test Coverage Report for Build 12488976839Warning: This coverage report may be inaccurate.This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Details
💛 - Coveralls |
加个 e2e case? |
annotationsNeedToAdd[fmt.Sprintf(util.IPAddressAnnotationTemplate, podNet.ProviderName)] = podNet.IPRequest | ||
} | ||
|
||
if podNet.MacRequest != "" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
如果用的不是 kubeovn 的 ip 和 mac,是不是补充个 log ? 不然查问题的话,不确定 mac 和 ip 哪里来的
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
已加
1 |
Signed-off-by: clyi <[email protected]>
92158c6
to
d21595e
Compare
Signed-off-by: clyi <[email protected]>
Pull Request
What type of this PR
Examples of user facing changes:
Which issue(s) this PR fixes
Fixes #(issue-number)