lists.ceph.io
Sign In
Sign Up
Sign In
Sign Up
Manage this list
×
Keyboard Shortcuts
Thread View
j
: Next unread message
k
: Previous unread message
j a
: Jump to all threads
j l
: Jump to MailingList overview
2024
May
April
March
February
January
2023
December
November
October
September
August
July
June
May
April
March
February
January
2022
December
November
October
September
August
July
June
May
April
March
February
January
2021
December
November
October
September
August
July
June
May
April
March
February
January
2020
December
November
October
September
August
July
June
May
April
March
February
January
2019
December
November
October
September
August
July
June
List overview
Download
thread
ceph API tests crashes when default value for shell is set to True
Rishabh Dave
28 Jan 2021
28 Jan '21
9:59 a.m.
Hi, PR #38443[1] sets default value of parameter (of vstart_runner.LocalRemoteProcess._do_run()) "shell" to "True" because teuthology adds '&&'[2] to every command it executes. However, setting shell to True leads the ceph API CI job to fail without any error message or traceback in the log. Following is the stack of method calls through which the ceph API test crashes/quits -
https://github.com/ceph/ceph/blob/master/qa/tasks/mgr/test_module_selftest.…
https://github.com/ceph/ceph/blob/master/qa/tasks/ceph_test_case.py#L85
https://github.com/ceph/ceph/blob/master/qa/tasks/vstart_runner.py#L974
https://github.com/ceph/ceph/blob/master/qa/tasks/vstart_runner.py#L404
https://github.com/ceph/ceph/blob/master/qa/tasks/vstart_runner.py#L467
https://github.com/ceph/ceph/blob/master/qa/tasks/vstart_runner.py#L183
https://github.com/ceph/ceph/blob/master/qa/tasks/vstart_runner.py#L221
test_module_selftest.TestModuleSelftest.test_selftest_cluster_log crashes at the very first iteration of the loop in it[3]. I have been able to trace these exact details by creating a clone[4] of the original[1] PR and adding extra log messages to it. [5] is the ceph API CI job for this PR. An alternate way to avoid the job failure is by setting shell to False in the call to run_ceph_w()[6]. Commit here[7] got that done for the original PR. Thanks, - Rishabh [1]
https://github.com/ceph/ceph/pull/38443
[2]
https://github.com/ceph/teuthology/blob/master/teuthology/orchestra/run.py#…
[3]
https://github.com/ceph/ceph/blob/master/qa/tasks/mgr/test_module_selftest.…
[4]
https://github.com/ceph/ceph/pull/38471
[5]
https://jenkins.ceph.com/job/ceph-api/10648/consoleFull#-85362248744e9240e-…
[6]
https://github.com/ceph/ceph/blob/master/qa/tasks/ceph_test_case.py#L109
[7]
https://github.com/ceph/ceph/pull/38443/commits/c7c5ad878e06d0fd76a43bcfa89…
0
0
Reply
Back to the thread
Back to the list