Skip to content

Some additions for Gtest TAPI#75

Open
okt-yurijp wants to merge 3 commits into
oktetlabs:mainfrom
okt-yurijp:gtest_fixes
Open

Some additions for Gtest TAPI#75
okt-yurijp wants to merge 3 commits into
oktetlabs:mainfrom
okt-yurijp:gtest_fixes

Conversation

@okt-yurijp

Copy link
Copy Markdown
Contributor

Add the following feature to Gtest TAPI:

  1. Add possibility to log Gtest command line
  2. Kill Gtest job if it lasts for too long.
  3. Add --complition_wait_multiplier option

Tested by running some tests which run Gtest TAPI and job hangs for too long - TAPI now kills such jobs. Also using of "--complition_wait_multiplier " option was added to some tests and they work as expected. And logging of Gtest command line also used in our tests.

@ol-andrewr ol-andrewr self-assigned this Jun 21, 2025
@oktetlabs oktetlabs deleted a comment from ol-sourcery-ai Jun 21, 2025
@ol-andrewr ol-andrewr requested review from ol-andrewr and ol-arteman and removed request for ol-sourcery-ai September 1, 2025 08:47
Comment thread lib/tapi_gtest/tapi_gtest.c Outdated
Comment thread lib/tapi_gtest/tapi_gtest.c Outdated
Comment thread lib/tapi_gtest/tapi_gtest.h Outdated
Signed-off-by: Yurij Plotnikov <yurij.plotnikov@arknetworks.am>
Reviewed-by: Ivan Malov <ivan.malov@arknetworks.am>
Reviewed-by: Roman Zhukov <roman.zhukov@arknetworks.am>
Add killing of job to tapi_gtest_wait() if it lasts more than timeout.

Signed-off-by: Yurij Plotnikov <yurij.plotnikov@arknetworks.am>
Reviewed-by: Oleg Karpov <oleg.karpov@arknetworks.am>
Add "--complition_wait_multiplier" option to gtest.

Signed-off-by: Yurij Plotnikov <yurij.plotnikov@arknetworks.am>
Reviewed-by: Denis Pryazhennikov <denis.pryazhennikov@arknetworks.am>
Reviewed-by: Oleg Karpov <oleg.karpov@arknetworks.am>
@okt-yurijp

Copy link
Copy Markdown
Contributor Author

All comments were corrected.

@okt-yurijp okt-yurijp requested a review from ol-arteman October 21, 2025 09:34
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