The Samba-Bugzilla – Attachment 16345 Details for
Bug 14486
vfs_glusterfs: smbtorture test smb2.rw.rw1 failure on distributed disperse volume
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Requests
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
follow up patch for 4.11
v4-11.patch (text/plain), 4.49 KB, created by
Anoop C S
on 2020-11-20 05:38:01 UTC
(
hide
)
Description:
follow up patch for 4.11
Filename:
MIME Type:
Creator:
Anoop C S
Created:
2020-11-20 05:38:01 UTC
Size:
4.49 KB
patch
obsolete
>From 76a1cdca38bdf8c74da4a6d923cde0324df31075 Mon Sep 17 00:00:00 2001 >From: =?UTF-8?q?G=C3=BCnther=20Deschner?= <gd@samba.org> >Date: Mon, 2 Nov 2020 16:10:44 +0100 >Subject: [PATCH 1/2] s3-vfs_glusterfs: always disable write-behind translator > >The "pass-through" option has now been merged upstream as of: >https://github.com/gluster/glusterfs/pull/1640 > >BUG: https://bugzilla.samba.org/show_bug.cgi?id=14486 > >Guenther > >Signed-off-by: Guenther Deschner <gd@samba.org> >Pair-Programmed-With: Anoop C S <anoopcs@samba.org> >Pair-Programmed-With: Sachin Prabhu <sprabhu@redhat.com> >Reviewed-by: Jeremy Allison <jra@samba.org> > >Autobuild-User(master): Jeremy Allison <jra@samba.org> >Autobuild-Date(master): Wed Nov 4 22:53:49 UTC 2020 on sn-devel-184 > >(cherry picked from commit a51cda69ec6a017ad04b5690a3ae67a5478deee9) > >Autobuild-User(v4-13-test): Karolin Seeger <kseeger@samba.org> >Autobuild-Date(v4-13-test): Thu Nov 5 13:54:25 UTC 2020 on sn-devel-184 >--- > source3/modules/vfs_glusterfs.c | 20 +++++++++++++++++--- > source3/wscript | 3 +++ > 2 files changed, 20 insertions(+), 3 deletions(-) > >diff --git a/source3/modules/vfs_glusterfs.c b/source3/modules/vfs_glusterfs.c >index 190235cf8ae..4e1dc169565 100644 >--- a/source3/modules/vfs_glusterfs.c >+++ b/source3/modules/vfs_glusterfs.c >@@ -365,6 +365,7 @@ static int vfs_gluster_connect(struct vfs_handle_struct *handle, > glfs_t *fs = NULL; > TALLOC_CTX *tmp_ctx; > int ret = 0; >+ bool write_behind_pass_through_set = false; > > tmp_ctx = talloc_new(NULL); > if (tmp_ctx == NULL) { >@@ -424,6 +425,17 @@ static int vfs_gluster_connect(struct vfs_handle_struct *handle, > goto done; > } > >+#ifdef HAVE_GFAPI_VER_7_9 >+ ret = glfs_set_xlator_option(fs, "*-write-behind", "pass-through", >+ "true"); >+ if (ret < 0) { >+ DBG_ERR("%s: Failed to set xlator option: pass-through\n", >+ volume); >+ goto done; >+ } >+ write_behind_pass_through_set = true; >+#endif >+ > ret = glfs_set_logging(fs, logfile, loglevel); > if (ret < 0) { > DEBUG(0, ("%s: Failed to set logfile %s loglevel %d\n", >@@ -438,9 +450,11 @@ static int vfs_gluster_connect(struct vfs_handle_struct *handle, > goto done; > } > >- ret = check_for_write_behind_translator(tmp_ctx, fs, volume); >- if (ret < 0) { >- goto done; >+ if (!write_behind_pass_through_set) { >+ ret = check_for_write_behind_translator(tmp_ctx, fs, volume); >+ if (ret < 0) { >+ goto done; >+ } > } > > ret = glfs_set_preopened(volume, handle->conn->connectpath, fs); >diff --git a/source3/wscript b/source3/wscript >index 7b257bcb845..47f97c60321 100644 >--- a/source3/wscript >+++ b/source3/wscript >@@ -1591,6 +1591,9 @@ main() { > conf.CHECK_CFG(package='glusterfs-api', args='"glusterfs-api >= 7.6" --cflags --libs', > msg='Checking for glusterfs-api >= 7.6', > uselib_store="GFAPI_VER_7_6") >+ conf.CHECK_CFG(package='glusterfs-api', args='"glusterfs-api >= 7.9" --cflags --libs', >+ msg='Checking for glusterfs-api >= 7.9', >+ uselib_store="GFAPI_VER_7_9") > else: > conf.SET_TARGET_TYPE('gfapi', 'EMPTY') > conf.undefine('HAVE_GLUSTERFS') >-- >2.28.0 > > >From 3799ef40bdb06a9ce15f064fc6641d2e26181b60 Mon Sep 17 00:00:00 2001 >From: Anoop C S <anoopcs@samba.org> >Date: Thu, 5 Nov 2020 16:12:09 +0530 >Subject: [PATCH 2/2] manpages/vfs_glusterfs: Mention silent skipping of > write-behind translator >MIME-Version: 1.0 >Content-Type: text/plain; charset=UTF-8 >Content-Transfer-Encoding: 8bit > >BUG: https://bugzilla.samba.org/show_bug.cgi?id=14486 > >Signed-off-by: Anoop C S <anoopcs@samba.org> >Reviewed-by: Andrew Bartlett <abartlet@samba.org> >Reviewed-by: Guenther Deschner <gd@samba.org> > >Autobuild-User(master): Günther Deschner <gd@samba.org> >Autobuild-Date(master): Mon Nov 9 13:30:06 UTC 2020 on sn-devel-184 > >(cherry picked from commit be03ce7d8bb213633eedcfc3299b8d9865a3c67f) >--- > docs-xml/manpages/vfs_glusterfs.8.xml | 5 +++++ > 1 file changed, 5 insertions(+) > >diff --git a/docs-xml/manpages/vfs_glusterfs.8.xml b/docs-xml/manpages/vfs_glusterfs.8.xml >index 7a4da1af919..ca6085a0595 100644 >--- a/docs-xml/manpages/vfs_glusterfs.8.xml >+++ b/docs-xml/manpages/vfs_glusterfs.8.xml >@@ -180,6 +180,11 @@ > Please disable the write-behind translator for the GlusterFS > volume to allow the plugin to connect to the volume. > </para> >+ <para> >+ With GlusterFS versions >= 9, we silently bypass write-behind >+ translator during intial connect and failure is avoided. >+ </para> >+ > </refsect1> > > >-- >2.28.0 >
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Raw
Actions:
View
Attachments on
bug 14486
:
16217
|
16218
|
16226
|
16319
|
16320
|
16322
|
16324
|
16333
|
16345
|
16346
|
16354
|
16355