Page Comparison

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

...

Highlights of the week  3rd Jan`23 from Rashmi:

1. Ported s_plus_p_gw_eps_bearer_context_information_t to protobuf structure and fixed the bugs found in unit test cases and while executing s1ap sanity test suite
2. Ported UE contexts to protobuf structure in spgw task. Validated the changes and changes are available in PR, https://github.com/magma/magma/pull/14782
3. Ported spgw_state contexts to protobuf structure in spgw task. Shall raise PR once previous PRs are merged to master.
4. Started working on removing common state manager dependency for spgw task.
5. Reviewed following PRs
   https://github.com/magma/magma/pull/14716
   https://github.com/magma/magma/pull/14769
   
   

Highlights of the week 26th Dec`22 from Ankit Kumar Aman:

• Raised Github Issue https://github.com/magma/magma/issues/14734 for pending hashlists conversion in follow up of slack discussion https://magmacore.slack.com/archives/C01B32Q9L76/p1671522770286319
• Addressed review comments and updated reference TODO comments for all pending hashlists in the code and committed it as part of PR https://github.com/magma/magma/pull/14638. Raised re-review request for the PR and got it merged.
• Raised PR https://github.com/magma/magma/pull/14670 for fixing display of wrong flaky retry flag status. Addressed review comments.
• Went through the code to understand the change needed for c structure to protobuf structure usage and from PR https://github.com/magma/magma/pull/13678
• Working on ESMContext proto structure usage instead of c usage.

...

Highlights of the week 21st Dec`22 from Pruthvi:

1. Completed migrating state_imsi_htble in mme_app to protobuf map https://github.com/magma/magma/pull/14716
2. Addressed review comments for PR https://github.com/magma/magma/pull/14716/
3. Reviewed PRs: https://github.com/magma/magma/pull/14717, https://github.com/magma/magma/pull/14406

Highlights of the week 19th Dec`22 from Ankit Kumar Aman:

• Addressed review comments on PR https://github.com/magma/magma/pull/14638 with guti_ue_context hashlist to protobuf map migration for mme_app task 
• Analysed hashlist need for hashlist g_e_dns_entries, raised PR https://github.com/magma/magma/pull/14665 for removal of hashlist.
• Started slack discussion for removal of hashlist https://magmacore.slack.com/archives/C01B32Q9L76/p1670564029595199 and considering the need of the hashlist, closed the PR https://github.com/magma/magma/pull/14665
• Analysed hashlist need for hashlists imei_htbl and sac_to_tacs_map_htbl 
• Raised PR https://github.com/magma/magma/pull/14725 for conversion of imei_htbl to protobuf map but the compilation is getting stuck for c files dependency with ngap/amf.
• Started slack discussion for conversion of hashlist from 5g team https://magmacore.slack.com/archives/C01B32Q9L76/p1671522770286319 and closed the PR https://github.com/magma/magma/pull/14725

...

Highlights of the week 12th Dec`22 from Rashmi:

1. Completed the porting of s_plus_p_gw_eps_bearer_context_information_t to protobuf structure. Fixed some of the bugs found in unit test cases

Highlights of the week 12th Dec`22 from Ankit Kumar Aman:

• Completed guti_ue_context hashlist to protobuf map migration for mme_app task and raised PR https://github.com/magma/magma/pull/14638
• Fixed conflicts after s11 tunel hashlist migration to ptotobuf map 
• Analyzing and understanding need of other hashlists in mme_app

...

Highlights of the week 5th Dec`22 from Ankit Kumar Aman:

• Debugged the issue using gdb and finally fixed the issue for gtpv1-u task cpp migrationPR https://github.com/magma/magma/pull/14290. Updated the PR and marked it ready for review.
• Code walkthrough and PR review to understand hashlist migration to protobuf map.
• Working on guti_ue_context hashlist migration to protobuf map for mme app.
• Leaves & holidays: Availed leave on 1st and 2nd December for personal reasons.

...

Highlights of the week 28th Nov`22 from Ankit Kumar Aman:

