Page Comparison

Highlights of the week 6th Aug`22 from Pruthvi:

...

1. As part of cpp migration in s1ap module, addressed review comments on PRs, https://github.com/magma/magma/pull/13508 , https://github.com/magma/magma/pull/13835 and https://github.com/magma/magma/pull/13678
2. As part of cpp migration in spgw module, ported teid hash list to map. Changes are available in PR, https://github.com/magma/magma/pull/13843 and porting of hash list of ue context is completed, some memory leaks are found. Need to fix them

Highlights of the week

...

5th Sept`22 from Ankit Kumar Aman:

• 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:

1. As part of cpp migration in s1ap module, ported enb context from C structure to protobuf structure. The changes are available in PR, https://github.com/magma/magma/pull/13728
2. As part of cpp migration in spgw module, started working on porting hash lists to protobuf map; On inclusion of proto_map.h in spgw_type.h file resulted into many compilation errors. Fixed them.
   Also removed some unused files and structures. All these changes are available in PR, https://github.com/magma/magma/pull/13764
3. Reviewed the below PRs, 
   https://github.com/magma/magma/pull/13754, https://github.com/magma/magma/pull/13714 and https://github.com/magma/magma/pull/1372813659As part of cpp migration in spgw module, started working on porting hash lists to protobuf map; On inclusion of proto_map.h in spgw_type.h file resulted into many compilation errors. Fixed them.
   Also removed some unused files and structures. All these changes are available in PR,

Highlights of the week 29th Aug`22 from Ankit Kumar Aman:

• Follow up with the CI Issues document https://docs.google.com/document/d/1eGPuYqQBqxn8TBH_pPSVGg77kzuUrKokgfi_CYOgYeE/edit?usp=sharing
• Fixed integ test case execution in ci because of incorrectly exported federated mode variable and raised PR https://github.com/magma/magma/pull/

...

• 13704
• Fixed CI debugging issue for LTE integ test because of mismatch in file, class and function names and raised PR https://github.com/magma/magma/pull/

...

• 13714
• Monitored CI issues, reported issue with lte integ test getting reports from feg integ test and followed up for the issue to be fixed over salck and PR https://github.com/magma/magma/pull

...

• /13706
• PR/Issues reviews and follow-ups: https://github.com/magma/magma/

...

• issues/13631
• Meeting and Discussion Involvements: Attended Monday Testing Weekly Meeting, Attended Monday Weekly TSC Meeting, Attended Wednesday 1.8 release meeting

...

Highlights of the week 23rd Aug`22 from Pruthvi:

...

• 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://wiki.magmacore.org/x/MwG7

...