Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

jewel: mds fails to respawn if executable has changed #11873

Merged
1 commit merged into from Nov 14, 2016
Merged

jewel: mds fails to respawn if executable has changed #11873

1 commit merged into from Nov 14, 2016

Conversation

ghost
Copy link

@ghost ghost commented Nov 9, 2016

This allows the MDS to respawn using the same executable file even if it
has since been deleted (on Linux). Otherwise, the execv fails because
the readlink returns "/path/to/deleted (deleted)". (There is no path to
the old executable.)

Fixes: http://tracker.ceph.com/issues/17531

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit 66a1220)
@ghost ghost self-assigned this Nov 9, 2016
@ghost ghost added this to the jewel milestone Nov 9, 2016
@ghost ghost added bug-fix cephfs Ceph File System labels Nov 9, 2016
@ghost ghost changed the base branch from jewel to jewel-next November 9, 2016 15:29
@ghost
Copy link
Author

ghost commented Nov 9, 2016

jenkins test this please (general jenkins failure)

@ghost
Copy link
Author

ghost commented Nov 9, 2016

jenkins test this please (osd-scrub-repair.sh failed)

ghost pushed a commit that referenced this pull request Nov 9, 2016
…has changed

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost
Copy link
Author

ghost commented Nov 14, 2016

@jcsp does this backport look good to merge ? It passed the fs suite http://tracker.ceph.com/issues/17851#note-5 (except for valgrind failures and one java test failure that also fail on the jewel branch ). Note that it will not be included in 10.2.4, reason why it targets jewel-next.

@ghost ghost assigned jcsp Nov 14, 2016
@ghost ghost merged commit 68146ea into ceph:jewel-next Nov 14, 2016
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-fix cephfs Ceph File System
Projects
None yet
2 participants