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:
- Completed migrating state_imsi_htble in mme_app to protobuf map https://github.com/magma/magma/pull/14716
- Addressed review comments for PR https://github.com/magma/magma/pull/14716/
- 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:
- 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:
- Porting of s_plus_p_gw_eps_bearer_context_information_t to protobuf structure is completed and testing and bug fixing are in progress
- 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
- Addressed review comments on PR, https://github.com/magma/magma/pull/14406
Highlights of the week 22nd Nov`22 from Pruthvi:
- Completed migrating files under include/ to cpp/hpp. PR: https://github.com/magma/magma/pull/14420 merged to master
- 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
- Working on migrating imsi_mme_ue_id_htbl in mme_app to proto map
- 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:
- Worked on migrating files under include/ to cpp/hpp. Raised PR: https://github.com/magma/magma/pull/14420
- 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
- Working on the review comments for https://github.com/magma/magma/pull/14420
- 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:
- 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
- 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:
- Migration of mme_app files to cpp/hpp PR : https://github.com/magma/magma/pull/14211 is merged to master
- 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
- 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:
- 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
- Incorporated review comments on PR, https://github.com/magma/magma/pull/14155 and https://github.com/magma/magma/pull/14342
- 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
Highlights of the week 7th Nov`22 from Ankit Kumar Aman:
- 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:
- Worked on issue, https://github.com/magma/magma/issues/14244 and fixes are available in PR, https://github.com/magma/magma/pull/14327
- 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.
- Reviewed the PR, https://github.com/magma/magma/pull/14211
Highlights of the week 31st Oct`22 from Ankit Kumar Aman:
- 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:
- Completed migration of mme_app files. Raised PR : https://github.com/magma/magma/pull/14211
- Started working on migrating imsi_mme_ue_id_htbl in mme_app to proto map
- Reviewed PRs: https://github.com/magma/magma/pull/14006, https://github.com/magma/magma/pull/14148, https://github.com/magma/magma/pull/13895
Highlights of the week 17th Oct`22 from Rashmi:
- Incorporated review comments on PR, https://github.com/magma/magma/pull/13895 and https://github.com/magma/magma/pull/14005
- Started working to stability testing. As part of stability testing, state_converter test case is modified to execute for 10K UEs.
- Reviewed the PR, https://github.com/magma/magma/pull/14008
Highlights of the week 17th Oct`22 from Ankit Kumar Aman:
- Worked on HA task cpp migration and raised a draft PR https://github.com/magma/magma/pull/14148
- Fixed compilation issues coming after HA and S6a task include guards are updated
- Worked on Github issue related to lte integ test stability in CI
- PR/Issues reviews and follow-ups: https://github.com/magma/magma/pull/14008
- Additional Activities: Completed 3 Meta mandatory annual trainings
Highlights of the week 10th Oct`22 from Rashmi:
- Incorporated review comments on PR, https://github.com/magma/magma/pull/13843 and https://github.com/magma/magma/pull/14072
- Started worked on issue, https://github.com/magma/magma/issues/11192 and changes for s1ap task are in PR https://github.com/magma/magma/pull/14116
- Reviewed the PR, https://github.com/magma/magma/pull/14089
- Was on PTO on 4th October
Highlights of the week 10th Oct`22 from Ankit Kumar Aman:
- Fixed issues with CPP migration and linking in S6a task and raised PR https://github.com/magma/magma/pull/14089.
- OAI Jenkins ci check was failing. Fixed issue with mme_oai FEATURES flag (Thanks Rashmi for providing the same) and updated the PR. One more issue was there that after fixing this features flag issue, make test_oai started failing. It needed to be exit out from the VM and re-login for fixing the issue (Thanks Rashmi for providing this solution). Also addressed all review comments.
- Faced issues with VM, make test_oai command was hanging and the VM was getting stuck with lack of space. Raised the ticket and fixed the issue.
- PR/Issues reviews and follow-ups: https://github.com/magma/magma/pull/14008, https://github.com/magma/magma/pull/14023, https://github.com/magma/magma/pull/14032 and https://github.com/magma/magma/pull/14034
- Leaves and holidays: Radisys holiday on 5th October on the occasion of Dassehra
Highlights of the week 3rd Oct`22 from Rashmi:
- Addressed review comment on PR, https://github.com/magma/magma/pull/13728
- On rebasing branch, rsarwad_move_s1ap_state_ctx resulted into too many conflicts. Resolved the conflicts and raised the PR, https://github.com/magma/magma/pull/14072
- Started working on issue, https://github.com/magma/magma/issues/11192 for s1ap tasks
Highlights of the week 3rd Oct`22 from Ankit Kumar Aman:
- 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
Highlights of the week 26th Sept`22 from Ankit Kumar Aman:
- Analyzing recent CI issues and follow up under Github Issue https://github.com/magma/magma/issues/13954
- Working on converting S6a task files to C++ and fixing relevant compilation issues.
- PR/Issues reviews and follow-ups: https://github.com/magma/magma/pull/13188, https://github.com/magma/magma/pull/13978, https://github.com/magma/magma/pull/13980, https://github.com/magma/magma/pull/14000, https://github.com/magma/magma/pull/14004 and https://github.com/magma/magma/pull/14009
- Meeting and Discussion Involvements: Attended Sync-up Meeting with Shruti and Rashmi for CPP migration activity
Highlights of the week 20th Sept`22 from Pruthvi:
- Worked on review comments on PR: https://github.com/magma/magma/pull/13904. PR is merged to master.
- Completed migration of nas/ies folder. Yet to raise PR for these changes
- Started working on cpp migration of mme_app files
- Reviewed PRs: https://github.com/magma/magma/pull/13943
Highlights of the week 19th Sept`22 from Rashmi:
- As part of cpp migration in S1ap module, worked on porting s1ap_state structure to protobuf structure, coding and testing is completed. Shall raise PR for review once it's parent PRs are merged.
- As part of cpp migration in spgw module, found some unused files, hash tables and structures and cleaned them up. Changes are available in PR, https://github.com/magma/magma/pull/13943
- As part of cpp migration in sgw_s8 module, started working on porting hash tables to protobuf map
- Reviewed the PRs, https://github.com/magma/magma/pull/13904 and https://github.com/magma/magma/pull/13768
Highlights of the week 19th 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
- Understanding of current cpp migration activity, selected the S6a task for migration and went through the code flow.
- Working on converting S6a task files to C++ and fixing relevant compilation issues.
Highlights of the week 13th Sept`22 from Pruthvi:
- Completed c++ migration of files under the following folders and raised PR : https://github.com/magma/magma/pull/13904
- nas (all files except ies folder)
- nas/api
- nas/emm
- nas/util
- Working on review comments on PR: https://github.com/magma/magma/pull/13904
- Working on migrating files under nas/ies folder
- Reviewed PRs: https://github.com/magma/magma/pull/13678, https://github.com/magma/magma/pull/13805, https://github.com/magma/magma/pull/13843
Highlights of the week 12th Sept`22 from Rashmi:
- As part of cpp migration in spgw module, fixed the memory leaks found while porting hash list of UE contexts to protobuf map. Manually verified the changes by executing the restart test cases. Changes are available in PR, https://github.com/magma/magma/pull/13895
- As part of cpp migration in S1ap module, started working on porting s1ap_state structure to protobuf structure
- Reviewed the PR, https://github.com/magma/magma/pull/13472, https://github.com/magma/magma/pull/13853
Highlights of the week 6th Sept`22 from Pruthvi:
- Completed c++ migration of nas/esm files. PR https://github.com/magma/magma/pull/13712 is merged to master
- Completed migration of nas/emm/msg files. PR https://github.com/magma/magma/pull/13754 is merged to master
- Working on migrating files under nas/emm/sap and nas/emm folders
- Reviewed PRs: https://github.com/magma/magma/pull/13805, https://github.com/magma/magma/pull/13508
Highlights of the week 5th Sept`22 from Rashmi:
- 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
- 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:
- 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
- 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 - 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/13659
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:
- Debugged test_ipv6_non_nat_dp_ul_tcp.py TC which is failing on CI with Christian Kramer from TNG. Found the following two issues:
- UE`s IPv6 address was getting added to eth3 interface of test vm with netmask 0. Fixed the issue.
- Sometimes neighbor between dev and trfserver was getting delayed or not happening at all. Earlier Pravin had given a fix for similar issue. But in this fix after receiving Neighbor Solicit msg from trfserver, agw was forwarding this msg to UE. UE would reply with Neighbor Adv msg. After this step, UE`s ipv6 address on eth3 interface of test vm used to become invalid and the test used to fail with "Cannot assign this address" error on the test VM. Christian Kramer fixed this issue so that the Neighbor Solicit msg from trfserver will be handled in dev VM itself and it will not be forwarded to UE.
- With the above fixes, a draft PR https://github.com/magma/magma/pull/13669 is created to verify if it works on CI. The test has passed in 3 consecutive runs. Will monitor the test for few more days before days.
- Started working c++ migration of files under nas/esm folder
- Reviewed PRs: https://github.com/magma/magma/pull/13508
Highlights of the week 22nd Aug`22 from Rashmi:
- Incorporated review comments on PR, https://github.com/magma/magma/pull/13508
- As part of cpp migration, working on porting eNB context from C structure to protobuf structure. After porting unit test case, GenerateUEContextReleaseCommand was failing because we added s1ap_enb_state as enum, previously it was integer, due to this change we need to add newly added parameter to encoded file. This issue is fixed. Coding is completed, need to validate the changes and cleanup the code.
Highlights of the week 22nd 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
- 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
Highlights of the week 15th Aug`22 from Ankit Kumar Aman:
- 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
Highlights of the week 9th Aug`22 from Pruthvi:
- As part of MOCN-GW design:
- Added design for handling success and failure scenarios during S1 and X2 handovers
- Retested IPv6 data test cases in PR https://github.com/magma/magma/pull/12646 which were earlier not working on master code. Without any local changes the tests are now working individually on multiple runs and also when added as part of integ tests. PR is merged to master.
- Working on IPv6 dedicated bearer test cases in PR https://github.com/magma/magma/pull/12892
- Reviewed PRs: https://github.com/magma/magma/pull/13423/, https://github.com/magma/magma/pull/13323, https://github.com/magma/magma/pull/13508, https://github.com/magma/magma/pull/13332, https://github.com/magma/magma/pull/13423
Highlights of the week 8th 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
- Raised Github Issue for need of CI workflow in S1APTesterhttps://github.com/magma/S1APTester/issues/87
- Fixed debugging enhancement in CI by fixing integ test status issue for failing executions and uploading of log artifacts and updated PRhttps://github.com/magma/magma/pull/13423
- PR reviews and follow-ups: https://github.com/magma/S1APTester/pull/86 and https://github.com/magma/magma/pull/13146
- Meeting and Discussion Involvements: Discussion with Govindarajan Mohandoss for testing S1APTester with arm
Highlights of the week 2nd Aug`22 from Pruthvi:
- As part of MOCN-GW design:
- Added design for handling failure scenarios for S1 setup, Initial attach, Idle mode and Reset procedures
- Modified handling of S1 setup procedure between MOCN-GW and MME. Updated the design to use preconfigured enb-id(per MME) at MOCN-GW instead of same Id for all MMEs
- Working on the design of HO scenarios
- IPv6 data test cases in PR https://github.com/magma/magma/pull/12646 are not working on master code. IPv6 data TC works if iperf is run between dev vm and trf vm on eth3 interface address prior to running the s1ap ipv6 test. The static ipv6 addresses configured on eth3 interfaces of dev and trf server vms become reachable when iperf is run between the 2 vms. Due to this ipv6 data test works if executed after running iperf. Trying to analyze why eth3 interface ipv6 addresses are not reachable if iperf is not run.
- Reviewed PRs: https://github.com/magma/magma/pull/13318, https://github.com/magma/magma/pull/13332
Highlights of the week 1st Aug`22 from Rashmi:
- Incorporated review comments on PR, https://github.com/magma/magma/pull/13335
- Locally brought up the Orc8r based subscriber DB. Issue was reported that on execution of basic attach detach test case from s1ap tester was resulting into Authentication Failure with emm_cause "sync failure"
But in my setup authentication is successful, but mobilityd is not able allocate UE ip address because subscriber is not configured. Checking the dependency of mobilityd on subscriber for IP allocation - As part of c++ migration, ported all c files to cpp for sgw_s8 module.
Highlights of the week 1st 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
- 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
Highlights of the week 26th July`22 from Pruthvi:
- As part of MOCN-GW design, added design for handling reset procedure
- IPv6 data test cases in PR https://github.com/magma/magma/pull/12646 are not working on master code. Debugging in progress
- Reviewed PRs: https://github.com/magma/magma/pull/13335, https://github.com/magma/S1APTester/pull/82, https://github.com/magma/magma/pull/13295 and https://github.com/magma/magma/pull/13138
Highlights of the week 25th July`22 from Rashmi:
- As part of issue, https://github.com/magma/magma/issues/11191 after converting C structure ue_description_t to protobuf structure UeDescription, memory leaks were occurring. Fixed these leaks in both unit test cases and end2end test cases
- Locally bringing up Orc8r and AGW and register AGW with Orc8r cloud.
Highlights of the week 25th July`22 from Ankit Kumar Aman:
- Follow up with the CI Issues document https://docs.google.com/document/d/1eGPuYqQBqxn8TBH_pPSVGg77kzuUrKokgfi_CYOgYeE/edit?usp=sharing
- Verified and monitored CI HTML report generation with pytest after merging PR https://github.com/magma/magma/pull/13060
- Found issue with TFW cleanup for successful testcases while testing flaky retry with pytest. Fixed the issue, added one new testcase to explicitly validate the scenario and raised PR https://github.com/magma/magma/pull/13332
- Some CI issues needed additional logs from VMs for better debugging. Followed up in devops-ci channel for getting access to those logs
- Addressed review comments over active PRs https://github.com/magma/magma/pull/13138, https://github.com/magma/magma/pull/13295, https://github.com/magma/magma/pull/13323 and https://github.com/magma/magma/pull/13332
- PR reviews and follow-ups: https://github.com/magma/magma/pull/13146, https://github.com/magma/magma/pull/12646 and https://github.com/magma/magma/pull/13243
- Meeting and Discussion Involvements: Attended Monday TSC Meeting and Wednesday 1.8 Release Meeting
Highlights of the week 22nd July`22 from Vikram:
- As part of C++ migration activity, raised PR: https://github.com/magma/magma/pull/13319 and fixed review comments.
Highlights of the week 19th July`22 from Pruthvi:
- As part of C++ migration activity , fixed review comments for PR : https://github.com/magma/magma/pull/13213. PR is merged to master.
- As part of MOCN-GW design, added design for s1 setup, initial attach, detach and idle mode procedures
- Reviewed PRs: https://github.com/magma/magma/pull/13138, https://github.com/magma/magma/pull/13295, https://github.com/magma/S1APTester/pull/82 and https://github.com/magma/magma/pull/13114
Highlights of the week 18th July`22 from Rashmi:
- As part of issue, https://github.com/magma/magma/issues/11191 converted C structure ue_description_t to protobuf structure UeDescription.
Verified manually the changes with basic test cases like test_attach_detach.py and test_attach_detach_with_mme_restart.py, But unit test case is failing because of memory leak. I shall continue to debug the same. - Reviewed the PRs, https://github.com/magma/S1APTester/pull/82, https://github.com/magma/magma/pull/13138, https://github.com/magma/magma/pull/13295
- Had internal discussion for designing MOCON-GW
Highlights of the week 18th July`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 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
Highlights of the week 13th July`22 from Pruthvi:
- As part of C++ migration activity , migrated all files under spgw task to cpp/hpp. Fixed review comments for PR : https://github.com/magma/magma/pull/13213
- Reviewed PRs: https://github.com/magma/magma/pull/13173, https://github.com/magma/magma/pull/13114
- Working on the design for handling S1 procedures on MOCN-GW
Highlights of the week 11th July`22 from Ankit Kumar Aman:
- Follow up with the CI Issues document https://docs.google.com/document/d/1eGPuYqQBqxn8TBH_pPSVGg77kzuUrKokgfi_CYOgYeE/edit?usp=sharing
- S1APTester Issue spotted by Guillaume Fraysse where wrong UE Id was printed in log for UEs larger than 255. The issue was causing detach to be unclean for UEs. Fixed the issue and raised the PR: https://github.com/magma/S1APTester/pull/82
- Found issue with S1APTester where DRB deletion is not happening for UE Id above 255. Raised the Github Issue https://github.com/magma/S1APTester/issues/83
- Addressed review comments on PR https://github.com/magma/S1APTester/pull/81 and got it merged
- Added reference testcase to support configurations for large number of UEs and raised PRs https://github.com/magma/magma/pull/13295 and https://github.com/magma/S1APTester/pull/84
- PR reviews and follow-ups:
Highlights of the week 11th July`22 from Rashmi:
- As part of c++ migration, incorporated review comments on PR, https://github.com/magma/magma/pull/13128
- On addressing the comment, https://github.com/magma/magma/pull/12973/#discussion_r896434249 "make test_oai" was failing for s1ap test suite. Debugged the same and shared the analysis on https://github.com/magma/magma/pull/12973/#discussion_r916084876
- Analyzed the issue, https://github.com/magma/magma/issues/9902 and shared the analysis in same github link
- Working on issue, https://github.com/magma/magma/issues/11191 for s1ap module
Highlights of the week 6th July`22 from Pruthvi:
- As part of C++ migration activity , migrated all files under spgw task to cpp/hpp. Fixed all the warning and errors. Raised PR for review: https://github.com/magma/magma/pull/13213
- Reviewed PR: https://github.com/magma/magma/pull/13173
- Reviewing PR: https://github.com/magma/magma/pull/13128
Highlights of the week 6th July`22 from Vikram:
- Working on C++ migration activity for renaming c to cpp in mme_app and nas.
Highlights of the week 4th July`22 from Ankit Kumar Aman:
- Analyzing an fixing CI related issues tracked with 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://github.com/magma/S1APTester/pull/81 and https://github.com/magma/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:
Highlights of the week 4th July`22 from Rashmi:
- 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.
- As part of issue, https://github.com/magma/magma/issues/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.
- Provided the fixes for the issue, https://github.com/magma/magma/issues/13086
- Reviewed the PRs, https://github.com/magma/magma/pull/13103 and https://github.com/magma/magma/pull/13142
Highlights of the week 29th June`22 from Vikram:
- 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 28th June`22 from Pruthvi:
- 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 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.