I think auto mark is useful, merged the change,
On 18 January 2016 at 14:31, ALeX Wang wrote:
> Marked committed patches as 'Accepted'
> Marked duplicates as 'Not Applicable'
>
> I have a patch that automates the marking above,
>
> https://github.com/ye
> 570946 New2016-01-20 22:33:17 [ovs-dev,v2,2/2] ovn-nbctl: Update
> show format for addresses.
> 570945 New2016-01-20 22:33:16 [ovs-dev,v2,1/2] ovn-nbctl: Help
> catch lport-set-addresses mistakes.
> 567326 New2016-01-14 11:29:08 [ovs-dev,v1,2/2] ofproto: Wildcard
> TTL on IP tunnels
> 567325 New2016-01-14 11:29:07 [ovs-dev,v1,1/2] odp-util: Accept
> fields with zero mask
>
>
--
Alex Wang,
Open vSwitch developer
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
Marked committed patches as 'Accepted'
Marked duplicates as 'Not Applicable'
I have a patch that automates the marking above,
https://github.com/yew011/openvswitch.patchwork.mgmt/pull/1
Ben, can I merge it and do it in the weekly cron task?
Thanks,
Alex Wang,
On 17 J
So so so sorry for spaming the dev mailing list,
Forgot to turn this off, just did,
Alex Wang,
On 17 January 2016 at 16:00, Ubuntu wrote:
> 30+ Day Old Patches
> ===
> ID State Date Name
> -- -
> 5480
my old understanding is that it won't do that automatically,
you can turn on the ACK detector which will set the flag.
other project uses that,
Thanks,
Alex Wang,
On 11 January 2016 at 06:30, Russell Bryant wrote:
> On 01/11/2016 02:01 AM, ALeX Wang wrote:
> > Marked "Co
Marked "Committed Patches" as Accepted
Marked "Duplicate Patches" as Not Applicable
Alex Wang,
On 10 January 2016 at 22:28, wrote:
> 30+ Day Old Patches
> ===
> ID State Date Name
> -- - ---
Thx, all make sense,
On 12 October 2015 at 11:25, Andy Zhou wrote:
> On Sun, Oct 11, 2015 at 10:33 PM, ALeX Wang wrote:
> > Hey Andy,
> >
> > Sorry for this late post, just very interested in the feature
> > implemented in this series, one question below,
>
parsing practice?
Thanks,
Alex Wang,
--
Alex Wang,
Open vSwitch developer
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
Thx for the review,
Applied to master~,
On 14 September 2015 at 16:46, Justin Pettit wrote:
> Looks good. Thanks!
>
> Acked-by: Justin Pettit
>
> --Justin
>
>
> > On Sep 13, 2015, at 2:39 PM, Alex Wang wrote:
> >
> > This commit extends
Thx for the review,
Applied to master~,
On 14 September 2015 at 16:12, Justin Pettit wrote:
>
> > On Sep 13, 2015, at 1:58 PM, Alex Wang wrote:
> >
> > --- a/ovn/controller-vtep/gateway.c
> > +++ b/ovn/controller-vtep/gateway.c
> > @@ -189,7 +189,8 @@ gatew
On 14 September 2015 at 08:33, Ben Pfaff wrote:
> On Sun, Sep 13, 2015 at 09:57:58AM -0700, ALeX Wang wrote:
> > On 11 September 2015 at 20:29, Ben Pfaff wrote:
> >
> > > This makes ovn-nbctl into a pretty slavish imitation of ovn-nbctl,
> using
> > > almost
d 'options' configuration.
Signed-off-by: Alex Wang
Acked-by: Russell Bryant
---
V9->V10
- fix gramma errors pointed out by Justin.
- fix underscore notation of option keys (vtep-logical-switch,
vtep-physical-switch).
- rebase to master and convert 'localnet' related docum
On 13 September 2015 at 12:15, Justin Pettit wrote:
> Go through ovn-sb to see what is also set by ovn-controller-vtep. For
> example, the "chassis" column in the Port_Binding record.
>
>
Done, thx for the reminder~!
>
> > On Aug 27, 2015
This commit extends the vtep module to support creating the
'Ucast_Macs_Remote' table entries in the vtep database for
MAC addresses on the ovn logical ports.
Signed-off-by: Alex Wang
Acked-by: Russell Bryant
---
V9->V10:
- Refine comments as suggested by Justin.
- Add check
This commit adds the vtep module to ovn-controller-vtep. The
module will scan through the Port_Binding table in OVN-SB database,
and update the vtep logcial switches tunnel keys.
Signed-off-by: Alex Wang
Acked-by: Russell Bryant
Acked-by: Justin Pettit
---
V9->V10:
- Refine comments w
On 13 September 2015 at 11:59, Justin Pettit wrote:
>
> > On Sep 13, 2015, at 11:35 AM, ALeX Wang wrote:
> >
> >> On 12 September 2015 at 16:43, Justin Pettit
> wrote:
> >>
> >> > On Aug 27, 2015, at 11:21 PM, Alex Wang wrote:
> >> >
On 13 September 2015 at 10:45, Justin Pettit wrote:
>
> > On Aug 27, 2015, at 11:21 PM, Alex Wang wrote:
> >
> >
> > +/* Searches the 'chassis_rec->encaps' for the first vtep tunnel
> > + * configuration, returns the 'ip'. */
> &
On 12 September 2015 at 16:43, Justin Pettit wrote:
>
> > On Aug 27, 2015, at 11:21 PM, Alex Wang wrote:
> >
> >
> > --- a/ovn/controller-vtep/binding.c
> > +++ b/ovn/controller-vtep/binding.c
> > @@ -226,7 +226,8 @@ binding_run(struct controller_vtep_ctx
ctl?
Also,
one thing I noticed is that there is no implementation
for 'struct nbctl_context' and 'cache'. so, there is no
guarantee that in a long cmd chain, the later cmd
will never see the invalid db states.
Thanks,
Alex Wang,
> ---
> o
First three patches all look good to me,
Acked-by: Alex Wang
On 11 September 2015 at 20:29, Ben Pfaff wrote:
> I find that it's nice to give functions for commands names specific to the
> utility, even though they're static, because occasionally it makes it
> easier to fi
All look good~,
Acked-by: Alex Wang
On 11 September 2015 at 20:46, Ben Pfaff wrote:
> These patches were originally in the series that adds end-to-end
> tests for OVN, but they don't need to be there so I'm reposting
> them separately.
>
> Ben Pfaff (4):
> Fi
Hey,
I would encourage you to try it yourself, using `ovs-appctl ofproto/trace`
command,
`ovs-appctl upcall/enable-megaflows` (also disable-megaflows) can be used
to turn it
on and off,
Thanks,
Alex Wang,
On 8 September 2015 at 00:28, openvswitcher wrote:
> Hi,all
>
>
> Could any
Hey all,
Glad to announce the patchwork site (hosted by ozlabs) for Open
vSwitch project.
https://patchwork.ozlabs.org/project/openvswitch/list/
The patchwork site link is also available on the index page of
http://openvswitch.org/
Special thank to orzlabs~!
Thanks,
--
Alex Wang,
Open vSwitch
tried this out, looks useful,
Look good to me,
Thanks,
Alex Wang,
On 3 September 2015 at 07:53, Flavio Leitner wrote:
> The plotnetcfg is an open source tool to visualy represent
> relationship between network interfaces on a single host.
>
> It helps to understand the path of a
Thx applied this patch to master~!
Thanks,
Alex Wang,
On 29 August 2015 at 11:37, Joe Stringer wrote:
> On 29 August 2015 at 00:42, Alex Wang wrote:
> > When dpdk configuration changes, all pmd threads are recreated
> > and rx queues of each port are reloaded. After this
On 1 September 2015 at 19:42, Joe Stringer wrote:
> Hi Alex, a minor comment, otherwise
>
> On 29 August 2015 at 18:44, Alex Wang wrote:
> > This commit adds logic using ovs barrier to allow main thread pause
> > all revalidators. This new feature will be used in a later
This commit adds a new key-value pair, 'punix_file_group=',
to the 'other_config' column in the 'Controller' table. This new config
allows user to change the punix socket file's group ownership, so that
non-root process can also connect to ovs bridge.
Sign
On 28 August 2015 at 08:58, Ben Pfaff wrote:
> On Fri, Aug 28, 2015 at 12:25:26AM -0700, Alex Wang wrote:
> > On Wed, Aug 26, 2015 at 2:13 PM, Ben Pfaff wrote:
> >
> > > The text in the log messages use " : " as separators but the common
> > > sty
On 28 August 2015 at 00:25, Alex Wang wrote:
> On Wed, Aug 26, 2015 at 2:13 PM, Ben Pfaff wrote:
>
> > On Fri, Aug 21, 2015 at 11:10:32PM -0700, Alex Wang wrote:
> > > This commit adds a new key-value pair, 'punix_file_group=',
> > > to the 'other_co
This commit adds logic using ovs barrier to allow main thread pause
all revalidators. This new feature will be used in a later patch.
Signed-off-by: Alex Wang
---
V3->V4:
- sync the pause_latch state among all revalidators~ thx Joe for point
out~!
V3:
- new patch.
---
ofproto/ofproto-d
Thx for pointing that out,
Totally ignore the existence of other barriers ;D
Let me address that,
Thanks,
Alex Wang,
On 29 August 2015 at 17:49, Joe Stringer wrote:
> On 29 August 2015 at 17:42, ALeX Wang wrote:
> >
> >
> > On 29 August 2015 at 16:45, Joe Stringer w
On 29 August 2015 at 16:45, Joe Stringer wrote:
> On 29 August 2015 at 15:24, Alex Wang wrote:
> >
> >
> > On Sat, Aug 29, 2015 at 11:34 AM, Joe Stringer
> > wrote:
> >>
> >> Thanks for working on this, Alex. I've considered implementing an
On Sat, Aug 29, 2015 at 11:34 AM, Joe Stringer
wrote:
> Thanks for working on this, Alex. I've considered implementing an
> approach like this before, but haven't had a strong reason to.
>
> On 29 August 2015 at 00:42, Alex Wang wrote:
> > This commit adds logic usi
the purging of datapath
(e.g. pmd thread deletion in dpif-netdev). So, the
ofproto-dpif-upcall module can react properly with deleting
the ukeys and with collecting flows' last stats.
Reported-by: Ilya Maximets
Signed-off-by: Alex Wang
Acked-by: Daniele Di Proietto
Tested-by: Danie
This commit adds logic using ovs barrier to allow main thread pause
all revalidators. This new feature will be used in a later patch.
Signed-off-by: Alex Wang
---
V3:
- new patch.
---
ofproto/ofproto-dpif-upcall.c | 46 +++
1 file changed, 46 insertions
be modify the
pmd thread destroy functions to avoid multiple times of revalidators
recreation,
Will try and send out V3 patch~
Thanks,
Alex Wang,
>
> > On 28/08/2015 06:25, "Alex Wang" wrote:
> >
> >>When dpdk configuration changes, all pmd threads are
Thx a lot for the quick review,
On 28 August 2015 at 09:41, Daniele Di Proietto
wrote:
> I've tested it and it appears to correctly keep all the stats.
>
> Two comments inline, otherwise:
>
> Acked-by: Daniele Di Proietto
>
> Thanks for fixing this Alex!
>
>
Thanks, fixed the typo and adopted your suggested change,~
On 28 August 2015 at 09:18, Jarno Rajahalme wrote:
>
> > On Aug 27, 2015, at 10:25 PM, Alex Wang wrote:
> >
> > When dpdk configuration changes, all pmd threads are recreated
> > and rx queues of each por
Thx, applied to master~
Alex Wang,
On 28 August 2015 at 14:07, Ben Pfaff wrote:
> On Fri, Aug 28, 2015 at 07:31:21AM +0000, Alex Wang wrote:
> > This commit relaxes the whitelist format for punix path for
> > service controller. Instead of only allowing
> > punix:/.contr
Thx, applied to master~
On 28 August 2015 at 09:12, Daniele Di Proietto
wrote:
> With the older version of this series I was able to reproduce the problem
> and this patch appears to fix it.
>
> Acked-by: Daniele Di Proietto
>
> Thanks!
>
> On 28/08/2015 06:25, "A
This commit relaxes the whitelist format for punix path for
service controller. Instead of only allowing
punix:/.controller, the new format
allows any suffix, like punix:/.*.
(except '/').
Signed-off-by: Alex Wang
---
PATCH->V2:
- prevent the punix path from specifying directory o
On Wed, Aug 26, 2015 at 2:13 PM, Ben Pfaff wrote:
> On Fri, Aug 21, 2015 at 11:10:32PM -0700, Alex Wang wrote:
> > This commit adds a new key-value pair, 'punix_file_group=',
> > to the 'other_config' column in the 'Controller' table. This new conf
This commit extends the vtep module to support creating the
'Ucast_Macs_Remote' table entries in the vtep database for
MAC addresses on the ovn logical ports.
Signed-off-by: Alex Wang
Acked-by: Russell Bryant
---
V8->V9:
- Add Ack from Russell.
- Minor adjustment based on Russell&
This commit adds the vtep module to ovn-controller-vtep. The
module will scan through the Port_Binding table in OVN-SB database,
and update the vtep logcial switches tunnel keys.
Signed-off-by: Alex Wang
Acked-by: Russell Bryant
---
V8->V9:
- Add Ack from Russell.
- Fix minor comments.
V7-
d 'options' configuration.
Signed-off-by: Alex Wang
Acked-by: Russell Bryant
---
V8->V9:
- rebase.
V7->V8:
- rebase.
V6->V7:
- adopt suggestions from Russell.
- leave out the documentation of feature that has not been implemented yet.
V5->V6:
- add description in ovn-architectur
Hey Russell,
Sorry for this very delayed reply,~
Please see my reply inline,
Thanks,
Alex Wang,
On Mon, Aug 24, 2015 at 10:48 AM, Russell Bryant wrote:
> On 08/23/2015 02:06 PM, Alex Wang wrote:
> > This commit extends the vtep module to support creating the
> > 'Ucas
the purging of datapath
(e.g. pmd thread deletion in dpif-netdev). So, the
ofproto-dpif-upcall module can react properly with deleting
the ukeys and with collecting flows' last stats.
Reported-by: Ilya Maximets
Signed-off-by: Alex Wang
---
PATCH->V2:
- PATCH does not fix the problem
happen, it is still good to make
push_ukey_ops__() just ignore the operation when it fails.
Signed-off-by: Alex Wang
---
V2:
- new patch.
---
ofproto/ofproto-dpif-upcall.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/ofproto/ofproto-dpif-upcall.c b/ofproto/ofproto-dpif-upcall.c
ind
Looks good to me,
Thx for fixing it~
On Thu, Aug 27, 2015 at 11:11 AM, Ben Pfaff wrote:
> "sed -i" isn't entirely portable, and we can avoid it by using the
> argument to check_logs as intended.
>
> Signed-off-by: Ben Pfaff
> Acked-by: Alex Wang
> ---
>
On Thu, Aug 27, 2015 at 5:15 PM, Justin Pettit wrote:
> Sadly:
>
> Acked-by: Justin Pettit
>
>
Thx, I also feel sad
Will be around~
Thanks,
Alex Wang,
> --Justin
>
>
> > On Aug 27, 2015, at 4:59 PM, Alex Wang wrote:
> >
> > Signed-off-by: Alex W
Signed-off-by: Alex Wang
---
AUTHORS | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/AUTHORS b/AUTHORS
index 254095e..4be28c4 100644
--- a/AUTHORS
+++ b/AUTHORS
@@ -7,7 +7,7 @@ Alexander Duyck alexander.h.du...@redhat.com
Alexandru Copot alex.miha
or/d
/timeval.*disk: [[0-9]]* reads, [[0-9]]* writes/d
On Wed, Aug 26, 2015 at 3:55 PM, Ben Pfaff wrote:
> "sed -i" isn't entirely portable, and we can avoid it by using the
> argument to check_logs as intended.
>
> Signed-off-by: Ben Pfaff
> Acked-by: Alex Wang
Thx for informing the portability issue.
I used 'sed -i' since I only saw this in ovsdb-server.log, so originally,
did not
want to miss same error logs in other *.log file,
But, since 'sed -i' is not portable, I think it is good to do it in
check_log()
Acked-by: Alex Wang
the purging of datapath
(e.g. pmd thread deletion in dpif-netdev). So, the
ofproto-dpif-upcall module can react properly with deleting
the ukeys and with collecting flows' last stats.
Reported-by: Ilya Maximets
Signed-off-by: Alex Wang
---
PATCH->V2:
- PATCH does not fix the problem
in each 'pmd' have
already
gone... So, we did not harvest the last bit of stats,
Instead, I think the right place to call purge_cb() is between pmd thread
stop
and deletion in dp_netdev_del_pmd(). Please see the V2 patch for details,
Thanks,
Alex Wang,
On Mon, Aug 17, 2015 at 11:0
Apologize for the delay, applied to master~
On Tue, Aug 25, 2015 at 4:53 AM, Traynor, Kevin
wrote:
>
> > -Original Message-
> > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Daniele Di
> > Proietto
> > Sent: Monday, August 24, 2015 7:08 PM
&g
And branch 2.1, 2.0
On Mon, Aug 24, 2015 at 10:12 PM, Alex Wang wrote:
> Just realized that this needs to be backported to branch-2.3,
>
> did that,~
>
> Thanks,
> Alex Wang,
>
> On Wed, Mar 11, 2015 at 2:05 PM, Alex Wang wrote:
>
>>
>> On Wed, Ma
Just realized that this needs to be backported to branch-2.3,
did that,~
Thanks,
Alex Wang,
On Wed, Mar 11, 2015 at 2:05 PM, Alex Wang wrote:
>
> On Wed, Mar 11, 2015 at 2:04 PM, Ben Pfaff wrote:
>
>> Acked-by: Ben Pfaff
>>
>
On Mon, Aug 24, 2015 at 2:46 PM, Russell Bryant wrote:
> On 08/24/2015 05:37 PM, Alex Wang wrote:
> >
> >
> > On Mon, Aug 24, 2015 at 11:30 AM, Russell Bryant > <mailto:rbry...@redhat.com>> wrote:
> >
> > On 08/22/2015 12:36 PM, Ale
On Mon, Aug 24, 2015 at 11:30 AM, Russell Bryant wrote:
> On 08/22/2015 12:36 PM, Alex Wang wrote:
> > This commit makes the OVN_CONTROLLER_VTEP_STOP remove all 'Broken pipe'
> > warning logs from ovsdb-server.log before running 'check_log'. This is
> in
d 'options' configuration.
Signed-off-by: Alex Wang
Acked-by: Russell Bryant
---
V7->V8:
- rebase.
V6->V7:
- adopt suggestions from Russell.
- leave out the documentation of feature that has not been implemented yet.
V5->V6:
- add description in ovn-architecture.
This commit extends the vtep module to support creating the
'Ucast_Macs_Remote' table entries in the vtep database for
MAC addresses on the ovn logical ports.
Signed-off-by: Alex Wang
---
V7->V8:
- rebase.
V6->V7:
- rebase.
- adopt suggestions from Russell.
V5->V6:
- rebase
This commit adds the vtep module to ovn-controller-vtep. The
module will scan through the Port_Binding table in OVN-SB database,
and update the vtep logcial switches tunnel keys.
Signed-off-by: Alex Wang
---
V7->V8:
- Fix bug pointed out by Russell.
V6->V7:
- change the assertion to VL
ovn-controller-vtep)
may immediately update the database; this later update may cause database
sending update back to *ctl command if *ctl has not proceeded to exit yet;
and if *ctl command exits before database calling send, the send will fail
with 'Broken pipe' error.
Signed-off
Sorry for the delay of pushing this and related dpdk patches,
Want to spend some time next week measuring the performance impact,~
Thanks,
Alex Wang,
On Fri, Aug 21, 2015 at 12:48 PM, Ben Pfaff wrote:
> On Thu, Aug 13, 2015 at 11:48:49AM -0700, Alex Wang wrote:
> > For performance
If we want to make ovsdb-server non-root, this change may not be need,~
On Fri, Aug 21, 2015 at 11:10 PM, Alex Wang wrote:
> This commit adds a new key-value pair, 'punix_file_group=',
> to the 'other_config' column in the 'Manager' table. This new conf
This commit adds a new key-value pair, 'punix_file_group=',
to the 'other_config' column in the 'Manager' table. This new config
allows user to change the punix socket file's group ownership, so that
non-root process can also connect to ovsdb-server.
Signed-of
This commit adds a new key-value pair, 'punix_file_group=',
to the 'other_config' column in the 'Controller' table. This new config
allows user to change the punix socket file's group ownership, so that
non-root process can also connect to ovs bridge.
Signed
This commit relaxes the whitelist format for punix path for
service controller. Instead of only allowing
punix:/.controller, the new format
allows any suffix, like punix:/.*.
Signed-off-by: Alex Wang
---
tests/ovs-vswitchd.at | 11 +++
vswitchd/bridge.c | 11 ++-
2
On Fri, Aug 21, 2015 at 3:02 PM, Ben Pfaff wrote:
> On Mon, Aug 17, 2015 at 11:24:48AM -0700, Alex Wang wrote:
> > Hey,
> >
> > Want to open a thread to discuss the following race I encountered while
> > unit testing ovn.
> >
> > The most simple case is
On Thu, Aug 20, 2015 at 9:40 AM, Russell Bryant wrote:
> On 08/20/2015 09:34 AM, Alex Wang wrote:
> >
> >
> > On Thu, Aug 20, 2015 at 9:09 AM, Russell Bryant > <mailto:rbry...@redhat.com>> wrote:
> >
> > Looks good to me except for the one thi
On Thu, Aug 20, 2015 at 9:09 AM, Russell Bryant wrote:
> Looks good to me except for the one thing I noticed that was introduced
> in the last patch.
>
> On 08/18/2015 05:58 PM, Alex Wang wrote:
> > This commit extends the vtep module to support creating the
> > 'Uca
As well as a test to cover that~ ;D
On Thu, Aug 20, 2015 at 9:06 AM, Alex Wang wrote:
>
>
> On Thu, Aug 20, 2015 at 8:24 AM, Russell Bryant
> wrote:
>
>> acked this earlier, but just came across something while reviewing the
>> next patch ... let me know if I'
On Thu, Aug 20, 2015 at 8:24 AM, Russell Bryant wrote:
> acked this earlier, but just came across something while reviewing the
> next patch ... let me know if I'm missing something.
>
> On 08/18/2015 05:58 PM, Alex Wang wrote:
> > This commit adds the vtep module to ov
On Wed, Aug 19, 2015 at 9:55 PM, Ben Pfaff wrote:
> On Tue, Aug 18, 2015 at 05:47:18PM -0700, Alex Wang wrote:
> > Signed-off-by: Alex Wang
> > ---
> > tests/ovn-controller-vtep.at |4 +++-
> > 1 file changed, 3 insertions(+), 1 deletion(-)
> >
> > di
On Wed, Aug 19, 2015 at 9:45 PM, Russell Bryant wrote:
>
>
> On Aug 19, 2015, at 9:00 PM, Alex Wang wrote:
>
>
>
> On Wed, Aug 19, 2015 at 8:58 PM, Ben Pfaff wrote:
>
>> On Wed, Aug 19, 2015 at 08:51:11PM -0700, Ben Pfaff wrote:
>> > I see that severa
Just want to follow the same fashion like 'del_bridge()', in ovs-vsctl.c .
Thanks,
Alex Wang,
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
be reviewed as-is, or is it appropriate to post a
> > rebased v6?
>
> Oh, never mind, I see now that there's a v6 already.
>
And V7 ;D, three patches left,
Thanks,
Alex Wang,
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
This commit extends the vtep module to support creating the
'Ucast_Macs_Remote' table entries in the vtep database for
MAC addresses on the ovn logical ports.
Signed-off-by: Alex Wang
---
V6->V7:
- rebase.
- adopt suggestions from Russell.
V5->V6:
- rebase.
V4->V5:
- reba
d 'options' configuration.
Signed-off-by: Alex Wang
---
V6->V7:
- adopt suggestions from Russell.
- leave out the documentation of feature that has not been implemented yet.
V5->V6:
- add description in ovn-architecture.
V5: new patch.
---
ovn/TODO | 21 +++
This commit adds the vtep module to ovn-controller-vtep. The
module will scan through the Port_Binding table in OVN-SB database,
and update the vtep logcial switches tunnel keys.
Signed-off-by: Alex Wang
---
V6->V7:
- change the assertion to VLOG_ERR in vtep_lswitch_run().
- refine
Signed-off-by: Alex Wang
---
tests/ovn-controller-vtep.at |4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/tests/ovn-controller-vtep.at b/tests/ovn-controller-vtep.at
index de97bf9..fe2fd4d 100644
--- a/tests/ovn-controller-vtep.at
+++ b/tests/ovn-controller-vtep.at
Thanks, applied to master,~
On Tue, Aug 18, 2015 at 11:05 AM, Russell Bryant wrote:
> On 08/18/2015 11:02 AM, Alex Wang wrote:
> > Found by inspection.
> >
> > Signed-off-by: Alex Wang
>
> Acked-by: Russell Bryant
>
> --
> Russell Bryant
>
Thx, applied to master,
On Tue, Aug 18, 2015 at 11:06 AM, Alex Wang wrote:
> Acked-by: Alex Wang
>
> On Tue, Aug 18, 2015 at 8:24 AM, Russell Bryant
> wrote:
>
>> The static result of default_db() was malloc'd but not freed before
>> exit. Make the static
Acked-by: Alex Wang
On Tue, Aug 18, 2015 at 8:24 AM, Russell Bryant wrote:
> The static result of default_db() was malloc'd but not freed before
> exit. Make the static result global and free it before exit.
>
> Signed-off-by: Russell Bryant
> ---
> ovn/n
Found by inspection.
Signed-off-by: Alex Wang
---
PATCH->V2:
- free 'default_db_'.
---
ovn/controller-vtep/ovn-controller-vtep.c | 19 ++-
1 file changed, 10 insertions(+), 9 deletions(-)
diff --git a/ovn/controller-vtep/ovn-controller-vtep.c
b/ovn/contro
On Tue, Aug 18, 2015 at 8:10 AM, Russell Bryant wrote:
> On 08/17/2015 11:29 PM, Alex Wang wrote:
> > Otherwise, binding_cleanup() will be a no-op since all related chassis
> > entries are deleted in gateway_cleanup().
> >
> > Found by inspection.
>
Found by inspection.
Signed-off-by: Alex Wang
---
ovn/controller-vtep/ovn-controller-vtep.c |8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/ovn/controller-vtep/ovn-controller-vtep.c
b/ovn/controller-vtep/ovn-controller-vtep.c
index ff25319..a42122b 100644
--- a
Otherwise, binding_cleanup() will be a no-op since all related chassis
entries are deleted in gateway_cleanup().
Found by inspection.
Signed-off-by: Alex Wang
---
ovn/controller-vtep/ovn-controller-vtep.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/ovn/controller
Found by inspection.
Signed-off-by: Alex Wang
---
ovn/controller-vtep/binding.c |4
1 file changed, 4 insertions(+)
diff --git a/ovn/controller-vtep/binding.c b/ovn/controller-vtep/binding.c
index cb7f389..652852d 100644
--- a/ovn/controller-vtep/binding.c
+++ b/ovn/controller-vtep
Wow, this is such an awesome change!!!
I totally like it~ Could you send the patch? Would like to know what Ben
thinks about this,
Thanks,
Alex Wang,
On Mon, Aug 17, 2015 at 9:43 AM, Russell Bryant wrote:
> On 08/17/2015 09:06 AM, Alex Wang wrote:
> >
> >
> > On Mon, Au
Thx for fixing this~!
Acked-by: Alex Wang
On Mon, Aug 17, 2015 at 3:44 PM, Gurucharan Shetty
wrote:
> A unit test for ovn-northd was hanging because ovn-northd
> running on Windows failed to detach. The detach function
> for Windows is implemented inside "service_start()&quo
Thx, applied to master~,
On Mon, Aug 17, 2015 at 3:11 PM, Gurucharan Shetty
wrote:
> On Mon, Aug 17, 2015 at 3:09 PM, Alex Wang wrote:
> > The 'ovs-vtep' simulator is not ported to Windows. So, for now,
> > just skip all tests in ovn-controller-vtep.at wh
Acked-by: Alex Wang
Thx for writing this fix,
Can you also add the same line to ovn/controller/ovn-controller.c and
ovn/controller-vtep/ovn-controller-vtep.c ?
Thanks,
Alex Wang,
On Mon, Aug 17, 2015 at 3:00 PM, Gurucharan Shetty
wrote:
> A unit test for ovn-northd was hanging because
The 'ovs-vtep' simulator is not ported to Windows. So, for now,
just skip all tests in ovn-controller-vtep.at when running in
Windows.
Signed-off-by: Alex Wang
---
tests/ovn-controller-vtep.at |6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/tests/ovn-
ovsdb_idl_txn_commit_block() and try waiting until receiving the
monitor request ack from ovsdb-server. After that ovsdb-server will
never try sending anything to "*-*ctl" commands,
Would like to hear what you think?~
Thanks,
Alex Wang,
___
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
e' is true, the obsolete ukeys should be
deleted at revaidator_sweep() stage. Right?
This sounds to me like a different issue and may require more thinking, so
I'd
like to keep it a separate issue,
Thanks,
Alex Wang,
> Thanks
>
> On 11/08/2015 03:47, "Alex Wang" wr
On Mon, Aug 17, 2015 at 6:37 AM, Russell Bryant wrote:
> On 08/16/2015 07:24 PM, Alex Wang wrote:
> >
> >
> > On Sun, Aug 16, 2015 at 3:26 PM, Russell Bryant > <mailto:rbry...@redhat.com>> wrote:
> >
> > On 08/09/2015 10:50 PM, Alex Wang wrote:
On Sun, Aug 16, 2015 at 3:38 PM, Russell Bryant wrote:
> Thanks for the docs! Just some minor comments/questions ...
>
> On 08/09/2015 10:50 PM, Alex Wang wrote:
> > This commit conducts the following documentation changes:
> >
> > * add a description in
On Sun, Aug 16, 2015 at 3:26 PM, Russell Bryant wrote:
> On 08/09/2015 10:50 PM, Alex Wang wrote:
> > This commit extends the vtep module to support creating the
> > 'Ucast_Macs_Remote' table entries in the vtep database for
> > MAC addresses on the ovn logical por
1 - 100 of 1759 matches
Mail list logo
