Skip to content

orchagent for generate SAI debug dump file#3301

Open
aviramd wants to merge 10 commits intosonic-net:masterfrom
Marvell-switching:SAI_DBG_GEN_DUMP_support
Open

orchagent for generate SAI debug dump file#3301
aviramd wants to merge 10 commits intosonic-net:masterfrom
Marvell-switching:SAI_DBG_GEN_DUMP_support

Conversation

@aviramd
Copy link

@aviramd aviramd commented Sep 25, 2024

This update enables the generation of a SAI debug dump file.
A new orchestration agent, DbgGenDumpOrch, has been introduced, which is triggered by updates in the APPL DB . It updates syncd by writing to the ASIC DB and waits for a response. Once received, it writes the result back to the APPL DB, allowing the calling application to retrieve the file.

HLD: sonic-net/SONiC#1846

What I did

Why I did it

How I verified it

Details if related

This update introduces support for the sai_dbg_gen_dump API
The general process is as follows:
The filename is written to configDB.
This action triggers the dbgGenDumpOrch to write the necessary data to ASIC DB.
The syncd process reads from ASIC DB and invokes the SAI.
SAI processes the request and returns a response.
syncd writes the response back to redis.
dbgGenDumpOrch retrieves the response and informs the caller by writing to configDB.
The caller then removes the data from configDB.
@aviramd aviramd requested a review from prsunny as a code owner September 25, 2024 14:39
@prsunny
Copy link
Collaborator

prsunny commented Oct 29, 2024

Please link HLD and also associated tests. Suggest move this to draft until then

@aviramd aviramd changed the title add orchagent for debug generate dump orchagent for generate SAI debug dump file Nov 6, 2024
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants