Yaniv Bronhaim has posted comments on this change.

Change subject: engine: Making all internal actions to have same correlationId
......................................................................


Patch Set 1: Looks good to me, but someone else must approve

(1 inline comment)

....................................................
Commit Message
Line 12: But, if action has some another internal actions they will have 
different correlationId.
Line 13: This patch will fix a problem.
Line 14: Discovered during work on provided bug
Line 15: This patch is addition to previous with Change-Id: 
I47ad8e04452b7f0e3f5f8477f84bc6b22e35ec98,
Line 16: in which I did some mistake. I have not explanation why I did such 
stupid mistake
you can remove the last sentence of your comment.
Line 17: 
Line 18: Change-Id: Ib12f9079a564bb507b3a3a0f84601e918bd92b3f
Line 19: Bug-Url: https://bugzilla.redhat.com/844335


--
To view, visit http://gerrit.ovirt.org/13101
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib12f9079a564bb507b3a3a0f84601e918bd92b3f
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Michael Kublin <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Michael Kublin <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to