search

sonic-net / sonic-sairedis

SAI object interface to Redis database, as used in the SONiC project
Other
56 stars 263 forks source link

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

Closed JunhongMao closed 6 months ago

JunhongMao commented 7 months ago

Why I did it During the process of fix PR1335, https://github.com/sonic-net/sonic-sairedis/pull/1335 There was an error: dpkg-checkbuilddeps: error: Unmet build dependencies: nlohmann-json3-dev.

I checked the building scripts and found that they didn't include nlohmann-json3-dev. So, I think I need to create a separate PR that only installs nlohmann-json3-dev. After merging that PR, the PR1335 could be checked successfully. sudo apt-get install -y libxml-simple-perl aspell ... ... libjansson-dev python

How I did it

JunhongMao commented 7 months ago

@kcudnik, there still was some errors:

image image

What's the reason? Please help to review them.

JunhongMao commented 7 months ago

@kcudnik @mlok-nokia , please help to process the Azure Pipelines / Azure.sonic-sairedis (Test vstest) error. Thanks.

JunhongMao commented 7 months ago

@kcudnik @mlok-nokia , please help to process the Azure Pipelines / Azure.sonic-sairedis (Test vstest) error. Thanks.

mlok-nokia commented 7 months ago

@kcudnik Please help to take a look if these failures is related to the change in this PR? Thanks

kcudnik commented 7 months ago

You can ignore this error not related to pr