• Updated header ordering and externs and addressed review comments on s11 task PR https://github.com/magma/magma/pull/14449
• Migrated sms_orc8r task to cpp and raised PR https://github.com/magma/magma/pull/14494
• Addressed review comments on gtpv1-u task PR https://github.com/magma/magma/pull/14290. Got stuck with gtpv1-u task migration to cpp. There were heap overflows during testing. 

...

Highlights of the week 22nd Nov`22 from Rashmi:

1. Porting of s_plus_p_gw_eps_bearer_context_information_t to protobuf structure is completed and testing and bug fixing are in progress
2. Reviewed the PRs, https://github.com/magma/magma/pull/14449, https://github.com/magma/magma/pull/14339 and https://github.com/magma/magma/pull/14420
3. Addressed review comments on PR, https://github.com/magma/magma/pull/14406

Highlights of the week 22nd Nov`22 from Pruthvi:

1. Completed migrating files under include/ to cpp/hpp. PR: https://github.com/magma/magma/pull/14420 merged to master
2. Completed removing of temporary file emm_headers.hpp which contains declarations of functions which were causing undefined references. PR: https://github.com/magma/magma/pull/14489 raised for review
3. Working on migrating imsi_mme_ue_id_htbl in mme_app to proto map
4. Reviewed PRs: https://github.com/magma/magma/pull/14449, https://github.com/magma/magma/pull/14339

Highlights of the week 21st Nov`22 from Ankit Kumar Aman:

• Resolved conflicts for S11 cpp migration, Raised PR https://github.com/magma/magma/pull/14449 and addressed review comments.
• Finished coding for service303 and raised PR https://github.com/magma/magma/pull/14436.
• The PR for sgs task cpp migration https://github.com/magma/magma/pull/14339 was showing bazel CI check failures with some unchanged files. Fixed the issues and addressed review comments. Re-requested review from Lars. Even after fixing the issue, CI checks were failing for python dependencies in bazel checks. Rebased PR multiple times to get rid of the issue.
• Multiple CI checks were failing for gtpv1 task PR https://github.com/magma/magma/pull/14290 which took sufficient time to debug as they were shown up in unrelated changes. Fixing the issues.

...

Highlights of the week 15th Nov`22 from Pruthvi:

1. Worked on migrating files under include/ to cpp/hpp. Raised PR: https://github.com/magma/magma/pull/14420
2. Worked on removing the temporary file emm_headers.hpp which contains declarations of functions which were causing undefined references. Yet to raise a PR for this
3. Working on the review comments for https://github.com/magma/magma/pull/14420
4. Reviewed PRs:  https://github.com/magma/magma/pull/14406, https://github.com/magma/magma/pull/14436, https://github.com/magma/magma/pull/14356

Highlights of the week 14th Nov`22 from Rashmi:

1. Reviewed the PRs, https://github.com/magma/magma/pull/14420, https://github.com/magma/magma/pull/14356 and https://github.com/magma/magma/pull/14422
2. Working on issue, https://github.com/magma/magma/issues/11191 to port s_plus_p_gw_eps_bearer_context_information_t to protobuf structure. Need to cover do dedicated bearer creation

Highlights of the week 14th Nov`22 from Ankit Kumar Aman:

• Finished code for s11 task cpp migration. Resolving final merge conflicts.
• Working on cpp migration for service303.
• The bazel ci checks for sgs task were showing conflicts in amf segment which were not related to the PR changes. Resolved the issues with the help of Rashmi. Updated the PR and requested re-review.
• Addressed review comments on PR https://github.com/magma/magma/pull/14290, https://github.com/magma/magma/pull/14339 and https://github.com/magma/magma/pull/14356 and raised re-review request

...

Highlights of the week 10th Nov`22 from Pruthvi:

1. Migration of mme_app files to cpp/hpp PR : https://github.com/magma/magma/pull/14211 is merged to master
2. Started working on migrating imsi_mme_ue_id_htbl in mme_app to proto map but this requires all the related files under include directory to be migrated to cpp. Working on migrating include/ files to cpp/hpp
3. Reviewed PRs:  https://github.com/magma/magma/pull/14356, https://github.com/magma/magma/pull/14342, https://github.com/magma/magma/pull/14006, https://github.com/magma/magma/pull/13335

Highlights of the week 7th Nov`22 from Rashmi:

