Skip to content

Conversation

@arnavk23
Copy link
Contributor

@arnavk23 arnavk23 commented Nov 8, 2025

Also:

  • Links the new document from config_upf.rst for discoverability.

@arnavk23 arnavk23 force-pushed the add-pfcp-dump-examples branch from 64a77fd to c172ba1 Compare November 8, 2025 05:57
@arnavk23 arnavk23 force-pushed the add-pfcp-dump-examples branch from a0c6bd0 to fe79b45 Compare November 8, 2025 07:15
@arnavk23 arnavk23 requested a review from gab-arrobo November 10, 2025 07:48
@arnavk23
Copy link
Contributor Author

@gab-arrobo As the linked file is merged, I think we can merge this also.

@gab-arrobo
Copy link
Contributor

@gab-arrobo As the linked file is merged, I think we can merge this also.

Yes, I am currently reviewing this. Thanks!

@arnavk23 arnavk23 force-pushed the add-pfcp-dump-examples branch from 67ac3bc to 750eb6b Compare November 12, 2025 18:48
@arnavk23 arnavk23 requested a review from gab-arrobo November 12, 2025 18:49
@gab-arrobo gab-arrobo requested review from a team and Copilot November 12, 2025 19:31
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds comprehensive documentation for the PFCP dump feature in the UPF configuration, providing users with clear examples for enabling PFCP raw-dump debugging capabilities. The documentation includes configuration snippets for both transient and persistent storage scenarios.

Key Changes:

  • Added new "PFCP Dump Examples" section with configuration examples for transient (emptyDir) and persistent (PVC) storage options
  • Included explanatory notes about maxBytes, maxFiles, and toLog behavior

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
dict.txt Added "emptyDir" to the spell-check dictionary
configuration/config_upf.rst Added new PFCP Dump Examples section with configuration snippets and usage notes
conf.py Updated linkcheck_ignore_uri pattern to support both http and https for opennetworking.org

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

arnavk23 and others added 11 commits November 13, 2025 10:38
Signed-off-by: Arnav Kapoor <kapoorarnav43@gmail.com>
Signed-off-by: Arnav Kapoor <kapoorarnav43@gmail.com>
Signed-off-by: Arnav Kapoor <kapoorarnav43@gmail.com>
…ude in toctree

Signed-off-by: Arnav Kapoor <kapoorarnav43@gmail.com>
… doc8

Signed-off-by: Arnav Kapoor <kapoorarnav43@gmail.com>
…lated failures

Signed-off-by: Arnav Kapoor <kapoorarnav43@gmail.com>
Signed-off-by: Arnav Kapoor <kapoorarnav43@gmail.com>
…move separate file

Signed-off-by: Arnav Kapoor <kapoorarnav43@gmail.com>
Signed-off-by: Arnav Kapoor <kapoorarnav43@gmail.com>
Signed-off-by: Arnav Kapoor <kapoorarnav43@gmail.com>
Co-authored-by: Gabriel Arrobo <gabriel.arrobo@intel.com>
Signed-off-by: Arnav Kapoor <kapoorarnav43@gmail.com>
@arnavk23 arnavk23 force-pushed the add-pfcp-dump-examples branch from 7fc940e to 474ca65 Compare November 13, 2025 05:09
@arnavk23 arnavk23 requested a review from gab-arrobo November 13, 2025 05:09
Copy link
Contributor

@gab-arrobo gab-arrobo left a comment

Choose a reason for hiding this comment

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

+1

@gab-arrobo gab-arrobo merged commit 6531a05 into omec-project:main Nov 13, 2025
6 checks passed
@gab-arrobo
Copy link
Contributor

@arnavk23, thanks for your contribution

@arnavk23 arnavk23 deleted the add-pfcp-dump-examples branch November 13, 2025 13:58
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.

2 participants