You are right, this is a bug.
I reverted the commit.
On Tue, Feb 12, 2013 at 12:01:04PM -0600, Saul St. John wrote:
> Previously, I was able to set setup multiple bridges on the same host to
> talk to the same controller like this:
>
> # ovs-vsctl set-controller br0 tcp:10.0.0.1:6633
> # ovs-vsctl set-controller br1 tcp:10.0.0.1:6633
>
> and it worked. Now, when I try to do that, I get this error:
>
> ovs-vsctl: transaction error: {"details":"Transaction causes multiple rows
> in
> \"Controller\" table to have identical values (\"tcp:10.0.0.1:6633\") for
> index on column \"target\". First row, with UUID
> 454b1ac9-0698-4035-b925-f8db4a475f4f, was inserted by this transaction.
> Second row, with UUID 23f3b80b-877e-4cdb-972d-8f9928c59b3a, existed in the
> database before this transaction and was not modified by the transaction.",
> "error":"constraint violation"}
>
> Although I can still do something like this to achieve the same effect:
>
> # ovs-vsctl -- set Bridge br1
> controller=23f3b80b-877e-4cdb-972d-8f9928c59b3a
>
> Should set-controller be handling the case where there already exists a
> Controller record for the given target?
>
> On Mon, Feb 11, 2013 at 01:47:12PM -0800, Ben Pfaff wrote:
> > Thank you, I applied this to master.
> >
> > On Mon, Feb 04, 2013 at 01:00:52PM -0800, Ethan Jackson wrote:
> > > Acked-by: Ethan Jackson <et...@nicira.com>
> > >
> > > On Fri, Feb 1, 2013 at 2:51 PM, Ben Pfaff <b...@nicira.com> wrote:
> > > > Commit cc7ecee48 (vswitchd: Add unique indexes for some columns.) says,
> > > > in part:
> > > >
> > > > With this commit, the database server itself rejects attempts to add
> > > > Port or Interface records with duplicate names or Controller or
> > > > Manager records with duplicate targets.
> > > >
> > > > but in fact didn't change the Controller table as described. This
> > > > commit
> > > > fixes that.
> > > >
> > > > This commit updates the schema version number's major version, because
> > > > this
> > > > is a potentially non-backward compatible change, if some user depended
> > > > on
> > > > the ability to add Controller records with duplicate targets. However,
> > > > if
> > > > anyone thinks this is a bad idea, then I'm open to discussion.
> > > >
> > > > Reported-by: Natasha Gude <nata...@nicira.com>
> > > > Signed-off-by: Ben Pfaff <b...@nicira.com>
> > > > ---
> > > > This is a repost of a patch originally posted Dec. 4, 2012.
> > > >
> > > > vswitchd/vswitch.ovsschema | 7 ++++---
> > > > 1 files changed, 4 insertions(+), 3 deletions(-)
> > > >
> > > > diff --git a/vswitchd/vswitch.ovsschema b/vswitchd/vswitch.ovsschema
> > > > index 16125a5..025686b 100644
> > > > --- a/vswitchd/vswitch.ovsschema
> > > > +++ b/vswitchd/vswitch.ovsschema
> > > > @@ -1,6 +1,6 @@
> > > > {"name": "Open_vSwitch",
> > > > - "version": "6.11.3",
> > > > - "cksum": "2234602985 17310",
> > > > + "version": "7.0.0",
> > > > + "cksum": "1525609876 17339",
> > > > "tables": {
> > > > "Open_vSwitch": {
> > > > "columns": {
> > > > @@ -428,7 +428,8 @@
> > > > "ephemeral": true},
> > > > "status": {
> > > > "type": {"key": "string", "value": "string", "min": 0, "max":
> > > > "unlimited"},
> > > > - "ephemeral": true}}},
> > > > + "ephemeral": true}},
> > > > + "indexes": [["target"]]},
> > > > "Manager": {
> > > > "columns": {
> > > > "target": {
> > > > --
> > > > 1.7.2.5
> > > >
> > > > _______________________________________________
> > > > dev mailing list
> > > > dev@openvswitch.org
> > > > http://openvswitch.org/mailman/listinfo/dev
> > _______________________________________________
> > dev mailing list
> > dev@openvswitch.org
> > http://openvswitch.org/mailman/listinfo/dev
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
