Skip to content

[Cherry-pick][202511]Speed up techsupport collection - #4845

Open
dovsianko-nvda wants to merge 1 commit into
sonic-net:202511from
dovsianko-nvda:techsupport_time_fix_202511
Open

[Cherry-pick][202511]Speed up techsupport collection#4845
dovsianko-nvda wants to merge 1 commit into
sonic-net:202511from
dovsianko-nvda:techsupport_time_fix_202511

Conversation

@dovsianko-nvda

Copy link
Copy Markdown
Contributor

generate_dump collects FEC histogram counters separately for every interface in two counter snapshots. On full-split systems, executing these commands sequentially significantly increases techsupport time.

Run the per-interface commands in bounded batches of eight and wait only for their recorded process IDs. Preserve the existing per-interface output files and unmatched-glob handling.

What I did

Cherry-pick of PR #4776

How I did it

How to verify it

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

generate_dump collects FEC histogram counters separately for every
interface in two counter snapshots. On full-split systems, executing
these commands sequentially significantly increases techsupport time.

Run the per-interface commands in bounded batches of eight and wait
only for their recorded process IDs. Preserve the existing per-interface
output files and unmatched-glob handling.

Signed-off-by: Dmytro Ovsianko <dovsianko@nvidia.com>
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

@dovsianko-nvda

Copy link
Copy Markdown
Contributor Author

Original PR #4776

@dovsianko-nvda

Copy link
Copy Markdown
Contributor Author

Hi @vaibhavhd, I’ve created this PR to cherry-pick the change into the 202511 branch. Could you please review and approve it? Thanks!

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.

2 participants