Re: [PATCH] Skip the Filter UI in Basic Storage mode (#598420)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On 07/22/2010 11:37 PM, David Cantrell wrote:
On Thu, 22 Jul 2010, Steffen Maier wrote:

On 07/22/2010 05:02 PM, Hans de Goede wrote:
---
pyanaconda/iw/filter_gui.py | 32 ++++++++------------------------
pyanaconda/iw/filter_type.py | 7 ++++---
2 files changed, 12 insertions(+), 27 deletions(-)


diff --git a/pyanaconda/iw/filter_type.py b/pyanaconda/iw/filter_type.py
index 940de7e..2b4e5d9 100644
--- a/pyanaconda/iw/filter_type.py
+++ b/pyanaconda/iw/filter_type.py
@@ -65,9 +65,10 @@ class FilterTypeWindow(InstallWindow):
"probably it."))
self.buttonGroup.addEntry("complex", _("Specialized Storage Devices"),
descr=_("Installs or upgrades to devices such as "
- "Storage Area Networks (SANs) or mainframe "
- "attached disks (DASD), usually in an "
- "enterprise environment"))
+ "Storage Area Networks (SANs). This option "

Now, I'm missing DASDs which are not necessarily understood under the
term of SAN since they do not use FCP.
Also the term "enterprise environment" seemed useful to me.

Some text noting enterprise storage might be useful here.


Added.

+ "will allow you to add FCoE / iSCSI / zFCP "
+ "disks and to filter out LUNs the "

Isn't the dialog about filtering disks, i.e. not just LUNs? After all,
it can filter DASDs which are usually not referred to as LUNs.
Therefore, I would like to suggest to make the wording more general:
"disks and to filter out disks the "

I would suggest "devices" rather than disks or LUNs.


Changed and committed.

Regards,

Hans

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list


[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux