[ovs-dev] [PATCH 3/8] ovn-nbctl: Update logical router port commands.
Ben Pfaff
blp at ovn.org
Sat Jun 11 02:07:21 UTC 2016
On Fri, Jun 10, 2016 at 05:03:31PM -0700, Justin Pettit wrote:
>
> > On Jun 9, 2016, at 5:09 PM, Ben Pfaff <blp at ovn.org> wrote:
> >
> > I think that lrp-add with --may-exist should report an error if the MAC
> > or NETWORK or [PEER] differ from the existing port's configuration.
>
> Currently, adding the port fails if the port belongs to a different
> router. I assume you are suggesting that we report an error if those
> fields are changing when re-adding to the same router, correct?
Yes.
The idea of --may-exist is to make the command idempotent: if the same
command has already been executed, then it turns it into a no-op. But
if a different command has been executed (with different MAC or NETWORK
or PEER), it's not what the user has asked for.
This follows the precedent set by the add-port command in ovs-vsctl,
which with --may-exist complains if, for example, the interfaces
specified for a bond are different from the ones that actually exist.
I think there is at least one other precedent, but that's the one that
comes to mind.
> When you say report an error, do you mean a fatal error (and prevent
> updating the addresses) or make the update and report a warning in the
> logs?
A fatal error, if we follow the ovs-vsctl precedent.
More information about the dev
mailing list