[Engine-devel] Advanced Nfs Options + NFSv4 GUI mockup

Please review the mock-up on the feature page: http://www.ovirt.org/wiki/Features/AdvancedNfsOptions#User_Experience Comments are welcome. ---- Thanks, Einav

Hi, On 05/21/2012 12:34 PM, Einav Cohen wrote:
Please review the mock-up on the feature page: http://www.ovirt.org/wiki/Features/AdvancedNfsOptions#User_Experience
Comments are welcome.
i was wondering why you have pulled out retrans and timeout? Are these the most important ones? Regards, Geert

----- Original Message ----- From: "Geert Jansen" <gjansen@redhat.com> Sent: Monday, May 21, 2012 1:40:40 PM
Hi,
On 05/21/2012 12:34 PM, Einav Cohen wrote:
Please review the mock-up on the feature page: http://www.ovirt.org/wiki/Features/AdvancedNfsOptions#User_Experience
Comments are welcome.
i was wondering why you have pulled out retrans and timeout? Are these the most important ones?
Sorry - not sure I got you. Can you please clarify your question?
Regards, Geert _______________________________________________ Engine-devel mailing list Engine-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel

On 05/21/2012 12:44 PM, Einav Cohen wrote:
i was wondering why you have pulled out retrans and timeout? Are these the most important ones?
Sorry - not sure I got you. Can you please clarify your question?
Sorry, I meant, why do retrans and timeout have their own input boxes, while the other options need to be set via the 'mount options' box. There's quite a few other important nfs options like tcp, rsize, wsize, and possibly nointr and hard. I was wondering why we took these specific two and gave them their separate boxes but not the other ones. Regards, Geert

----- Original Message ----- From: "Geert Jansen" <gjansen@redhat.com> Sent: Monday, May 21, 2012 2:44:09 PM
On 05/21/2012 12:44 PM, Einav Cohen wrote:
i was wondering why you have pulled out retrans and timeout? Are these the most important ones?
Sorry - not sure I got you. Can you please clarify your question?
Sorry, I meant, why do retrans and timeout have their own input boxes, while the other options need to be set via the 'mount options' box. There's quite a few other important nfs options like tcp, rsize, wsize, and possibly nointr and hard. I was wondering why we took these specific two and gave them their separate boxes but not the other ones.
Retrans and Timeout got separate boxes since they have separate parameters in the api (GUI was determined by api). Ayal - any reason for these specific two values having parameters of their own?
Regards, Geert _______________________________________________ Engine-devel mailing list Engine-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel

On 21/05/12 14:49, Einav Cohen wrote:
----- Original Message ----- From: "Geert Jansen" <gjansen@redhat.com> Sent: Monday, May 21, 2012 2:44:09 PM
On 05/21/2012 12:44 PM, Einav Cohen wrote:
i was wondering why you have pulled out retrans and timeout? Are these the most important ones?
Sorry - not sure I got you. Can you please clarify your question?
Sorry, I meant, why do retrans and timeout have their own input boxes, while the other options need to be set via the 'mount options' box. There's quite a few other important nfs options like tcp, rsize, wsize, and possibly nointr and hard. I was wondering why we took these specific two and gave them their separate boxes but not the other ones.
Retrans and Timeout got separate boxes since they have separate parameters in the api (GUI was determined by api). Ayal - any reason for these specific two values having parameters of their own?
Reading some of the thread and reviews I can say that these 3 options (version, retrans and timeo) are advanced NFS features, which in most cases should not be used at all. VDSM has very good defaults, and in some specific cases there's a need to change these defaults. Only in such case these options should be modified.
Regards, Geert _______________________________________________ Engine-devel mailing list Engine-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel
_______________________________________________ Engine-devel mailing list Engine-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel
-- /d "Hi, my name is Any Key. Please don't hit me!"

