[sssd PR#810][opened] TESTS: ability to run unit tests under valgrind (sssd-1-16 backport)
URL: https://github.com/SSSD/sssd/pull/810 Author: alexey-tikhonov Title: #810: TESTS: ability to run unit tests under valgrind (sssd-1-16 backport) Action: opened
PR body: """ Adds ability to run unit tests under valgrind with handy makefile targets. Enabled with --enable-valgrind passed to ./configure script.
Reviewed-by: Pavel Březina pbrezina@redhat.com (cherry picked from commit e812151511f7b09908efb3c8ebed294a26efebc7) """
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/810/head:pr810 git checkout pr810
URL: https://github.com/SSSD/sssd/pull/810 Title: #810: TESTS: ability to run unit tests under valgrind (sssd-1-16 backport)
alexey-tikhonov commented: """ retest this please """
See the full comment at https://github.com/SSSD/sssd/pull/810#issuecomment-488685326
URL: https://github.com/SSSD/sssd/pull/810 Title: #810: TESTS: ability to run unit tests under valgrind (sssd-1-16 backport)
alexey-tikhonov commented: """ retest this please """
See the full comment at https://github.com/SSSD/sssd/pull/810#issuecomment-488998014
URL: https://github.com/SSSD/sssd/pull/810 Title: #810: TESTS: ability to run unit tests under valgrind (sssd-1-16 backport)
alexey-tikhonov commented: """ retest this please """
See the full comment at https://github.com/SSSD/sssd/pull/810#issuecomment-489600169
URL: https://github.com/SSSD/sssd/pull/810 Author: alexey-tikhonov Title: #810: [WiP] TESTS: ability to run unit tests under valgrind (sssd-1-16 backport) Action: edited
Changed field: title Original value: """ TESTS: ability to run unit tests under valgrind (sssd-1-16 backport) """
URL: https://github.com/SSSD/sssd/pull/810 Title: #810: [WiP] TESTS: ability to run unit tests under valgrind (sssd-1-16 backport)
alexey-tikhonov commented: """ This PR is constantly failing on CentOS CI (intg-tests). It doesn't fail on other machines (including similar CentOS). To figure out a reason of fail it is required to (temporary) patch PR to export CI logs. """
See the full comment at https://github.com/SSSD/sssd/pull/810#issuecomment-491789744
URL: https://github.com/SSSD/sssd/pull/810 Title: #810: [WiP] TESTS: ability to run unit tests under valgrind (sssd-1-16 backport)
pbrezina commented: """ What is the current status? """
See the full comment at https://github.com/SSSD/sssd/pull/810#issuecomment-527064577
URL: https://github.com/SSSD/sssd/pull/810 Title: #810: [WiP] TESTS: ability to run unit tests under valgrind (sssd-1-16 backport)
alexey-tikhonov commented: """
What is the current status?
Unchanged since https://github.com/SSSD/sssd/pull/810#issuecomment-491789744
I am not sure if it worth the effort. """
See the full comment at https://github.com/SSSD/sssd/pull/810#issuecomment-527120947
URL: https://github.com/SSSD/sssd/pull/810 Title: #810: [WiP] TESTS: ability to run unit tests under valgrind (sssd-1-16 backport)
alexey-tikhonov commented: """
What is the current status?
Unchanged since https://github.com/SSSD/sssd/pull/810#issuecomment-491789744
I am not sure if it worth the effort. Maybe we should just close this PR. """
See the full comment at https://github.com/SSSD/sssd/pull/810#issuecomment-527120947
URL: https://github.com/SSSD/sssd/pull/810 Title: #810: [WiP] TESTS: ability to run unit tests under valgrind (sssd-1-16 backport)
Label: +Deferred
URL: https://github.com/SSSD/sssd/pull/810 Title: #810: [WiP] TESTS: ability to run unit tests under valgrind (sssd-1-16 backport)
pbrezina commented: """ Ok, let's close this PR and reopen if the work is resumed. """
See the full comment at https://github.com/SSSD/sssd/pull/810#issuecomment-527124002
URL: https://github.com/SSSD/sssd/pull/810 Author: alexey-tikhonov Title: #810: [WiP] TESTS: ability to run unit tests under valgrind (sssd-1-16 backport) Action: closed
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/810/head:pr810 git checkout pr810
sssd-devel@lists.fedorahosted.org
-
alexey-tikhonov -
pbrezina