Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

dpvs session synchronization #476

Open
wants to merge 4 commits into
base: devel
Choose a base branch
from
Open

dpvs session synchronization #476

wants to merge 4 commits into from

Conversation

weiyanhua100
Copy link
Contributor

No description provided.

@kldeng
make it possible to bind laddr to lcore by adding laddr_lcore_mapping
f394d1b 
option. In this mode, dest addr is needed in FDIR instead of dest addr and dest port mask.

NOTICE:
number of laddrs should be greater or equal to number of slave lcores.
Or, some slave lcores will have no laddr causing FNAT fowarding failed in
those slave lcores.

Co-authored-by: kldeng <[email protected]>
Co-authored-by: lixiaoxiao <[email protected]>
mscbg
mscbg reviewed Sep 11, 2019
View reviewed changes
doc/tutorial.md Show resolved Hide resolved
mscbg
mscbg reviewed Sep 11, 2019
View reviewed changes
include/ipvs/conn.h Outdated Show resolved Hide resolved
mscbg
mscbg reviewed Sep 12, 2019
View reviewed changes
src/ipvs/ip_vs_sync_msg.c Show resolved Hide resolved
src/ipvs/ip_vs_proto_tcp.c Outdated Show resolved Hide resolved
@weiyanhua100 weiyanhua100 force-pushed the sync branch 6 times, most recently from 34c91b2 to 7224d8e Compare September 23, 2019 12:48
kldeng and others added 3 commits September 23, 2019 20:56
@kldeng
session_sync: Add session sync to decrease the connection break due t…
066bc17 
…o the DPVS node failure in the cluster.

1.there are two synchronization modes: full and incremental.
2.incremental synchronization is used for new session.
3.full synchronization is used for existing session.

Co-authored-by: lixiaoxiao [email protected]
Add identifiers for Synchronized sessions.
f73ea54
add readme for session sync.
a3130ed
@hulinfan
Copy link

hulinfan commented Sep 30, 2019 •
edited
Loading

Hi,
Have you ever tested the performance of dpvs while enabling sessions synchronization?
What's the PPS and CPS?
I doubt it harms performance a lot.

@ywc689 ywc689 added pr/to-confirm-needs consider whether the feature of pr is needed pr/needs-confirmed the feature in the pr is what we need，and list what cases should be checked in later stages pr/to-review-codes review codes line by line and check if problem exists. and removed pr/to-confirm-needs consider whether the feature of pr is needed labels Jul 16, 2020
@yfming
Copy link
Contributor

yfming commented Feb 17, 2022 •
edited
Loading

你好，看了代码后有几个疑问：
1、需要同步session的机器运行ipvsadm --conn-sync命令？假如a执行了这个命令，这时候b和c同时收到了probe报文，都回应了reply怎么确定应该从c还是b同步呢？假如c同时收到了a和b的probe报文，那怎么能给两台都同步全量呢？
2、需要同步的机器都要执行一下ipvsadm --conn-sync才会做全量和增量同步吧？
3、同步后的conn为什么不能用lport添加fdir，必须要支持lip方式呢？
4、session建立之后为什么要等待（sync_session_elapse），然后才进行同步呢？

@ywc689 ywc689 removed the pr/to-review-codes review codes line by line and check if problem exists. label Feb 16, 2023
@ywc689 ywc689 mentioned this pull request Oct 10, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mscbg mscbg mscbg left review comments

@August90dd August90dd August90dd left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
pr/needs-confirmed the feature in the pr is what we need，and list what cases should be checked in later stages
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@weiyanhua100 @hulinfan @yfming @August90dd @mscbg @ywc689 @kldeng