Bug 1304824

Summary: [Scale] UI blocked with spinner for 2-3 minutes during long running Storage Attachment
Product: [oVirt] ovirt-engine Reporter: mlehrer
Component: Frontend.WebAdminAssignee: Tal Nisan <tnisan>
Status: CLOSED CURRENTRELEASE QA Contact: guy chen <guchen>
Severity: high Docs Contact:
Priority: unspecified    
Version: 3.6.2.5CC: acanan, amureini, bugs, eberman, mlehrer, rgolan, tnisan, ylavi
Target Milestone: ovirt-4.1.0-betaKeywords: Performance
Target Release: ---Flags: amureini: ovirt-4.1?
ylavi: exception?
ylavi: planning_ack+
rule-engine: devel_ack?
rule-engine: testing_ack?
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-01-09 17:21:20 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Storage RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
UI blocked none

Description mlehrer 2016-02-04 17:31:37 UTC
Created attachment 1121170 [details]
UI blocked

Description of problem:

When an ovirt admin attempts to execute a long running storage operation such as adding a new domain or activate an existing large storage domain, the admin user is blocked for up to 2.5 - 3.5 minutes by a spinner in a dialog box.

Version-Release number of selected component (if applicable):


How reproducible:
Very

Steps to Reproduce:
1. Create a large data ISCSI SD of multiple luns (tested with 7, 16, and 40+)
2. Click Attach from Web UI
3. User is blocked from 2-4+ mins

Actual results:
User is blocked from 2-4+ mins

Expected results:
That the operation occur in the background and the admin can continue to browse/use the web admin.




Additional info:

Comment 1 mlehrer 2016-02-04 17:33:54 UTC
Marking it as [high] from UX/usability perspective, obviously if you wait the operation will complete, adjust severity as necessary.

Comment 3 Allon Mureinik 2016-02-08 13:21:12 UTC
Isn't this just a specific case of bug 1259353?

Comment 4 Tal Nisan 2016-02-09 15:18:13 UTC
Not sure since the problem in bug 1259353 is in the opening of the dialog and here it's in the saving, might be the same since in the other bug it's when editing the domain.
In any way further investigation is needed so I keep this one open

Comment 5 Allon Mureinik 2016-03-03 15:00:18 UTC
This has been this dialog's behavior since forever. If we're still investing in the current webadmin it's worth fixing, but it's not a good fit for a z-stream.

Comment 6 Yaniv Kaul 2016-11-21 12:49:32 UTC
Can you update with latest findings in logs (from your recent 4.1 tests)?

Comment 7 Roy Golan 2016-12-01 09:28:05 UTC
I'm quite sure this isn't the dialog or the UI but the engine command itself.

What the 'add storage' is doing specifically is to wait till AddStorageServerConnectionCommand is done, because the behaviour is that if the connection is failing (can't connect to the host or any other reason) the dialog remains presented, and letting you amend the path in case it is wrong etc

So the responsiveness here is bounded to how fast the engine will perform the connect command (can be also other things like db latency and so)

So we have a tradeoff: catch execution failures and leave the dialog in place to prevent retyping all details again, against quick dialog dismiss and let the user track the succeed/fail using the event tab

Comment 8 Roy Golan 2017-01-04 06:57:42 UTC
ping

Comment 9 guy chen 2017-01-09 14:47:51 UTC
I have tested the bug under 4.1 build 4 with SD with 10 luns, the symptoms does not reproduced, the spinner takes about 25 seconds and the UI is not blocked - can close the spinner and continue working.
Adding the relevant logs.