1. Reviewed the PR, https://github.com/magma/magma/pull/14356 , https://github.com/magma/magma/pull/14339 and https://github.com/magma/magma/pull/14211
2. Incorporated review comments on PR, https://github.com/magma/magma/pull/14155 and https://github.com/magma/magma/pull/14342
3. Started working porting structure, s_plus_p_gw_eps_bearer_context_information_t to protobuf structure, done with Create session procedure and continue to work other procedures

...

• Finished code for sgs task cpp migration and raised PR https://github.com/magma/magma/pull/14339
• Finished code for sctp task cpp migration and raised PR https://github.com/magma/magma/pull/14356
• Faced issues with Magma test vm bring-up. Took time to get it fixed.
• Some ci checks are failing for raised PRs for amf_test with bazel. Got more bazel test commands to test with for the raised PRs to detect and fix ci failures. Fixed the CI check failures with bazel and updated the PR.
• Leaves and holidays: Personal leave on 31st October on the occasion of Chhath Puja

...

Highlights of the week 31st Oct`22 from Rashmi:

1. Worked on issue, https://github.com/magma/magma/issues/14244 and fixes are available in PR, https://github.com/magma/magma/pull/14327
2. As part of stability testing, scripted several test cases for s1ap task with large number of UEs and eNBs. But we need to continue this activity once s1ap task has ported all it's contexts to protobuf structure.
3. Reviewed the PR, https://github.com/magma/magma/pull/14211

...

• Finished code for GTPv1 task cpp migration and raised PR https://github.com/magma/magma/pull/14290
• Started working on SGS task cpp migration
• Leaves and holidays: Radisys holidays on 24 & 25th (Diwali)

...

Highlights of the week 24th Oct`22 from Ankit Kumar Aman:

• Started working on GTPv1 task for cpp migration
• Found compilation issues with S6a and HA tasks for pragma once guard. Marked the issues and revised the PRs. The PRs are merged https://github.com/magma/magma/pull/14089 and https://github.com/magma/magma/pull/14148
• Meeting and Discussion Involvements: Attended Monday Bi-Weekly Testing Meeting
• Leaves and holidays: Radisys holidays from Friday till Tuesday -21st (Global Wellness day), 24 & 25th (Diwali)

...

Highlights of the week 18th Oct`22 from Pruthvi:

...

• Migrated S6a c files to Cpp. Some linking issue is seen during make run. Fixing the issue and will raise the PR max by Tuesday.
• Worked on analyzing the CI issues marked under Github Issue https://github.com/magma/magma/issues/13954, trying to regenerate and fix the issue.
• PR/Issues reviews and follow-ups: https://github.com/magma/magma/pull/13980, https://github.com/magma/magma/pull/13188, https://github.com/magma/magma/pull/14000, https://github.com/magma/magma/pull/14004, https://github.com/magma/magma/pull/14008, https://github.com/magma/magma/pull/14013 and https://github.com/magma/magma/pull/14015
• Meeting and Discussion Involvements: Attended Monday Weekly TSC Meeting, Attended Sync-up Meeting with Shruti on setting activities priority

...

• Follow up with the CI Issues document https://docs.google.com/document/d/1eGPuYqQBqxn8TBH_pPSVGg77kzuUrKokgfi_CYOgYeE/edit?usp=sharing
• Debugged issue with Authentication failure for retrying test cases because of setting the integrity protected message flag for 2nd attempt.
• Created testcases for large number of UEs with mixed scenarios and tested with local setup in iteration
• Preparing the setup for profiling and performance check with cpp migration activity
• PR/Issues reviews and follow-ups: https://github.com/magma/magma/pull/13781

...

Highlights of the week 29th Aug`22 from Rashmi:

...

