[ovs-dev] [PATCH v2] ovs-tcpdump: Support vlan option.

2024-04-09 Thread Daniel Ding
select_vlan column, and also marks the port as native tagged with VLAN. Signed-off-by: Daniel Ding --- utilities/ovs-tcpdump.in | 35 +++ 1 file changed, 31 insertions(+), 4 deletions(-) diff --git a/utilities/ovs-tcpdump.in b/utilities/ovs-tcpdump.in index

Re: [ovs-dev] [PATCH] ovs-tcpdump: Support vlan option.

2024-03-30 Thread Daniel Ding
> 2024年3月30日 上午2:43,Aaron Conole 写道: > > Daniel Ding mailto:danieldin...@gmail.com>> writes: > >> When I try filter geneve protocol with a vlan, the warning message >> occurs that tell me the kernel cann't support this combination. >

[ovs-dev] [PATCH] ovs-tcpdump: Support vlan option.

2024-03-23 Thread Daniel Ding
with a vlan tag, which let datapath to pop its tag. 2. the traffic will be mirrored with mirror's select_vlan, and that don't care about will not be received on the mirror-to interface. Signed-off-by: Daniel Ding --- utilities/ovs-tcpdump.in | 37 + 1 file changed

[ovs-dev] [PATCH v3] ovs-tcpdump: Fix cleanup mirror failed with twice fatal signals.

2024-03-19 Thread Daniel Ding
gistered to the signal handler. When received CTRL+C again, the program was broken, and calling hook could not be executed completely. Signed-off-by: Daniel Ding --- python/ovs/fatal_signal.py | 24 +--- utilities/ovs-tcpdump.in | 32 +++- 2 files

Re: [ovs-dev] [PATCH v2] ovs-tcpdump: Fix cleanup mirror failed with twice fatal signals.

