contiv / netplugin

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

make Kubernetes use standalone OVS container #1106

Closed unclejack closed 6 years ago

unclejack commented 6 years ago

This PR makes the netplugin Docker image not use the embedded OVS by default. This makes it possible to run OVS in a separate container.

tiewei commented 6 years ago

build PR

tiewei commented 6 years ago

build PR

dseevr commented 6 years ago

build pr

unclejack commented 6 years ago

build PR

unclejack commented 6 years ago

build PR

tiewei commented 6 years ago

build PR

unclejack commented 6 years ago

build PR

tiewei commented 6 years ago

LGTM !

tiewei commented 6 years ago

build PR

chrisplo commented 6 years ago

LGTM

unclejack commented 6 years ago

I've got some more changes for this.

unclejack commented 6 years ago

build PR

chrisplo commented 6 years ago

thinking, this pod is going to be marked ready before switch is ready to accept traffic, might be room for improvement here using k8s features?

tiewei commented 6 years ago

build PR

unclejack commented 6 years ago

build PR

unclejack commented 6 years ago

build PR

unclejack commented 6 years ago

The CI status is green and the PR has been reviewed. I'll merge this now.