-
Notifications
You must be signed in to change notification settings - Fork 584
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
create multus kubeconfig for in case of non auto flag. #1317
create multus kubeconfig for in case of non auto flag. #1317
Conversation
This PR is based on #1302 , after rebase on master and some streamlining of the test to reduce code changes in thin_entrypoint. |
@dougbtv PTAL |
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.
thanks for the rework Adrian!
"testing" | ||
|
||
. "github.com/onsi/ginkgo/v2" //nolint:golint | ||
. "github.com/onsi/gomega" //nolint:golint | ||
) | ||
|
||
// chrootTestHelper performs chroot syscall, returns func to get back to original root or error if occurred | ||
func chrootTestHelper(path string) (func() error, error) { |
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.
smart!
@@ -585,6 +585,11 @@ func main() { | |||
var masterConfigFilePath string | |||
// copy user specified multus conf to CNI conf directory | |||
if opt.MultusConfFile != "auto" { | |||
caHash, saTokenHash, err = opt.createKubeConfig(nil, nil) |
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.
nice and efficient.
This is a small rework of PR#1302 to reduce code changes for test.
Fixes #1295