JunhongMao

Results 17 comments of JunhongMao

[VOQ][saidump] Enhance saidump with new option -r to parser the JSON file and displays/format the right output, fix compiling error

@kcudnik , how to update this PR's checking policy to bypass the coverage check blocking?

[VOQ][saidump] Enhance saidump with new option -r to parser the JSON file and displays/format the right output, fix compiling error

@kcudnik, @gechiang @mlok-nokia , could we bypass this code-testing coverage issue to merge it first and create a new PR to fix the code-testing coverage issue in the future? I'm...

[VOQ][saidump] Enhance saidump with new option -r to parser the JSON file and displays/format the right output, fix compiling error

@kcudnik Thanks. I'm working on it.

[VOQ][saidump] Enhance saidump with new option -r to parser the JSON file and displays/format the right output, fix compiling error

Hi @kcudnik, I have added the unit test codes in sonic-sairedis\unittest\saidump, but the Coverage check results showed that the new unit test codes were not called. Could you help to...

[VOQ][saidump] Enhance saidump with new option -r to parser the JSON file and displays/format the right output, fix compiling error

Hi @kcudnik, I have converted the saidump.cpp to libsaidump.a. But there was still an error below: dpkg-checkbuilddeps: error: Unmet build dependencies: nlohmann-json3-dev. I checked the building scripts and found that...

[VOQ][saidump] Enhance saidump with new option -r to parser the JSON file and displays/format the right output, fix compiling error

I created new PR: https://github.com/sonic-net/sonic-sairedis/pull/1350

[VOQ][saidump] Enhance saidump with new option -r to parser the JSON file and displays/format the right output, fix compiling error

@kcudnik @gechiang I found the failures of the below checks are wired. Azure.sonic-sairedis (Build amd64) Failing after 33m — Build amd64 failed Azure.sonic-sairedis Failing after 33m — Build #20240416.3 failed...