Bug 1306585 - Network Import dialogue will not close on Cancel
Summary: Network Import dialogue will not close on Cancel
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Frontend.WebAdmin
Version: 3.6.2
Hardware: Unspecified
OS: Unspecified
high
medium
Target Milestone: ovirt-3.6.5
: 3.6.5
Assignee: Marcin Mirecki
QA Contact: Petr Matyáš
URL:
Whiteboard:
: 1215709 1313191 (view as bug list)
Depends On:
Blocks: 1327382
TreeView+ depends on / blocked
 
Reported: 2016-02-11 11:34 UTC by Petr Matyáš
Modified: 2016-04-21 14:42 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-04-21 14:42:47 UTC
oVirt Team: Network
Embargoed:
rule-engine: ovirt-3.6.z+
ylavi: planning_ack+
danken: devel_ack+
rule-engine: testing_ack+


Attachments (Terms of Use)
dialogue screenshot (35.07 KB, image/png)
2016-02-11 11:34 UTC, Petr Matyáš
no flags Details
engine log (180.47 KB, text/plain)
2016-02-11 11:35 UTC, Petr Matyáš
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 53479 0 master MERGED webadmin: Network Import dialog open/cancel fails 2016-02-29 07:29:04 UTC
oVirt gerrit 54279 0 ovirt-engine-3.6 MERGED webadmin: Network Import dialog open/cancel fails 2016-03-10 08:07:17 UTC

Description Petr Matyáš 2016-02-11 11:34:05 UTC
Created attachment 1123124 [details]
dialogue screenshot

Description of problem:
When I open network import dialogue and then click on cancel, nothing happens. Also when I close this dialogue with close button on top right I can't open any more dialogues.

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

How reproducible:
always

Steps to Reproduce:
1. go to networks tab in webadmin
2. click on import button
3. click on Cancel button

Actual results:
nothing happens

Expected results:
dialogue closes

Additional info:
In console log I can see only too much recursion message with:
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892
EventLoop.prototype.enter()
 script.js:347
ThreadActor.prototype.unsafeSynchronize()
 script.js:1085
ThreadActor.prototype._addSource()
 script.js:1945
ThreadActor.prototype.onNewScript()
 script.js:1892

And in engine.log only this message seems relevant:
2016-02-11 12:30:20,536 WARN  [org.ovirt.engine.core.bll.hostdeploy.VdsDeployBase] (VdsDeploy) [] Cannot validate host name settings, reason: resolved host does not match any of the local addresses

Comment 1 Petr Matyáš 2016-02-11 11:35:24 UTC
Created attachment 1123126 [details]
engine log

Comment 2 Red Hat Bugzilla Rules Engine 2016-02-12 09:46:32 UTC
Bug tickets must have version flags set prior to targeting them to a release. Please ask maintainer to set the correct version flags and only then set the target milestone.

Comment 3 Marcin Mirecki 2016-02-12 15:39:28 UTC
One more problem was that the dialog could only be opened once. When closed (using the dialog x icon, which worked), it would never open again.
Related is the external network provider network dialog (same code), which should also be checked.

Comment 4 Dan Kenigsberg 2016-03-02 09:32:04 UTC
*** Bug 1313191 has been marked as a duplicate of this bug. ***

Comment 5 Dan Kenigsberg 2016-03-02 09:33:06 UTC
(moving back to POST, as a backport is required)

Comment 6 Michael Burman 2016-03-02 09:35:16 UTC
Dan

Please note that the problems starting once pressing the 'Import' button. from here many parts of the UI will stop working(including the 'Cancel' button)

Comment 7 Marcin Mirecki 2016-03-07 08:57:21 UTC
*** Bug 1215709 has been marked as a duplicate of this bug. ***

Comment 8 Petr Matyáš 2016-03-31 08:59:21 UTC
Verified on 3.6.5-1


Note You need to log in before you can comment on or make changes to this bug.