Re: [ovs-dev] ovn-controller doesn't bring re-added container ports up?

2021-02-03 Thread Dumitru Ceara
On 2/2/21 8:17 PM, Dumitru Ceara wrote: On 2/2/21 7:18 PM, Numan Siddique wrote: On Tue, Feb 2, 2021 at 11:26 PM Numan Siddique wrote: On Tue, Feb 2, 2021 at 11:11 PM Ben Pfaff wrote: The test "ovn -- nested containers" adds, removes, and then re-adds a container port bar2.  It appears to

Re: [ovs-dev] ovn-controller doesn't bring re-added container ports up?

2021-02-02 Thread 0-day Robot
Bleep bloop. Greetings Ben Pfaff, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. git-am: error: sha1 information is lacking or useless (tests/ovn.at). error: could not build fake ancestor hint

Re: [ovs-dev] ovn-controller doesn't bring re-added container ports up?

2021-02-02 Thread 0-day Robot
Bleep bloop. Greetings Ben Pfaff, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. git-am: error: sha1 information is lacking or useless (tests/ovn.at). error: could not build fake ancestor hint

Re: [ovs-dev] ovn-controller doesn't bring re-added container ports up?

2021-02-02 Thread Dumitru Ceara
On 2/2/21 7:18 PM, Numan Siddique wrote: On Tue, Feb 2, 2021 at 11:26 PM Numan Siddique wrote: On Tue, Feb 2, 2021 at 11:11 PM Ben Pfaff wrote: The test "ovn -- nested containers" adds, removes, and then re-adds a container port bar2. It appears to me that ovn-controller never brings the r

Re: [ovs-dev] ovn-controller doesn't bring re-added container ports up?

2021-02-02 Thread Numan Siddique
On Tue, Feb 2, 2021 at 11:26 PM Numan Siddique wrote: > > On Tue, Feb 2, 2021 at 11:11 PM Ben Pfaff wrote: > > > > The test "ovn -- nested containers" adds, removes, and then re-adds a > > container port bar2. It appears to me that ovn-controller never brings > > the re-added port up. I don't k

Re: [ovs-dev] ovn-controller doesn't bring re-added container ports up?

2021-02-02 Thread Numan Siddique
On Tue, Feb 2, 2021 at 11:11 PM Ben Pfaff wrote: > > The test "ovn -- nested containers" adds, removes, and then re-adds a > container port bar2. It appears to me that ovn-controller never brings > the re-added port up. I don't know why. > > To see the problem, apply the following patch then run

[ovs-dev] ovn-controller doesn't bring re-added container ports up?

2021-02-02 Thread Ben Pfaff
The test "ovn -- nested containers" adds, removes, and then re-adds a container port bar2. It appears to me that ovn-controller never brings the re-added port up. I don't know why. To see the problem, apply the following patch then run that test (currently test 88): diff --git a/tests/ovn.at b/