mirror of
https://github.com/retspen/webvirtcloud
synced 2026-11-08 19:26:48 +00:00
Disk edits: - edit_disk built a new <disk> from the form and replaced the old one, losing what the form does not show (e.g. encryption and virtio driver options); a bus change detached and re-attached the disk. The disk of the persistent definition is now edited in place and the domain defined once; a bus change picks a free device name and drops the old address and virtio-only options. Pending persistent changes of a running VM are kept. - Clone omits empty disk devices, refuses a disk without a supported source and destination, and checks the name before allocating storage. Locks: - Every VM change held its compute's exclusive lock, so one slow operation blocked all VMs on that compute. A VM change now holds its own VM lock plus a shared compute lock; reconciliation, migration and cloning hold the compute exclusively (two clones of different VMs must not pass the name check together). - Quota-checked changes of one user (CPU, memory and disk resize, clone) run one at a time under a per-user lock, also across computes, so concurrent requests cannot all pass the check against the old allocation. It is the outermost lock; superusers, who have no quota, skip it. - A busy lock answers 409 to the API and a message in the web UI. API power and destroy actions return the view's error instead of success.
223 lines
8.4 KiB
Python
223 lines
8.4 KiB
Python
from contextlib import contextmanager
|
|
import fcntl
|
|
import logging
|
|
import os
|
|
import stat
|
|
import tempfile
|
|
import threading
|
|
import time
|
|
from django.db import transaction
|
|
from computes.models import Compute
|
|
from instances.models import Instance
|
|
|
|
logger = logging.getLogger(__name__)
|
|
|
|
_COMPUTE_THREAD_LOCKS = {}
|
|
_COMPUTE_THREAD_LOCKS_GUARD = threading.Lock()
|
|
|
|
|
|
def _get_compute_thread_lock(compute_pk):
|
|
with _COMPUTE_THREAD_LOCKS_GUARD:
|
|
if compute_pk not in _COMPUTE_THREAD_LOCKS:
|
|
_COMPUTE_THREAD_LOCKS[compute_pk] = threading.Lock()
|
|
return _COMPUTE_THREAD_LOCKS[compute_pk]
|
|
|
|
|
|
def _get_lock_directory():
|
|
candidates = [
|
|
"/srv/webvirtcloud/data/locks",
|
|
os.path.join(tempfile.gettempdir(), f"webvirtcloud_locks_{os.geteuid()}"),
|
|
]
|
|
for candidate in candidates:
|
|
try:
|
|
if os.path.islink(candidate):
|
|
continue
|
|
if not os.path.exists(candidate):
|
|
os.makedirs(candidate, mode=0o700, exist_ok=True)
|
|
if not os.path.isdir(candidate) or os.path.islink(candidate):
|
|
continue
|
|
|
|
stat_info = os.stat(candidate)
|
|
# Ensure candidate directory is owned by the current process user
|
|
if stat_info.st_uid != os.geteuid():
|
|
continue
|
|
# Ensure candidate directory has private permissions (mode 0700)
|
|
if (stat.S_IMODE(stat_info.st_mode) & 0o077) != 0:
|
|
try:
|
|
os.chmod(candidate, 0o700)
|
|
except OSError:
|
|
continue
|
|
stat_info = os.stat(candidate)
|
|
if (stat.S_IMODE(stat_info.st_mode) & 0o077) != 0:
|
|
continue
|
|
|
|
test_file = os.path.join(candidate, f".test_write_{os.getpid()}_{threading.get_ident()}")
|
|
with open(test_file, "w") as tf:
|
|
tf.write("1")
|
|
os.unlink(test_file)
|
|
return candidate
|
|
except (OSError, PermissionError):
|
|
continue
|
|
return None
|
|
|
|
|
|
_thread_local = threading.local()
|
|
|
|
|
|
def _get_thread_held_locks():
|
|
if not hasattr(_thread_local, "compute_locks"):
|
|
_thread_local.compute_locks = {}
|
|
return _thread_local.compute_locks
|
|
|
|
|
|
@contextmanager
|
|
def _libvirt_lock(key, filename, timeout=15.0, shared=False):
|
|
"""Use flock for both threads and processes; exclusive locks also use a mutex."""
|
|
held_locks = _get_thread_held_locks()
|
|
if key in held_locks:
|
|
if held_locks[key] and not shared:
|
|
raise RuntimeError("Cannot upgrade a shared compute lock inside a VM operation")
|
|
yield
|
|
return
|
|
|
|
thread_lock = None if shared else _get_compute_thread_lock(key)
|
|
start_time = time.monotonic()
|
|
if thread_lock is not None and not thread_lock.acquire(timeout=timeout):
|
|
raise TimeoutError(f"Timeout waiting for lock on {key}")
|
|
try:
|
|
lock_dir = _get_lock_directory()
|
|
if not lock_dir:
|
|
raise OSError(f"Could not acquire secure lock directory for {key}")
|
|
path = os.path.join(lock_dir, filename)
|
|
if os.path.islink(path):
|
|
raise PermissionError(f"Lock file {path} is a symlink")
|
|
fd = os.open(path, os.O_CREAT | os.O_RDWR | os.O_NOFOLLOW, 0o600)
|
|
acquired = False
|
|
try:
|
|
mode = fcntl.LOCK_SH if shared else fcntl.LOCK_EX
|
|
while True:
|
|
try:
|
|
fcntl.flock(fd, mode | fcntl.LOCK_NB)
|
|
acquired = True
|
|
break
|
|
except BlockingIOError:
|
|
if time.monotonic() - start_time >= timeout:
|
|
raise TimeoutError(f"Timeout waiting for lock on {key}")
|
|
time.sleep(0.05)
|
|
held_locks[key] = shared
|
|
try:
|
|
yield
|
|
finally:
|
|
held_locks.pop(key, None)
|
|
finally:
|
|
try:
|
|
if acquired:
|
|
fcntl.flock(fd, fcntl.LOCK_UN)
|
|
finally:
|
|
os.close(fd)
|
|
finally:
|
|
if thread_lock is not None:
|
|
thread_lock.release()
|
|
|
|
|
|
@contextmanager
|
|
def libvirt_compute_lock(compute, timeout=15.0, *, shared=False):
|
|
"""Exclude reconciliation/migration while allowing independent VM operations."""
|
|
compute_pk = getattr(compute, "pk", getattr(compute, "id", None))
|
|
if not compute_pk:
|
|
yield
|
|
return
|
|
with _libvirt_lock(compute_pk, f"sync_compute_{compute_pk}.lock", timeout, shared):
|
|
yield
|
|
|
|
|
|
@contextmanager
|
|
def user_quota_lock(user, timeout=15.0):
|
|
"""One user's quota check and the change it allows run one at a time, on every compute.
|
|
Take it before any compute or VM lock."""
|
|
with _libvirt_lock(("quota", user.pk), f"quota_user_{user.pk}.lock", timeout):
|
|
yield
|
|
|
|
|
|
@contextmanager
|
|
def libvirt_instance_lock(instance, timeout=15.0):
|
|
"""Serialize one VM and share the compute barrier with other VM operations."""
|
|
with libvirt_compute_lock(instance.compute, timeout, shared=True):
|
|
with _libvirt_lock(("instance", instance.pk), f"instance_{instance.pk}.lock", timeout):
|
|
yield
|
|
|
|
|
|
def refresh_instance_database(compute):
|
|
"""
|
|
Synchronizes the WebVirtCloud database with libvirt domain state.
|
|
Libvirt is the single source of truth:
|
|
- VMs removed in virt-manager/virsh are deleted from the database.
|
|
- VMs created in virt-manager/virsh are added to the database.
|
|
- VMs renamed in virt-manager/virsh have their name updated without losing
|
|
UserInstance permission relationships (matched by UUID).
|
|
- If libvirt is unreachable (connection/network error), the database is left
|
|
untouched to avoid false deletions.
|
|
"""
|
|
compute_pk = getattr(compute, "pk", getattr(compute, "id", None))
|
|
if not compute_pk:
|
|
return
|
|
|
|
try:
|
|
with libvirt_compute_lock(compute, timeout=15.0):
|
|
with transaction.atomic():
|
|
comp_locked = Compute.objects.select_for_update().filter(pk=compute_pk).first()
|
|
if not comp_locked:
|
|
return
|
|
|
|
if hasattr(compute, "status") and compute.status is False:
|
|
return
|
|
|
|
try:
|
|
domains = compute.proxy.wvm.listAllDomains()
|
|
except Exception as e:
|
|
logger.warning(
|
|
"Failed to retrieve domains from compute %s (%s): %s",
|
|
getattr(compute, "name", compute),
|
|
getattr(compute, "hostname", ""),
|
|
e,
|
|
)
|
|
return
|
|
|
|
# Map libvirt UUIDs to current domain names
|
|
host_domains = {}
|
|
for d in domains:
|
|
try:
|
|
host_domains[d.UUIDString()] = d.name()
|
|
except Exception as e:
|
|
logger.warning(
|
|
"Error reading domain metadata from compute %s: %s",
|
|
getattr(compute, "name", compute),
|
|
e,
|
|
)
|
|
return
|
|
|
|
db_instances = list(Instance.objects.filter(compute=compute).select_for_update())
|
|
db_uuids = set(inst.uuid for inst in db_instances)
|
|
|
|
# 1. Update name for any VM whose name changed out-of-band in virsh/virt-manager
|
|
for inst in db_instances:
|
|
if inst.uuid in host_domains:
|
|
new_name = host_domains[inst.uuid]
|
|
if inst.name != new_name:
|
|
inst.name = new_name
|
|
inst.save(update_fields=["name"])
|
|
|
|
# 2. Delete instances that were genuinely undefined/deleted in virsh/virt-manager
|
|
Instance.objects.filter(compute=compute).exclude(uuid__in=host_domains.keys()).delete()
|
|
|
|
# 3. Create or update new instances that were created out-of-band in virsh/virt-manager
|
|
for uuid, name in host_domains.items():
|
|
if uuid not in db_uuids:
|
|
Instance.objects.get_or_create(
|
|
compute=compute,
|
|
uuid=uuid,
|
|
defaults={"name": name},
|
|
)
|
|
except (OSError, PermissionError, TimeoutError) as e:
|
|
logger.error("Could not synchronize instances for compute %s: %s; failing closed", compute_pk, e)
|
|
return
|