Bug 878943 - engine: no task id in engine log for revert task
Summary: engine: no task id in engine log for revert task
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-engine
Version: unspecified
Hardware: x86_64
OS: Linux
high
high
Target Milestone: ---
: 3.2.0
Assignee: Dafna Ron
QA Contact: Dafna Ron
URL:
Whiteboard: infra
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-11-21 15:41 UTC by Dafna Ron
Modified: 2016-02-10 19:32 UTC (History)
11 users (show)

Fixed In Version: sf4
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-06-11 09:18:21 UTC
oVirt Team: Infra
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
logs (696.94 KB, application/x-gzip)
2012-11-21 15:41 UTC, Dafna Ron
no flags Details


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 10804 0 None None None Never

Description Dafna Ron 2012-11-21 15:41:17 UTC
Created attachment 649280 [details]
logs

Description of problem:

2012-11-21 17:05:27,297 ERROR [org.ovirt.engine.core.bll.lsm.LiveMigrateDiskCommand] (pool-3-thread-29) Reverting task handler: org.ovirt.engine.core.bll.lsm.VmReplicateDiskSta
rtTaskHandler


there is no task ID which will be helpful in debugging whicg task failed and reverted - especially when we have multiple tasks sent.  

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

si24.4

How reproducible:

100%

Steps to Reproduce:
1. live migrate several disks at the same time -> this currently cause some of the tasks to fail. 
2.
3.
  
Actual results:

we get a revert task with no task id


Expected results:

user will find it difficult to know which task failed if multiple tasks are sent


Additional info:logs

Comment 1 Yair Zaslavsky 2012-12-30 10:40:27 UTC
This requires infra change to TaskHandler mechanism.
Not sure we will make it for 3.2 timeframe.
The information can be deduced from async_tasks table by querying
select task_type from async_tasks where command_id = <command_id_of_live_storage_migration>.
Suggesting this for future.

Comment 3 Eli Mesika 2013-01-09 10:48:18 UTC
http://gerrit.ovirt.org/#/c/10804/

Comment 4 Eli Mesika 2013-01-10 12:31:55 UTC
fixed in commit: 3def841

Comment 5 Dafna Ron 2013-02-05 10:59:27 UTC
verified on sf5

2013-02-03 16:33:28,816 INFO  [org.ovirt.engine.core.bll.lsm.CreateImagePlaceholderTaskHandler] (pool-3-thread-44) [46464d54] Reverting task deleteImage with ID 933ec268-d720-4989-a6f0-15df5dc84953 on DataCenter b08d859c-f654-4976-8c7c-
e102d4c252fb.

Comment 6 Itamar Heim 2013-06-11 09:18:21 UTC
3.2 has been released

Comment 7 Itamar Heim 2013-06-11 09:42:19 UTC
3.2 has been released


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