openshift / openshift-sdn

Apache License 2.0
69 stars 63 forks source link

Fail more cleanly on script errors #314

Closed danwinship closed 8 years ago

danwinship commented 8 years ago

If we fail to get info from docker/ip/etc, fail immediately rather than failing with a syntax error due to substituting in an empty value later.

inspired by https://bugzilla.redhat.com/show_bug.cgi?id=1333393, though as noted there, we probably want a better solution eventually to ensure we don't leave cruft in OVS

@openshift/networking PTAL

pravisankar commented 8 years ago

LGTM

dcbw commented 8 years ago

LGTM