-
Notifications
You must be signed in to change notification settings - Fork 76
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
clean the /et/cni/net.d folder before we start the k8s provsion #190
Conversation
…sion Signed-off-by: Hui Chen <[email protected]>
Signed-off-by: Hui Chen <[email protected]>
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.
Some nits only, LGTM overall.
Co-authored-by: Dharmjit Singh <[email protected]>
Co-authored-by: Dharmjit Singh <[email protected]>
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.
LGTM.
Signed-off-by: Hui Chen <[email protected]>
Signed-off-by: Hui Chen <[email protected]>
…b runner, and caused failure of test-agent case Signed-off-by: Hui Chen <[email protected]>
…2e case failed. Only clean all the files or directories under /etc/cni/net.d directory Signed-off-by: Hui Chen <[email protected]>
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.
overall lgtm.
Wondering about the unit tests for dir clean function.
Signed-off-by: Hui Chen <[email protected]>
It's hard to test this in unit tests. All I call is system function, let's have some confidence about it. |
we should clean the /et/cni/net.d folder before we start the k8s provsion. It will save us from a lot of potential network issues, especially we try to re-use a byoh host.