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.

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

Flow steering based on SDF and precedence #132

Merged
merged 17 commits into from
Sep 4, 2020
Merged

Conversation

krsna1729
Copy link
Member

Certain flows of a session can be directed to N9 vs N6 based on SDF rules in PDI.
We now take leverage precedence field of wildcardmatch table too.
Swapped layout of dst/src ip ports to be consistent everywhere.

@krsna1729
Copy link
Member Author

Opened a ticket with spirent regarding overlapping PDRs for DL with same precedence.
image

@krsna1729
Copy link
Member Author

Currently tested only with sim mode

@hyunsun
Copy link
Contributor

hyunsun commented Sep 1, 2020

retest this please

1 similar comment
@yoooou
Copy link

yoooou commented Sep 1, 2020

retest this please

@krsna1729 krsna1729 marked this pull request as draft September 1, 2020 22:58
@krsna1729 krsna1729 force-pushed the topics/far_seid branch 2 times, most recently from 985f0b4 to 82ae230 Compare September 2, 2020 03:01
ajamshed and others added 16 commits September 2, 2020 20:14
Signed-off-by: Muhammad Asim Jamshed <muhammad.jamshed@intel.com>
Signed-off-by: Muhammad Asim Jamshed <muhammad.jamshed@intel.com>
Signed-off-by: Muhammad Asim Jamshed <muhammad.jamshed@intel.com>
Signed-off-by: Muhammad Asim Jamshed <muhammad.jamshed@intel.com>
Signed-off-by: Muhammad Asim Jamshed <muhammad.jamshed@intel.com>
src followed by dst.

Signed-off-by: Muhammad Asim Jamshed <muhammad.jamshed@intel.com>
src ip/port[s] followed by dst ip/port[s].

Signed-off-by: Muhammad Asim Jamshed <muhammad.jamshed@intel.com>
Signed-off-by: Saikrishna Edupuganti <saikrishna.edupuganti@intel.com>
Signed-off-by: Saikrishna Edupuganti <saikrishna.edupuganti@intel.com>
Signed-off-by: Saikrishna Edupuganti <saikrishna.edupuganti@intel.com>
Signed-off-by: Saikrishna Edupuganti <saikrishna.edupuganti@intel.com>
Signed-off-by: Saikrishna Edupuganti <saikrishna.edupuganti@intel.com>
Signed-off-by: Saikrishna Edupuganti <saikrishna.edupuganti@intel.com>
Signed-off-by: Muhammad Asim Jamshed <muhammad.jamshed@intel.com>
Seq number is already provided in the API.
Core IP will overwrite the accessIP because the it accepts only one UserPlaneIPResourceInformation
https://github.com/wmnsk/go-pfcp/blob/master/message/association-setup-response.go#L51-L52
Copy link
Contributor

@badhrinathpa badhrinathpa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@krsna1729 krsna1729 marked this pull request as ready for review September 3, 2020 23:57
@krsna1729
Copy link
Member Author

Default N6 test cases are working fine. Merging now. Will fix any N9 parsing issues in follow-on PRs

@krsna1729 krsna1729 merged commit 7824302 into master Sep 4, 2020
@krsna1729 krsna1729 deleted the topics/far_seid branch September 4, 2020 00:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants