Project

General

Profile

Actions

Bug #10413

closed

samba: coredumps after tests run

Added by Greg Farnum over 9 years ago. Updated about 5 years ago.

Status:
Resolved
Priority:
High
Assignee:
Category:
-
Target version:
-
% Done:

0%

Source:
Q/A
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
Labels (FS):
Samba/CIFS
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

These tests had coredumps without corresponding failures in teuthology.log or backtraces in any of the logs. The client logs on the machines with coredumps ended abruptly during shutdown, so they're probably client crashes of some kind.

http://pulpito.ceph.com/teuthology-2014-12-17_23:14:01-samba-master-testing-basic-multi/667135/
http://pulpito.ceph.com/teuthology-2014-12-17_23:14:01-samba-master-testing-basic-multi/667133/

Actions #1

Updated by Zheng Yan over 9 years ago

  • Status changed from New to In Progress

smbd can call cephwrap_{getcwd,chdir,stat} after umount

Actions #2

Updated by Zheng Yan over 9 years ago

  • Status changed from In Progress to Fix Under Review
Actions #3

Updated by Greg Farnum over 9 years ago

That looks good to me (I guess?), but I don't think putting it into our own samba repo is the right place. IIRC we're pulling down samba from their upstream git repo rather than our own for the nightlies, and we certainly need to get the patch upstream. Can you do that?

Actions #4

Updated by Greg Farnum over 9 years ago

  • Status changed from Fix Under Review to 15
  • Assignee set to Zheng Yan

We talked about this in standup; Zheng is going to send it to the Samba guys.

Actions #5

Updated by Zheng Yan over 9 years ago

samba guys proposes a fix on their side, the new fix should go into upstream soon

Actions #7

Updated by Zheng Yan about 9 years ago

  • Status changed from 15 to Resolved

commit 81464daea71e5fa3067ec7d5f5c69c890c0f7949 smbd: Stop using vfs_Chdir after SMB_VFS_DISCONNECT" should fix this issue

Actions #8

Updated by Patrick Donnelly about 5 years ago

  • Category deleted (43)
  • Labels (FS) Samba/CIFS added
Actions

Also available in: Atom PDF