Skip to content

Commit

Permalink
tests: increase p11_child_timeout
Browse files Browse the repository at this point in the history
With recent version of valgrind some tests failed during a CI run with a
timeout. To avoid this the related p11_child_timeout is increased for
the affected tests.

Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
  • Loading branch information
sumit-bose authored and jhrozek committed Nov 26, 2018
1 parent 08bba3a commit 1617f3e
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/tests/cmocka/test_cert_utils.c
Expand Up @@ -54,7 +54,7 @@
/* When run under valgrind with --trace-children=yes we have to increase the
* timeout not because p11_child needs much more time under valgrind but
* because of the way valgrind handles the children. */
#define P11_CHILD_TIMEOUT 40
#define P11_CHILD_TIMEOUT 80

/* TODO: create a certificate for this test */
const uint8_t test_cert_der[] = {
Expand Down
2 changes: 1 addition & 1 deletion src/tests/cmocka/test_ssh_srv.c
Expand Up @@ -223,7 +223,7 @@ static int ssh_test_setup(void **state)
* the timeout not because p11_child needs much more time under valgrind
* but because of the way valgrind handles the children. */
struct sss_test_conf_param ssh_params[] = {
{ "p11_child_timeout", "40" },
{ "p11_child_timeout", "80" },
{ NULL, NULL }, /* Sentinel */
};

Expand Down

0 comments on commit 1617f3e

Please sign in to comment.