Qi Luo

Results 63 comments of Qi Luo

Could you merge latest master? It will also retrigger the PR checkers.

Please merge latest master branch. The semgrep checker is added after this PR raised, so it will always block.

/azp run Azure.sonic-utilities #Closed

Waiting on @prsunny approval.

I assume you are talking about caclmgrd. Could you try improve below code? https://github.com/Azure/sonic-buildimage/blob/bba5df05233e17ab3eb8cd378e4629923b30ff16/files/image_config/caclmgrd/caclmgrd#L553 After each select, there are lots of data in SubscribeStateTable. You can keep pop() until you...

@mikelazar Could you help review?

1. > support switching between standalone mode and cluser mode in runtime You mentioned there are 2 modes. Are they mutually exclusive? Can we have them at the same time?...

@marian-pritsak Could you help check this issue?

> "LOGLEVEL_DB" : { There is a ongoing PR to remove this DB. Possible to use it? --- In reply to: [1260486336](https://github.com/sonic-net/sonic-swss-common/pull/683#issuecomment-1260486336) [](http://example.com/codeflow?ancestors=1260486336) --- In reply to: [1260486336](https://github.com/sonic-net/sonic-swss-common/pull/683#issuecomment-1260486336) [](http://example.com/codeflow?ancestors=1260486336) ---...