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
- Updated reference to above PR in the code as todo 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.
- 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.
...
- 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:
...
- 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://wiki.magmacore.org/x/MwG7
...
- Started working on C++ migration activity for SPGW task. Fixed return type mismatch errors in all the files under SPGW task
- Changed file extension to .cpp and .hpp which is resulting into lot of compilation errors and warnings. Working on fixing them
- IPv6 PR https://github.com/magma/magma/pull/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/14734 for pending hashlists for conversion in follow up of slack discussion https://magmacore.slack.com/archives/C01B32Q9L76/p1671522770286319
- Fixed conflicts after s11 tunel hashlist migration to ptotobuf map and addressed Review Comments on Updated reference to above PR in the code as todo and committed it as part of PR https://github.com/magma/magma/pull/14638Understanding need of other haslists in mme_app. Raised re-review request for the PR and got it merged.
- 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.