sonic-swss icon indicating copy to clipboard operation
sonic-swss copied to clipboard

Published 20 hours ago • verified publisher icon sonic-net

[ci][asan] add DVS tests run with ASAN

Open Yakiv-Huryk opened this issue 2 years ago • 6 comments

This adds 3 new stages:

  • BuildAsan: same as 'Build' but with ENABLE_ASAN=y
  • BuildDockerAsan: same as 'BuildDocker' but uses asan-enabled swss and adds swss-dbg package. It also uses asan-enabled docker-sonic-vs from the sonic-buildimage ci.
  • TestAsan: same as 'Test' but uses asan-enabled docker and publishes the asan reports if any.

What I did Added new CI stages to run the DVS tests with ASAN and check the ASAN reports.

Why I did it To utilize ASAN for catching memory usage issues on PR.

How I verified it Run the CI on my fork

Details if related

Yakiv-Huryk avatar Jun 30 '22 11:06 Yakiv-Huryk

This requires the https://github.com/Azure/sonic-buildimage/pull/11297

Yakiv-Huryk avatar Jun 30 '22 11:06 Yakiv-Huryk

/azpw run Azure.sonic-swss

oleksandrivantsiv avatar Jul 04 '22 09:07 oleksandrivantsiv

/azp run

oleksandrivantsiv avatar Jul 04 '22 09:07 oleksandrivantsiv

Commenter does not have sufficient privileges for PR 2363 in repo Azure/sonic-swss

azure-pipelines[bot] avatar Jul 04 '22 09:07 azure-pipelines[bot]

@Yakiv-Huryk could you please handle the conflicts and verify checkers are passing?

liat-grozovik avatar Jul 14 '22 06:07 liat-grozovik

@liushilongbuaa kindly reminder to review and approve

liat-grozovik avatar Aug 10 '22 15:08 liat-grozovik

LGTM @Yakiv-Huryk , please resolve conflicts.

liushilongbuaa avatar Aug 22 '22 04:08 liushilongbuaa

LGTM @Yakiv-Huryk , please resolve conflicts.

It says 'This branch has no conflicts with the base branch' for me

Yakiv-Huryk avatar Aug 22 '22 10:08 Yakiv-Huryk