Project

General

Profile

Bug #10413

samba: coredumps after tests run

Added by Greg Farnum over 4 years ago. Updated 5 months ago.

Status:
Resolved
Priority:
High
Assignee:
Category:
-
Target version:
-
Start date:
12/22/2014
Due date:
% 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:

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/

History

#1 Updated by Zheng Yan over 4 years ago

  • Status changed from New to In Progress

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

#2 Updated by Zheng Yan over 4 years ago

  • Status changed from In Progress to Need Review

#3 Updated by Greg Farnum over 4 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?

#4 Updated by Greg Farnum over 4 years ago

  • Status changed from Need Review to Pending Upstream
  • Assignee set to Zheng Yan

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

#5 Updated by Zheng Yan over 4 years ago

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

#7 Updated by Zheng Yan over 4 years ago

  • Status changed from Pending Upstream to Resolved

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

#8 Updated by Patrick Donnelly 5 months ago

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

Also available in: Atom PDF