add support for "sai_dbg_generate_dump" API

[aviramd]

This update adds support for the sai_dbg_gen_dump API, enabling the generation of a debug dump file by the SAI when there is no SAI failure.

add support to the global API sai_dbg_generate_dump to the SaiInterface class.
add a new operation to the syncd to support SAI debug generate dump

HLD: sonic-net/SONiC#1846

[kcudnik]

please make build success, after that i will review this

[kcudnik]

please rebase to latest master

[aviramd]

please rebase to latest master

Done

[aviramd]

add HLD PR: sonic-net/SONiC#1846

[kcudnik]

@aviramd can you fix whitespaces issue:

Checkig for white spaces ...
../proxylib/Sai.h:195:                  
../syncd/VendorSai.h:218:                    _In_ const char *dump_file_name) override;                    
../syncd/Syncd.cpp:412:    
../syncd/Syncd.cpp:434:        
../lib/ClientSai.cpp:1566:    
../lib/ClientSai.cpp:1568: 
../lib/Sai.cpp:788:    return SAI_STATUS_FAILURE; 
../lib/RedisRemoteSaiInterface.cpp:1923:    
../lib/RedisRemoteSaiInterface.cpp:1926: 
../lib/RedisRemoteSaiInterface.cpp:1932:        m_recorder->recordDbgGenDumpResponse(status);        
../lib/RedisRemoteSaiInterface.cpp:1936:    return SAI_STATUS_SUCCESS;  
../lib/Recorder.h:262:                    _In_ sai_status_t status);        
../lib/ClientServerSai.cpp:648:     
../lib/ClientServerSai.h:183:                    _In_ const char *dump_file_name) override;       
../lib/ServerSai.h:188:                    _In_ const char *dump_file_name) override;       
../lib/ServerSai.h:301:                    _In_ const swss::KeyOpFieldsValuesTuple &kco); 
../vslib/VirtualSwitchSaiInterface.h:196:                    _In_ const char *dump_file_name) override;               
../vslib/VirtualSwitchSaiInterface.h:197:                                    
../vslib/Sai.h:198:                    
../meta/SaiInterface.h:338:            
../meta/SaiInterface.h:341:                    
../meta/Meta.cpp:1427:    
ERROR: some files contain white spaces at the end of line, please fix
FAIL: checkwhitespace.sh

[kcudnik]

You will also need to add some unit tests to satisfy code coverage

[kcudnik]

i hink you made some mistake, you only touch test file

[aviramd]

i hink you made some mistake, you only touch test file

Yes you are right , i am working to fix it

[kcudnik]

you can try build this on your local machine to check whether it will compile first

[aviramd]

I have some git issues on my local machine
I am working to solve it , please ignore the last commit

[aviramd]

Is it possible to execute the sairedis unit test locally? if yes how?

[kcudnik]

Is it possible to execute the sairedis unit test locally? if yes how?

Yes, type make check in local console

[kcudnik]

to avoid so many push commits please test everything locally
type "make" and "make check" to run unittests locally

[aviramd]

to avoid so many push commits please test everything locally type "make" and "make check" to run unittests locally

Is this the correct procedure for testing?

1. Build the target.
2. Run sonic-slave-bookworm-user.
3. Inside the slave container, navigate to sonic/src/sonic-sairedis/tests.
4. Run "make check".
   Should I install any packages after entering the slave Docker container?

Also, after building the target, a cleanup occurs, and the make file (which is auto-generated during the build) gets deleted. How can I prevent this from happening?

[kcudnik]

no, you don't need to build entire docker
you can locally ./configure --with-sai=vs locally
on your machine