93c9da0a74
Change bed303e9b47b5aef3a9e2d6a5da4b5891e9e6689 introduced the ability to set timeout without blocking, but this caused a few problems. - The acquire method has now different arguments according to a backend driver being used - The change broke the existing usage with blocking=<number> This addresses these two problems. Change-Id: I8cefa2f4a940082d17cde43d132a818940422176
111 lines
3.8 KiB
Python
111 lines
3.8 KiB
Python
# -*- coding: utf-8 -*-
|
|
#
|
|
# Copyright (C) 2014 eNovance Inc. All Rights Reserved.
|
|
#
|
|
# Licensed under the Apache License, Version 2.0 (the "License"); you may
|
|
# not use this file except in compliance with the License. You may obtain
|
|
# a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
|
|
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
|
|
# License for the specific language governing permissions and limitations
|
|
# under the License.
|
|
import abc
|
|
|
|
import tooz
|
|
from tooz import coordination
|
|
|
|
|
|
class _LockProxy(object):
|
|
def __init__(self, lock, *args, **kwargs):
|
|
self.lock = lock
|
|
self.args = args
|
|
self.kwargs = kwargs
|
|
|
|
def __enter__(self):
|
|
return self.lock.__enter__(*self.args, **self.kwargs)
|
|
|
|
def __exit__(self, exc_type, exc_val, exc_tb):
|
|
self.lock.__exit__(exc_type, exc_val, exc_tb)
|
|
|
|
|
|
class Lock(object, metaclass=abc.ABCMeta):
|
|
def __init__(self, name):
|
|
if not name:
|
|
raise ValueError("Locks must be provided a name")
|
|
self._name = name
|
|
|
|
@property
|
|
def name(self):
|
|
return self._name
|
|
|
|
def __call__(self, *args, **kwargs):
|
|
return _LockProxy(self, *args, **kwargs)
|
|
|
|
def __enter__(self, *args, **kwargs):
|
|
acquired = self.acquire(*args, **kwargs)
|
|
if not acquired:
|
|
msg = u'Acquiring lock %s failed' % self.name
|
|
raise coordination.LockAcquireFailed(msg)
|
|
|
|
return self
|
|
|
|
def __exit__(self, exc_type, exc_val, exc_tb):
|
|
self.release()
|
|
|
|
def is_still_owner(self):
|
|
"""Checks if the lock is still owned by the acquiree.
|
|
|
|
:returns: returns true if still acquired (false if not) and
|
|
false if the lock was never acquired in the first place
|
|
or raises ``NotImplemented`` if not implemented.
|
|
"""
|
|
raise tooz.NotImplemented
|
|
|
|
@abc.abstractmethod
|
|
def release(self):
|
|
"""Attempts to release the lock, returns true if released.
|
|
|
|
The behavior of releasing a lock which was not acquired in the first
|
|
place is undefined (it can range from harmless to releasing some other
|
|
users lock)..
|
|
|
|
:returns: returns true if released (false if not)
|
|
:rtype: bool
|
|
"""
|
|
|
|
def break_(self):
|
|
"""Forcefully release the lock.
|
|
|
|
This is mostly used for testing purposes, to simulate an out of
|
|
band operation that breaks the lock. Backends may allow waiters to
|
|
acquire immediately if a lock is broken, or they should raise an
|
|
exception. Releasing should be successful for objects that believe
|
|
they hold the lock but do not have the lock anymore. However,
|
|
they should be careful not to re-break the lock by releasing it,
|
|
since they may not be the holder anymore.
|
|
|
|
:returns: returns true if forcefully broken (false if not)
|
|
or raises ``NotImplemented`` if not implemented.
|
|
"""
|
|
raise tooz.NotImplemented
|
|
|
|
@abc.abstractmethod
|
|
def acquire(self, blocking=True, shared=False, timeout=None):
|
|
"""Attempts to acquire the lock.
|
|
|
|
:param blocking: If True, blocks until the lock is acquired. If False,
|
|
returns right away. Otherwise, the value is used as a
|
|
timeout value and the call returns maximum after this
|
|
number of seconds.
|
|
:param shared: If False, the lock is exclusive. If True, the lock can
|
|
be shareable or raises ``NotImplemented`` if not
|
|
implemented.
|
|
:param timeout: Timeout to acquire a lock.
|
|
:returns: returns true if acquired (false if not)
|
|
:rtype: bool
|
|
"""
|