[XL] Change in ...nfs-ganesha[next]: Fix: Implementation of registering ganesha service to ceph.
by Name of user not set (GerritHub)
sragraha(a)redhat.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1230117?usp=email )
Change subject: Fix: Implementation of registering ganesha service to ceph.
......................................................................
Fix: Implementation of registering ganesha service to ceph.
Change-Id: I6f2938ea523f75a705d158f3035a802643d2301e
Signed-off-by: Sreedhar Agraharam <sragraha(a)redhat.com>
---
M src/FSAL/FSAL_CEPH/internal.h
M src/FSAL/FSAL_CEPH/main.c
M src/FSAL/commonlib.c
M src/FSAL/default_methods.c
M src/MainNFSD/CMakeLists.txt
M src/MainNFSD/libganesha_nfsd.ver
M src/MainNFSD/nfs_init.c
M src/config_parsing/conf_url_rados.c
M src/include/fsal.h
M src/include/fsal_api.h
10 files changed, 614 insertions(+), 557 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/17/1230117/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1230117?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Change-Id: I6f2938ea523f75a705d158f3035a802643d2301e
Gerrit-Change-Number: 1230117
Gerrit-PatchSet: 1
Gerrit-Owner: sragraha(a)redhat.com
2 weeks
[XS] Change in ...nfs-ganesha[next]: Set proper mode for file/directory at creation time
by Sachin Punadikar (GerritHub)
Sachin Punadikar has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1230067?usp=email )
Change subject: Set proper mode for file/directory at creation time
......................................................................
Set proper mode for file/directory at creation time
Files created without an explicit mode defaulted to 0600 (owner-only)
and directories to 0700. This kind of strict mode breaks tools like git,
where it creates temporary files and reads back.
When git creates temporary files (like tmp_pack_24gqju), it writes to
them, closes them, and then reopens them for reading. With mode 0600,
the file was too restrictive for this workflow.
Change-Id: I22c37a45f2a4f2080581100466bc86f5a1245b03
Signed-off-by: Sachin Punadikar <sachin.punadikar(a)ibm.com>
---
M src/Protocols/NFS/nfs3_create.c
M src/Protocols/NFS/nfs4_op_create.c
M src/Protocols/NFS/nfs4_op_open.c
3 files changed, 4 insertions(+), 4 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/67/1230067/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1230067?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Change-Id: I22c37a45f2a4f2080581100466bc86f5a1245b03
Gerrit-Change-Number: 1230067
Gerrit-PatchSet: 1
Gerrit-Owner: Sachin Punadikar <sachin.punadikar(a)ibm.com>
2 weeks, 1 day
Weekly Community Call
by Frank Filz
Greetings all,
Lately we haven't been having much participation in the weekly calls. On top
of this, every other week I have a conflict, and then those weeks I don't
have a conflict it has been hard for me to make it on time.
I know it's hard for folks to commit to the meeting, and the time probably
doesn't help. Unfortunately my flexibility is very limited because getting
up any earlier doesn't work for my family dynamics and an hour later I'm
driving kids to school. Any later than that becomes way too late for India
and is certainly past normal working hours for Europe and east. Those time
constraints of course make it tough to avoid conflicts with IBM internal
meetings.
If anyone has any input on other times or days that would work for them,
maybe we can find a time more folks can make it.
At the least, I would like to schedule the meeting 30 minutes later (7:30 AM
Pacific Time) until the US goes on Daylight Savings Time in the spring. This
would avoid the conflict and make it easier for me to make the meeting. But
I'm open to other times and days if something works for a good number of
folks.
All that said, the alternate meeting at 4:00 PM Pacific Time is going well
(except when I forget about it), but I'm not sure that works well for anyone
other than those in Australia (it certainly is middle of the night for
India).
Thanks
Frank Filz
2 weeks, 2 days
[S] Change in ...nfs-ganesha[next]: Add default no-op implementation for fsal_enable_delegations
by Lior Suliman (GerritHub)
Attention is currently required from: Frank Filz.
Hello Frank Filz,
I'd like you to do a code review.
Please visit
https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1229518?usp=email
to review the following change.
Change subject: Add default no-op implementation for fsal_enable_delegations
......................................................................
Add default no-op implementation for fsal_enable_delegations
This commit adds a default fsal_enable_delegations function to
default_methods.c, which logs an unsupported operation and does
nothing. This function is also added to the def_fsal_ops struct
to provide a default handler for this operation in FSAL modules.
Change-Id: Ibc8ebaa4bb079edec30c17225320a23b002a7a79
Signed-off-by: Lior Suliman <liorsu(a)gmail.com>
---
M src/FSAL/default_methods.c
1 file changed, 13 insertions(+), 0 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/18/1229518/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1229518?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Change-Id: Ibc8ebaa4bb079edec30c17225320a23b002a7a79
Gerrit-Change-Number: 1229518
Gerrit-PatchSet: 1
Gerrit-Owner: Lior Suliman <liorsu(a)google.com>
Gerrit-Reviewer: Frank Filz <ffilzlnx(a)mindspring.com>
Gerrit-Attention: Frank Filz <ffilzlnx(a)mindspring.com>
3 weeks
[S] Change in ...nfs-ganesha[next]: Add IP-based client owner separation for NFSv4.1+.
by Lior Suliman (GerritHub)
Attention is currently required from: Frank Filz.
Hello Frank Filz,
I'd like you to do a code review.
Please visit
https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1229517?usp=email
to review the following change.
Change subject: Add IP-based client owner separation for NFSv4.1+.
......................................................................
Add IP-based client owner separation for NFSv4.1+.
This change introduces a new configuration option,
"Ip_Based_Client_Owner_Separation", which, when enabled,
causes Ganesha to use the client's IP address in addition
to the provided client owner identifier to uniquely identify
NFSv4.1+ clients. The `get_client_record` function is updated
to accept and store the client's IP address in the client record.
Change-Id: Ie2cbedc3178156eaa49b36a945db2e5f04492de0
Signed-off-by: Lior Suliman <liorsu(a)gmail.com>
---
M src/Protocols/NFS/nfs4_op_exchange_id.c
M src/Protocols/NFS/nfs4_op_setclientid.c
M src/SAL/nfs4_clientid.c
M src/doc/man/ganesha-core-config.rst
M src/include/gsh_config.h
M src/include/sal_data.h
M src/include/sal_functions.h
M src/support/nfs_read_conf.c
8 files changed, 39 insertions(+), 5 deletions(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/17/1229517/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1229517?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Change-Id: Ie2cbedc3178156eaa49b36a945db2e5f04492de0
Gerrit-Change-Number: 1229517
Gerrit-PatchSet: 1
Gerrit-Owner: Lior Suliman <liorsu(a)google.com>
Gerrit-Reviewer: Frank Filz <ffilzlnx(a)mindspring.com>
Gerrit-Attention: Frank Filz <ffilzlnx(a)mindspring.com>
3 weeks
[XS] Change in ...nfs-ganesha[next]: Allow read locks on exported RO filesystems.
by Name of user not set (GerritHub)
pradeepthomas(a)gmail.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1229412?usp=email )
Change subject: Allow read locks on exported RO filesystems.
......................................................................
Allow read locks on exported RO filesystems.
When a Read-Only filesystem is exported through Ganesha, taking
read-locks on the NFS mount fails with ROFS error. This is because
of the open mode escalation in fsal_start_io. This patch handles
the ROFS error and retries the open with read.
Change-Id: I6452334ee1eae0faa8559fda2e218839ec57e1ae
Signed-off-by: Pradeep <pradeepthomas(a)gmail.com>
---
M src/FSAL/commonlib.c
1 file changed, 2 insertions(+), 1 deletion(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/12/1229412/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1229412?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Change-Id: I6452334ee1eae0faa8559fda2e218839ec57e1ae
Gerrit-Change-Number: 1229412
Gerrit-PatchSet: 1
Gerrit-Owner: pradeepthomas(a)gmail.com
3 weeks, 2 days
[XS] Change in ...nfs-ganesha[next]: Add Content-Type header to Prometheus metrics response
by Name of user not set (GerritHub)
sanyam230902(a)gmail.com has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1229379?usp=email )
Change subject: Add Content-Type header to Prometheus metrics response
......................................................................
Add Content-Type header to Prometheus metrics response
Latest Prometheus rejects scrape targets without Content-Type,
marking Files pod metrics as non-compliant.
Include text/plain; version=0.0.4; charset=utf-8 in the
exposer response to satisfy the scraper.
Change-Id: Ic4495c49d5caacdfc913c9ac233cc254df7b5002
Signed-off-by: Sanyam <sanyam230902(a)gmail.com>
---
M src/monitoring/prometheus_exposer.cc
1 file changed, 3 insertions(+), 1 deletion(-)
git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/79/1229379/1
--
To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/1229379?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: ffilz/nfs-ganesha
Gerrit-Branch: next
Gerrit-Change-Id: Ic4495c49d5caacdfc913c9ac233cc254df7b5002
Gerrit-Change-Number: 1229379
Gerrit-PatchSet: 1
Gerrit-Owner: sanyam230902(a)gmail.com
3 weeks, 2 days