August 2019 - Devel - NFS Ganesha List Archives

Change in ...nfs-ganesha[next]: FSAL_RGW - Track openflags per state
by Daniel Gryniewicz (GerritHub) 12 Aug '19

12 Aug '19

Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/464947 ) Change subject: FSAL_RGW - Track openflags per state ...................................................................... FSAL_RGW - Track openflags per state Change-Id: I4573a50eddc20a1c6943d4c0401fa3f348ac618e Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com> --- M src/FSAL/FSAL_RGW/handle.c M src/FSAL/FSAL_RGW/internal.h 2 files changed, 13 insertions(+), 4 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/47/464947/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/464947 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I4573a50eddc20a1c6943d4c0401fa3f348ac618e Gerrit-Change-Number: 464947 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com> Gerrit-MessageType: newchange

1 0

Announce Push of V2.9-dev.4
by Frank Filz 12 Aug '19

12 Aug '19

Branch next Tag:V2.9-dev.4 Release Highlights * MDCACHE - Try to recover chunk after dropping lock * Fix malloc cast related warnings * XFS - libhandle is external, so it doesn't get ASAN * FSAL_RGW: restore unconditional close of opened files Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com> Contents: 8392cae Frank S. Filz V2.9-dev.4 1b9dafe Matt Benjamin FSAL_RGW: restore unconditional close of opened files 5a5b622 Daniel Gryniewicz XFS - libhandle is external, so it doesn't get ASAN 124697f Daniel Gryniewicz Fix malloc cast related warnings 92b4a45 Daniel Gryniewicz MDCACHE - Try to recover chunk after dropping lock

1 0

Change in ...nfs-ganesha[next]: ganeshactl: more fixes for selected pylint warnings and errors
by Kaleb KEITHLEY (GerritHub) 12 Aug '19

12 Aug '19

Kaleb KEITHLEY has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/464902 ) Change subject: ganeshactl: more fixes for selected pylint warnings and errors ...................................................................... ganeshactl: more fixes for selected pylint warnings and errors using pylint-1.3.1 from CentOS SIG repo. Note: EPEL has pylint-1.6.5, Fedora 30+ has pylint-2.3.1. Later versions do emit more warnings and errors, but pylint-1.3.1 is what's available in the SIG repos and thus what's available for eventual testing in the CentOS CI. This doesn't attempt to fix all the warnings. Mostly it's indentation, white space, long lines, and other low risk fixes. Signed-off-by: Kaleb S. KEITHLEY <kkeithle(a)redhat.com> Change-Id: I39531d691df5a908962105b479b6514754ec7dc7 --- M src/scripts/ganeshactl/Ganesha/QtUI/clients_table.py M src/scripts/ganeshactl/Ganesha/QtUI/exports_table.py M src/scripts/ganeshactl/Ganesha/QtUI/log_settings.py M src/scripts/ganeshactl/Ganesha/admin.py M src/scripts/ganeshactl/Ganesha/client_mgr.py M src/scripts/ganeshactl/Ganesha/config_editor.py M src/scripts/ganeshactl/Ganesha/export_mgr.py M src/scripts/ganeshactl/Ganesha/ganesha_mgr_utils.py M src/scripts/ganeshactl/Ganesha/glib_dbus_stats.py M src/scripts/ganeshactl/Ganesha/log_mgr.py 10 files changed, 276 insertions(+), 294 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/02/464902/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/464902 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I39531d691df5a908962105b479b6514754ec7dc7 Gerrit-Change-Number: 464902 Gerrit-PatchSet: 1 Gerrit-Owner: Kaleb KEITHLEY <kaleb(a)redhat.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: op_ctx->ctx_export->fsal_export maybe dereference error
by sepia-liu (GerritHub) 10 Aug '19

10 Aug '19

