Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove KAFKA_HOST_TEST from compute-sanitizer check #12831

Merged

Conversation

davidwendt
Copy link
Contributor

Description

Removes the KAFKA_HOST_TEST from the compute-sanitizer memcheck nighly runs.
The following error occurs when running this host test.

Running compute-sanitizer on KAFKA_HOST_TEST
========= COMPUTE-SANITIZER
Running main() from gmock_main.cc
[==========] Running 2 tests from 1 test suite.
[----------] Global test environment set-up.
[----------] 2 tests from KafkaDatasourceTest
[ RUN      ] KafkaDatasourceTest.MissingGroupID
[       OK ] KafkaDatasourceTest.MissingGroupID (0 ms)
[ RUN      ] KafkaDatasourceTest.InvalidConfigValues
[       OK ] KafkaDatasourceTest.InvalidConfigValues (0 ms)
[----------] 2 tests from KafkaDatasourceTest (0 ms total)

[----------] Global test environment tear-down
[==========] 2 tests from 1 test suite ran. (0 ms total)
[  PASSED  ] 2 tests.
========= Error: Target application terminated before first instrumented API call
========= Tracking kernels launched by child processes requires the --target-processes all option.

Adding the --target-processes all option gives the same error.

Disabling the check of this test since it is a host test that checks error conditions and does not appear to make any device calls.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@davidwendt davidwendt added bug Something isn't working 3 - Ready for Review Ready for review by team libcudf Affects libcudf (C++/CUDA) code. non-breaking Non-breaking change labels Feb 23, 2023
@davidwendt davidwendt self-assigned this Feb 23, 2023
@github-actions github-actions bot added ci and removed libcudf Affects libcudf (C++/CUDA) code. labels Feb 23, 2023
@davidwendt davidwendt marked this pull request as ready for review February 23, 2023 20:53
@davidwendt davidwendt requested a review from a team as a code owner February 23, 2023 20:53
@davidwendt
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit 12e4501 into rapidsai:branch-23.04 Feb 24, 2023
@davidwendt davidwendt deleted the sanitizer-disable-kafka-tests branch February 24, 2023 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Ready for Review Ready for review by team bug Something isn't working non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants