Skip to content
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 to port s1ap_imsi_map to protobuf structure #14155

Merged
merged 133 commits into from
Nov 8, 2022

Conversation

rsarwad
Copy link
Contributor

@rsarwad rsarwad commented Oct 13, 2022

Summary

fix(agw): Modified code to port s1ap_imsi_map to protobuf structure

Test Plan

Executed s1ap sanity test suite and unit test cases

Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
…tion_10869

Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
…_c_to_cpp

Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
…tion_10869

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>
…_c_to_cpp

Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
…ad/magma into rsarwad_s1ap_integrate_proto_map

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>
…ad/magma into rsarwad_s1ap_integrate_proto_map

Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
…_c_to_cpp

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>
…tion_10869

Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
…tion_10869

Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
…odule

Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
…tion_10869

Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
@github-actions
Copy link
Contributor

github-actions bot commented Oct 13, 2022

feg-workflow

    2 files  203 suites   40s ⏱️
374 tests 374 ✔️ 0 💤 0
388 runs  388 ✔️ 0 💤 0

Results for commit 3c494e1.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 13, 2022

dp-workflow

14 tests   14 ✔️  2m 17s ⏱️
  1 suites    0 💤
  1 files      0

Results for commit 3c494e1.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 13, 2022

agw-workflow

615 tests   611 ✔️  3m 45s ⏱️
    2 suites      4 💤
    2 files        0

Results for commit 3c494e1.

♻️ This comment has been updated with latest results.

Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
…_map

Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
@rsarwad rsarwad requested a review from ajahl October 28, 2022 06:14
…_map

Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
@rsarwad rsarwad requested review from a team and electronjoe October 28, 2022 07:08
@LKreutzer LKreutzer requested review from LKreutzer and removed request for electronjoe October 28, 2022 07:13
Copy link
Contributor

@LKreutzer LKreutzer left a comment

Choose a reason for hiding this comment

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

Bazel changes lgtm!

…er.cpp

Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
…_map

Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
Copy link
Collaborator

@ssanadhya ssanadhya left a comment

Choose a reason for hiding this comment

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

LGTM

…_map

Signed-off-by: Rashmi <rashmi.sarwad@radisys.com>
@rsarwad rsarwad merged commit 1226ced into magma:master Nov 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: agw Access gateway-related issue size/L Denotes a Pull Request that changes 100-499 lines.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants