[Gluster-devel] NFS Ganesha fails to export a volume
Ravishankar N
ravishankar at redhat.com
Wed Nov 18 07:11:03 UTC 2020
On 18/11/20 12:17 pm, Strahil Nikolov wrote:
> Nope, it's a deeper s**t.
> I had to edit the ".spec.in" file so it has Source0 point to local tar.gz.
> The I edit the requires in both ".spec" & ".spec.in" and also I had to remove an obsolete stanza in the glusterfs section.
>
> In the end, I got the source - extracted, copied the spec & spec.in , and then tar.gz-ed again and put it into the dir.
>
> Only then the rpms were properly built.
>
> The proposed patch is fixing the issue.
Thanks for confirming!
>
> Why do we have line 285 in https://raw.githubusercontent.com/gluster/glusterfs/devel/glusterfs.spec.in ?
>
> I guess I need to open 2 issues for the glusterfs:
> - that obsolete stanza is useless
Using git blame points me to
https://github.com/gluster/glusterfs/commit/f9118c2c9389e0793951388c2d69ce0350bb9318.
Adding Shwetha to confirm if the change was intended.
-Ravi
>
>
> Best Regards,
> Strahil Nikolov
>
>
>
> В вторник, 17 ноември 2020 г., 14:16:36 Гринуич+2, Ravishankar N <ravishankar at redhat.com> написа:
>
>
>
>
>
> Hi Strahil,
>
> I would have imagined editing the 'Requires' section in
> glusterfs.spec.in would have sufficed. Do you need rpms though? A source
> install is not enough?
>
> Regards,
> Ravi
>
> On 17/11/20 5:32 pm, Strahil Nikolov wrote:
>> Hi Ravi,
>>
>>
>> Any idea how to make the glusterfs-ganesha.x86_64 require resource-agents >= 4.1.0 (instead of 4.2.0) ?
>> I 've replaced every occurance I found and still it tries to grab resource-agents 4.2 (which is not available on EL8).
>>
>> Best Regards,
>> Strahil Nikolov
>>
>>
>>
>>
>>
>>
>> В понеделник, 16 ноември 2020 г., 13:15:54 Гринуич+2, Ravishankar N<ravishankar at redhat.com> написа:
>>
>>
>>
>>
>>
>>
>> I am surprised too that it wasn't caught earlier.
>>
>>
>> Steps:
>>
>> 1. Clone the gluster repo
>>
>> 2. Compile the sourcehttps://docs.gluster.org/en/latest/Developer-guide/Building-GlusterFS/
>>
>> 3. Make the changes (in a different branch if you prefer), compile again and install
>>
>> 4. Test it out:
>>
>> [root at linuxpad glusterfs]# gluster v create testvol 127.0.0.2:/home/ravi/bricks/brick{1..2} force
>> volume create: testvol: success: please start the volume to access data
>> [root at linuxpad glusterfs]#
>> [root at linuxpad glusterfs]# gluster v start testvol
>> volume start: testvol: success
>> [root at linuxpad glusterfs]#
>> [root at linuxpad glusterfs]# gluster v set testvol ganesha.enable on
>> volume set: failed: The option nfs-ganesha should be enabled before setting ganesha.enable.
>> [root at linuxpad glusterfs]#
>>
>>
>> I just tried the change and it looks like some new error shows up. Not too familiar with these settings; I will need to debug further.
>>
>> Thanks,
>>
>> Ravi
>>
>>
>> On 16/11/20 4:05 pm, Strahil Nikolov wrote:
>>
>>
>>> I can try to help with the testing (I'm quite new to that).
>>> Can someone share documentation of that process ?
>>>
>>> yet we have another problem -> ganesha is deployed with ocf:heartbeat:portblock which supports only IPTABLES, while EL8 uses NFTABLES ...
>>>
>>> Best Regards,
>>> Strahil Nikolov
>>>
>>>
>>>
>>>
>>>
>>>
>>> В понеделник, 16 ноември 2020 г., 10:47:43 Гринуич+2, Yaniv Kaul<ykaul at redhat.com> написа:
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>> On Mon, Nov 16, 2020 at 10:26 AM Ravishankar N<ravishankar at redhat.com> wrote:
>>>
>>>> On 15/11/20 8:24 pm, Strahil Nikolov wrote:
>>>>
>>>>> Hello All,
>>>>>
>>>>> did anyone get a chance to look athttps://github.com/gluster/glusterfs/issues/1778 ?
>>>>>
>>>> A look at
>>>> https://review.gluster.org/#/c/glusterfs/+/23648/4/xlators/mgmt/glusterd/src/glusterd-op-sm.c@1117
>>>> seems to indicate this could be due to a typo error. Do you have a
>>>> source install where you can apply this simple diff and see if it fixes
>>>> the issue?
>>>>
>>> I think you are right - I seem to have introduced it as part ofhttps://github.com/gluster/glusterfs/commit/e081ac683b6a5bda54891318fa1e3ffac981e553 - my bad.
>>>
>>> However, it was merged ~1 year ago, and no one has complained thus far... :-/
>>> 1. Is no one using NFS Ganesha?
>>> 2. We are lacking tests for NFS Ganesha - code coverage indicates this path is not covered.
>>>
>>> Y.
>>>
>>>
>>>>
>>>> diff --git a/xlators/mgmt/glusterd/src/glusterd-op-sm.c
>>>> b/xlators/mgmt/glusterd/src/glusterd-op-sm.c
>>>> index 558f04fb2..d7bf96adf 100644
>>>> --- a/xlators/mgmt/glusterd/src/glusterd-op-sm.c
>>>> +++ b/xlators/mgmt/glusterd/src/glusterd-op-sm.c
>>>> @@ -1177,7 +1177,7 @@ glusterd_op_stage_set_volume(dict_t *dict, char
>>>> **op_errstr)
>>>> }
>>>> } else if (len_strcmp(key, keylen, "ganesha.enable")) {
>>>> key_matched = _gf_true;
>>>> - if (!strcmp(value, "off") == 0) {
>>>> + if (strcmp(value, "off") == 0) {
>>>> ret = ganesha_manage_export(dict, "off", _gf_true,
>>>> op_errstr);
>>>> if (ret)
>>>> goto out;
>>>>
>>>> Thanks,
>>>>
>>>> Ravi
>>>>
>>>>> It's really strange that NFS Ganesha has ever passed the tests.
>>>>> How do we test NFS Ganesha exporting ?
>>>>>
>>>>> Best Regards,
>>>>> Strahil Nikolov
>>>>> _______________________________________________
>>>>>
>>>>> Community Meeting Calendar:
>>>>>
>>>>> Schedule -
>>>>> Every 2nd and 4th Tuesday at 14:30 IST / 09:00 UTC
>>>>> Bridge:https://bluejeans.com/441850968
>>>>>
>>>>>
>>>>>
>>>>>
>>>>> Gluster-devel mailing list
>>>>> Gluster-devel at gluster.org
>>>>> https://lists.gluster.org/mailman/listinfo/gluster-devel
>>>>>
>>>>>
>>>> _______________________________________________
>>>>
>>>> Community Meeting Calendar:
>>>>
>>>> Schedule -
>>>> Every 2nd and 4th Tuesday at 14:30 IST / 09:00 UTC
>>>> Bridge:https://bluejeans.com/441850968
>>>>
>>>>
>>>>
>>>>
>>>> Gluster-devel mailing list
>>>> Gluster-devel at gluster.org
>>>> https://lists.gluster.org/mailman/listinfo/gluster-devel
>>>>
>>>>
>>>>
More information about the Gluster-devel
mailing list