Chiradeep,

Looks like I have a rookie mistake in S3-backed Secondary Storage.  I will 
investigate, and send a patch to lock on the management server side.

Thanks,
-John

On Jun 10, 2013, at 1:30 PM, Chiradeep Vittal <chiradeep.vit...@citrix.com> 
wrote:

> 
>> 
>> From a code perspective, I think it would behove us to implement a more
>> functional approach to command execution in order to ensure reference
>> counting, error handling, resource management are handled in a consistent
>> manner.  I implemented such an approach in
>> com.cloud.utils.db.GlobalLock#executeWithLock where locking around a
>> particular operation is managed separately form the actual operation
>> being performed.
> 
> This one confused me: this is called from NfsSecondaryStorageResource
> which does not have access to the management server database. Since if you
> call the lock() function, it will try to access a db row.
> 

Reply via email to