----- Original Message ----- From: "Doron Fediuck" <dfediuck@redhat.com> Sent: Monday, May 21, 2012 3:59:07 PM
On 21/05/12 14:49, Einav Cohen wrote:
----- Original Message ----- From: "Geert Jansen" <gjansen@redhat.com> Sent: Monday, May 21, 2012 2:44:09 PM
On 05/21/2012 12:44 PM, Einav Cohen wrote:
i was wondering why you have pulled out retrans and timeout? Are these the most important ones?
Sorry - not sure I got you. Can you please clarify your question?
Sorry, I meant, why do retrans and timeout have their own input boxes, while the other options need to be set via the 'mount options' box. There's quite a few other important nfs options like tcp, rsize, wsize, and possibly nointr and hard. I was wondering why we took these specific two and gave them their separate boxes but not the other ones.
Retrans and Timeout got separate boxes since they have separate parameters in the api (GUI was determined by api). Ayal - any reason for these specific two values having parameters of their own?
Reading some of the thread and reviews I can say that these 3 options (version, retrans and timeo) are advanced NFS features, which in most cases should not be used at all. VDSM has very good defaults, and in some specific cases there's a need to change these defaults. Only in such case these options should be modified.
No question that vdsm has very good defaults and only rarely should they be modified. However, why not change everything via "mount options"? why do we have separate parameters specifically for "retrans" and "timeout" (in addition to "mount options")? As Geert said, we have also tcp, rsize, wsize, etc. So why "retrans" and "timeout" got parameters of their own and the others didn't (i.e. if there is a need to change tcp, rsize, wsize, the only way to do it is via the generic "mount options")?
Regards, Geert _______________________________________________ Engine-devel mailing list Engine-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel
_______________________________________________ Engine-devel mailing list Engine-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel
--
/d
"Hi, my name is Any Key. Please don't hit me!" _______________________________________________ Engine-devel mailing list Engine-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel

On 21/05/12 16:09, Einav Cohen wrote:
----- Original Message ----- From: "Doron Fediuck" <dfediuck@redhat.com> Sent: Monday, May 21, 2012 3:59:07 PM
On 21/05/12 14:49, Einav Cohen wrote:
----- Original Message ----- From: "Geert Jansen" <gjansen@redhat.com> Sent: Monday, May 21, 2012 2:44:09 PM
On 05/21/2012 12:44 PM, Einav Cohen wrote:
i was wondering why you have pulled out retrans and timeout? Are these the most important ones?
Sorry - not sure I got you. Can you please clarify your question?
Sorry, I meant, why do retrans and timeout have their own input boxes, while the other options need to be set via the 'mount options' box. There's quite a few other important nfs options like tcp, rsize, wsize, and possibly nointr and hard. I was wondering why we took these specific two and gave them their separate boxes but not the other ones.
Retrans and Timeout got separate boxes since they have separate parameters in the api (GUI was determined by api). Ayal - any reason for these specific two values having parameters of their own?
Reading some of the thread and reviews I can say that these 3 options (version, retrans and timeo) are advanced NFS features, which in most cases should not be used at all. VDSM has very good defaults, and in some specific cases there's a need to change these defaults. Only in such case these options should be modified.
No question that vdsm has very good defaults and only rarely should they be modified. However, why not change everything via "mount options"? why do we have separate parameters specifically for "retrans" and "timeout" (in addition to "mount options")? As Geert said, we have also tcp, rsize, wsize, etc. So why "retrans" and "timeout" got parameters of their own and the others didn't (i.e. if there is a need to change tcp, rsize, wsize, the only way to do it is via the generic "mount options")? I think it has to do w/ the vdsm implementation of it, maybe also connection management in the backend. But I think Livnat may have some better understandings of that.
Regards, Geert _______________________________________________ Engine-devel mailing list Engine-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel
_______________________________________________ Engine-devel mailing list Engine-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel
--
/d
"Hi, my name is Any Key. Please don't hit me!" _______________________________________________ Engine-devel mailing list Engine-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel
-- /d Why doesn't DOS ever say "EXCELLENT command or filename!"

