Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1248090

Summary: Tasks drawer doesn't display jobs/step tree correctly
Product: [Retired] oVirt Reporter: Shmuel Melamud <smelamud>
Component: ovirt-engine-webadminAssignee: Moti Asayag <masayag>
Status: CLOSED CURRENTRELEASE QA Contact: Petr Kubica <pkubica>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 3.6CC: amureini, bugs, ecohen, gklein, lsurette, masayag, mgoldboi, oourfali, pkubica, rbalakri, smelamud, yeylon
Target Milestone: ---   
Target Release: 3.6.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: infra
Fixed In Version: 3.6.0-12 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-11-04 11:25:53 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Infra RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Shmuel Melamud 2015-07-29 15:01:25 UTC
Description of problem:

"Tasks" drawer at the bottom of the webadmin should display tree of jobs, steps and substeps. But currently, it displays substeps several times - once in the correct place in the tree and also on all previous levels. The corresponding structure in the DB is correct.

Also, when user click on "+" to open a branch, it closes shortly, so it's practically impossible to open the whole tree.

How reproducible:
100%

Steps to Reproduce:
1. Create a VM pool with one or several VMs.
2. Open the "Tasks" drawer at the bottom.

Actual results:
The drawer shows steps and substeps mixed. The topmost job "Creating VM Pool" contains several Validating, Executing and Finalizing steps and so on.

Expected results:
The drawer shows the correct jobs/steps tree.

Additional info:

Comment 1 Oved Ourfali 2015-07-30 07:00:31 UTC
Shmuel - I guess you refer to recursive hierarchy?
Moti - do we support recursiveness there?

Comment 2 Shmuel Melamud 2015-07-30 10:17:13 UTC
(In reply to Oved Ourfali from comment #1)
> Shmuel - I guess you refer to recursive hierarchy?

Yes.

Comment 3 Moti Asayag 2015-08-02 10:53:52 UTC
(In reply to Oved Ourfali from comment #1)
> Shmuel - I guess you refer to recursive hierarchy?
> Moti - do we support recursiveness there?

yes, the steps are implemented as a composite pattern - each step may have sub steps.

There were few changes around the fetch method of sub-steps by commit:
92dca0ea7fff25c8e8e8e509241df1158928e180

Is it possible to verify the same scenario before that commit was introduced ?

Comment 4 Oved Ourfali 2015-08-05 05:43:35 UTC
Liran - please sync with Moti on that one, and check it out.

Comment 5 Petr Kubica 2015-09-15 08:38:35 UTC
Verified in 3.6.0-0.15.master.el6

Comment 6 Sandro Bonazzola 2015-11-04 11:25:53 UTC
oVirt 3.6.0 has been released on November 4th, 2015 and should fix this issue.
If problems still persist, please open a new BZ and reference this one.