CI: Improve error handling in Iperf3 JSON analysis functions

Handle exception when error occurs during the iperf3 JSON log opening and decoding.

Prevent errors observed in these runs:

• https://jenkins-oai.eurecom.fr/job/RAN-SA-B200-Module-SABOX-Container/5268/
• https://jenkins-oai.eurecom.fr/job/RAN-Container-Parent/9533/

added BUILD-ONLY label

requested review from @schmidtr

there is some duplicated code, we could also harmonize the opening, and give to the functions one json object that is valid

requested changes

OAI RAN-Container-Parent build (9556): passed (https://jenkins-oai.eurecom.fr/job/RAN-Container-Parent/9556/)

added 2 commits

• 05e92c3a - CI: create get_JSON() function to check validity and return JSON object
• 7bcca305 - CI: Simplify iperf3 analysis functions

Compare with previous version

OAI RAN-Container-Parent build (9567): passed (https://jenkins-oai.eurecom.fr/job/RAN-Container-Parent/9567/)

added 4G-LTE 5G-NR labels

removed BUILD-ONLY label

marked this merge request as draft

marked this merge request as ready

OAI RAN-Container-Parent build (9572): failed (https://jenkins-oai.eurecom.fr/job/RAN-Container-Parent/9572/)

List of failing test stages:

• RAN-SA-AW2S-CN5G
• RAN-SA-2x2-Module-CN5G
• RAN-LTE-FDD-OAIUE-OAICN4G-Container
• RAN-PhySim-Cluster
• RAN-RF-Sim-Test-5G
• RAN-LTE-TDD-2x2-Container
• RAN-SA-B200-Module-SABOX-Container

marked this merge request as draft

marked this merge request as ready

OAI RAN-Container-Parent build (9619): failed (https://jenkins-oai.eurecom.fr/job/RAN-Container-Parent/9619/)

List of failing test stages:

• RAN-SA-AW2S-CN5G
• RAN-LTE-FDD-OAIUE-OAICN4G-Container
• RAN-RF-Sim-Test-5G
• RAN-SA-B200-Module-SABOX-Container
• RAN-LTE-TDD-LTEBOX-Container

please ensure proper spacing (blocking for this MR)

added 2 commits

• 5f684d7f - Improve error handling in Iperf_analyzeV3TCPJson()
• 9acf60a3 - Improve error handling in Iperf_analyzeV3BIDIRJson()

Compare with previous version

changed milestone to %REVIEW_CAN_START

added 1 commit

• 39070f65 - Minor fix in Iperf_analyzeV3TCPJson()

Compare with previous version

OAI RAN-Container-Parent build (9634): failed (https://jenkins-oai.eurecom.fr/job/RAN-Container-Parent/9634/)

List of failing test stages:

• RAN-SA-AW2S-CN5G
• RAN-LTE-FDD-OAIUE-OAICN4G-Container
• RAN-SA-B200-Module-SABOX-Container
• RAN-NSA-B200-Module-LTEBOX-Container
• RAN-RF-Sim-Test-5G
• RAN-RF-Sim-Test-4G
• RAN-L2-Sim-Test-4G

added 1 commit

• 12229ef5 - Smaller except blocks, add JSONDecodeError

Compare with previous version

resolved all threads

@jfiedler please have a look at what I did. I tested with the faulty iperf from the first link, and also some other logs that I mangled to trigger some problems. Worked for me. Let's check the CI, and maybe you can play with it briefly and we merge afterwards?

btw, better error handling but overall one line less.

Thank you @schmidtr, your modifications looks OK to me. I tested a bit with my log files and it worked well. I will check CI results tomorrow.

changed milestone to %REVIEW_IN_PROGRESS

OAI RAN-Container-Parent build (9638): failed (https://jenkins-oai.eurecom.fr/job/RAN-Container-Parent/9638/)

List of failing test stages:

• RAN-SA-AW2S-CN5G
• RAN-gNB-N300-Timing-Phytest-LDPC
• RAN-RF-Sim-Test-5G
• RAN-LTE-FDD-OAIUE-OAICN4G-Container
• RAN-SA-B200-Module-SABOX-Container
• RAN-L2-Sim-Test-4G

OAI RAN-Container-Parent build (9644): failed (https://jenkins-oai.eurecom.fr/job/RAN-Container-Parent/9644/)

List of failing test stages:

• RAN-LTE-FDD-OAIUE-OAICN4G-Container
• RAN-SA-AW2S-CN5G
• RAN-SA-B200-Module-SABOX-Container
• RAN-NSA-B200-Module-LTEBOX-Container
• RAN-RF-Sim-Test-5G
• RAN-LTE-TDD-LTEBOX-Container

• SA-B200 replay --> https://jenkins-oai.eurecom.fr/job/RAN-SA-B200-Module-SABOX-Container/5331/
• NSA-B200 replay --> https://jenkins-oai.eurecom.fr/job/RAN-NSA-B200-Module-LTEBOX-Container/6504/

mentioned in merge request !2894 (merged)

changed milestone to %REVIEW_COMPLETED_AND_APPROVED

approved this merge request

changed milestone to %OK_TO_BE_MERGED

mentioned in commit d47eb536

merged with merge request !2894 (merged)