Ying Xie

Results 46 comments of Ying Xie

> @yxieca @prgeor - It's been waiting long time, btw Prince George is code owner for this repo. Can you guys approve this PR or assign to right owner. Thanks...

@Pterosaur @rlhui can you assign someone to review this test plan?

@vmittal-msft cherry-picking this PR to 202205 branch causes build failure, please raise separate PR. Failure message: portsorch.cpp:7795:12: note: suggested alternative: 'gbport_stat_ids'return m_port_voq_ids[port.m_alias];^~~~~~~~~~~~~~gbport_stat_ids

> Should this be a part of doc: https://github.com/sonic-net/SONiC/blob/master/doc/sonic-port-name.md ? I agree with this suggestion. The design sounds benight but the actual change touches quite some code. Can we call...

@vaibhavhd to check from upgrade perspective. @xumia to check from secure boot/upgrade perspective.

@jcaiMR can you update the PR title to make it more meaningful?

@prgeor, @sujinmkang please review this change. Thanks.

@StormLiangMS this change is much bigger than the title. Are you sure you want to merge 99 commits?

@longhuan-cisco this change cannot be cherry-picked to 202205 cleanly. Please raise separate PR. Thanks!