Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix: selectAnchorUPF() skip some links when searching #172

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Sep 4, 2025
Merged

fix: selectAnchorUPF() skip some links when searching #172

merged 1 commit into from
Sep 4, 2025

Conversation

@blackcat-118
Copy link
Contributor

@blackcat-118 blackcat-118 commented Aug 13, 2025

Description

selectAnchorUPF() should perform a BFS over the topology defined in smfcfg.yaml for all UPF nodes and return a list of qualified UPFs.

In the previous implementation, it stopped upon finding the first new UPF node, ignoring other links.

Test case:

 # the topology graph of userplane, A and B represent the two nodes of each link
links:
      - A: gNB1
        B: UPF1
      - A: gNB1
        B: UPF2
      - A: gNB1
        B: UPF3

Alonza0314
Alonza0314 approved these changes Aug 15, 2025
View reviewed changes
Copy link
Contributor

@Alonza0314 Alonza0314 left a comment

Choose a reason for hiding this comment

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

LGTM
It has passed ULCL TI test and basic tests(under free5GC repo).

@ianchen0119 ianchen0119 merged commit fc513eb into free5gc:main Sep 4, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Alonza0314 Alonza0314 Alonza0314 approved these changes

@ianchen0119 ianchen0119 Awaiting requested review from ianchen0119

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@blackcat-118 @Alonza0314 @ianchen0119