longhuan-cisco

Results 28 comments of longhuan-cisco

The below two pre-commit check failures are due to baseline, because they started to happen right after merging master to local branch https://github.com/sonic-net/sonic-mgmt/pull/13108/commits/b0dd014f6bba79df0a9384e703499b1907b4656b: https://github.com/sonic-net/sonic-mgmt/pull/13108/checks?check_run_id=27298781282 https://github.com/sonic-net/sonic-mgmt/pull/13108/checks?check_run_id=27298780950

> @longhuan-cisco Can you please test the changes on a device with breakout cable and ensure all subports are linked up after one of the subport has gone through reset....

The pre-commit sanity check failure [Azure.sonic-mgmt (Test onboarding t1 testcases by Elastictest - optional) ](https://github.com/sonic-net/sonic-mgmt/pull/13108/checks?check_run_id=29855101954) seems to be some baseline issue, also saw it on other PRs': https://github.com/sonic-net/sonic-mgmt/pull/14474/checks?check_run_id=29863617790 https://github.com/sonic-net/sonic-mgmt/pull/14472/checks?check_run_id=29853135820

> @prgeor for viz @longhuan-cisco I am seeing that after executing the `test_check_sfputil_reset` testcase individually, a config reload is being triggered due to a failure seen while executing `core_dump_and_config_check`. Are...

> @longhuan-cisco Can you please help in addressing the below per our discussion? [#13108 (comment)](https://github.com/sonic-net/sonic-mgmt/pull/13108#discussion_r1785387333) @mihirpat1 I addressed as what we discussed. And I also added logic to skip `config...

@kevinskwang I've addressed conflicts and raised cherry-pick PR already https://github.com/sonic-net/sonic-mgmt/pull/15543

> @longhuan-cisco Hi, we need this changes also for 202311 branch, can you please also make PR for 202311 ? @eyakubch Could you please check with them if it's OK...

@prgeor I have done OIR tests with the latest diff. Tried below sequence on 400G AOC: 1. Move from non-breakout mode (1x400G) to 4x100G breakout mode via DPB CLI 2....