• Follow up with the CI Issues document https://docs.google.com/document/d/1eGPuYqQBqxn8TBH_pPSVGg77kzuUrKokgfi_CYOgYeE/edit?usp=sharing
• PR/Issues reviews and follow-ups: https://github.com/magma/magma/pull/13531, https://github.com/magma/magma/pull/13188, https://github.com/magma/magma/pull/13146
• Meeting and Discussion Involvements: Attended Monday Testing Weekly Meeting, Attended Wednesday Sync-up Meeting with Shruti-RadisysTeam and Attended Wednesday DevOps Meeting
• Leaves & Holidays: Radisys holiday on 15th and 18th August 2022, On leave on 16th August 2022 for personal work

...

• Fixed issue in publishing lte integ status update for ci and raised PRhttps://github.com/magma/magma/pull/13568
• Verified PR https://github.com/magma/S1APTester/pull/86 changes on local 3 VM setup, updated report to the PR and merged the PR
• Verified changes for fixing failing non-sanity test cases and added the fixed testcase as part of review to PRhttps://github.com/magma/magma/pull/13543
• Follow-ups & addressed review comments over my active PRs:https://github.com/magma/magma/pull/13323,https://github.com/magma/magma/pull/13332,https://github.com/magma/magma/pull/13423 and https://github.com/magma/magma/pull/13568
• PR/Issues reviews and follow-ups: https://github.com/magma/S1APTester/pull/86,https://github.com/magma/magma/pull/12646,https://github.com/magma/magma/pull/12892,https://github.com/magma/magma/pull/13531, https://github.com/magma/magma/pull/13543, https://github.com/magma/magma/issues/13595 and https://github.com/magma/magma/issues/13596 
• Meeting and Discussion Involvements: Attended Wednesday 1.8 Release Meeting and Attended Thursday Magma-Arm Meeting, Discussion and follow-ups with Balaji on TRF server bring-up issue

...

• Follow up with the CI Issues document https://docs.google.com/document/d/1eGPuYqQBqxn8TBH_pPSVGg77kzuUrKokgfi_CYOgYeE/edit?usp=sharing
• Analyzed for recent CI issues and updated CI issues doc with latest major CI issues
• Monitored CI reports and followed up in slack channel on log artifacts not reported to CI. Debugged and found the issue with the status update for failing executions
• Raised PRhttps://github.com/magma/magma/pull/13423 for enhancing debugging in CI by fetching more logs from vms
• PR reviews and follow-ups:  https://github.com/magma/magma/pull/12646
• Meeting and Discussion Involvements: Attended Monday Testing Weekly Meeting, Attended Monday Magma Weekly TSC Meeting and Attended Wednesday 1.8 Release Meeting

...

• Follow up with the CI Issues document https://docs.google.com/document/d/1eGPuYqQBqxn8TBH_pPSVGg77kzuUrKokgfi_CYOgYeE/edit?usp=sharing
• Debugged on Router Advertisement Issue for IPv6 testcases and OVS api call key error issue. Trying to get more logs for debugging.
• Found Issue with S1APTester that the attach retry was integrity protected and causing the test case failure with assertion. Raised Github Issue https://github.com/magma/S1APTester/issues/85
• Fixed CI issue with attach accept and TAU Accept for all the test cases and updated the PR https://github.com/magma/magma/pull/13139 (Merged)
• Addressed review comments over active PRshttps://github.com/magma/magma/pull/13138,https://github.com/magma/magma/pull/13295 andhttps://github.com/magma/S1APTester/pull/82
• Raised PRhttps://github.com/magma/magma/pull/13323 for reducing the overall sanity execution for failure scenarios
• PR reviews and follow-ups: https://github.com/magma/magma/pull/13060
• Meeting and Discussion Involvements: Attended Wednesday DevOps Meeting and followed up on Release 1.8 meeting notes https://wikilf-magma.magmacoreatlassian.orgnet/wiki/x/MwG7wZV5

...