sepia-liu has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/464738 ) Change subject: op_ctx->ctx_export->fsal_export maybe dereference error ...................................................................... op_ctx->ctx_export->fsal_export maybe dereference error Change-Id: Ibb91decceef2e8e15b7683e4fc2f7006594f6878 Signed-off-by: sepia-liu <liuwei_coder(a)163.com> --- M src/Protocols/NFS/nfs4_op_readdir.c 1 file changed, 1 insertion(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/38/464738/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/464738 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: Ibb91decceef2e8e15b7683e4fc2f7006594f6878 Gerrit-Change-Number: 464738 Gerrit-PatchSet: 1 Gerrit-Owner: sepia-liu <liuwei_coder(a)163.com> Gerrit-MessageType: newchange

1 0

No merge this week, maybe early next week
by Frank Filz 10 Aug '19

10 Aug '19

I ran out of time, and there are a bunch of patches that have been submitted together to Gerrithub in various versions so I can't pull a branch and be assured of getting all the latest patches, and pulling patches individually is very time consuming and error prone. Frank

1 0

Change in ...nfs-ganesha[next]: ganeshactl: fix selected pylint warnings and errors
by Kaleb KEITHLEY (GerritHub) 09 Aug '19

09 Aug '19

Kaleb KEITHLEY has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/464715 ) Change subject: ganeshactl: fix selected pylint warnings and errors ...................................................................... ganeshactl: fix selected pylint warnings and errors using pylint-1.3.1 from CentOS SIG repo. Note: EPEL has pylint-1.6.5, Fedora 30+ has pylint-2.3.1. Later versions do emit more warnings and errors, but pylint-1.3.1 is what's available in the SIG repos and thus what's available for eventual testing in the CentOS CI. This doesn't attempt to fix all the warnings. Mostly it's indentation, white space, long lines, and other low risk fixes. Signed-off-by: Kaleb S. KEITHLEY <kkeithle(a)redhat.com> Change-Id: I1e12f24579cacbbd6400e997efa15c2439f33ece --- M src/scripts/ganeshactl/client_stats_9pOps.py M src/scripts/ganeshactl/export_stats_9pOps.py M src/scripts/ganeshactl/fake_recall.py M src/scripts/ganeshactl/ganesha-admin.py M src/scripts/ganeshactl/ganesha_conf.py M src/scripts/ganeshactl/ganesha_mgr.py M src/scripts/ganeshactl/ganesha_stats.py M src/scripts/ganeshactl/ganeshactl.py M src/scripts/ganeshactl/grace_period.py M src/scripts/ganeshactl/knfs2ganesha-exports.py M src/scripts/ganeshactl/manage_clients.py M src/scripts/ganeshactl/manage_exports.py M src/scripts/ganeshactl/manage_logger.py 13 files changed, 265 insertions(+), 214 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/15/464715/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/464715 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I1e12f24579cacbbd6400e997efa15c2439f33ece Gerrit-Change-Number: 464715 Gerrit-PatchSet: 1 Gerrit-Owner: Kaleb KEITHLEY <kaleb(a)redhat.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: FSAL_RGW: restore unconditional close of opened files
by Matt Benjamin (GerritHub) 09 Aug '19

09 Aug '19

Matt Benjamin has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/464703 ) Change subject: FSAL_RGW: restore unconditional close of opened files ...................................................................... FSAL_RGW: restore unconditional close of opened files This behavior was inadvertently changed in a5db2bed7, restore it. Required to...store data with NFSv4. Change-Id: I68887e9c7265ff80c36c25d1f3e2c7de98d1e423 Signed-off-by: Matt Benjamin <mbenjamin(a)redhat.com> --- M src/FSAL/FSAL_RGW/handle.c 1 file changed, 3 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/03/464703/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/464703 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I68887e9c7265ff80c36c25d1f3e2c7de98d1e423 Gerrit-Change-Number: 464703 Gerrit-PatchSet: 1 Gerrit-Owner: Matt Benjamin <mbenjamin(a)redhat.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: XFS - libhandle is external, so it doesn't get ASAN
by Daniel Gryniewicz (GerritHub) 08 Aug '19

08 Aug '19

Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/464585 ) Change subject: XFS - libhandle is external, so it doesn't get ASAN ...................................................................... XFS - libhandle is external, so it doesn't get ASAN Change-Id: I60999d36e860ef9ebcbb9107844e103aa452df49 Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com> --- M src/FSAL/FSAL_VFS/xfs/CMakeLists.txt 1 file changed, 0 insertions(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/85/464585/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/464585 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I60999d36e860ef9ebcbb9107844e103aa452df49 Gerrit-Change-Number: 464585 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: Fix malloc cast related warnings
by Daniel Gryniewicz (GerritHub) 08 Aug '19

08 Aug '19

Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/464559 ) Change subject: Fix malloc cast related warnings ...................................................................... Fix malloc cast related warnings Change-Id: I992b12427f0a026cfab74d8f5497c6b6dd262871 Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com> --- M src/include/abstract_mem.h M src/include/nfsv41.h 2 files changed, 3 insertions(+), 3 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/59/464559/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/464559 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I992b12427f0a026cfab74d8f5497c6b6dd262871 Gerrit-Change-Number: 464559 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com> Gerrit-MessageType: newchange

1 0

Change in ...nfs-ganesha[next]: MDCACHE - Try to recover chunk after dropping lock
by Daniel Gryniewicz (GerritHub) 08 Aug '19

08 Aug '19

Daniel Gryniewicz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/464558 ) Change subject: MDCACHE - Try to recover chunk after dropping lock ...................................................................... MDCACHE - Try to recover chunk after dropping lock whence_is_name benifits greatly from having the prev chunk when populating a new chunk. We can't keep the chunk across dropping the lock, since the directory could be invalidated, but we can save a cookie from the chunk and try to reload it afterwards. Change-Id: I50dc296bda1d13963e5312b4705c42a11242b007 Signed-off-by: Daniel Gryniewicz <dang(a)redhat.com> --- M src/FSAL/Stackable_FSALs/FSAL_MDCACHE/mdcache_helpers.c 1 file changed, 21 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/58/464558/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/464558 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I50dc296bda1d13963e5312b4705c42a11242b007 Gerrit-Change-Number: 464558 Gerrit-PatchSet: 1 Gerrit-Owner: Daniel Gryniewicz <dang(a)redhat.com> Gerrit-MessageType: newchange

1 0