ovn-nbctl: Help catch lport-set-addresses mistakes.
authorRussell Bryant <russell@ovn.org>
Thu, 14 Jan 2016 15:47:18 +0000 (10:47 -0500)
committerRussell Bryant <russell@ovn.org>
Thu, 21 Jan 2016 14:06:33 +0000 (09:06 -0500)
commit69e73316e9c3c997fe353abd12a2fe1d5a21f5f7
tree666f907531408ca243797771ccbd8a1aadeb7b1e
parent667bb1fbf97774cc0627eeca9972c4f8d219fe54
ovn-nbctl: Help catch lport-set-addresses mistakes.

While debugging a broken OVN environment yesterday, the problem turned
out to be invalid entries in the logical port addresses column.  In
particular, the following command had been used:

  $ ovn-nbctl lport-set-addresses lp0 MAC IP

instead of:

  $ ovn-nbctl lport-set-addresses lp0 "MAC IP"

This is really easy to mess up, so add some simple validation to the
lport-set-addresses command.  If the beginning of an argument is ever
an IP address, it's wrong.

In passing, also add a note to the ovn-nb db documentation to note that
the order of "MAC IP" is required, as "IP MAC" is not valid.

Signed-off-by: Russell Bryant <russell@ovn.org>
Acked-by: Ben Pfaff <blp@ovn.org>
ovn/ovn-nb.xml
ovn/utilities/ovn-nbctl.c