From 6c7340dac78e565f530c82ae2f7ed52df971ddbd Mon Sep 17 00:00:00 2001 From: Michael Tremer Date: Thu, 27 Mar 2025 17:51:35 +0000 Subject: [PATCH] release monitoring: Manually fetch the Pakfire user The property does not support the async stuff. Signed-off-by: Michael Tremer --- src/buildservice/releasemonitoring.py | 7 ++++++- src/buildservice/users.py | 13 ------------- 2 files changed, 6 insertions(+), 14 deletions(-) diff --git a/src/buildservice/releasemonitoring.py b/src/buildservice/releasemonitoring.py index 221037b5..3f9e903f 100644 --- a/src/buildservice/releasemonitoring.py +++ b/src/buildservice/releasemonitoring.py @@ -781,6 +781,11 @@ class Monitoring(database.Base, database.BackendMixin, database.SoftDeleteMixin) """ Creates a new release for this package """ + # Fetch the pakfire user + owner = await self.backend.users.get_by_email("pakfire@ipfire.org") + if not owner: + raise RuntimeError("Pakfire user does not exist") + # XXX Do we need to check whether we are going backwards? # Raise an error if the release already exists @@ -804,7 +809,7 @@ class Monitoring(database.Base, database.BackendMixin, database.SoftDeleteMixin) if self.create_builds: await release._create_build( build = build, - owner = self.backend.users.pakfire, + owner = owner, ) # Create a bug report diff --git a/src/buildservice/users.py b/src/buildservice/users.py index 1ae34ce9..dbf3fe64 100644 --- a/src/buildservice/users.py +++ b/src/buildservice/users.py @@ -317,19 +317,6 @@ class Users(base.Object): # Return as list return await self.db.fetch_as_list(stmt) - # Pakfire - - @property - def pakfire(self): - """ - This is a c - """ - user = self.get_by_email("pakfire@ipfire.org") - if not user: - raise RuntimeError("Missing Pakfire user") - - return user - @functools.cached_property def build_counts(self): """ -- 2.47.3