On 05/21/2012 04:09 PM, Einav Cohen wrote:
----- Original Message ----- From: "Doron Fediuck"<dfediuck@redhat.com> Sent: Monday, May 21, 2012 3:59:07 PM
On 21/05/12 14:49, Einav Cohen wrote:
----- Original Message ----- From: "Geert Jansen"<gjansen@redhat.com> Sent: Monday, May 21, 2012 2:44:09 PM
On 05/21/2012 12:44 PM, Einav Cohen wrote:
i was wondering why you have pulled out retrans and timeout? Are these the most important ones? Sorry - not sure I got you. Can you please clarify your question? Sorry, I meant, why do retrans and timeout have their own input boxes, while the other options need to be set via the 'mount options' box. There's quite a few other important nfs options like tcp, rsize, wsize, and possibly nointr and hard. I was wondering why we took these specific two and gave them their separate boxes but not the other ones. Retrans and Timeout got separate boxes since they have separate parameters in the api (GUI was determined by api). Ayal - any reason for these specific two values having parameters of their own?
Reading some of the thread and reviews I can say that these 3 options (version, retrans and timeo) are advanced NFS features, which in most cases should not be used at all. VDSM has very good defaults, and in some specific cases there's a need to change these defaults. Only in such case these options should be modified. No question that vdsm has very good defaults and only rarely should they be modified. However, why not change everything via "mount options"? why do we have separate parameters specifically for "retrans" and "timeout" (in addition to "mount options")? As Geert said, we have also tcp, rsize, wsize, etc. So why "retrans" and "timeout" got parameters of their own and the others didn't (i.e. if there is a need to change tcp, rsize, wsize, the only way to do it is via the generic "mount options")?
While I agree they could be collapsed, practically, we've never heard of requests to any other parameters but those two, plus: - hard mount instead of soft. Probably not going to happen soon. Requires a lot more changes everywhere. - udp instead of tcp - bahhh.. We are in the 21st century, shouldn't be needed. But there are other params we may need in the future, which is why we have the additional options field. Y.
Regards, Geert _______________________________________________ Engine-devel mailing list Engine-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel
_______________________________________________ Engine-devel mailing list Engine-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel
--
/d
"Hi, my name is Any Key. Please don't hit me!" _______________________________________________ Engine-devel mailing list Engine-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel
_______________________________________________ Engine-devel mailing list Engine-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel

On 05/21/2012 01:34 PM, Einav Cohen wrote:
Please review the mock-up on the feature page: http://www.ovirt.org/wiki/Features/AdvancedNfsOptions#User_Experience
Comments are welcome.
Very minor comment - IMHO Retrans and Timeo should be bigger than 0.
---- Thanks, Einav _______________________________________________ Engine-devel mailing list Engine-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel

----- Original Message ----- From: "Yair Zaslavsky" <yzaslavs@redhat.com> Sent: Monday, May 21, 2012 1:49:57 PM
On 05/21/2012 01:34 PM, Einav Cohen wrote:
Please review the mock-up on the feature page: http://www.ovirt.org/wiki/Features/AdvancedNfsOptions#User_Experience
Comments are welcome.
Very minor comment - IMHO Retrans and Timeo should be bigger than 0.
Timeout - right. Retrans - not sure; Ayal?
---- Thanks, Einav _______________________________________________ Engine-devel mailing list Engine-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel
_______________________________________________ Engine-devel mailing list Engine-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel

Need to have units on the Retrans and Timout. Is tooltip will be available with the correct values? ----- Original Message -----
From: "Einav Cohen" <ecohen@redhat.com> To: "Ayal Baron" <abaron@redhat.com>, "Eldan Hildesheim" <ehildesh@redhat.com>, "Eldan Hildesheim" <info@eldanet.com>, "Miki Kenneth" <mkenneth@redhat.com>, "Andrew Cathrow" <acathrow@redhat.com>, "Simon Grinberg" <sgrinber@redhat.com>, "Alexey Chub" <achub@redhat.com> Cc: engine-devel@ovirt.org Sent: Monday, May 21, 2012 1:34:32 PM Subject: Advanced Nfs Options + NFSv4 GUI mockup
Please review the mock-up on the feature page: http://www.ovirt.org/wiki/Features/AdvancedNfsOptions#User_Experience
Comments are welcome.
---- Thanks, Einav

----- Original Message ----- From: "Miki Kenneth" <mkenneth@redhat.com> Sent: Monday, May 21, 2012 1:54:25 PM
Need to have units on the Retrans and Timout. Is tooltip will be available with the correct values?
Retrans doesn't have units. Timeout should be in seconds (Yair: backend api parameter is in seconds?) No need for a tool-tip - we can add it in parenthesis. mock-up updated.
----- Original Message -----
From: "Einav Cohen" <ecohen@redhat.com> To: "Ayal Baron" <abaron@redhat.com>, "Eldan Hildesheim" <ehildesh@redhat.com>, "Eldan Hildesheim" <info@eldanet.com>, "Miki Kenneth" <mkenneth@redhat.com>, "Andrew Cathrow" <acathrow@redhat.com>, "Simon Grinberg" <sgrinber@redhat.com>, "Alexey Chub" <achub@redhat.com> Cc: engine-devel@ovirt.org Sent: Monday, May 21, 2012 1:34:32 PM Subject: Advanced Nfs Options + NFSv4 GUI mockup
Please review the mock-up on the feature page: http://www.ovirt.org/wiki/Features/AdvancedNfsOptions#User_Experience
Comments are welcome.
---- Thanks, Einav
_______________________________________________ Engine-devel mailing list Engine-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel

