Skip to content

Add free5GC as an example CNF#2377

Merged
martin-mat merged 7 commits intomainfrom
feature/free5gc-example-cnf
Apr 29, 2026
Merged

Add free5GC as an example CNF#2377
martin-mat merged 7 commits intomainfrom
feature/free5gc-example-cnf

Conversation

@marketa-zejdova
Copy link
Copy Markdown
Collaborator

Description

Introduce free5GC as an example CNF that is fully deployable and testable via the CNTi test suite.

Issues:

Refs: #2374

How has this been tested:

  • Covered by existing integration testing
  • Added integration testing to cover
  • Verified all A/C passes
    • develop
    • master
    • tag/other branch
  • Test environment
    • Shared Packet K8s cluster
    • New Packet K8s cluster
    • Kind cluster
  • Have not tested

Types of changes:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

Checklist:

Documentation

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • No updates required.

Code Review

  • Does the test handle fatal exceptions, ie. rescue block

Issue

  • Tasks in issue are checked off

@collivier
Copy link
Copy Markdown
Collaborator

@marketa-zejdova could we leverage public helm charts instead of duplicating all files in our tree ?

@collivier
Copy link
Copy Markdown
Collaborator

+ it would be great to cover this example cnf via gate jobs

@marketa-zejdova
Copy link
Copy Markdown
Collaborator Author

@marketa-zejdova could we leverage public helm charts instead of duplicating all files in our tree ?

For now, I used a local copy of the free5GC Helm chart as a temporary solution. The upstream chart is still missing some changes needed for this deployment to work reliably (mainly MongoDB storage and kind compatibility).

I also had to make a few adjustments on our side to get it running on kind.

Because of that, I didn’t want to pull the chart directly from their development branch, as it could introduce breaking changes.

I agree that copying the chart is not ideal long-term. Would it be better to reference a public Helm repository here, or possibly publish a stabilized version of this chart somewhere (if we have an internal Helm registry)?

@marketa-zejdova
Copy link
Copy Markdown
Collaborator Author

marketa-zejdova commented Apr 16, 2026

  • it would be great to cover this example cnf via gate jobs

I agree this would be useful. We are planning to handle it in a follow-up PR.

Comment thread example-cnfs/free5gc/README.md Outdated
Comment thread example-cnfs/free5gc/README.md Outdated
Comment thread example-cnfs/free5gc/README.md Outdated
Comment thread example-cnfs/free5gc/README.md
@marketa-zejdova marketa-zejdova force-pushed the feature/free5gc-example-cnf branch from 40c9b18 to 7153c56 Compare April 28, 2026 12:20
Comment thread example-cnfs/free5gc/README.md Outdated
Copy link
Copy Markdown
Collaborator

@collivier collivier left a comment

Choose a reason for hiding this comment

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

That seems very good but Isn't the gtp part missing in this PR ?

2026-04-28T14:32:31.378359855Z [INFO][UPF][CFG] ==================================================
2026-04-28T14:32:31.378365705Z [INFO][UPF][Main] Log level is set to [info]
2026-04-28T14:32:31.378371000Z [INFO][UPF][Main] Report Caller is set to [false]
2026-04-28T14:32:31.378393793Z [INFO][UPF][Main] starting Gtpu Forwarder [gtp5g]
2026-04-28T14:32:31.378401749Z [INFO][UPF][Main] GTP Address: "10.244.2.3:2152"
2026-04-28T14:32:31.381402985Z [ERRO][UPF][Main] UPF Cli Run Error: open Gtp5g: open link: create: operation not supported

Is he kernel module loaded in your env ?

Comment thread example-cnfs/free5gc/README.md Outdated
@marketa-zejdova
Copy link
Copy Markdown
Collaborator Author

That seems very good but Isn't the gtp part missing in this PR ?

2026-04-28T14:32:31.378359855Z [INFO][UPF][CFG] ==================================================
2026-04-28T14:32:31.378365705Z [INFO][UPF][Main] Log level is set to [info]
2026-04-28T14:32:31.378371000Z [INFO][UPF][Main] Report Caller is set to [false]
2026-04-28T14:32:31.378393793Z [INFO][UPF][Main] starting Gtpu Forwarder [gtp5g]
2026-04-28T14:32:31.378401749Z [INFO][UPF][Main] GTP Address: "10.244.2.3:2152"
2026-04-28T14:32:31.381402985Z [ERRO][UPF][Main] UPF Cli Run Error: open Gtp5g: open link: create: operation not supported

Is he kernel module loaded in your env ?

You’re right, I had it loaded in my env. I added UPF kernel requirements part into the README. Is that sufficient like that?

@martin-mat martin-mat self-requested a review April 29, 2026 19:26
Copy link
Copy Markdown
Collaborator

@martin-mat martin-mat left a comment

Choose a reason for hiding this comment

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

lgtm

@github-project-automation github-project-automation Bot moved this from In progress to Reviewer approved in CNF Test Suite Apr 29, 2026
@martin-mat martin-mat merged commit 0622c82 into main Apr 29, 2026
100 checks passed
@github-project-automation github-project-automation Bot moved this from Reviewer approved to Done in CNF Test Suite Apr 29, 2026
@martin-mat martin-mat deleted the feature/free5gc-example-cnf branch April 29, 2026 19:28
cd gtp5g
make
sudo make install
sudo modprobe gtp5g
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

make install is enough

@collivier
Copy link
Copy Markdown
Collaborator

collivier commented Apr 30, 2026

@marketa-zejdova thank you for the update. please take my last comment into a 2nd PR if needed. It was merged very quickly. good work!

https://github.com/free5gc/gtp5g/blob/master/Makefile#L88

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