Bug 13684 - CTDB autobuild can fail on certain CI machines
Summary: CTDB autobuild can fail on certain CI machines
Status: RESOLVED FIXED
Alias: None
Product: Samba 4.1 and newer
Classification: Unclassified
Component: CTDB (show other bugs)
Version: 4.9.0
Hardware: All All
: P5 normal (vote)
Target Milestone: ---
Assignee: Karolin Seeger
QA Contact: Samba QA Contact
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-11-14 21:58 UTC by Tim Beale
Modified: 2018-11-27 13:19 UTC (History)
3 users (show)

See Also:


Attachments
Patch for v4.9 (2.33 KB, patch)
2018-11-18 23:55 UTC, Martin Schwenke
amitay: review+
timbeale: review+
Details
Possible patch for 4.8 (2.34 KB, patch)
2018-11-18 23:56 UTC, Martin Schwenke
amitay: review-
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Tim Beale 2018-11-14 21:58:01 UTC
The CTDB simple/90_debug_hung_script.sh test-case fails when run on certain host machines. Basically it only passes if the sysctl setting yama.ptrace_scope=0 is present on the host OS. If yama.ptrace_scope=1 the test fails because it can't cat /prod/<pid>/stack.

More details on mailing-list thread:
https://lists.samba.org/archive/samba-technical/2018-November/131113.html

We manually set yama.ptrace_scope=0 on the gitlab CI machines. However, we don't necessarily have control over this setting on other cloud machines (or host PCs) that we want to run CI on.

This bug is to backport a fix for the test-case to maintenance branches, so that we can potentially run CI on other cloud machines.
Comment 1 Martin Schwenke 2018-11-18 23:55:16 UTC
Created attachment 14665 [details]
Patch for v4.9
Comment 2 Martin Schwenke 2018-11-18 23:56:41 UTC
Created attachment 14666 [details]
Possible patch for 4.8
Comment 3 Martin Schwenke 2018-11-18 23:59:21 UTC
I've added a patch for 4.8 but I don't think we want it.  In <=4.8 the test is in CTDB's "complex" test suite, which is not run during autobuild (as it requires a real cluster).  I've attached it for completeness but probably delete it...  :-)
Comment 4 Amitay Isaacs 2018-11-19 00:07:31 UTC
(In reply to Martin Schwenke from comment #3)

I agree.  We don't need to patch the test for 4.8.
Comment 5 Martin Schwenke 2018-11-19 22:34:00 UTC
Comment on attachment 14666 [details]
Possible patch for 4.8

OK, marking this one obsolete to avoid confusion...
Comment 6 Martin Schwenke 2018-11-19 22:34:35 UTC
Hi Karolin,

This is ready for 4.9.

Thanks...
Comment 7 Karolin Seeger 2018-11-20 11:30:26 UTC
(In reply to Martin Schwenke from comment #6)
Pushed to autobuild-v4-9-test.
Comment 8 Karolin Seeger 2018-11-27 13:19:20 UTC
(In reply to Karolin Seeger from comment #7)
Pushed to v4-9-test.
Closing out bug report.

Thanks!