* [lustre-devel] Fwd: llmount.sh does not use fs-specific paths to the ost(s) and mds it mounts
[not found] <5731EAC3.5070207@cea.fr>
@ 2016-05-10 14:14 ` Quentin BOUGET
2016-05-10 19:42 ` Dilger, Andreas
0 siblings, 1 reply; 6+ messages in thread
From: Quentin BOUGET @ 2016-05-10 14:14 UTC (permalink / raw)
To: lustre-devel
Hello!
While trying to mount two different lustre filesystems on a local node
with the "llmount.sh" script, I noticed that the path to mount mds and
ost(s) was computed like this:
eval mntpt=${!var:-${MOUNT%/*}/$facet} # var is not defined by
default, so this removes
# everything after the last '/' of $MOUNT
The code can be found at test-framework.sh in the function "facet_mntpt"
around the line 3226.
By default MOUNT is initialized like this:
MOUNT=${MOUNT:-/mnt/${FSNAME}} # default: "/mnt/lustre"
Thus the default mountpoints for mds and ost(s) are "/mnt/mds1" and
"/mnt/ost1".
Because of that, when trying to do two successive calls to "llmount.sh"
with two different values for FSNAME, the second fs to be mounted will
unmount the first's mds and ost(s) (paths are identical).
Is there any reason to truncate the MOUNT variable like this ? Shouldn't
the mountpoint path look more like "/mnt/<fsname>-mds1" and
"/mnt/<fsname>-ost1" ?
Regards,
Quentin Bouget
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.lustre.org/pipermail/lustre-devel-lustre.org/attachments/20160510/809629ec/attachment.htm>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [lustre-devel] Fwd: llmount.sh does not use fs-specific paths to the ost(s) and mds it mounts
2016-05-10 14:14 ` [lustre-devel] Fwd: llmount.sh does not use fs-specific paths to the ost(s) and mds it mounts Quentin BOUGET
@ 2016-05-10 19:42 ` Dilger, Andreas
2016-05-11 1:17 ` [lustre-devel] " Drokin, Oleg
0 siblings, 1 reply; 6+ messages in thread
From: Dilger, Andreas @ 2016-05-10 19:42 UTC (permalink / raw)
To: lustre-devel
On 2016/05/10, 08:14, "Quentin BOUGET" <quentin.bouget.ocre@cea.fr> wrote:
>
>Hello!
>
>While trying to mount two different lustre filesystems on a local node
>with the "llmount.sh" script, I noticed that the path to mount mds and
>ost(s) was computed like this:
>
>eval mntpt=${!var:-${MOUNT%/*}/$facet} # var is not defined by default,
>so this removes
> #
>everything after the last '/' of $MOUNT
>
>
>The code can be found at test-framework.sh in the function "facet_mntpt"
>around the line 3226.
>
>
>By default MOUNT is initialized like this:
>
>MOUNT=${MOUNT:-/mnt/${FSNAME}} # default: "/mnt/lustre"
>
>
>Thus the default mountpoints for mds and ost(s) are "/mnt/mds1" and
>"/mnt/ost1".
>Because of that, when trying to do two successive calls to "llmount.sh"
>with two different values for FSNAME, the second fs to be mounted will
>unmount the first's mds and ost(s) (paths are identical).
>
>
>Is there any reason to truncate the MOUNT variable like this ? Shouldn't
>the mountpoint path look more like "/mnt/<fsname>-mds1" and
>"/mnt/<fsname>-ost1"?
I don't think there is any particular reason it was implemented this way,
just that it has worked for many years without the need to include the
fsname into the MDT/OST mount points... :-)
If you want to expand it to allow exporting multiple filesystems from the
same MDS/OSS nodes then your suggestion to use $FSNAME-$facet in
facet_mntpt() is reasonable. Please submit a patch to verify this is
working correctly. There may be some places in the tests that aren't
using facet_mntpt() yet.
Cheers, Andreas
--
Andreas Dilger
Lustre Principal Architect
Intel High Performance Data Division
^ permalink raw reply [flat|nested] 6+ messages in thread
* [lustre-devel] llmount.sh does not use fs-specific paths to the ost(s) and mds it mounts
2016-05-10 19:42 ` Dilger, Andreas
@ 2016-05-11 1:17 ` Drokin, Oleg
2016-05-11 7:07 ` Quentin BOUGET
0 siblings, 1 reply; 6+ messages in thread
From: Drokin, Oleg @ 2016-05-11 1:17 UTC (permalink / raw)
To: lustre-devel
On May 10, 2016, at 3:42 PM, Dilger, Andreas wrote:
> On 2016/05/10, 08:14, "Quentin BOUGET" <quentin.bouget.ocre@cea.fr> wrote:
>>
>> Hello!
>>
>> While trying to mount two different lustre filesystems on a local node
>> with the "llmount.sh" script, I noticed that the path to mount mds and
>> ost(s) was computed like this:
>>
>> eval mntpt=${!var:-${MOUNT%/*}/$facet} # var is not defined by default,
>> so this removes
>> #
>> everything after the last '/' of $MOUNT
>>
>>
>> The code can be found at test-framework.sh in the function "facet_mntpt"
>> around the line 3226.
>>
>>
>> By default MOUNT is initialized like this:
>>
>> MOUNT=${MOUNT:-/mnt/${FSNAME}} # default: "/mnt/lustre"
>>
>>
>> Thus the default mountpoints for mds and ost(s) are "/mnt/mds1" and
>> "/mnt/ost1".
>> Because of that, when trying to do two successive calls to "llmount.sh"
>> with two different values for FSNAME, the second fs to be mounted will
>> unmount the first's mds and ost(s) (paths are identical).
>>
>>
>> Is there any reason to truncate the MOUNT variable like this ? Shouldn't
>> the mountpoint path look more like "/mnt/<fsname>-mds1" and
>> "/mnt/<fsname>-ost1"?
>
> I don't think there is any particular reason it was implemented this way,
> just that it has worked for many years without the need to include the
> fsname into the MDT/OST mount points... :-)
>
> If you want to expand it to allow exporting multiple filesystems from the
> same MDS/OSS nodes then your suggestion to use $FSNAME-$facet in
> facet_mntpt() is reasonable. Please submit a patch to verify this is
> working correctly. There may be some places in the tests that aren't
> using facet_mntpt() yet.
I guess to expand on this, since llmount.sh is primarily a testing
infrastructure and typically only one filesystem at a time is tested,
the more than one mountpoint was never bothering anyone till you
discovered this bit, I guess.
Bye,
Oleg
^ permalink raw reply [flat|nested] 6+ messages in thread
* [lustre-devel] llmount.sh does not use fs-specific paths to the ost(s) and mds it mounts
2016-05-11 1:17 ` [lustre-devel] " Drokin, Oleg
@ 2016-05-11 7:07 ` Quentin BOUGET
2016-05-11 15:58 ` James Simmons
0 siblings, 1 reply; 6+ messages in thread
From: Quentin BOUGET @ 2016-05-11 7:07 UTC (permalink / raw)
To: lustre-devel
Le 11/05/2016 03:17, Drokin, Oleg a ?crit :
> On May 10, 2016, at 3:42 PM, Dilger, Andreas wrote:
>
>> On 2016/05/10, 08:14, "Quentin BOUGET" <quentin.bouget.ocre@cea.fr> wrote:
>>> Hello!
>>>
>>> While trying to mount two different lustre filesystems on a local node
>>> with the "llmount.sh" script, I noticed that the path to mount mds and
>>> ost(s) was computed like this:
>>>
>>> eval mntpt=${!var:-${MOUNT%/*}/$facet} # var is not defined by default,
>>> so this removes
>>> #
>>> everything after the last '/' of $MOUNT
>>>
>>>
>>> The code can be found at test-framework.sh in the function "facet_mntpt"
>>> around the line 3226.
>>>
>>>
>>> By default MOUNT is initialized like this:
>>>
>>> MOUNT=${MOUNT:-/mnt/${FSNAME}} # default: "/mnt/lustre"
>>>
>>>
>>> Thus the default mountpoints for mds and ost(s) are "/mnt/mds1" and
>>> "/mnt/ost1".
>>> Because of that, when trying to do two successive calls to "llmount.sh"
>>> with two different values for FSNAME, the second fs to be mounted will
>>> unmount the first's mds and ost(s) (paths are identical).
>>>
>>>
>>> Is there any reason to truncate the MOUNT variable like this ? Shouldn't
>>> the mountpoint path look more like "/mnt/<fsname>-mds1" and
>>> "/mnt/<fsname>-ost1"?
>> I don't think there is any particular reason it was implemented this way,
>> just that it has worked for many years without the need to include the
>> fsname into the MDT/OST mount points... :-)
>>
>> If you want to expand it to allow exporting multiple filesystems from the
>> same MDS/OSS nodes then your suggestion to use $FSNAME-$facet in
>> facet_mntpt() is reasonable. Please submit a patch to verify this is
>> working correctly. There may be some places in the tests that aren't
>> using facet_mntpt() yet.
> I guess to expand on this, since llmount.sh is primarily a testing
> infrastructure and typically only one filesystem at a time is tested,
> the more than one mountpoint was never bothering anyone till you
> discovered this bit, I guess.
>
> Bye,
> Oleg
>
>
Ok, thank you. I will submit a patch about it.
Bye,
Quentin
^ permalink raw reply [flat|nested] 6+ messages in thread
* [lustre-devel] llmount.sh does not use fs-specific paths to the ost(s) and mds it mounts
2016-05-11 7:07 ` Quentin BOUGET
@ 2016-05-11 15:58 ` James Simmons
2016-05-12 7:17 ` Quentin BOUGET
0 siblings, 1 reply; 6+ messages in thread
From: James Simmons @ 2016-05-11 15:58 UTC (permalink / raw)
To: lustre-devel
> Le 11/05/2016 03:17, Drokin, Oleg a ?crit :
> > On May 10, 2016, at 3:42 PM, Dilger, Andreas wrote:
> >
> > > On 2016/05/10, 08:14, "Quentin BOUGET" <quentin.bouget.ocre@cea.fr> wrote:
> > > > Hello!
> > > >
> > > > While trying to mount two different lustre filesystems on a local node
> > > > with the "llmount.sh" script, I noticed that the path to mount mds and
> > > > ost(s) was computed like this:
> > > >
> > > > eval mntpt=${!var:-${MOUNT%/*}/$facet} # var is not defined by default,
> > > > so this removes
> > > > #
> > > > everything after the last '/' of $MOUNT
> > > >
> > > >
> > > > The code can be found at test-framework.sh in the function "facet_mntpt"
> > > > around the line 3226.
> > > >
> > > >
> > > > By default MOUNT is initialized like this:
> > > >
> > > > MOUNT=${MOUNT:-/mnt/${FSNAME}} # default: "/mnt/lustre"
> > > >
> > > >
> > > > Thus the default mountpoints for mds and ost(s) are "/mnt/mds1" and
> > > > "/mnt/ost1".
> > > > Because of that, when trying to do two successive calls to "llmount.sh"
> > > > with two different values for FSNAME, the second fs to be mounted will
> > > > unmount the first's mds and ost(s) (paths are identical).
> > > >
> > > >
> > > > Is there any reason to truncate the MOUNT variable like this ? Shouldn't
> > > > the mountpoint path look more like "/mnt/<fsname>-mds1" and
> > > > "/mnt/<fsname>-ost1"?
> > > I don't think there is any particular reason it was implemented this way,
> > > just that it has worked for many years without the need to include the
> > > fsname into the MDT/OST mount points... :-)
> > >
> > > If you want to expand it to allow exporting multiple filesystems from the
> > > same MDS/OSS nodes then your suggestion to use $FSNAME-$facet in
> > > facet_mntpt() is reasonable. Please submit a patch to verify this is
> > > working correctly. There may be some places in the tests that aren't
> > > using facet_mntpt() yet.
> > I guess to expand on this, since llmount.sh is primarily a testing
> > infrastructure and typically only one filesystem at a time is tested,
> > the more than one mountpoint was never bothering anyone till you
> > discovered this bit, I guess.
> >
> > Bye,
> > Oleg
> >
> >
> Ok, thank you. I will submit a patch about it.
Please post a link to the JIRA ticket here. Thanks for the work.
> Bye,
> Quentin
> _______________________________________________
> lustre-devel mailing list
> lustre-devel at lists.lustre.org
> http://lists.lustre.org/listinfo.cgi/lustre-devel-lustre.org
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [lustre-devel] llmount.sh does not use fs-specific paths to the ost(s) and mds it mounts
2016-05-11 15:58 ` James Simmons
@ 2016-05-12 7:17 ` Quentin BOUGET
0 siblings, 0 replies; 6+ messages in thread
From: Quentin BOUGET @ 2016-05-12 7:17 UTC (permalink / raw)
To: lustre-devel
Le 11/05/2016 17:58, James Simmons a ?crit :
>> Le 11/05/2016 03:17, Drokin, Oleg a ?crit :
>>> On May 10, 2016, at 3:42 PM, Dilger, Andreas wrote:
>>>
>>>> On 2016/05/10, 08:14, "Quentin BOUGET" <quentin.bouget.ocre@cea.fr> wrote:
>>>>> Hello!
>>>>>
>>>>> While trying to mount two different lustre filesystems on a local node
>>>>> with the "llmount.sh" script, I noticed that the path to mount mds and
>>>>> ost(s) was computed like this:
>>>>>
>>>>> eval mntpt=${!var:-${MOUNT%/*}/$facet} # var is not defined by default,
>>>>> so this removes
>>>>> #
>>>>> everything after the last '/' of $MOUNT
>>>>>
>>>>>
>>>>> The code can be found at test-framework.sh in the function "facet_mntpt"
>>>>> around the line 3226.
>>>>>
>>>>>
>>>>> By default MOUNT is initialized like this:
>>>>>
>>>>> MOUNT=${MOUNT:-/mnt/${FSNAME}} # default: "/mnt/lustre"
>>>>>
>>>>>
>>>>> Thus the default mountpoints for mds and ost(s) are "/mnt/mds1" and
>>>>> "/mnt/ost1".
>>>>> Because of that, when trying to do two successive calls to "llmount.sh"
>>>>> with two different values for FSNAME, the second fs to be mounted will
>>>>> unmount the first's mds and ost(s) (paths are identical).
>>>>>
>>>>>
>>>>> Is there any reason to truncate the MOUNT variable like this ? Shouldn't
>>>>> the mountpoint path look more like "/mnt/<fsname>-mds1" and
>>>>> "/mnt/<fsname>-ost1"?
>>>> I don't think there is any particular reason it was implemented this way,
>>>> just that it has worked for many years without the need to include the
>>>> fsname into the MDT/OST mount points... :-)
>>>>
>>>> If you want to expand it to allow exporting multiple filesystems from the
>>>> same MDS/OSS nodes then your suggestion to use $FSNAME-$facet in
>>>> facet_mntpt() is reasonable. Please submit a patch to verify this is
>>>> working correctly. There may be some places in the tests that aren't
>>>> using facet_mntpt() yet.
>>> I guess to expand on this, since llmount.sh is primarily a testing
>>> infrastructure and typically only one filesystem at a time is tested,
>>> the more than one mountpoint was never bothering anyone till you
>>> discovered this bit, I guess.
>>>
>>> Bye,
>>> Oleg
>>>
>>>
>> Ok, thank you. I will submit a patch about it.
> Please post a link to the JIRA ticket here. Thanks for the work.
>
>> Bye,
>> Quentin
>> _______________________________________________
>> lustre-devel mailing list
>> lustre-devel at lists.lustre.org
>> http://lists.lustre.org/listinfo.cgi/lustre-devel-lustre.org
Here is a link to the JIRA ticket:
https://jira.hpdd.intel.com/browse/LU-8132
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2016-05-12 7:17 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <5731EAC3.5070207@cea.fr>
2016-05-10 14:14 ` [lustre-devel] Fwd: llmount.sh does not use fs-specific paths to the ost(s) and mds it mounts Quentin BOUGET
2016-05-10 19:42 ` Dilger, Andreas
2016-05-11 1:17 ` [lustre-devel] " Drokin, Oleg
2016-05-11 7:07 ` Quentin BOUGET
2016-05-11 15:58 ` James Simmons
2016-05-12 7:17 ` Quentin BOUGET
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.