search

contiv / netplugin

Container networking for various use cases
Apache License 2.0
513 stars 177 forks source link

[WIP] Support secure etcd connection with tls cert #1077

Closed BSWANG closed 6 years ago

BSWANG commented 6 years ago

Description of the changes

Type of fix: New feature

Fixes #777

Please describe:

TODO

dseevr commented 6 years ago

build pr

kahou82 commented 6 years ago

build PR

dseevr commented 6 years ago

@BSWANG This PR is hitting a compile error here:

# github.com/contiv/netplugin/netmaster/objApi
netmaster/objApi/objapi_test.go:137: not enough arguments in call to NewAPIController

You can leave this PR open and just force push your branch when you make changes. There's no problem with that.

If you want to run the tests locally, you can run make system-test. It will spin up 3 VMs using VirtualBox and run all the tests. You will need 16gb ram, though. It will be faster than waiting for us to check our CI and post the errors here. 😃