Skip to content

[tests] [asan] extend graceful stop flag to also stop syncd#2491

Merged
liat-grozovik merged 2 commits intosonic-net:masterfrom
Yakiv-Huryk:tests-graceful-stop-syncd
Nov 8, 2022
Merged

[tests] [asan] extend graceful stop flag to also stop syncd#2491
liat-grozovik merged 2 commits intosonic-net:masterfrom
Yakiv-Huryk:tests-graceful-stop-syncd

Conversation

@Yakiv-Huryk
Copy link
Contributor

What I did
Extended DVS test "--graceful-stop" flag to also stop syncd.

Why I did it
So ASAN can generate reports for syncd.

How I verified it
Run the tests with --graceful-stop, observe that the syncd process is stopped via SIGTERM

Details if related

Now, when running DVS tests with --graceful-stop, both swss and syncd
are stopped via SIGTERM. This is needed for asan to generate reports.

Signed-off-by: Yakiv Huryk <[email protected]>
@Yakiv-Huryk Yakiv-Huryk requested a review from prsunny as a code owner October 20, 2022 10:10
@prsunny prsunny requested a review from xumia October 20, 2022 22:39
@liat-grozovik
Copy link
Collaborator

@xumia kindly reminder to review this pr

@liat-grozovik
Copy link
Collaborator

/azp run Azure.sonic-swss

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@liat-grozovik liat-grozovik merged commit 724f914 into sonic-net:master Nov 8, 2022
Janetxxx pushed a commit to Janetxxx/sonic-swss that referenced this pull request Nov 10, 2025
…t#2491)

- What I did
Extended DVS test "--graceful-stop" flag to also stop syncd.

- Why I did it
So ASAN can generate reports for syncd.

- How I verified it
Run the tests with --graceful-stop, observe that the syncd process is stopped via SIGTERM

Signed-off-by: Yakiv Huryk <[email protected]>
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