[PATCH 4/4 rhel7-branch] gui: add Refresh button to network storage UI (#1065716)
Radek Vykydal
rvykydal at redhat.com
Wed Sep 17 07:46:45 UTC 2014
On 09/16/2014 08:10 PM, Vratislav Podzimek wrote:
> On Tue, 2014-09-16 at 14:51 +0200, Radek Vykydal wrote:
>> On 09/16/2014 09:39 AM, Radek Vykydal wrote:
>>> Resolves: rhbz#1065716
>>>
>>> Add Refresh button to advanced storage (filter) screen. The use case is
>>> slow connection to FCoE SAN where our 10s sleep is not enough. This used
>>> to be worked around by adding the FCoE LUN for second time in UI so that the
>>> devicetree is repopulated once more after another 10s (or more) sleep [1].
>>>
>>> Ideally we would refresh the screen (blivet devicetree) based on an event of
>>> new FCoE SAN devices appearing. But it is not happening in near future afaik.
>>> Even some kind of polling for new device would be quite hard to implement
>>> correctly.
>>>
>>> So ... the Refresh button.
>>> Should we add a tooltip?
>>> Should we make it less prominent?
>>>
>>> [1] Details:
>>>
>>> Note 10s sleep after fipvlan returns successfully which is not enough for
>>> device for LUN connected to p2p1 to appear (you can ignore p2p2). Devicetree is
>>> repopulated with new fcoe device only after the discovery on p2p1 for the
>>> second time.
>>>
>>> 12:11:59,162 INFO anaconda: Running Thread: AnaFCOEThread (140047382103808)
>>> 12:11:59,162 INFO blivet: Activating FCoE SAN attached to p2p1, dcb: False autovlan: True
>>> 12:11:59,200 INFO program: Running... fipvlan -c -s -f -fcoe p2p1
>>> 12:12:03,207 INFO program: Created VLAN device p2p1.802-fcoe
>>> 12:12:03,207 INFO program: Starting FCoE on interface p2p1.802-fcoe
>>> 12:12:13,231 INFO anaconda: Thread Done: AnaFCOEThread (140047382103808)
>>> 12:12:13,235 INFO blivet: DeviceTree.populate: ignoredDisks is [] ; exclusiveDisks is []
>>> 12:14:25,264 INFO anaconda: Running Thread: AnaFCOEThread (140047382103808)
>>> 12:14:25,264 INFO blivet: Activating FCoE SAN attached to p2p2, dcb: False autovlan: True
>>> 12:14:25,277 INFO program: Running... fipvlan -c -s -f -fcoe p2p2
>>> 12:14:29,279 INFO program: Created VLAN device p2p2.802-fcoe
>>> 12:14:29,279 INFO program: Starting FCoE on interface p2p2.802-fcoe
>>> 12:14:39,301 INFO anaconda: Thread Done: AnaFCOEThread (140047382103808)
>>> 12:14:39,305 INFO blivet: DeviceTree.populate: ignoredDisks is [] ; exclusiveDisks is []
>>> 12:15:51,411 INFO anaconda: Running Thread: AnaFCOEThread (140047382103808)
>>> 12:15:51,411 INFO blivet: Activating FCoE SAN attached to p2p1, dcb: False autovlan: True
>>> 12:15:51,424 INFO program: Running... fipvlan -c -s -f -fcoe p2p1
>>> 12:15:52,161 INFO program: Starting FCoE on interface p2p1.802-fcoe
>>> 12:16:02,183 INFO anaconda: Thread Done: AnaFCOEThread (140047382103808)
>>> 12:16:02,187 INFO blivet: DeviceTree.populate: ignoredDisks is [] ; exclusiveDisks is []
>>> 12:16:02,294 INFO blivet: scanning sdb (/devices/virtual/net/p2p1.802-fcoe/ctlr_0/host7/rport-7:0-5/target7:0:2/7:0:2:0/block/sdb)...
>>> 12:16:02,297 WARN blivet: unexpected sysfs_path of bnx2fc device: /devices/virtual/net/p2p1.802-fcoe/ctlr_0/host7/rport-7:0-5/target7:0:2/7:0:2:0/block/sdb
>>> 12:16:02,297 INFO blivet: sdb is an fcoe disk
>>> ---
>>> pyanaconda/ui/gui/spokes/filter.glade | 15 +++++++++++++++
>>> pyanaconda/ui/gui/spokes/filter.py | 7 ++++++-
>>> 2 files changed, 21 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/pyanaconda/ui/gui/spokes/filter.glade b/pyanaconda/ui/gui/spokes/filter.glade
>>> index ff0426d..d8569fb 100644
>>> --- a/pyanaconda/ui/gui/spokes/filter.glade
>>> +++ b/pyanaconda/ui/gui/spokes/filter.glade
>>> @@ -1668,6 +1668,21 @@
>>> <property name="position">2</property>
>>> </packing>
>>> </child>
>>> + <child>
>>> + <object class="GtkButton" id="refresh">
>>> + <property name="label" translatable="yes">Refres_h</property>
>>>
>> Damn, the accelerator collides with Searc_h and all letters from Refresh
>> are used.
> Can't we change "Searc_h" to something else instead?
>
I was thinking about this option too ('r' and 'c' are free), but
I was hesitant to propose to change it after it was already released and
perhaps even used?
More information about the anaconda-patches
mailing list