-
Notifications
You must be signed in to change notification settings - Fork 592
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
fix(agw): Modified code in sgw_s8 task to port ue context hash table to protobuf map #14406
Conversation
…edure_id to protobuf map Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
After rebase revisit this branch Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
…d_hl Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
…d_hl Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
…d_hl Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
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
|
✔️ The Semantic PR check ended with status success. See instructions on formatting your commit and pull request titles. |
} | ||
free_cpp_wrapper((void**)ptr); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[cpplint] reported by reviewdog 🐶
Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please finx cpplint errors
return; | ||
} | ||
state_cache_p->imsi_ue_context_map.map = | ||
new google::protobuf::Map<unsigned long int, struct spgw_ue_context_s*>(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[cpplint] reported by reviewdog 🐶
Use int16/int64/etc, rather than the C type long [runtime/int] [4]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix this error
} | ||
free_cpp_wrapper((void**)ptr); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please finx cpplint errors
return; | ||
} | ||
state_cache_p->imsi_ue_context_map.map = | ||
new google::protobuf::Map<unsigned long int, struct spgw_ue_context_s*>(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix this error
…ask and sgw_s8 task Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
d62ab0b
to
a49ee75
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…ctx_hl Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
9c35a36
to
2b844f5
Compare
Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
Summary
fix(agw): Modified code in sgw_s8 task to port ue context hash table to protobuf map
Removed dependency of spgw task and sgw_s8 task on hash table
Test Plan
Executed unit test cases and built with "make build_oai" and executed s1ap sanity test suite
We will not be able to verify inbound roaming feature in local setup.
Additional Information
Needs to be tested federated integ_tests once PCRF is integrated