----- Original Message -----
From: "Einav Cohen" <ecohen@redhat.com> To: "Miki Kenneth" <mkenneth@redhat.com>, "Yair Zaslavsky" <yzaslavs@redhat.com> Cc: "Eldan Hildesheim" <info@eldanet.com>, engine-devel@ovirt.org, "Simon Grinberg" <sgrinber@redhat.com>, "Eldan Hildesheim" <ehildesh@redhat.com> Sent: Monday, May 21, 2012 2:09:42 PM Subject: Re: [Engine-devel] Advanced Nfs Options + NFSv4 GUI mockup
----- Original Message ----- From: "Miki Kenneth" <mkenneth@redhat.com> Sent: Monday, May 21, 2012 1:54:25 PM
Need to have units on the Retrans and Timout. Is tooltip will be available with the correct values?
Retrans doesn't have units. i guess we need to explain something like # of retransmits? Timeout should be in seconds (Yair: backend api parameter is in seconds?) No need for a tool-tip - we can add it in parenthesis. mock-up updated.
----- Original Message -----
From: "Einav Cohen" <ecohen@redhat.com> To: "Ayal Baron" <abaron@redhat.com>, "Eldan Hildesheim" <ehildesh@redhat.com>, "Eldan Hildesheim" <info@eldanet.com>, "Miki Kenneth" <mkenneth@redhat.com>, "Andrew Cathrow" <acathrow@redhat.com>, "Simon Grinberg" <sgrinber@redhat.com>, "Alexey Chub" <achub@redhat.com> Cc: engine-devel@ovirt.org Sent: Monday, May 21, 2012 1:34:32 PM Subject: Advanced Nfs Options + NFSv4 GUI mockup
Please review the mock-up on the feature page: http://www.ovirt.org/wiki/Features/AdvancedNfsOptions#User_Experience
Comments are welcome.
---- Thanks, Einav
_______________________________________________ Engine-devel mailing list Engine-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel

----- Original Message ----- From: "Miki Kenneth" <mkenneth@redhat.com> Sent: Monday, May 21, 2012 2:12:12 PM
----- Original Message -----
From: "Einav Cohen" <ecohen@redhat.com> To: "Miki Kenneth" <mkenneth@redhat.com>, "Yair Zaslavsky" <yzaslavs@redhat.com> Cc: "Eldan Hildesheim" <info@eldanet.com>, engine-devel@ovirt.org, "Simon Grinberg" <sgrinber@redhat.com>, "Eldan Hildesheim" <ehildesh@redhat.com> Sent: Monday, May 21, 2012 2:09:42 PM Subject: Re: [Engine-devel] Advanced Nfs Options + NFSv4 GUI mockup
----- Original Message ----- From: "Miki Kenneth" <mkenneth@redhat.com> Sent: Monday, May 21, 2012 1:54:25 PM
Need to have units on the Retrans and Timout. Is tooltip will be available with the correct values?
Retrans doesn't have units. i guess we need to explain something like # of retransmits?
Updated mock-up. [I believe that nothing more is needed; people that define an NFS storage domain should generally know what retrans is, I think]
Timeout should be in seconds (Yair: backend api parameter is in seconds?) No need for a tool-tip - we can add it in parenthesis. mock-up updated.
----- Original Message -----
From: "Einav Cohen" <ecohen@redhat.com> To: "Ayal Baron" <abaron@redhat.com>, "Eldan Hildesheim" <ehildesh@redhat.com>, "Eldan Hildesheim" <info@eldanet.com>, "Miki Kenneth" <mkenneth@redhat.com>, "Andrew Cathrow" <acathrow@redhat.com>, "Simon Grinberg" <sgrinber@redhat.com>, "Alexey Chub" <achub@redhat.com> Cc: engine-devel@ovirt.org Sent: Monday, May 21, 2012 1:34:32 PM Subject: Advanced Nfs Options + NFSv4 GUI mockup
Please review the mock-up on the feature page: http://www.ovirt.org/wiki/Features/AdvancedNfsOptions#User_Experience
Comments are welcome.
---- Thanks, Einav
_______________________________________________ Engine-devel mailing list Engine-devel@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-devel
participants (6)
-
Doron Fediuck
-
Einav Cohen
-
Geert Jansen
-
Miki Kenneth
-
Yair Zaslavsky
-
Yaniv Kaul