Skip to content

[lte][agw] Fixing attach type update on guti identity attach proc#6114

Merged
ardzoht merged 1 commit intomagma:masterfrom
ardzoht:fix_guti_attach_type
Apr 13, 2021
Merged

[lte][agw] Fixing attach type update on guti identity attach proc#6114
ardzoht merged 1 commit intomagma:masterfrom
ardzoht:fix_guti_attach_type

Conversation

@ardzoht
Copy link
Copy Markdown
Contributor

@ardzoht ardzoht commented Apr 13, 2021

Signed-off-by: Alex Rodriguez [email protected]

Summary

Test Plan

  • tested locally by modifying test_attach_via_guti with specifying EPS/IMSI attach type to ensure attach result contains same attach type on the request

Additional Information

  • This change is backwards-breaking

@pull-request-size pull-request-size bot added the size/XS Denotes a PR that changes 0-9 lines. label Apr 13, 2021
@magmabot magmabot added the component: agw Access gateway-related issue label Apr 13, 2021
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 13, 2021

Codecov Report

Merging #6114 (29a76c5) into master (dc0812f) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6114      +/-   ##
==========================================
- Coverage   33.91%   33.91%   -0.01%     
==========================================
  Files        1133     1133              
  Lines      100116   100117       +1     
  Branches     1307     1307              
==========================================
  Hits        33951    33951              
- Misses      62811    62812       +1     
  Partials     3354     3354              
Flag Coverage Δ
c_cpp 8.13% <0.00%> (-0.01%) ⬇️
cloud_lint 66.38% <ø> (ø)
feg-lint 56.26% <ø> (ø)
lte-test 72.85% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
lte/gateway/c/oai/tasks/nas/emm/Attach.c 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dc0812f...29a76c5. Read the comment docs.

@ulaskozat ulaskozat added the apply-v1.4 Needs to be applied to v1.4 release branch as well label Apr 13, 2021
@ardzoht ardzoht merged commit f68edcf into magma:master Apr 13, 2021
@tmdzk tmdzk removed the apply-v1.4 Needs to be applied to v1.4 release branch as well label Apr 15, 2021
electronjoe pushed a commit to electronjoe/magma that referenced this pull request Apr 16, 2021
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/XS Denotes a PR that changes 0-9 lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants