Bug 987915 - Missing correlation-Id for adding new host
Summary: Missing correlation-Id for adding new host
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: oVirt
Classification: Retired
Component: ovirt-engine-core
Version: unspecified
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: m1
: 3.3.2
Assignee: Mooli Tayer
QA Contact:
URL:
Whiteboard: infra
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-07-24 12:12 UTC by Moti Asayag
Modified: 2013-12-19 14:24 UTC (History)
5 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2013-12-19 14:24:16 UTC
oVirt Team: ---
Embargoed:


Attachments (Terms of Use)
engine.log (523.34 KB, text/x-log)
2013-07-24 12:12 UTC, Moti Asayag
no flags Details


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

Description Moti Asayag 2013-07-24 12:12:31 UTC
Created attachment 777758 [details]
engine.log

Description of problem:
A correlation id isn't created for 'Add Host' action.
Without the correlation-id, it is less convenient to track the installation flow on the engine log and since setup network is used to configure the network on vdsm, that value doesn't pass to vdsm (although mostly relevant for first installation where the log file is relatively short at that point of time) 
Version-Release number of selected component (if applicable):

How reproducible:
always

Steps to Reproduce:
1. Add host via the UI
2. Open engine log and track 'AddVdsCommand'

Actual results:
No correlation id for the command, although invoked by an external user.
2013-07-24 09:57:26,728 INFO  [org.ovirt.engine.core.bll.AddVdsCommand] (ajp--
127.0.0.1-8702-10) Running command: AddVdsCommand internal: false. Entities affected :  ID: 99408929-82cf-4dc7-a532-9d998063fa95 Type: VdsGroups

Expected results:
2013-07-24 09:57:26,728 INFO  [org.ovirt.engine.core.bll.AddVdsCommand] (ajp--
127.0.0.1-8702-10) Running command: AddVdsCommand internal: false. Entities affected :  ID: 99408929-82cf-4dc7-a532-9d998063fa95 Type: VdsGroups


Additional info:
The RemoveVdsCommand which also invoked by the user does contain the correlation id. However the RemoveVdsCommand called by the UI is invoked as a MultipleAction, therefore its threads are created from the engine thread-pool-util (pool-6-thread-50) which the AddVdsCommand allocated thread is taken from other thread pool (ajp--127.0.0.1-8702).

2013-07-24 09:53:47,398 INFO  [org.ovirt.engine.core.bll.RemoveVdsCommand] (pool-6-thread-50) [63a05356] Lock freed to object EngineLock [exclusiveLocks= key: a32d0adf-8f13-4f98-9d06-594c62859827 value: VDS
, sharedLocks= ]
2013-07-24 09:57:26,728 INFO  [org.ovirt.engine.core.bll.AddVdsCommand] (ajp--127.0.0.1-8702-10) Running command: AddVdsCommand internal: false. Entities affected :  ID: 99408929-82cf-4dc7-a532-9d998063fa95 Type: VdsGroups

Comment 1 Mooli Tayer 2013-10-14 12:48:25 UTC
A bug was introduces in http://gerrit.ovirt.org/#/c/15795/16 causing all commands 1.) not executed by MultipleActionRunner 
AND
2.) not provided with correlationId in initial parameters
to not print a correlationId to the log. 

proposing patch upstream.

Comment 2 Mooli Tayer 2013-10-20 11:42:21 UTC
Submitted under ovirt-engine-3.3

Comment 3 Sandro Bonazzola 2013-12-13 16:01:52 UTC
rc build done, moving to ON_QA, packages will be available on updates-testing repository.

Comment 4 Sandro Bonazzola 2013-12-19 14:24:16 UTC
oVirt 3.3.2 has been released resolving the problem described in this bug report.


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