docs: Add IPFIX docs #12078
Merged
docs: Add IPFIX docs #12078
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
pull-request-size
bot
added
the
size/L
|
Thanks for opening a PR! 💯
Howto
More infoPlease take a moment to read through the Magma project's
If this is your first Magma PR, also consider reading
|
|
Oops! Looks like you failed the Howto
♻️ Updated: ✅ The check is passing the Semantic PR after the last commit. |
|
Oops! Looks like you failed the Howto
♻️ Updated: ✅ The check is passing the Markdown lint check after the last commit. |
koolzz
force-pushed
the
ipfix_doc
branch
2 times, most recently
from
0af3d0f
to
6f04d31
Compare
pshelar
approved these changes
Mar 11, 2022
| This is done by leveraging the linux conntrack events via the ConntrackD service and generating an internal packet every time a conntrack event is generated. | ||
| The internal packet is recognized by the OVS flowtable as a special packet, gets forwarded to a specific IPFIX sampling table, hits the sampling rule(creating the IPFIX entry), and is then dropped. | ||
|
|
||
| ## Default mode |
There was a problem hiding this comment.
Can you mention advantage on typical setup with contract-mode.
koolzz
force-pushed
the
ipfix_doc
branch
3 times, most recently
from
5ac2036
to
5520f42
Compare
themarwhal
approved these changes
Mar 11, 2022
|
is this meant to be visible through docusaurus? |
Sure why not, do I need to run some docusaurus gen? |
Ya you just have to add an entry here: docs/docusaurus/sidebars.json |
koolzz
force-pushed
the
ipfix_doc
branch
2 times, most recently
from
ba12408
to
d6fa07f
Compare
Signed-off-by: Nick Yurchenko <koolzz@fb.com>
electronjoe
approved these changes
Mar 11, 2022
ardzoht
pushed a commit
that referenced
this pull request
Mar 30, 2022
Signed-off-by: Nick Yurchenko <koolzz@fb.com>
emakeev
pushed a commit
to emakeev/magma
that referenced
this pull request
Aug 5, 2022
Signed-off-by: Nick Yurchenko <koolzz@fb.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Nick Yurchenko koolzz@fb.com
Summary
Test Plan
Additional Information