[Engine-devel] LockManager maybe not be really taking locks in your flows...
Yair Zaslavsky
yzaslavs at redhat.com
Sun Aug 11 14:03:05 UTC 2013
Hi all,
Thanks to Alon Bar Lev's efforts for preventing to concurrent host installation for the same host entity + Roy Golan's check of the code, we saw that for commands that override
getExclusiveLocks() or getSharedLocks() the locking mechanism does not work (lock is not being acquired) if there is no annotation of @LockIdNameAttribute on the class.
A bug was filed for removing this annotation (leftover from some historical code ) , but until it is fixed - bare in mind you need to add this annotation in current commands you are working on
in order to utilize the mechanism.
See RemoveVmCommand (has the annotation) vs AddDiskCommand (which doesn't have the annotation)
Cheers,
Yair
More information about the Engine-devel
mailing list