[ovs-dev] [RFC] ovs-tcpdump: Add a tcpdump wrapper utility

Markos Chandras mchandras at suse.de
Wed May 25 11:40:00 UTC 2016


Hi Aaron,

On 05/24/2016 09:35 PM, Aaron Conole wrote:
> [...]
> +    except OVSDBException as oe:
> +        print "ERROR: Unable to properly setup the mirror: %s." % str(oe)
> +        sys.exit(1)
> +
> +    time.sleep(1)

I am curious about this 1s delay. Is this necessary? And if so, maybe
worth adding a comment explaining why otherwise it's likely to be
forgotten and removed in the future.

-- 
markos

SUSE LINUX GmbH | GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg) Maxfeldstr. 5, D-90409, Nürnberg



More information about the dev mailing list