[Kimchi-devel] [PATCH v4] [Kimchi] Storage Volume management

Tue May 24 14:39:58 UTC 2016

On 05/24/2016 10:27 AM, Samuel Henrique De Oliveira Guimaraes wrote:
>
> -----Original Message-----
> From: kimchi-devel-bounces at ovirt.org [mailto:kimchi-devel-bounces at ovirt.org] On Behalf Of Aline Manera
> Sent: segunda-feira, 23 de maio de 2016 17:41
> To: sguimaraes943 at gmail.com; Kimchi Devel <kimchi-devel at ovirt.org>
> Subject: Re: [Kimchi-devel] [PATCH v4] [Kimchi] Storage Volume management
>
> Hi Samuel,
>
> Some comments:
>
> 1. When I switch to the "Gallery view" I can see all volumes selected and after one second, they are deselected
>
> Hi. I tried to reproduce this here but it is working fine. Tested with Firefox and Chrome and watched for DOM changes and after a minute the checkboxes were still selected. Even If I run an asynchronous task in another Storage Pool it won't change the volumes in another Storage Pool.

I am saying, when you are in the storage volumes list view without any 
storage volume selected and switch to the gallery view, all the storage 
volumes appear selected for a second and then the page is refreshed to 
deselect all them.

> 2. When you select multiple volumes to perform an action, there is no feedback to user that an operation will be done on those selected volumes.
>       I can select others, deselect the ones I have selected before and so. We should block the volume selected, add a loading icon, for example, and a mark check when the operation is completed.
>
> I think my machine is running these tasks too fast so that's why I didn't foresaw this as a requirement. I'll disable the checkboxes and add the spinner icon.
>
> 3. Usually we don't use success messages on Wok and its plugins. Any reason to add them now?
>
> I've added some to System Services and OVS Bridges there was no feedback to the user pointing a change in the tables or that an action was being performed but with multiple selection sometimes it may render multiple messages in the area so I'll remove them.
>
> 4. The message are duplicated. I can see 2 messages when trying to delete multiple items "Volume selected were deleted" + "Volume X was successfully deleted".
>
> See above.
>
> 5. The same messages for delete operation are shown when I wipe volumes.
>
> See above.
>
> 6. The wipe operation will change the allocation value and so it must be updated in the volume box.
>
> You mean the Storage Pool line in the parent table? Once the wipe process is completed the line is updated with 0.0B in the allocated column, however the parent %Used and Allocated columns are not refreshed. Are these two the only fields that should be updated? Should it be done for Clone and Delete as well?

I was talking about the storage volume settings. When I do wipe multiple 
volumes, only one of them get the settings updated.

About what you said, I agree we should update the storage pool settings 
as new volumes or deleting volumes may change the storage pool 
allocation. I think only %used and allocated must be updated.

> 7. The scroll bar should only scroll the volumes. It is hard to get back all time to have the Actions menu available.
>
> I'm ok with this for the list view appending a scrollbar on the "table" element but with the Gallery view I think it looks odd. See attached screenshots.

IMO It is OK for me on both views (list and gallery).

Do you have any other idea to have the same behavior on gallery view 
which does not look odd as you said?

>
> I think that is all from my side.
>
> Regards,
> Aline Manera
>
> On 05/23/2016 12:46 PM, sguimaraes943 at gmail.com wrote:
>> From: Samuel Guimarães <sguimaraes943 at gmail.com>
>>
>> This patch adds Storage Volume management functions Wipe, Clone, Resize and Delete with multiple selection. It also includes a filter input for each Storage Pool and Gallery View for Storage Volumes.
>>
>> Changes from [RFC] version:
>>
>> v1:
>>    - HTML and CSS
>>
>> v2:
>>    - Delete and Wipe with multi-selection
>>    - Confirm messages with list of selected volumes when wiping or deleting volumes (requires SCSS/CSS patch sent to Wok)
>>    - Filter working
>>    - Removed "Add Volume" link from Storage Pool action button
>>    - Added "Add Volume" to Volume box action button
>>
>> v3:
>>    - Clone function working with multiple selection
>>    - Progress bar working for clone and create volume
>>    - Temporary volume added to the volumes when cloning
>>    - Seamless refresh on the volumes once each task is finished
>>    - Fixed issue when list wouldn't refresh when all volumes are removed from the storage pool.
>>
>> v4:
>>    - Prevent scroll when Drop-down in volumes list is clicked
>>    - Dropdown is not clipped from volumes list when there's only one or two items on the list
>>    - Added Media Queries for small screen resolutions
>>
>> Samuel Guimarães (1):
>>     Storage Volume management
>>
>>    model/storagevolumes.py                            |   2 +-
>>    ui/css/kimchi.css                                  | 369 ++++++++++++++--
>>    ui/css/src/modules/_storage.scss                   | 347 ++++++++++++---
>>    ui/js/src/kimchi.api.js                            |  50 +++
>>    ui/js/src/kimchi.storage_main.js                   | 465 +++++++++++++++++----
>>    ui/js/src/kimchi.storagepool_add_volume_main.js    |   2 +-
>>    ui/js/src/kimchi.storagepool_resize_volume_main.js |  59 +++
>>    ui/pages/i18n.json.tmpl                            |   6 +
>>    ui/pages/storagepool-resize-volume.html.tmpl       |  51 +++
>>    ui/pages/tabs/storage.html.tmpl                    | 155 +++----
>>    10 files changed, 1249 insertions(+), 257 deletions(-)
>>    create mode 100644 ui/js/src/kimchi.storagepool_resize_volume_main.js
>>    create mode 100644 ui/pages/storagepool-resize-volume.html.tmpl
>>
> _______________________________________________
> Kimchi-devel mailing list
> Kimchi-devel at ovirt.org
> http://lists.ovirt.org/mailman/listinfo/kimchi-devel