2024-03-18 Thread Daniel Ding
> 2024年3月16日 上午9:17,Ilya Maximets 写道: > > On 2/23/24 04:37, Daniel Ding wrote: >> After running ovs-tcpdump and inputs multiple CTRL+C, the program will >> raise the following exception. >> >> Error in atexit._run_exitfuncs: >> Traceback (most recent cal

[ovs-dev] [PATCH v2] ovs-tcpdump: Fix cleanup mirror failed with twice fatal signals.

2024-02-22 Thread Daniel Ding
ot register the signal handler successfully. When received CTRL+C again, the program be break, and calling hook can't execute completely. Signed-off-by: Daniel Ding --- python/ovs/fatal_signal.py | 24 +--- utilities/ovs-tcpdump.in | 38 +- 2 files

Re: [ovs-dev] [PATCH v1] ovs-tcpdump: Cleanup mirror failed with twice fatal signals

2024-02-22 Thread Daniel Ding
> 2024年2月22日 下午7:20,Ilya Maximets 写道: > > On 2/22/24 11:53, Ilya Maximets wrote: >> On 2/22/24 11:02, Daniel Ding wrote: >>> >>> >>>> 2024年2月22日 上午1:55,Ilya Maximets >>> <mailto:i.maxim...@ovn.org>> 写道: >>>> &g

Re: [ovs-dev] [PATCH v1] ovs-tcpdump: Cleanup mirror failed with twice fatal signals

2024-02-22 Thread Daniel Ding
> 2024年2月22日 上午1:55,Ilya Maximets 写道: > > On 2/21/24 15:27, Aaron Conole wrote: >> Daniel Ding writes: >> >>> After running ovs-tcpdump and inputs multiple CTRL+C, the program will >>> raise the following exception. >>> >>> Error

[ovs-dev] [PATCH v1] ovs-tcpdump: Cleanup mirror failed with twice fatal signals

2024-02-20 Thread Daniel Ding
dump", line 168, in __init__ OVSDB.wait_for_db_change(self._idl_conn) # Initial Sync with DB File "/usr/bin/ovs-tcpdump", line 155, in wait_for_db_change while idl.change_seqno == seq and not idl.run(): Signed-off-by: Daniel Ding --- utilities/ovs-tcpdump.in | 15 +

[ovs-dev] [PATCH ovn v4] northd: forward arp request to lrp snat on.

2023-12-08 Thread Daniel Ding
omain whenever possible.") Reported-at: https://github.com/ovn-org/ovn/issues/209 Signed-off-by: Daniel Ding --- northd/northd.c | 31 +++ tests/ovn-northd.at | 12 2 files changed, 43 insertions(+) diff --git a/northd/northd.c b/northd/nort

Re: [ovs-dev] [PATCH ovn v3] northd: forward arp request to lrp snat on.

2023-12-08 Thread Daniel Ding
On Thu, 7 Dec 2023 at 21:26, Dumitru Ceara wrote: > On 12/6/23 02:56, Daniel Ding wrote: > > Hi Dumitru! > > > > On Tue, 5 Dec 2023 at 23:59, Dumitru Ceara wrote: > > > >> On 12/5/23 13:58, Daniel Ding wrote: > >>> > >>> >

Re: [ovs-dev] [PATCH ovn v3] northd: forward arp request to lrp snat on.

2023-12-05 Thread Daniel Ding
Hi Dumitru! On Tue, 5 Dec 2023 at 23:59, Dumitru Ceara wrote: > On 12/5/23 13:58, Daniel Ding wrote: > > > > > > On Tue, 5 Dec 2023 at 18:58, Dumitru Ceara > <mailto:dce...@redhat.com>> wrote: > > > > Hi Daniel, > > > > Thanks

Re: [ovs-dev] [PATCH ovn v3] northd: forward arp request to lrp snat on.

2023-12-05 Thread Daniel Ding
On Tue, 5 Dec 2023 at 18:58, Dumitru Ceara wrote: > Hi Daniel, > > Thanks for this new revision but why is it v3? I don't think I saw v2 > posted anywhere but maybe I missed it. > > Sorry, that is my mistake. > On 12/5/23 08:33, Daniel Ding wrote: > > If the rout

[ovs-dev] [PATCH ovn v3] northd: forward arp request to lrp snat on.

2023-12-04 Thread Daniel Ding
omain whenever possible.") Reported-at: https://github.com/ovn-org/ovn/issues/209 Signed-off-by: Daniel Ding Acked-by: Dumitru Ceara --- northd/northd.c | 18 +- tests/ovn-northd.at | 12 2 files changed, 29 insertions(+), 1 deletion(-) diff --git a/north

Re: [ovs-dev] [PATCH ovn] northd: forward arp request to lrp snat on.

2023-12-04 Thread Daniel Ding
On Fri, 10 Nov 2023 at 22:51, Dumitru Ceara wrote: > On 10/24/23 09:39, Daniel Ding wrote: > > If the router has a snat rule and it's external ip isn't lrp address, > > when the arp request from other router for this external ip, will > > be drop, because of this external ip

[ovs-dev] [PATCH ovn] northd: forward arp request to lrp snat on.

2023-10-24 Thread Daniel Ding
omain whenever possible.") Reported-at: https://github.com/ovn-org/ovn/issues/209 Signed-off-by: Daniel Ding --- northd/northd.c | 18 +- tests/ovn-northd.at | 12 2 files changed, 29 insertions(+), 1 deletion(-) diff --git a/northd/northd.c b/northd/nort

[ovs-dev] [PATCH v2] ovsschema: Set bfd_status to ephemeral.

2022-10-27 Thread Daniel Ding
When restart openvswitch, the bfd status will be kept before ovs-vswitchd running. And if the ovs-vswitchd has high workload, which will defer updating bfd status, which not we excepted. Signed-off-by: Daniel Ding --- vswitchd/vswitch.ovsschema | 7 --- 1 file changed, 4 insertions(+), 3

[ovs-dev] [PATCH v1] ovsschema: Set bfd_status to ephemeral.

2022-10-25 Thread Daniel Ding
When restart openvswitch, the bfd status will be kept before ovs-vswitchd running. And if the ovs-vswitchd has high workload, which will defer updating bfd status, which not we excepted. --- vswitchd/vswitch.ovsschema | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git

[ovs-dev] [PATCH v5] ovs-tcpdump: Cleanup mirror port on SIGHUP/SIGTERM

2022-09-13 Thread Daniel Ding
If ovs-tcpdump received HUP or TERM signal, mirror and mirror interface should be destroyed. This often happens, when controlling terminal is closed, like ssh session closed, and other users use kill to terminate it. Acked-by: Mike Pattrick Signed-off-by: Daniel Ding --- utilities/ovs

Re: [ovs-dev] [PATCH v4] ovs-tcpdump: Cleanup mirror port on SIGHUP/SIGTERM

2022-09-12 Thread Daniel Ding
On 2022/9/10 1:43, Ilya Maximets wrote: On 9/8/22 09:37, Daniel Ding wrote: If ovs-tcpdump received HUP or TERM signal, mirror and mirror interface should be destroyed. This often happens, when controlling terminal is closed, like ssh session closed, and other users use kill to terminate

[ovs-dev] [PATCH v4] ovs-tcpdump: Cleanup mirror port on SIGHUP/SIGTERM

2022-09-08 Thread Daniel Ding
If ovs-tcpdump received HUP or TERM signal, mirror and mirror interface should be destroyed. This often happens, when controlling terminal is closed, like ssh session closed, and other users use kill to terminate it. Acked-by: Mike Pattrick Signed-off-by: Daniel Ding --- utilities/ovs

Re: [ovs-dev] [PATCH v3] ovs-tcpdump: Cleanup mirror port on SIGHUP/SIGTERM

2022-09-08 Thread Daniel Ding
Excuse me, This patch is invalid because of indentation. On 2022/9/8 15:07, Daniel Ding wrote: If ovs-tcpdump received HUP or TERM signal, mirror and mirror interface should be destroyed. This often happens, when controlling terminal is closed, like ssh session closed, and other users use

[ovs-dev] [PATCH v3] ovs-tcpdump: Cleanup mirror port on SIGHUP/SIGTERM

2022-09-08 Thread Daniel Ding
If ovs-tcpdump received HUP or TERM signal, mirror and mirror interface should be destroyed. This often happens, when controlling terminal is closed, like ssh session closed, and other users use kill to terminate it. Acked-by: Mike Pattrick Signed-off-by: Daniel Ding --- utilities/ovs

[ovs-dev] [PATCH v2] ovs-tcpdump: Cleanup mirror port on SIGHUP/SIGTERM

2022-08-25 Thread Daniel Ding
If ovs-tcpdump received HUP or TERM signal, mirror and mirror interface should be destroyed. This often happens, when controlling terminal is closed, like ssh session closed, and other users use kill to terminate it. Signed-off-by: Daniel Ding --- utilities/ovs-tcpdump.in | 30

[ovs-dev] [PATCH] utilities/ovs-tcpdump.in: Fix not destroy mirror (Daniel Ding)

2022-08-23 Thread Daniel Ding
If ovs-tcpdump received HUP or TERM signal, mirror and mirror interface should be destroyed. This often happens, when controlling terminal is closed, like ssh session closed, and other users use kill to terminate it. Signed-off-by: Daniel Ding --- utilities/ovs-tcpdump.in | 12 1