Highlights of the week 13^th July`22 from Pruthvi:

...

• Analyzing an fixing CI related issues tracked with the documentwith the document: https://docs.google.com/document/d/1eGPuYqQBqxn8TBH_pPSVGg77kzuUrKokgfi_CYOgYeE/edit?usp=sharing
• Raised PR with fix for CI issue with assertion failure for initial context setup and attach accept: https://github.com/magma/magma/pull/13139
• Raised PR with fix for enb complete reset test case: https://docs.googlegithub.com/magma/S1APTester/pull/81 and https://github.com/documentmagma/d/1eGPuYqQBqxn8TBH_pPSVGg77kzuUrKokgfi_CYOgYeE/edit?usp=sharingRaised PR with fix for CI issue with assertion failure for initial context setup and attach accept: magma/pull/13138
• Debugged on issue with running traffic server. Raised Github Issue https://github.com/magma/magma/issues/13203 for tracking the activity of upgradation of TRF Server VM to Ubuntu 20.04
• Reviewed the PRs:
  • https://github.com/magma/magma/pull/12646
  • https://github.com/magma/magma/pull/
  13139Raised PR with fix for enb complete reset test case:
  • 13060
  • https://github.com/magma/
  S1APTester
  • magma/pull/
  81 and 
  • 13202
  • https://github.com/magma/magma/pull/
  13138Debugged on issue with running traffic server. Raised Github Issue
  • 13094

Highlights of the week 4th July`22 from Rashmi: 

1. As part of c++ migration, completed porting of hashlist, state_ue_ht to protobuf ma. Manually verified the changes. Once previous PRs are merged. I shall create new PR.
   
2. As part of issue, https://github.com/magma/magma/issues/

...

1. 9902. As part of mme restart I did some analysis. Got to know that mme2associd was not updated. Modifying the code based on this observation didn't seem to work. 
2. Provided the fixes for the issue, https://github.com/magma/magma/

...

1. issues/

...

1. 13086
2. Reviewed the PRs, https://github.com/magma/magma/pull/

...

1. 13103 and https://github.com/magma/magma/pull/

...

1. 13142

...

Highlights of the week 29^th June`22 from Vikram:

1. Working on C++ migration activity for renaming c to cpp in mme_app.

Highlights of the week

...

28th June`22 from Rashmi: 

...

•  Incorporated review comments on PR, https://github.com/magma/magma/pull/

...

• 12965/

...

As part

...

of c++ migration, completed porting of hashlist, mme_ue_id_imsi_htbl to protobuf map. Shall raise PR for the same

•  As part of issue, https://github.com/magma/magma/issues/

...

• 9902 Analyzed the logs. From the logs it appears that mme is going through multiple scenarios like mme is handling multi UE and multi eNB with different types of procedures like request for handover, UE entering to idle state and back to active state; sctp disconnection. From logs I re-created few of scenario, but looks to be good.  Right now we shall replace assert with error log and return.

Highlights of the week 2928^th June`22 from VikramPruthvi:

1. Working Started working on C++ migration activity for renaming c to cpp in mme_app.

Highlights of the week 28th June`22 from Rashmi: 

...

1. SPGW task. Fixed return type mismatch errors in all the files under SPGW task
2. Changed file extension to .cpp and .hpp which is resulting into lot of compilation errors and warnings. Working on fixing them
3. IPv6 PR https://github.com/magma/magma/pull

...

As part of c++ migration, completed porting of hashlist, mme_ue_id_imsi_htbl to protobuf map. Shall raise PR for the same

...

1. /13010 is merged to master. But vms were not coming up on local VMs, updated quick start guide to add IPv6 address range in /etc/vbox/networks.con

...

[Current Week] Highlights of the week 26th Dec`22 from Ankit Kumar Aman:

• Raised Github Issue https://github.com/magma/magma/issues/

...

Highlights of the week 28^th June`22 from Pruthvi:

...

• 14734 for pending hashlists conversion in follow up of slack discussion https://magmacore.slack.com/archives/C01B32Q9L76/p1671522770286319
• Addressed review comments and updated reference TODO comments for all pending hashlists in the code and committed it as part of PR https://github.com/magma/magma/pull/14638. Raised re-review request for the PR and got it merged.
• Raised PR https://github.com/magma/magma/pull/

...

• 14670 for fixing display of wrong flaky retry flag status. Addressed review comments.
• Went through the code to understand the change needed for c structure to protobuf structure usage and from PR https://github.com/magma/magma/pull/13678
• Working on ESMContext proto structure usage instead of c usage.