openshift / openshift-sdn

Apache License 2.0
69 stars 63 forks source link

Simplify service tracking by using kapi.NamespaceAll #171

Closed danwinship closed 9 years ago

danwinship commented 9 years ago

We don't need to track services per-namespace; we can just use the namespace kapi.NamespaceAll (aka ""), like "oc get --all-namespaces" does.

Closes #158

@pravisankar

rajatchopra commented 9 years ago

Good stroke. I was searching for this one in the beginning but couldn't think of this obvious one.

Lgtm.

pravisankar commented 9 years ago

Nice, clean solution. lgtm