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

Port Mirroring Exercise #335

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

Port Mirroring Exercise #335

wants to merge 4 commits into from

Conversation

khooi8913
Copy link
Contributor

I would like to propose an exercise to introduce the port mirroring features of P4 that involves the clone function/ extern, which many would probably find useful for many cases.

@jnfoster
Copy link
Contributor

This is super cool! Can I ask if you are an existing P4 member or ONF contributor?

@khooi8913
Copy link
Contributor Author

Hi Nate, nope, I'm not. :)

@jnfoster
Copy link
Contributor

Can you send me a note at [email protected]? I'll get you set up.

@khooi8913
Copy link
Contributor Author

Sure thing! I've just dropped you an email.

@huhub
Copy link

huhub commented Mar 16, 2020

hello, now I want to use resubmit and clone to make a packet send to the same detination after egress edit several times, can you tell me how to set clone session to make clone function effective? if you can send me you code at [email protected] ,thank you!

@khooi8913
Copy link
Contributor Author

Hi @huhub
In short, (1) you simply need to define a session X (that can be any number), when specifying your clone action along with the CloneType. Then, (2) you will need to assign the corresponding outports to session X, that where you would like to send the cloned packets to using the control plan/ CLI.

You may refer to my commit for further details.
I have included a sample solution as well. I hope it helps! 😄

@huhub
Copy link

huhub commented Mar 17, 2020 via email

@khooi8913
Copy link
Contributor Author

Hi @jnfoster , can you review my proposed exercise (I have made some slight changes since the PR was opened) and possibly merge it? Thanks!

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.

3 participants