Release 2.19.1

(#150)

Current implementation did it in wrong way. First of all, there was a
lookup through pkgbase instead of pkgname, which lead to errors, because
aur api doesn't allow to search by pkgbase (as well as provides is
basically pkgname instead)

It also was found that dependencies resolution lookup has been performed
by using raw packages array, which can include versions, descriptions
etc

docs/ahriman.application.handlers.rst

@@ -100,14 +100,6 @@ ahriman.application.handlers.rebuild module
    :no-undoc-members:
    :show-inheritance:
 
-ahriman.application.handlers.reload module
-------------------------------------------
-
-.. automodule:: ahriman.application.handlers.reload
-   :members:
-   :no-undoc-members:
-   :show-inheritance:
-
 ahriman.application.handlers.remove module
 ------------------------------------------
 

docs/ahriman.core.housekeeping.rst

@@ -1,21 +0,0 @@
-ahriman.core.housekeeping package
-=================================
-
-Submodules
-----------
-
-ahriman.core.housekeeping.logs\_rotation\_trigger module
---------------------------------------------------------
-
-.. automodule:: ahriman.core.housekeeping.logs_rotation_trigger
-   :members:
-   :no-undoc-members:
-   :show-inheritance:
-
-Module contents
----------------
-
-.. automodule:: ahriman.core.housekeeping
-   :members:
-   :no-undoc-members:
-   :show-inheritance:

docs/ahriman.core.rst

@@ -15,7 +15,6 @@ Subpackages
    ahriman.core.distributed
    ahriman.core.formatters
    ahriman.core.gitremote
-   ahriman.core.housekeeping
    ahriman.core.http
    ahriman.core.log
    ahriman.core.report

docs/ahriman.web.schemas.rst

@@ -44,14 +44,6 @@ ahriman.web.schemas.changes\_schema module
    :no-undoc-members:
    :show-inheritance:
 
-ahriman.web.schemas.configuration\_schema module
-------------------------------------------------
-
-.. automodule:: ahriman.web.schemas.configuration_schema
-   :members:
-   :no-undoc-members:
-   :show-inheritance:
-
 ahriman.web.schemas.counters\_schema module
 -------------------------------------------
 
@@ -148,14 +140,6 @@ ahriman.web.schemas.logs\_schema module
    :no-undoc-members:
    :show-inheritance:
 
-ahriman.web.schemas.logs\_search\_schema module
------------------------------------------------
-
-.. automodule:: ahriman.web.schemas.logs_search_schema
-   :members:
-   :no-undoc-members:
-   :show-inheritance:
-
 ahriman.web.schemas.oauth2\_schema module
 -----------------------------------------
 

docs/ahriman.web.views.v1.service.rst

@@ -12,14 +12,6 @@ ahriman.web.views.v1.service.add module
    :no-undoc-members:
    :show-inheritance:
 
-ahriman.web.views.v1.service.config module
-------------------------------------------
-
-.. automodule:: ahriman.web.views.v1.service.config
-   :members:
-   :no-undoc-members:
-   :show-inheritance:
-
 ahriman.web.views.v1.service.logs module
 ----------------------------------------
 

docs/architecture.rst

@@ -40,7 +40,6 @@ This package contains everything required for the most of application actions an
 * ``ahriman.core.distributed`` package with triggers and helpers for distributed build system.
 * ``ahriman.core.formatters`` package provides ``Printer`` sub-classes for printing data (e.g. package properties) to stdout which are used by some handlers.
 * ``ahriman.core.gitremote`` is a package with remote PKGBUILD triggers. Should not be called directly.
-* ``ahriman.core.housekeeping`` package provides few triggers for removing old data.
 * ``ahriman.core.http`` package provides HTTP clients which can be used later by other classes.
 * ``ahriman.core.log`` is a log utils package. It includes logger loader class, custom HTTP based logger and some wrappers.
 * ``ahriman.core.report`` is a package with reporting triggers. Should not be called directly.

docs/configuration.rst

@@ -81,6 +81,7 @@ Base configuration settings.
 * ``apply_migrations`` - perform database migrations on the application start, boolean, optional, default ``yes``. Useful if you are using git version. Note, however, that this option must be changed only if you know what to do and going to handle migrations manually.
 * ``database`` - path to the application SQLite database, string, required.
 * ``include`` - path to directory with configuration files overrides, string, optional. Files will be read in alphabetical order.
+* ``keep_last_logs`` - amount of build logs to be kept for each package, integer, optional ,default ``0``. Logs will be cleared at the end of each process.
 * ``logging`` - path to logging configuration, string, required. Check ``logging.ini`` for reference.
 
 ``alpm:*`` groups
@@ -165,7 +166,6 @@ Reporting to web service related settings. In most cases there is fallback to we
 Web server settings. This feature requires ``aiohttp`` libraries to be installed.
 
 * ``address`` - optional address in form ``proto://host:port`` (``port`` can be omitted in case of default ``proto`` ports), will be used instead of ``http://{host}:{port}`` in case if set, string, optional. This option is required in case if ``OAuth`` provider is used.
-* ``autorefresh_intervals`` - enable page auto refresh options, space separated list of integers, optional. The first defined interval will be used as default. If no intervals set, the auto refresh buttons will be disabled. If first element of the list equals ``0``, auto refresh will be disabled by default.
 * ``enable_archive_upload`` - allow to upload packages via HTTP (i.e. call of ``/api/v1/service/upload`` uri), boolean, optional, default ``no``.
 * ``host`` - host to bind, string, optional.
 * ``index_url`` - full URL of the repository index page, string, optional.
@@ -179,7 +179,7 @@ Web server settings. This feature requires ``aiohttp`` libraries to be installed
 * ``wait_timeout`` - wait timeout in seconds, maximum amount of time to be waited before lock will be free, integer, optional.
 
 ``keyring`` group
------------------
+--------------------
 
 Keyring package generator plugin.
 
@@ -197,13 +197,6 @@ Keyring generator plugin
 * ``revoked`` - list of revoked packagers keys, space separated list of strings, optional.
 * ``trusted`` - list of master keys, space separated list of strings, optional, if not set, the ``key`` option from ``sign`` group will be used.
 
-``housekeeping`` group
-----------------------
-
-This section describes settings for the ``ahriman.core.housekeeping.LogsRotationTrigger`` plugin.
-
-* ``keep_last_logs`` - amount of build logs to be kept for each package, integer, optional ,default ``0``. Logs will be cleared at the end of each process.
-
 ``mirrorlist`` group
 --------------------
 

package/archlinux/PKGBUILD

@@ -2,7 +2,7 @@
 
 pkgbase='ahriman'
 pkgname=('ahriman' 'ahriman-core' 'ahriman-triggers' 'ahriman-web')
-pkgver=2.19.0
+pkgver=2.19.1
 pkgrel=1
 pkgdesc="ArcH linux ReposItory MANager"
 arch=('any')
@@ -40,7 +40,6 @@ package_ahriman-core() {
                 'rsync: sync by using rsync')
     install="$pkgbase.install"
     backup=('etc/ahriman.ini'
-            'etc/ahriman.ini.d/00-housekeeping.ini'
             'etc/ahriman.ini.d/logging.ini')
 
     cd "$pkgbase-$pkgver"
@@ -50,7 +49,6 @@ package_ahriman-core() {
 
     # keep usr/share configs as reference and copy them to /etc
     install -Dm644 "$pkgdir/usr/share/$pkgbase/settings/ahriman.ini" "$pkgdir/etc/ahriman.ini"
-    install -Dm644 "$pkgdir/usr/share/$pkgbase/settings/ahriman.ini.d/00-housekeeping.ini" "$pkgdir/etc/ahriman.ini.d/00-housekeeping.ini"
     install -Dm644 "$pkgdir/usr/share/$pkgbase/settings/ahriman.ini.d/logging.ini" "$pkgdir/etc/ahriman.ini.d/logging.ini"
 
     install -Dm644 "$srcdir/$pkgbase.sysusers" "$pkgdir/usr/lib/sysusers.d/$pkgbase.conf"

package/lib/systemd/system/ahriman-web.service

@@ -5,7 +5,6 @@ After=network.target
 [Service]
 Type=simple
 ExecStart=/usr/bin/ahriman web
-ExecReload=/usr/bin/ahriman web-reload
 User=ahriman
 Group=ahriman
 

package/share/ahriman/settings/ahriman.ini

@@ -7,6 +7,8 @@ logging = ahriman.ini.d/logging.ini
 ;apply_migrations = yes
 ; Path to the application SQLite database.
 database = ${repository:root}/ahriman.db
+; Keep last build logs for each package
+keep_last_logs = 5
 
 [alpm]
 ; Path to pacman system database cache.
@@ -43,11 +45,9 @@ triggers[] = ahriman.core.gitremote.RemotePullTrigger
 triggers[] = ahriman.core.report.ReportTrigger
 triggers[] = ahriman.core.upload.UploadTrigger
 triggers[] = ahriman.core.gitremote.RemotePushTrigger
-triggers[] = ahriman.core.housekeeping.LogsRotationTrigger
 ; List of well-known triggers. Used only for configuration purposes.
 triggers_known[] = ahriman.core.gitremote.RemotePullTrigger
 triggers_known[] = ahriman.core.gitremote.RemotePushTrigger
-triggers_known[] = ahriman.core.housekeeping.LogsRotationTrigger
 triggers_known[] = ahriman.core.report.ReportTrigger
 triggers_known[] = ahriman.core.upload.UploadTrigger
 ; Maximal age in seconds of the VCS packages before their version will be updated with its remote source.

package/share/ahriman/settings/ahriman.ini.d/00-housekeeping.ini

@@ -1,3 +0,0 @@
-[logs-rotation]
-; Keep last build logs for each package
-keep_last_logs = 5

package/share/ahriman/settings/ahriman.ini.d/00-web.ini

@@ -28,10 +28,6 @@ allow_read_only = yes
 ; External address of the web service. Will be used for some features like OAuth. If none set will be generated as
 ;     address = http://${web:host}:${web:port}
 ;address = http://${web:host}:${web:port}
-; Enable page auto refresh. Intervals are given in seconds. Default interval is always the first element of the list.
-; If no intervals set, auto refresh will be disabled. 0 can only be the first element and will disable auto refresh
-; by default.
-autorefresh_intervals = 5 1 10 30 60
 ; Enable file upload endpoint used by some triggers.
 ;enable_archive_upload = no
 ; Address to bind the server.

package/share/ahriman/templates/build-status.jinja2

@@ -80,28 +80,10 @@
                 <button type="button" class="btn btn-secondary" onclick="reload()">
                     <i class="bi bi-arrow-clockwise"></i><span class="d-none d-sm-inline"> reload</span>
                 </button>
-
-                {% if autorefresh_intervals %}
-                    <div class="btn-group">
-                        <input id="table-autoreload-button" type="checkbox" class="btn-check" autocomplete="off" onclick="toggleTableAutoReload()" checked>
-                        <label for="table-autoreload-button" class="btn btn-outline-secondary" title="toggle auto reload"><i class="bi bi-clock"></i></label>
-                        <button type="button" class="btn btn-outline-secondary dropdown-toggle dropdown-toggle-split" data-bs-toggle="dropdown" aria-expanded="false">
-                            <span class="visually-hidden">select interval</span>
-                        </button>
-                        <ul id="table-autoreload-input" class="dropdown-menu">
-                            {% for interval in autorefresh_intervals %}
-                                <li><a class="dropdown-item {{ "active" if interval.is_active }}" onclick="toggleTableAutoReload({{ interval.interval }})" data-interval="{{ interval.interval }}">{{ interval.text }}</a></li>
-                            {% endfor %}
-                        </ul>
-                    </div>
-                {% endif %}
             </div>
 
             <table id="packages"
                    data-classes="table table-hover"
-                   data-cookie="true"
-                   data-cookie-id-table="ahriman-packages"
-                   data-cookie-storage="localStorage"
                    data-export-options='{"fileName": "packages"}'
                    data-filter-control="true"
                    data-filter-control-visible="false"
@@ -120,8 +102,8 @@
                    data-sortable="true"
                    data-sort-name="base"
                    data-sort-order="asc"
-                   data-toolbar="#toolbar"
-                   data-unique-id="id">
+                   data-toggle="table"
+                   data-toolbar="#toolbar">
                 <thead class="table-primary">
                     <tr>
                         <th data-checkbox="true"></th>

package/share/ahriman/templates/build-status/alerts.jinja2

@@ -3,9 +3,7 @@
 
     function createAlert(title, message, clz, action, id) {
         id ??= md5(title + message); // MD5 id from the content
-        if (alertPlaceholder.querySelector(`#alert-${id}`)) {
-            return; // check if there are duplicates
-        }
+        if (alertPlaceholder.querySelector(`#alert-${id}`)) return; // check if there are duplicates
 
         const wrapper = document.createElement("div");
         wrapper.id = `alert-${id}`;

package/share/ahriman/templates/build-status/dashboard.jinja2

@@ -51,87 +51,6 @@
     const dashboardPackagesCountChartCanvas = document.getElementById("dashboard-packages-count-chart");
     let dashboardPackagesCountChart = null;
 
-    function statusLoad() {
-        const badgeClass = status => {
-            if (status === "pending") return "btn-outline-warning";
-            if (status === "building") return "btn-outline-warning";
-            if (status === "failed") return "btn-outline-danger";
-            if (status === "success") return "btn-outline-success";
-            return "btn-outline-secondary";
-        };
-
-        makeRequest(
-            "/api/v1/status",
-            {
-                query: {
-                    architecture: repository.architecture,
-                    repository: repository.repository,
-                },
-                convert: response => response.json(),
-            },
-            data => {
-                versionBadge.innerHTML = `<i class="bi bi-github"></i> ahriman ${safe(data.version)}`;
-
-                dashboardButton.classList.remove(...dashboardButton.classList);
-                dashboardButton.classList.add("btn");
-                dashboardButton.classList.add(badgeClass(data.status.status));
-
-                dashboardModalHeader.classList.remove(...dashboardModalHeader.classList);
-                dashboardModalHeader.classList.add("modal-header");
-                headerClass(data.status.status).forEach(clz => dashboardModalHeader.classList.add(clz));
-
-                dashboardName.textContent = data.repository;
-                dashboardArchitecture.textContent = data.architecture;
-                dashboardStatus.textContent = data.status.status;
-                dashboardStatusTimestamp.textContent = new Date(1000 * data.status.timestamp).toISOStringShort();
-
-                if (dashboardPackagesStatusesChart) {
-                    const labels = [
-                        "unknown",
-                        "pending",
-                        "building",
-                        "failed",
-                        "success",
-                    ];
-                    dashboardPackagesStatusesChart.config.data = {
-                        labels: labels,
-                        datasets: [{
-                            label: "packages in status",
-                            data: labels.map(label => data.packages[label]),
-                            backgroundColor: [
-                                "rgb(55, 58, 60)",
-                                "rgb(255, 117, 24)",
-                                "rgb(255, 117, 24)",
-                                "rgb(255, 0, 57)",
-                                "rgb(63, 182, 24)",  // copy-paste from current style
-                            ],
-                        }],
-                    };
-                    dashboardPackagesStatusesChart.update();
-                }
-
-                if (dashboardPackagesCountChart) {
-                    dashboardPackagesCountChart.config.data = {
-                        labels: ["packages"],
-                        datasets: [
-                            {
-                                label: "archives",
-                                data: [data.stats.packages],
-                            },
-                            {
-                                label: "bases",
-                                data: [data.stats.bases],
-                            },
-                        ],
-                    };
-                    dashboardPackagesCountChart.update();
-                }
-
-                dashboardCanvas.hidden = data.status.total > 0;
-            },
-        );
-    }
-
     ready(_ => {
         dashboardPackagesStatusesChart = new Chart(dashboardPackagesStatusesChartCanvas, {
             type: "pie",

package/share/ahriman/templates/build-status/package-add-modal.jinja2

@@ -148,19 +148,8 @@
 
         packageAddInput.addEventListener("keyup", _ => {
             clearTimeout(packageAddInput.requestTimeout);
-
-            // do not update datalist if search string didn't change yet
-            const value = packageAddInput.value;
-            const previousValue = packageAddInput.dataset.previousValue;
-            if (value === previousValue) {
-                return;
-            }
-
-            // store current search string in attributes
-            packageAddInput.dataset.previousValue = value;
-
-            // perform data list update
             packageAddInput.requestTimeout = setTimeout(_ => {
+                const value = packageAddInput.value;
 
                 if (value.length >= 3) {
                     makeRequest(

package/share/ahriman/templates/build-status/package-info-modal.jinja2

@@ -80,7 +80,8 @@
                                data-classes="table table-hover"
                                data-sortable="true"
                                data-sort-name="timestamp"
-                               data-sort-order="desc">
+                               data-sort-order="desc"
+                               data-toggle="table">
                             <thead class="table-primary">
                                 <tr>
                                     <th data-align="right" data-field="timestamp">date</th>
@@ -97,24 +98,10 @@
                     <input id="package-info-refresh-input" type="checkbox" class="form-check-input" value="" checked>
                     <label for="package-info-refresh-input" class="form-check-label">update pacman databases</label>
 
-                    <button id="package-info-update-button" type="submit" class="btn btn-success" onclick="packageInfoUpdate()"><i class="bi bi-play"></i><span class="d-none d-sm-inline"> update</span></button>
+                    <button id="package-info-update-button" type="submit" class="btn btn-success" onclick="packageInfoUpdate()" data-bs-dismiss="modal"><i class="bi bi-play"></i><span class="d-none d-sm-inline"> update</span></button>
                     <button id="package-info-remove-button" type="submit" class="btn btn-danger" onclick="packageInfoRemove()" data-bs-dismiss="modal"><i class="bi bi-trash"></i><span class="d-none d-sm-inline"> remove</span></button>
                 {% endif %}
-                {% if autorefresh_intervals %}
                 <button type="button" class="btn btn-secondary" onclick="showPackageInfo()"><i class="bi bi-arrow-clockwise"></i><span class="d-none d-sm-inline"> reload</span></button>
-                    <div class="btn-group dropup">
-                        <input id="package-info-autoreload-button" type="checkbox" class="btn-check" autocomplete="off" onclick="togglePackageInfoAutoReload()" checked>
-                        <label for="package-info-autoreload-button" class="btn btn-outline-secondary" title="toggle auto reload"><i class="bi bi-clock"></i></label>
-                        <button type="button" class="btn btn-outline-secondary dropdown-toggle dropdown-toggle-split" data-bs-toggle="dropdown" aria-expanded="false">
-                            <span class="visually-hidden">select interval</span>
-                        </button>
-                        <ul id="package-info-autoreload-input" class="dropdown-menu">
-                            {% for interval in autorefresh_intervals %}
-                                <li><a class="dropdown-item {{ "active" if interval.is_active }}" onclick="togglePackageInfoAutoReload({{ interval.interval }})" data-interval="{{ interval.interval }}">{{ interval.text }}</a></li>
-                            {% endfor %}
-                        </ul>
-                    </div>
-                {% endif %}
                 <button type="button" class="btn btn-primary" data-bs-dismiss="modal"><i class="bi bi-x"></i><span class="d-none d-sm-inline"> close</span></button>
             </div>
         </div>
@@ -153,10 +140,6 @@
 
     const packageInfoRefreshInput = document.getElementById("package-info-refresh-input");
 
-    const packageInfoAutoReloadButton = document.getElementById("package-info-autoreload-button");
-    const packageInfoAutoReloadInput = document.getElementById("package-info-autoreload-input");
-    let packageInfoAutoReloadTask = null;
-
     function clearChart() {
         packageInfoEventsUpdateChartCanvas.hidden = true;
         if (packageInfoEventsUpdateChart) {
@@ -165,13 +148,6 @@
         }
     }
 
-    function convertLogs(data, filter) {
-        return data
-            .filter((filter || Boolean))
-            .map(log_record => `[${new Date(1000 * log_record.created).toISOString()}] ${log_record.message}`)
-            .join("\n");
-    }
-
     async function copyChanges() {
         const changes = packageInfoChangesInput.textContent;
         await copyToClipboard(changes, packageInfoChangesCopyButton);
@@ -315,69 +291,6 @@
     }
 
     function loadLogs(packageBase, onFailure) {
-        const sortFn = (left, right) => left.process_id.localeCompare(right.process_id) || left.version.localeCompare(right.version);
-        const compareFn = (left, right) => left.process_id === right.process_id && left.version === right.version;
-
-        makeRequest(
-            `/api/v2/packages/${packageBase}/logs`,
-            {
-                query: {
-                    architecture: repository.architecture,
-                    head: true,
-                    repository: repository.repository,
-                },
-                convert: response => response.json(),
-            },
-            data => {
-                const currentVersions = Array.from(packageInfoLogsVersions.children)
-                    .map(el => {
-                        return {
-                            process_id: el.dataset.processId,
-                            version: el.dataset.version,
-                        };
-                    })
-                    .sort(sortFn);
-                const newVersions = data
-                    .map(el => {
-                        return {
-                            process_id: el.process_id,
-                            version: el.version,
-                        };
-                    })
-                    .sort(sortFn);
-
-                if (currentVersions.equals(newVersions, compareFn))
-                    loadLogsActive(packageBase);
-                else
-                    loadLogsAll(packageBase, onFailure);
-            },
-        )
-    }
-
-    function loadLogsActive(packageBase) {
-        const activeLogSelector = packageInfoLogsVersions.querySelector(".active");
-
-        if (activeLogSelector) {
-            makeRequest(
-                `/api/v2/packages/${packageBase}/logs`,
-                {
-                    query: {
-                        architecture: repository.architecture,
-                        repository: repository.repository,
-                        version: activeLogSelector.dataset.version,
-                        process_id: activeLogSelector.dataset.processId,
-                    },
-                    convert: response => response.json(),
-                },
-                data => {
-                    activeLogSelector.dataset.logs = convertLogs(data);
-                    activeLogSelector.click();
-                },
-            );
-        }
-    }
-
-    function loadLogsAll(packageBase, onFailure) {
         makeRequest(
             `/api/v2/packages/${packageBase}/logs`,
             {
@@ -406,19 +319,15 @@
                         const link = document.createElement("a");
                         link.classList.add("dropdown-item");
 
-                        link.dataset.version = version.version;
-                        link.dataset.processId = version.process_id;
-                        link.dataset.logs = convertLogs(data, log_record => log_record.version === version.version && log_record.process_id === version.process_id);
-
                         link.textContent = new Date(1000 * version.created).toISOStringShort();
                         link.href = "#";
                         link.onclick = _ => {
-                            // check if we are at the bottom of the code block
-                            const isScrolledToBottom = packageInfoLogsInput.scrollTop + packageInfoLogsInput.clientHeight >= packageInfoLogsInput.scrollHeight;
-                            packageInfoLogsInput.textContent = link.dataset.logs;
+                            const logs = data
+                                .filter(log_record => log_record.version === version.version && log_record.process_id === version.process_id)
+                                .map(log_record => `[${new Date(1000 * log_record.created).toISOString()}] ${log_record.message}`);
+
+                            packageInfoLogsInput.textContent = logs.join("\n");
                             highlight(packageInfoLogsInput);
-                            if (isScrolledToBottom)
-                                packageInfoLogsInput.scrollTop = packageInfoLogsInput.scrollHeight; // scroll to the new end
 
                             Array.from(packageInfoLogsVersions.children).forEach(el => el.classList.remove("active"));
                             link.classList.add("active");
@@ -494,12 +403,12 @@
     }
 
     function packageInfoRemove() {
-        const packageBase = packageInfoModal.dataset.package;
+        const packageBase = packageInfoModal.package;
         packagesRemove([packageBase]);
     }
 
     function packageInfoUpdate() {
-        const packageBase = packageInfoModal.dataset.package;
+        const packageBase = packageInfoModal.package;
         packagesAdd(packageBase, [], repository, {refresh: packageInfoRefreshInput.checked});
     }
 
@@ -507,10 +416,10 @@
         const isPackageBaseSet = packageBase !== undefined;
         if (isPackageBaseSet) {
             // set package base as currently used
-            packageInfoModal.dataset.package = packageBase;
+            packageInfoModal.package = packageBase;
         } else {
             // read package base from the current window attribute
-            packageBase = packageInfoModal.dataset.package;
+            packageBase = packageInfoModal.package;
         }
 
         const onFailure = error => {
@@ -529,27 +438,10 @@
 
         if (isPackageBaseSet) {
             bootstrap.Modal.getOrCreateInstance(packageInfoModal).show();
-            {% if autorefresh_intervals %}
-                togglePackageInfoAutoReload();
-            {% endif %}
         }
     }
 
-    function togglePackageInfoAutoReload(interval) {
-        clearInterval(packageInfoAutoReloadTask);
-        packageInfoAutoReloadTask = toggleAutoReload(packageInfoAutoReloadButton, interval, packageInfoAutoReloadInput, _ => {
-            if (!hasActiveSelection()) {
-                const packageBase = packageInfoModal.dataset.package;
-                // we only poll status and logs here
-                loadPackage(packageBase);
-                loadLogs(packageBase);
-            }
-        });
-    }
-
     ready(_ => {
-        packageInfoEventsTable.bootstrapTable({});
-
         packageInfoEventsUpdateChart = new Chart(packageInfoEventsUpdateChartCanvas, {
             type: "line",
             data: {},
@@ -576,11 +468,6 @@
             packageInfoChangesInput.textContent = "";
             packageInfoEventsTable.bootstrapTable("load", []);
             clearChart();
-
-            clearInterval(packageInfoAutoReloadTask);
-            packageInfoAutoReloadTask = null; // not really required (?) but lets clear everything
         });
-
-        restoreAutoReloadSettings(packageInfoAutoReloadButton, packageInfoAutoReloadInput);
     });
 </script>

package/share/ahriman/templates/build-status/table.jinja2

@@ -10,10 +10,6 @@
     const dashboardButton = document.getElementById("dashboard-button");
     const versionBadge = document.getElementById("badge-version");
 
-    const tableAutoReloadButton = document.getElementById("table-autoreload-button");
-    const tableAutoReloadInput = document.getElementById("table-autoreload-input");
-    let tableAutoReloadTask = null;
-
     function doPackageAction(uri, packages, repository, successText, failureText, data) {
         makeRequest(
             uri,
@@ -59,41 +55,6 @@
         return table.bootstrapTable("getSelections").map(row => row.id);
     }
 
-    function packagesLoad(onFailure) {
-        makeRequest(
-            "/api/v1/packages",
-            {
-                query: {
-                    architecture: repository.architecture,
-                    repository: repository.repository,
-                },
-                convert: response => response.json(),
-            },
-            data => {
-                const payload = data
-                    .map(description => {
-                        const package_base = description.package.base;
-                        const web_url = description.package.remote.web_url;
-                        return {
-                            id: package_base,
-                            base: web_url ? safeLink(web_url, package_base, package_base).outerHTML : safe(package_base),
-                            version: safe(description.package.version),
-                            packager: description.package.packager ? safe(description.package.packager) : "",
-                            packages: listToTable(Object.keys(description.package.packages)),
-                            groups: listToTable(extractListProperties(description.package, "groups")),
-                            licenses: listToTable(extractListProperties(description.package, "licenses")),
-                            timestamp: new Date(1000 * description.status.timestamp).toISOStringShort(),
-                            status: description.status.status,
-                        };
-                    });
-
-                updateTable(table, payload);
-                table.bootstrapTable("hideLoading");
-            },
-            onFailure,
-        );
-    }
-
     function packagesRemove(packages) {
         packages = packages ?? getSelection();
         const onSuccess = update => `Packages ${update} have been removed`;
@@ -127,7 +88,46 @@
 
     function reload() {
         table.bootstrapTable("showLoading");
-        const onFailure = error => {
+
+        const badgeClass = status => {
+            if (status === "pending") return "btn-outline-warning";
+            if (status === "building") return "btn-outline-warning";
+            if (status === "failed") return "btn-outline-danger";
+            if (status === "success") return "btn-outline-success";
+            return "btn-outline-secondary";
+        };
+
+        makeRequest(
+            "/api/v1/packages",
+            {
+                query: {
+                    architecture: repository.architecture,
+                    repository: repository.repository,
+                },
+                convert: response => response.json(),
+            },
+            data => {
+                const payload = data.map(description => {
+                    const package_base = description.package.base;
+                    const web_url = description.package.remote.web_url;
+                    return {
+                        id: package_base,
+                        base: web_url ? safeLink(web_url, package_base, package_base).outerHTML : safe(package_base),
+                        version: safe(description.package.version),
+                        packager: description.package.packager ? safe(description.package.packager) : "",
+                        packages: listToTable(Object.keys(description.package.packages)),
+                        groups: listToTable(extractListProperties(description.package, "groups")),
+                        licenses: listToTable(extractListProperties(description.package, "licenses")),
+                        timestamp: new Date(1000 * description.status.timestamp).toISOStringShort(),
+                        status: description.status.status,
+                    };
+                });
+
+                table.bootstrapTable("load", payload);
+                table.bootstrapTable("uncheckAll");
+                table.bootstrapTable("hideLoading");
+            },
+            error => {
                 if ((error.status === 401) || (error.status === 403)) {
                     // authorization error
                     const text = "In order to see statuses you must login first.";
@@ -139,10 +139,79 @@
                     const message = details => `Could not load list of packages: ${details}`;
                     showFailure("Load failure", message, error);
                 }
-        };
+            },
+        );
 
-        packagesLoad(onFailure);
-        statusLoad();
+        makeRequest(
+            "/api/v1/status",
+            {
+                query: {
+                    architecture: repository.architecture,
+                    repository: repository.repository,
+                },
+                convert: response => response.json(),
+            },
+            data => {
+                versionBadge.innerHTML = `<i class="bi bi-github"></i> ahriman ${safe(data.version)}`;
+
+                dashboardButton.classList.remove(...dashboardButton.classList);
+                dashboardButton.classList.add("btn");
+                dashboardButton.classList.add(badgeClass(data.status.status));
+
+                dashboardModalHeader.classList.remove(...dashboardModalHeader.classList);
+                dashboardModalHeader.classList.add("modal-header");
+                headerClass(data.status.status).forEach(clz => dashboardModalHeader.classList.add(clz));
+
+                dashboardName.textContent = data.repository;
+                dashboardArchitecture.textContent = data.architecture;
+                dashboardStatus.textContent = data.status.status;
+                dashboardStatusTimestamp.textContent = new Date(1000 * data.status.timestamp).toISOStringShort();
+
+                if (dashboardPackagesStatusesChart) {
+                    const labels = [
+                        "unknown",
+                        "pending",
+                        "building",
+                        "failed",
+                        "success",
+                    ];
+                    dashboardPackagesStatusesChart.config.data = {
+                        labels: labels,
+                        datasets: [{
+                            label: "packages in status",
+                            data: labels.map(label => data.packages[label]),
+                            backgroundColor: [
+                                "rgb(55, 58, 60)",
+                                "rgb(255, 117, 24)",
+                                "rgb(255, 117, 24)",
+                                "rgb(255, 0, 57)",
+                                "rgb(63, 182, 24)",  // copy-paste from current style
+                            ],
+                        }],
+                    };
+                    dashboardPackagesStatusesChart.update();
+                }
+
+                if (dashboardPackagesCountChart) {
+                    dashboardPackagesCountChart.config.data = {
+                        labels: ["packages"],
+                        datasets: [
+                            {
+                                label: "archives",
+                                data: [data.stats.packages],
+                            },
+                            {
+                                label: "bases",
+                                data: [data.stats.bases],
+                            },
+                        ],
+                    };
+                    dashboardPackagesCountChart.update();
+                }
+
+                dashboardCanvas.hidden = data.status.total > 0;
+            },
+        );
     }
 
     function selectRepository() {
@@ -161,24 +230,7 @@
         return {classes: cellClass(value)};
     }
 
-    function toggleTableAutoReload(interval) {
-        clearInterval(tableAutoReloadTask);
-        tableAutoReloadTask = toggleAutoReload(tableAutoReloadButton, interval, tableAutoReloadInput, _ => {
-            if (!hasActiveModal() &&
-                !hasActiveDropdown()) {
-                packagesLoad();
-                statusLoad();
-            }
-        });
-    }
-
     ready(_ => {
-        const onCheckFunction = function () {
-            if (packageRemoveButton) {
-                packageRemoveButton.disabled = !getSelection().length;
-            }
-        };
-
         document.querySelectorAll("#repositories a").forEach(element => {
             element.onclick = _ => {
                 repository = {
@@ -193,16 +245,18 @@
             };
         });
 
-        table.bootstrapTable({
-            onCheck: onCheckFunction,
-            onCheckAll: onCheckFunction,
-            onClickRow: (data, row, cell) => {
+        table.on("check.bs.table uncheck.bs.table check-all.bs.table uncheck-all.bs.table", _ => {
+            if (packageRemoveButton) {
+                packageRemoveButton.disabled = !table.bootstrapTable("getSelections").length;
+            }
+        });
+        table.on("click-row.bs.table", (self, data, row, cell) => {
             if (0 === cell || "base" === cell) {
                 const method = data[0] === true ? "uncheckBy" : "checkBy"; // fck javascript
                 table.bootstrapTable(method, {field: "id", values: [data.id]});
             } else showPackageInfo(data.id);
-            },
-            onCreatedControls: _ => {
+        });
+        table.on("created-controls.bs.table", _ => {
             new easepick.create({
                 element: document.querySelector(".bootstrap-table-filter-control-timestamp"),
                 css: [
@@ -232,16 +286,8 @@
                     };
                 },
             });
-            },
-            onUncheck: onCheckFunction,
-            onUncheckAll: onCheckFunction,
         });
 
-        restoreAutoReloadSettings(tableAutoReloadButton, tableAutoReloadInput);
-
         selectRepository();
-        {% if autorefresh_intervals %}
-            toggleTableAutoReload();
-        {% endif %}
     });
 </script>

package/share/ahriman/templates/repo-index.jinja2

@@ -53,7 +53,8 @@ SigLevel = Database{% if has_repo_signed %}Required{% else %}Never{% endif %} Pa
                    data-show-search-clear-button="true"
                    data-sortable="true"
                    data-sort-name="base"
-                   data-sort-order="asc">
+                   data-sort-order="asc"
+                   data-toggle="table">
                 <thead class="table-primary">
                     <tr>
                         <th data-sortable="true" data-switchable="false" data-field="name" data-filter-control="input" data-filter-control-placeholder="(any package)">package</th>
@@ -127,8 +128,7 @@ SigLevel = Database{% if has_repo_signed %}Required{% else %}Never{% endif %} Pa
             }
 
             ready(_ => {
-                table.bootstrapTable({
-                    onCreatedControls: _ => {
+                table.on("created-controls.bs.table", _ => {
                     new easepick.create({
                         element: document.querySelector(".bootstrap-table-filter-control-timestamp"),
                         css: [
@@ -158,7 +158,6 @@ SigLevel = Database{% if has_repo_signed %}Required{% else %}Never{% endif %} Pa
                             };
                         },
                     });
-                    },
                 });
             });
         </script>

package/share/ahriman/templates/utils/bootstrap-scripts.jinja2

@@ -1,24 +1,23 @@
 <script src="https://cdn.jsdelivr.net/npm/jquery@3.7.1/dist/jquery.min.js" crossorigin="anonymous" type="application/javascript"></script>
 <script src="https://cdn.jsdelivr.net/npm/js-md5@0.8.3/src/md5.min.js" crossorigin="anonymous" type="application/javascript"></script>
 
-<script src="https://cdn.jsdelivr.net/npm/tableexport.jquery.plugin@1.33.0/tableExport.min.js" crossorigin="anonymous" type="application/javascript"></script>
+<script src="https://cdn.jsdelivr.net/npm/tableexport.jquery.plugin@1.30.0/tableExport.min.js" crossorigin="anonymous" type="application/javascript"></script>
 
 <script src="https://cdn.jsdelivr.net/npm/jquery-resizable-columns@0.2.3/dist/jquery.resizableColumns.min.js" crossorigin="anonymous" type="application/javascript"></script>
 
 <script src="https://cdn.jsdelivr.net/npm/@popperjs/core@2.11.8/dist/umd/popper.min.js" crossorigin="anonymous" type="application/javascript"></script>
-<script src="https://cdn.jsdelivr.net/npm/bootstrap@5.3.7/dist/js/bootstrap.min.js" crossorigin="anonymous" type="application/javascript"></script>
-<script src="https://cdn.jsdelivr.net/npm/bootstrap-table@1.24.1/dist/bootstrap-table.min.js" crossorigin="anonymous" type="application/javascript"></script>
+<script src="https://cdn.jsdelivr.net/npm/bootstrap@5.3.3/dist/js/bootstrap.min.js" crossorigin="anonymous" type="application/javascript"></script>
+<script src="https://cdn.jsdelivr.net/npm/bootstrap-table@1.23.2/dist/bootstrap-table.min.js" crossorigin="anonymous" type="application/javascript"></script>
 
-<script src="https://cdn.jsdelivr.net/npm/bootstrap-table@1.24.1/dist/extensions/export/bootstrap-table-export.min.js" crossorigin="anonymous" type="application/javascript"></script>
-<script src="https://cdn.jsdelivr.net/npm/bootstrap-table@1.24.1/dist/extensions/resizable/bootstrap-table-resizable.js" crossorigin="anonymous" type="application/javascript"></script>
-<script src="https://cdn.jsdelivr.net/npm/bootstrap-table@1.24.1/dist/extensions/filter-control/bootstrap-table-filter-control.js" crossorigin="anonymous" type="application/javascript"></script>
-<script src="https://cdn.jsdelivr.net/npm/bootstrap-table@1.24.1/dist/extensions/cookie/bootstrap-table-cookie.min.js" crossorigin="anonymous" type="application/javascript"></script>
+<script src="https://cdn.jsdelivr.net/npm/bootstrap-table@1.23.2/dist/extensions/export/bootstrap-table-export.min.js" crossorigin="anonymous" type="application/javascript"></script>
+<script src="https://cdn.jsdelivr.net/npm/bootstrap-table@1.23.2/dist/extensions/resizable/bootstrap-table-resizable.js" crossorigin="anonymous" type="application/javascript"></script>
+<script src="https://cdn.jsdelivr.net/npm/bootstrap-table@1.23.2/dist/extensions/filter-control/bootstrap-table-filter-control.js" crossorigin="anonymous" type="application/javascript"></script>
 
 <script src="https://cdn.jsdelivr.net/npm/@easepick/bundle@1.2.1/dist/index.umd.min.js" crossorigin="anonymous" type="application/javascript"></script>
 
-<script src="https://cdn.jsdelivr.net/gh/highlightjs/cdn-release@11.11.1/build/highlight.min.js" crossorigin="anonymous" type="application/javascript"></script>
+<script src="https://cdn.jsdelivr.net/gh/highlightjs/cdn-release@11.10.0/build/highlight.min.js" crossorigin="anonymous" type="application/javascript"></script>
 
-<script src="https://cdn.jsdelivr.net/npm/chart.js@4.5.0/dist/chart.umd.min.js" crossorigin="anonymous" type="application/javascript"></script>
+<script src="https://cdn.jsdelivr.net/npm/chart.js@4.4.4/dist/chart.umd.min.js" crossorigin="anonymous" type="application/javascript"></script>
 
 <script>
     async function copyToClipboard(text, button) {
@@ -59,20 +58,6 @@
         return value.includes(dataList[index].toLowerCase());
     }
 
-    function hasActiveSelection() {
-        return !document.getSelection().isCollapsed; // not sure if it is a valid way, but I guess so
-    }
-
-    function hasActiveDropdown() {
-        return Array.from(document.querySelectorAll(".dropdown-menu"))
-            .some(el => el.classList.contains("show"));
-    }
-
-    function hasActiveModal() {
-        return Array.from(document.querySelectorAll(".modal"))
-            .some(el => el.classList.contains("show"));
-    }
-
     function headerClass(status) {
         if (status === "pending") return ["bg-warning"];
         if (status === "building") return ["bg-warning"];
@@ -121,12 +106,6 @@
             .catch(error => onFailure && onFailure(error));
     }
 
-    function readOptional(extractor, callback) {
-        for (let value = extractor(); !!value; value = null) {
-            callback(value);
-        }
-    }
-
     function ready(fn) {
         if (document.readyState === "complete" || document.readyState === "interactive") {
             setTimeout(fn, 1);
@@ -135,11 +114,6 @@
         }
     }
 
-    function restoreAutoReloadSettings(toggle, intervalSelector) {
-        readOptional(() => localStorage.getItem(`ahriman-${toggle.id}-refresh-enabled`), value => toggle.checked = value === "true");
-        readOptional(() => localStorage.getItem(`ahriman-${toggle.id}-refresh-interval`), value => toggleActiveElement(intervalSelector, "interval", value));
-    }
-
     function safe(string) {
         return String(string)
             .replace(/&/g, "&amp;")
@@ -159,86 +133,7 @@
         return element;
     }
 
-    function toggleActiveElement(selector, dataType, value) {
-        const targetElement = selector.querySelector(`a[data-${dataType}="${value}"]`);
-        if (targetElement?.classList?.contains("active")) {
-            return; // element is already active, skip processing
-        }
-
-        Array.from(selector.children).forEach(il => {
-            Array.from(il.children).forEach(el => el.classList.remove("active"));
-        });
-        targetElement?.classList?.add("active");
-    }
-
-    function toggleAutoReload(toggle, interval, intervalSelector, callback) {
-        if (interval) {
-            toggle.checked = true; // toggle reload
-        } else {
-            interval = intervalSelector.querySelector(".active")?.dataset?.interval; // find active element
-        }
-
-        let intervalId = null;
-        if (interval) {
-            if (toggle.checked) {
-                // refresh UI
-                toggleActiveElement(intervalSelector, "interval", interval);
-                // finally create timer task
-                intervalId = setInterval(callback, interval);
-            }
-        } else {
-            toggle.checked = false; // no active interval found, disable toggle
-        }
-
-        localStorage.setItem(`ahriman-${toggle.id}-refresh-enabled`, toggle.checked);
-        localStorage.setItem(`ahriman-${toggle.id}-refresh-interval`, interval);
-        return intervalId;
-    }
-
-    function updateTable(table, rows) {
-        // instead of using load method here, we just update rows manually to avoid table reinitialization
-        const currentData = table.bootstrapTable("getData").reduce((accumulator, row) => {
-            accumulator[row.id] = row["0"];
-            return accumulator;
-        }, {});
-        // insert or update rows
-        rows.forEach(row => {
-            if (Object.hasOwn(currentData, row.id)) {
-                row["0"] = currentData[row.id]; // copy checkbox state
-                table.bootstrapTable("updateByUniqueId", {
-                    id: row.id,
-                    row: row,
-                    replace: true,
-                });
-            } else {
-                table.bootstrapTable("insertRow", {index: 0, row: row});
-            }
-        });
-        // remove old rows
-        const newData = rows.map(value => value.id);
-        Object.keys(currentData).forEach(id => {
-            if (!newData.includes(id)) {
-                table.bootstrapTable("removeByUniqueId", id);
-            }
-        });
-    }
-
-    Array.prototype.equals = function (right, comparator) {
-        let index = this.length;
-        if (index !== right.length) {
-            return false;
-        }
-
-        while (index--) {
-            if (!comparator(this[index], right[index])) {
-                return false;
-            }
-        }
-
-        return true;
-    }
-
-    Date.prototype.toISOStringShort = function () {
+    Date.prototype.toISOStringShort = function() {
         const pad = number => String(number).padStart(2, "0");
         return `${this.getFullYear()}-${pad(this.getMonth() + 1)}-${pad(this.getDate())} ${pad(this.getHours())}:${pad(this.getMinutes())}:${pad(this.getSeconds())}`;
     }

package/share/ahriman/templates/utils/style.jinja2

@@ -1,15 +1,15 @@
-<link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/bootstrap@5.3.7/dist/css/bootstrap.min.css" crossorigin="anonymous" type="text/css">
-<link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/bootstrap-icons@1.13.1/font/bootstrap-icons.css" crossorigin="anonymous" type="text/css">
+<link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/bootstrap@5.3.3/dist/css/bootstrap.min.css" crossorigin="anonymous" type="text/css">
+<link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/bootstrap-icons@1.11.1/font/bootstrap-icons.css" crossorigin="anonymous" type="text/css">
 
-<link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/bootstrap-table@1.24.1/dist/bootstrap-table.min.css" crossorigin="anonymous" type="text/css">
+<link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/bootstrap-table@1.23.2/dist/bootstrap-table.min.css" crossorigin="anonymous" type="text/css">
 
 <link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/jquery-resizable-columns@0.2.3/dist/jquery.resizableColumns.css" crossorigin="anonymous" type="text/css">
 
-<link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/bootstrap-table@1.24.1/dist/extensions/filter-control/bootstrap-table-filter-control.css" crossorigin="anonymous" type="text/css">
+<link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/bootstrap-table@1.23.2/dist/extensions/filter-control/bootstrap-table-filter-control.css" crossorigin="anonymous" type="text/css">
 
-<link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/bootswatch@5.3.7/dist/cosmo/bootstrap.min.css" crossorigin="anonymous" type="text/css">
+<link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/bootswatch@5.3.3/dist/cosmo/bootstrap.min.css" crossorigin="anonymous" type="text/css">
 
-<link rel="stylesheet" href="https://cdn.jsdelivr.net/gh/highlightjs/cdn-release@11.11.1/build/styles/github.min.css" crossorigin="anonymous" type="text/css">
+<link rel="stylesheet" href="https://cdn.jsdelivr.net/gh/highlightjs/cdn-release@11.10.0/build/styles/github.min.css" crossorigin="anonymous" type="text/css">
 
 <style>
     .pre-scrollable {

package/share/man/man1/ahriman.1

@@ -1,4 +1,4 @@
-.TH AHRIMAN "1" "2025\-06\-29" "ahriman 2.19.0" "ArcH linux ReposItory MANager"
+.TH AHRIMAN "1" "2025\-07\-14" "ahriman 2.19.1" "ArcH linux ReposItory MANager"
 .SH NAME
 ahriman \- ArcH linux ReposItory MANager
 .SH SYNOPSIS

recipes/check/compose.yml

@@ -8,7 +8,7 @@ services:
       AHRIMAN_OUTPUT: console
       AHRIMAN_PASSWORD: ${AHRIMAN_PASSWORD}
       AHRIMAN_PORT: 8080
-      AHRIMAN_POSTSETUP_COMMAND: (cat /run/secrets/password; echo; cat /run/secrets/password) | sudo -u ahriman ahriman user-add demo -R full
+      AHRIMAN_PRESETUP_COMMAND: (cat /run/secrets/password; echo; cat /run/secrets/password) | sudo -u ahriman ahriman user-add demo -R full
       AHRIMAN_REPOSITORY: ahriman-demo
       AHRIMAN_UNIX_SOCKET: /var/lib/ahriman/ahriman/ahriman.sock
 

recipes/distributed-manual/compose.yml

@@ -8,7 +8,7 @@ services:
       AHRIMAN_OUTPUT: console
       AHRIMAN_PASSWORD: ${AHRIMAN_PASSWORD}
       AHRIMAN_PORT: 8080
-      AHRIMAN_POSTSETUP_COMMAND: (cat /run/secrets/password; echo; cat /run/secrets/password) | sudo -u ahriman ahriman user-add demo -R full
+      AHRIMAN_PRESETUP_COMMAND: (cat /run/secrets/password; echo; cat /run/secrets/password) | sudo -u ahriman ahriman user-add demo -R full
       AHRIMAN_REPOSITORY: ahriman-demo
       AHRIMAN_UNIX_SOCKET: /var/lib/ahriman/ahriman/ahriman.sock
 

recipes/distributed/compose.yml

@@ -8,7 +8,7 @@ services:
       AHRIMAN_OUTPUT: console
       AHRIMAN_PASSWORD: ${AHRIMAN_PASSWORD}
       AHRIMAN_PORT: 8080
-      AHRIMAN_POSTSETUP_COMMAND: (cat /run/secrets/password; echo; cat /run/secrets/password) | sudo -u ahriman ahriman user-add demo -R full
+      AHRIMAN_PRESETUP_COMMAND: (cat /run/secrets/password; echo; cat /run/secrets/password) | sudo -u ahriman ahriman user-add demo -R full
       AHRIMAN_REPOSITORY: ahriman-demo
       AHRIMAN_UNIX_SOCKET: /var/lib/ahriman/ahriman/ahriman.sock
 
@@ -62,7 +62,7 @@ services:
       AHRIMAN_OUTPUT: console
       AHRIMAN_PASSWORD: ${AHRIMAN_PASSWORD}
       AHRIMAN_PORT: 8080
-      AHRIMAN_POSTSETUP_COMMAND: (cat /run/secrets/password; echo; cat /run/secrets/password) | sudo -u ahriman ahriman user-add demo -R full
+      AHRIMAN_PRESETUP_COMMAND: (cat /run/secrets/password; echo; cat /run/secrets/password) | sudo -u ahriman ahriman user-add demo -R full
       AHRIMAN_REPOSITORY: ahriman-demo
       AHRIMAN_REPOSITORY_SERVER: http://frontend/repo/$$repo/$$arch
 

recipes/i686/compose.yml

@@ -12,7 +12,7 @@ services:
       AHRIMAN_PACMAN_MIRROR: https://de.mirror.archlinux32.org/$$arch/$$repo
       AHRIMAN_PASSWORD: ${AHRIMAN_PASSWORD}
       AHRIMAN_PORT: 8080
-      AHRIMAN_POSTSETUP_COMMAND: (cat /run/secrets/password; echo; cat /run/secrets/password) | sudo -u ahriman ahriman user-add demo -R full
+      AHRIMAN_PRESETUP_COMMAND: (cat /run/secrets/password; echo; cat /run/secrets/password) | sudo -u ahriman ahriman user-add demo -R full
       AHRIMAN_REPOSITORY: ahriman-demo
       AHRIMAN_UNIX_SOCKET: /var/lib/ahriman/ahriman/ahriman.sock
 

recipes/multirepo/compose.yml

@@ -8,8 +8,8 @@ services:
       AHRIMAN_OUTPUT: console
       AHRIMAN_PASSWORD: ${AHRIMAN_PASSWORD}
       AHRIMAN_PORT: 8080
-      AHRIMAN_POSTSETUP_COMMAND: (cat /run/secrets/password; echo; cat /run/secrets/password) | sudo -u ahriman ahriman user-add demo -R full
-      AHRIMAN_PRESETUP_COMMAND: ahriman --architecture x86_64 --repository another-demo service-setup --build-as-user ahriman --packager 'ahriman bot <ahriman@example.com>'
+      AHRIMAN_POSTSETUP_COMMAND: ahriman --architecture x86_64 --repository another-demo service-setup --build-as-user ahriman --packager 'ahriman bot <ahriman@example.com>'
+      AHRIMAN_PRESETUP_COMMAND: (cat /run/secrets/password; echo; cat /run/secrets/password) | sudo -u ahriman ahriman user-add demo -R full
       AHRIMAN_REPOSITORY: ahriman-demo
       AHRIMAN_UNIX_SOCKET: /var/lib/ahriman/ahriman/ahriman.sock
 

recipes/oauth/compose.yml

@@ -9,7 +9,7 @@ services:
       AHRIMAN_OAUTH_CLIENT_SECRET: ${AHRIMAN_OAUTH_CLIENT_SECRET}
       AHRIMAN_OUTPUT: console
       AHRIMAN_PORT: 8080
-      AHRIMAN_POSTSETUP_COMMAND: sudo -u ahriman ahriman user-add ${AHRIMAN_OAUTH_USER} -R full -p ""
+      AHRIMAN_PRESETUP_COMMAND: sudo -u ahriman ahriman user-add ${AHRIMAN_OAUTH_USER} -R full -p ""
       AHRIMAN_REPOSITORY: ahriman-demo
       AHRIMAN_UNIX_SOCKET: /var/lib/ahriman/ahriman/ahriman.sock
 

recipes/sign/compose.yml

@@ -6,7 +6,7 @@ services:
     environment:
       AHRIMAN_DEBUG: yes
       AHRIMAN_OUTPUT: console
-      AHRIMAN_POSTSETUP_COMMAND: sudo -u ahriman gpg --import /run/secrets/key
+      AHRIMAN_PRESETUP_COMMAND: sudo -u ahriman gpg --import /run/secrets/key
       AHRIMAN_REPOSITORY: ahriman-demo
 
     configs:

recipes/web/compose.yml

@@ -8,7 +8,7 @@ services:
       AHRIMAN_OUTPUT: console
       AHRIMAN_PASSWORD: ${AHRIMAN_PASSWORD}
       AHRIMAN_PORT: 8080
-      AHRIMAN_POSTSETUP_COMMAND: (cat /run/secrets/password; echo; cat /run/secrets/password) | sudo -u ahriman ahriman user-add demo -R full
+      AHRIMAN_PRESETUP_COMMAND: (cat /run/secrets/password; echo; cat /run/secrets/password) | sudo -u ahriman ahriman user-add demo -R full
       AHRIMAN_REPOSITORY: ahriman-demo
       AHRIMAN_UNIX_SOCKET: /var/lib/ahriman/ahriman/ahriman.sock
 

src/ahriman/__init__.py

@@ -17,4 +17,4 @@
 # You should have received a copy of the GNU General Public License
 # along with this program. If not, see <http://www.gnu.org/licenses/>.
 #
-__version__ = "2.19.0"
+__version__ = "2.19.1"

src/ahriman/application/handlers/reload.py

@@ -1,70 +0,0 @@
-#
-# Copyright (c) 2021-2025 ahriman team.
-#
-# This file is part of ahriman
-# (see https://github.com/arcan1s/ahriman).
-#
-# This program is free software: you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation, either version 3 of the License, or
-# (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program. If not, see <http://www.gnu.org/licenses/>.
-#
-import argparse
-
-from ahriman.application.application import Application
-from ahriman.application.handlers.handler import Handler, SubParserAction
-from ahriman.core.configuration import Configuration
-from ahriman.models.repository_id import RepositoryId
-
-
-class Reload(Handler):
-    """
-    web server reload handler
-    """
-
-    ALLOW_MULTI_ARCHITECTURE_RUN = False  # system-wide action
-
-    @classmethod
-    def run(cls, args: argparse.Namespace, repository_id: RepositoryId, configuration: Configuration, *,
-            report: bool) -> None:
-        """
-        callback for command line
-
-        Args:
-            args(argparse.Namespace): command line args
-            repository_id(RepositoryId): repository unique identifier
-            configuration(Configuration): configuration instance
-            report(bool): force enable or disable reporting
-        """
-        application = Application(repository_id, configuration, report=True)
-        client = application.repository.reporter
-        client.configuration_reload()
-
-    @staticmethod
-    def _set_web_reload_parser(root: SubParserAction) -> argparse.ArgumentParser:
-        """
-        add parser for web reload subcommand
-
-        Args:
-            root(SubParserAction): subparsers for the commands
-
-        Returns:
-            argparse.ArgumentParser: created argument parser
-        """
-        parser = root.add_parser("web-reload", help="reload configuration",
-                                 description="reload web server configuration",
-                                 epilog="This method forces the web server to reload its configuration. "
-                                        "Note, however, that this method does not apply all configuration changes "
-                                        "(like ports, authentication, etc)")
-        parser.set_defaults(architecture="", lock=None, quiet=True, report=False, repository="", unsafe=True)
-        return parser
-
-    arguments = [_set_web_reload_parser]

src/ahriman/application/handlers/setup.py

@@ -72,7 +72,6 @@ class Setup(Handler):
 
         application = Application(repository_id, configuration, report=report)
 
-        with application.repository.paths.preserve_owner():
         Setup.configuration_create_makepkg(args.packager, args.makeflags_jobs, application.repository.paths)
         Setup.executable_create(application.repository.paths, repository_id)
         repository_server = f"file://{application.repository.paths.repository}" if args.server is None else args.server
@@ -281,5 +280,6 @@ class Setup(Handler):
         command = Setup.build_command(paths.root, repository_id)
         command.unlink(missing_ok=True)
         command.symlink_to(Setup.ARCHBUILD_COMMAND_PATH)
+        paths.chown(command)  # we would like to keep owner inside ahriman's home
 
     arguments = [_set_service_setup_parser]

src/ahriman/core/alpm/pacman.py

@@ -130,8 +130,8 @@ class Pacman(LazyLogging):
             return  # database for some reason deos not exist
 
         self.logger.info("copy pacman database %s from operating system root to ahriman's home %s", src, dst)
-        with self.repository_paths.preserve_owner(dst.parent):
         shutil.copy(src, dst)
+        self.repository_paths.chown(dst)
 
     def database_init(self, handle: Handle, repository: str, architecture: str) -> DB:
         """

src/ahriman/core/alpm/remote/remote.py

@@ -94,15 +94,6 @@ class Remote(SyncHttpClient):
                 for package in portion
                 if package.name in packages or not packages
             }
-
-        # simple check for duplicates. This method will remove all packages under base if there is
-        # a package named exactly as its base
-        packages = {
-            package.name: package
-            for package in packages.values()
-            if package.package_base not in packages or package.package_base == package.name
-        }
-
         return list(packages.values())
 
     @classmethod

src/ahriman/core/configuration/configuration.py

@@ -17,7 +17,6 @@
 # You should have received a copy of the GNU General Public License
 # along with this program. If not, see <http://www.gnu.org/licenses/>.
 #
-# pylint: disable=too-many-public-methods
 import configparser
 import shlex
 import sys
@@ -86,10 +85,9 @@ class Configuration(configparser.RawConfigParser):
             empty_lines_in_values=not allow_multi_key,
             interpolation=ShellInterpolator(),
             converters={
-                "intlist": lambda value: list(map(int, shlex.split(value))),
                 "list": shlex.split,
                 "path": self._convert_path,
-                "pathlist": lambda value: list(map(self._convert_path, shlex.split(value))),
+                "pathlist": lambda value: [self._convert_path(element) for element in shlex.split(value)],
             },
         )
 
@@ -238,8 +236,6 @@ class Configuration(configparser.RawConfigParser):
 
     # pylint and mypy are too stupid to find these methods
     # pylint: disable=missing-function-docstring,unused-argument
-    def getintlist(self, *args: Any, **kwargs: Any) -> list[int]: ...  # type: ignore[empty-body]
-
     def getlist(self, *args: Any, **kwargs: Any) -> list[str]: ...  # type: ignore[empty-body]
 
     def getpath(self, *args: Any, **kwargs: Any) -> Path: ...  # type: ignore[empty-body]

src/ahriman/core/configuration/schema.py

@@ -45,6 +45,11 @@ CONFIGURATION_SCHEMA: ConfigurationSchema = {
                 "path_exists": True,
                 "path_type": "dir",
             },
+            "keep_last_logs": {
+                "type": "integer",
+                "coerce": "integer",
+                "min": 0,
+            },
             "logging": {
                 "type": "path",
                 "coerce": "absolute_path",
@@ -319,15 +324,6 @@ CONFIGURATION_SCHEMA: ConfigurationSchema = {
                 "empty": False,
                 "is_url": ["http", "https"],
             },
-            "autorefresh_intervals": {
-                "type": "list",
-                "coerce": "list",
-                "schema": {
-                    "type": "integer",
-                    "coerce": "integer",
-                    "min": 0,
-                },
-            },
             "enable_archive_upload": {
                 "type": "boolean",
                 "coerce": "boolean",

src/ahriman/core/database/migrations/m011_repository_name.py

@@ -203,6 +203,8 @@ def migrate_package_repository(connection: Connection, configuration: Configurat
         configuration(Configuration): configuration instance
     """
     _, repository_id = configuration.check_loaded()
+    if repository_id.is_empty:
+        return  # no repository available yet
 
     connection.execute("""update build_queue set repository = :repository""", {"repository": repository_id.id})
     connection.execute("""update package_bases set repository = :repository""", {"repository": repository_id.id})

src/ahriman/core/database/operations/logs_operations.py

@@ -29,15 +29,13 @@ class LogsOperations(Operations):
     logs operations
     """
 
-    def logs_get(self, package_base: str, version: str | None = None, process_id: str | None = None,
-                 limit: int = -1, offset: int = 0, repository_id: RepositoryId | None = None) -> list[LogRecord]:
+    def logs_get(self, package_base: str, limit: int = -1, offset: int = 0,
+                 repository_id: RepositoryId | None = None) -> list[LogRecord]:
         """
         extract logs for specified package base
 
         Args:
             package_base(str): package base to extract logs
-            version(str | None, optional): package version to filter (Default value = None)
-            process_id(str | None, optional): process identifier to filter (Default value = None)
             limit(int, optional): limit records to the specified count, -1 means unlimited (Default value = -1)
             offset(int, optional): records offset (Default value = 0)
             repository_id(RepositoryId, optional): repository unique identifier override (Default value = None)
@@ -54,17 +52,12 @@ class LogsOperations(Operations):
                     """
                     select created, message, version, process_id from (
                         select * from logs
-                        where package_base = :package_base
-                          and repository = :repository
-                          and (:version is null or version = :version)
-                          and (:process_id is null or process_id = :process_id)
+                        where package_base = :package_base and repository = :repository
                         order by created desc limit :limit offset :offset
                     ) order by created asc
                     """,
                     {
                         "package_base": package_base,
-                        "version": version,
-                        "process_id": process_id,
                         "repository": repository_id.id,
                         "limit": limit,
                         "offset": offset,

src/ahriman/core/database/sqlite.py

@@ -94,13 +94,9 @@ class SQLite(
         sqlite3.register_adapter(list, json.dumps)
         sqlite3.register_converter("json", json.loads)
 
-        if not self._configuration.getboolean("settings", "apply_migrations", fallback=True):
-            return
-        if self._repository_id.is_empty:
-            return  # do not perform migration on empty repository identifier (e.g. multirepo command)
-
-        with self._repository_paths.preserve_owner():
+        if self._configuration.getboolean("settings", "apply_migrations", fallback=True):
             self.with_connection(lambda connection: Migrations.migrate(connection, self._configuration))
+        self._repository_paths.chown(self.path)
 
     def package_clear(self, package_base: str, repository_id: RepositoryId | None = None) -> None:
         """

src/ahriman/core/housekeeping/__init__.py

@@ -1,20 +0,0 @@
-#
-# Copyright (c) 2021-2025 ahriman team.
-#
-# This file is part of ahriman
-# (see https://github.com/arcan1s/ahriman).
-#
-# This program is free software: you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation, either version 3 of the License, or
-# (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program. If not, see <http://www.gnu.org/licenses/>.
-#
-from ahriman.core.housekeeping.logs_rotation_trigger import LogsRotationTrigger

src/ahriman/core/housekeeping/logs_rotation_trigger.py

@@ -1,87 +0,0 @@
-#
-# Copyright (c) 2021-2025 ahriman team.
-#
-# This file is part of ahriman
-# (see https://github.com/arcan1s/ahriman).
-#
-# This program is free software: you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation, either version 3 of the License, or
-# (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program. If not, see <http://www.gnu.org/licenses/>.
-#
-from ahriman.core import context
-from ahriman.core.configuration import Configuration
-from ahriman.core.status import Client
-from ahriman.core.triggers import Trigger
-from ahriman.models.package import Package
-from ahriman.models.repository_id import RepositoryId
-from ahriman.models.result import Result
-
-
-class LogsRotationTrigger(Trigger):
-    """
-    rotate logs after build processes
-
-    Attributes:
-        keep_last_records(int): number of last records to keep
-    """
-
-    CONFIGURATION_SCHEMA = {
-        "logs-rotation": {
-            "type": "dict",
-            "schema": {
-                "keep_last_logs": {
-                    "type": "integer",
-                    "required": True,
-                    "coerce": "integer",
-                    "min": 0,
-                },
-            },
-        },
-    }
-
-    def __init__(self, repository_id: RepositoryId, configuration: Configuration) -> None:
-        """
-        Args:
-            repository_id(RepositoryId): repository unique identifier
-            configuration(Configuration): configuration instance
-        """
-        Trigger.__init__(self, repository_id, configuration)
-
-        section = next(iter(self.configuration_sections(configuration)))
-        self.keep_last_records = configuration.getint(  # read old-style first and then fallback to new style
-            "settings", "keep_last_logs",
-            fallback=configuration.getint(section, "keep_last_logs"))
-
-    @classmethod
-    def configuration_sections(cls, configuration: Configuration) -> list[str]:
-        """
-        extract configuration sections from configuration
-
-        Args:
-            configuration(Configuration): configuration instance
-
-        Returns:
-            list[str]: read configuration sections belong to this trigger
-        """
-        return list(cls.CONFIGURATION_SCHEMA.keys())
-
-    def on_result(self, result: Result, packages: list[Package]) -> None:
-        """
-        run trigger
-
-        Args:
-            result(Result): build result
-            packages(list[Package]): list of all available packages
-        """
-        ctx = context.get()
-        reporter = ctx.get(Client)
-        reporter.logs_rotate(self.keep_last_records)

src/ahriman/core/log/http_log_handler.py

@@ -17,6 +17,7 @@
 # You should have received a copy of the GNU General Public License
 # along with this program. If not, see <http://www.gnu.org/licenses/>.
 #
+import atexit
 import logging
 import uuid
 
@@ -36,6 +37,7 @@ class HttpLogHandler(logging.Handler):
     method
 
     Attributes:
+        keep_last_records(int): number of last records to keep
         reporter(Client): build status reporter instance
         suppress_errors(bool): suppress logging errors (e.g. if no web server available)
     """
@@ -54,6 +56,7 @@ class HttpLogHandler(logging.Handler):
 
         self.reporter = Client.load(repository_id, configuration, report=report)
         self.suppress_errors = suppress_errors
+        self.keep_last_records = configuration.getint("settings", "keep_last_logs", fallback=0)
 
     @classmethod
     def load(cls, repository_id: RepositoryId, configuration: Configuration, *, report: bool) -> Self:
@@ -80,6 +83,7 @@ class HttpLogHandler(logging.Handler):
         root.addHandler(handler)
 
         LogRecordId.DEFAULT_PROCESS_ID = str(uuid.uuid4())  # assign default process identifier for log records
+        atexit.register(handler.rotate)
 
         return handler
 
@@ -100,3 +104,9 @@ class HttpLogHandler(logging.Handler):
             if self.suppress_errors:
                 return
             self.handleError(record)
+
+    def rotate(self) -> None:
+        """
+        rotate log records, removing older ones
+        """
+        self.reporter.logs_rotate(self.keep_last_records)

src/ahriman/core/status/client.py

@@ -81,11 +81,6 @@ class Client:
 
         return make_local_client()
 
-    def configuration_reload(self) -> None:
-        """
-        reload configuration
-        """
-
     def event_add(self, event: Event) -> None:
         """
         create new event
@@ -208,15 +203,12 @@ class Client:
         """
         # this method does not raise NotImplementedError because it is actively used as dummy client for http log
 
-    def package_logs_get(self, package_base: str, version: str | None = None, process_id: str | None = None,
-                         limit: int = -1, offset: int = 0) -> list[LogRecord]:
+    def package_logs_get(self, package_base: str, limit: int = -1, offset: int = 0) -> list[LogRecord]:
         """
         get package logs
 
         Args:
             package_base(str): package base
-            version(str | None, optional): package version to search (Default value = None)
-            process_id(str | None, optional): process identifier to search (Default value = None)
             limit(int, optional): limit records to the specified count, -1 means unlimited (Default value = -1)
             offset(int, optional): records offset (Default value = 0)
 

src/ahriman/core/status/local_client.py

@@ -152,22 +152,19 @@ class LocalClient(Client):
         """
         self.database.logs_insert(log_record, self.repository_id)
 
-    def package_logs_get(self, package_base: str, version: str | None = None, process_id: str | None = None,
-                         limit: int = -1, offset: int = 0) -> list[LogRecord]:
+    def package_logs_get(self, package_base: str, limit: int = -1, offset: int = 0) -> list[LogRecord]:
         """
         get package logs
 
         Args:
             package_base(str): package base
-            version(str | None, optional): package version to search (Default value = None)
-            process_id(str | None, optional): process identifier to search (Default value = None)
             limit(int, optional): limit records to the specified count, -1 means unlimited (Default value = -1)
             offset(int, optional): records offset (Default value = 0)
 
         Returns:
             list[LogRecord]: package logs
         """
-        return self.database.logs_get(package_base, version, process_id, limit, offset, self.repository_id)
+        return self.database.logs_get(package_base, limit, offset, self.repository_id)
 
     def package_logs_remove(self, package_base: str, version: str | None) -> None:
         """

src/ahriman/core/status/watcher.py

@@ -109,7 +109,7 @@ class Watcher(LazyLogging):
 
     package_logs_add: Callable[[LogRecord], None]
 
-    package_logs_get: Callable[[str, str | None, str | None, int, int], list[LogRecord]]
+    package_logs_get: Callable[[str, int, int], list[LogRecord]]
 
     package_logs_remove: Callable[[str, str | None], None]
 

src/ahriman/core/status/web_client.py

@@ -17,7 +17,6 @@
 # You should have received a copy of the GNU General Public License
 # along with this program. If not, see <http://www.gnu.org/licenses/>.
 #
-# pylint: disable=too-many-public-methods
 import contextlib
 
 from urllib.parse import quote_plus as url_encode
@@ -166,13 +165,6 @@ class WebClient(Client, SyncAhrimanClient):
         """
         return f"{self.address}/api/v1/status"
 
-    def configuration_reload(self) -> None:
-        """
-        reload configuration
-        """
-        with contextlib.suppress(Exception):
-            self.make_request("POST", f"{self.address}/api/v1/service/config")
-
     def event_add(self, event: Event) -> None:
         """
         create new event
@@ -334,15 +326,12 @@ class WebClient(Client, SyncAhrimanClient):
         self.make_request("POST", self._logs_url(log_record.log_record_id.package_base),
                           params=self.repository_id.query(), json=payload, suppress_errors=True)
 
-    def package_logs_get(self, package_base: str, version: str | None = None, process_id: str | None = None,
-                         limit: int = -1, offset: int = 0) -> list[LogRecord]:
+    def package_logs_get(self, package_base: str, limit: int = -1, offset: int = 0) -> list[LogRecord]:
         """
         get package logs
 
         Args:
             package_base(str): package base
-            version(str | None, optional): package version to search (Default value = None)
-            process_id(str | None, optional): process identifier to search (Default value = None)
             limit(int, optional): limit records to the specified count, -1 means unlimited (Default value = -1)
             offset(int, optional): records offset (Default value = 0)
 
@@ -350,10 +339,6 @@ class WebClient(Client, SyncAhrimanClient):
             list[LogRecord]: package logs
         """
         query = self.repository_id.query() + [("limit", str(limit)), ("offset", str(offset))]
-        if version is not None:
-            query.append(("version", version))
-        if process_id is not None:
-            query.append(("process_id", process_id))
 
         with contextlib.suppress(Exception):
             response = self.make_request("GET", self._logs_url(package_base), params=query)

src/ahriman/core/utils.py

@@ -51,7 +51,6 @@ __all__ = [
     "parse_version",
     "partition",
     "pretty_datetime",
-    "pretty_interval",
     "pretty_size",
     "safe_filename",
     "srcinfo_property",
@@ -354,28 +353,6 @@ def pretty_datetime(timestamp: datetime.datetime | float | int | None) -> str:
     return timestamp.strftime("%Y-%m-%d %H:%M:%S")
 
 
-def pretty_interval(interval: int) -> str:
-    """
-    convert time interval to string
-
-    Args:
-        interval(int): time interval in seconds
-
-    Returns:
-        str: pretty printable interval as string
-    """
-    minutes, seconds = divmod(interval, 60)
-    hours, minutes = divmod(minutes, 60)
-    return " ".join([
-        f"{value} {description}{"s" if value > 1 else ""}"
-        for value, description in [
-            (hours, "hour"),
-            (minutes, "minute"),
-            (seconds, "second"),
-        ] if value > 0
-    ])
-
-
 def pretty_size(size: float | None, level: int = 0) -> str:
     """
     convert size to string

src/ahriman/models/repository_paths.py

@@ -17,7 +17,6 @@
 # You should have received a copy of the GNU General Public License
 # along with this program. If not, see <http://www.gnu.org/licenses/>.
 #
-import contextlib
 import os
 import shutil
 
@@ -222,14 +221,22 @@ class RepositoryPaths(LazyLogging):
         stat = path.stat()
         return stat.st_uid, stat.st_gid
 
-    def _chown(self, path: Path) -> None:
+    def cache_for(self, package_base: str) -> Path:
+        """
+        get path to cached PKGBUILD and package sources for the package base
+
+        Args:
+            package_base(str): package base name
+
+        Returns:
+            Path: full path to directory for specified package base cache
+        """
+        return self.cache / package_base
+
+    def chown(self, path: Path) -> None:
         """
         set owner of path recursively (from root) to root owner
 
-        Notes:
-            More likely you don't want to call this method explicitly, consider using :func:`preserve_owner`
-            as context manager instead
-
         Args:
             path(Path): path to be chown
 
@@ -249,56 +256,6 @@ class RepositoryPaths(LazyLogging):
             set_owner(path)
             path = path.parent
 
-    def cache_for(self, package_base: str) -> Path:
-        """
-        get path to cached PKGBUILD and package sources for the package base
-
-        Args:
-            package_base(str): package base name
-
-        Returns:
-            Path: full path to directory for specified package base cache
-        """
-        return self.cache / package_base
-
-    @contextlib.contextmanager
-    def preserve_owner(self, path: Path | None = None) -> Generator[None, None, None]:
-        """
-        perform any action preserving owner for any newly created file or directory
-
-        Args:
-            path(Path | None, optional): use this path as root instead of repository root (Default value = None)
-
-        Examples:
-            This method is designed to use as context manager when you are going to perform operations which might
-            change filesystem, especially if you are doing it under unsafe flag, e.g.::
-
-                >>> with paths.preserve_owner():
-                >>>     paths.tree_create()
-
-            Note, however, that this method doesn't handle any exceptions and will eventually interrupt
-            if there will be any.
-        """
-        path = path or self.root
-
-        def walk(root: Path) -> Generator[Path, None, None]:
-            # basically walk, but skipping some content
-            for child in root.iterdir():
-                yield child
-                if child in (self.chroot.parent,):
-                    yield from child.iterdir()  # we only yield top-level in chroot directory
-                elif child.is_dir():
-                    yield from walk(child)
-
-        # get current filesystem and run action
-        previous_snapshot = set(walk(path))
-        yield
-
-        # get newly created files and directories and chown them
-        new_entries = set(walk(path)).difference(previous_snapshot)
-        for entry in new_entries:
-            self._chown(entry)
-
     def tree_clear(self, package_base: str) -> None:
         """
         clear package specific files
@@ -317,8 +274,6 @@ class RepositoryPaths(LazyLogging):
         """
         if self.repository_id.is_empty:
             return  # do not even try to create tree in case if no repository id set
-
-        with self.preserve_owner():
         for directory in (
                 self.cache,
                 self.chroot,
@@ -327,3 +282,4 @@ class RepositoryPaths(LazyLogging):
                 self.repository,
         ):
             directory.mkdir(mode=0o755, parents=True, exist_ok=True)
+            self.chown(directory)

src/ahriman/web/schemas/__init__.py

@@ -22,7 +22,6 @@ from ahriman.web.schemas.aur_package_schema import AURPackageSchema
 from ahriman.web.schemas.auth_schema import AuthSchema
 from ahriman.web.schemas.build_options_schema import BuildOptionsSchema
 from ahriman.web.schemas.changes_schema import ChangesSchema
-from ahriman.web.schemas.configuration_schema import ConfigurationSchema
 from ahriman.web.schemas.counters_schema import CountersSchema
 from ahriman.web.schemas.dependencies_schema import DependenciesSchema
 from ahriman.web.schemas.error_schema import ErrorSchema
@@ -35,7 +34,6 @@ from ahriman.web.schemas.log_schema import LogSchema
 from ahriman.web.schemas.login_schema import LoginSchema
 from ahriman.web.schemas.logs_rotate_schema import LogsRotateSchema
 from ahriman.web.schemas.logs_schema import LogsSchema
-from ahriman.web.schemas.logs_search_schema import LogsSearchSchema
 from ahriman.web.schemas.oauth2_schema import OAuth2Schema
 from ahriman.web.schemas.package_name_schema import PackageNameSchema
 from ahriman.web.schemas.package_names_schema import PackageNamesSchema

src/ahriman/web/schemas/aur_package_schema.py

@@ -25,11 +25,11 @@ class AURPackageSchema(Schema):
     response AUR package schema
     """
 
-    description = fields.String(required=True, metadata={
-        "description": "Package description",
-        "example": "ArcH linux ReposItory MANager",
-    })
     package = fields.String(required=True, metadata={
         "description": "Package base",
         "example": "ahriman",
     })
+    description = fields.String(required=True, metadata={
+        "description": "Package description",
+        "example": "ArcH linux ReposItory MANager",
+    })

src/ahriman/web/schemas/changes_schema.py

@@ -25,10 +25,10 @@ class ChangesSchema(Schema):
     response package changes schema
     """
 
-    changes = fields.String(metadata={
-        "description": "Package changes in patch format",
-    })
     last_commit_sha = fields.String(metadata={
         "description": "Last recorded commit hash",
         "example": "f1875edca1eb8fc0e55c41d1cae5fa05b6b7c6",
     })
+    changes = fields.String(metadata={
+        "description": "Package changes in patch format",
+    })

src/ahriman/web/schemas/configuration_schema.py

@@ -1,39 +0,0 @@
-#
-# Copyright (c) 2021-2025 ahriman team.
-#
-# This file is part of ahriman
-# (see https://github.com/arcan1s/ahriman).
-#
-# This program is free software: you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation, either version 3 of the License, or
-# (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program. If not, see <http://www.gnu.org/licenses/>.
-#
-from ahriman.web.apispec import Schema, fields
-
-
-class ConfigurationSchema(Schema):
-    """
-    response configuration schema
-    """
-
-    key = fields.String(required=True, metadata={
-        "description": "Configuration key",
-        "example": "host",
-    })
-    section = fields.String(required=True, metadata={
-        "description": "Configuration section",
-        "example": "web",
-    })
-    value = fields.String(required=True, metadata={
-        "description": "Configuration value",
-        "example": "127.0.0.1",
-    })

src/ahriman/web/schemas/counters_schema.py

@@ -25,6 +25,18 @@ class CountersSchema(Schema):
     response package counters schema
     """
 
+    total = fields.Integer(required=True, metadata={
+        "description": "Total amount of packages",
+        "example": 6,
+    })
+    _unknown = fields.Integer(data_key="unknown", required=True, metadata={
+        "description": "Amount of packages in unknown state",
+        "example": 0,
+    })
+    pending = fields.Integer(required=True, metadata={
+        "description": "Amount of packages in pending state",
+        "example": 2,
+    })
     building = fields.Integer(required=True, metadata={
         "description": "Amount of packages in building state",
         "example": 1,
@@ -33,19 +45,7 @@ class CountersSchema(Schema):
         "description": "Amount of packages in failed state",
         "example": 1,
     })
-    pending = fields.Integer(required=True, metadata={
-        "description": "Amount of packages in pending state",
-        "example": 2,
-    })
     success = fields.Integer(required=True, metadata={
         "description": "Amount of packages in success state",
         "example": 3,
     })
-    total = fields.Integer(required=True, metadata={
-        "description": "Total amount of packages",
-        "example": 6,
-    })
-    unknown_ = fields.Integer(data_key="unknown", required=True, metadata={
-        "description": "Amount of packages in unknown state",
-        "example": 0,
-    })

src/ahriman/web/schemas/event_schema.py

@@ -30,17 +30,17 @@ class EventSchema(Schema):
         "description": "Event creation timestamp",
         "example": 1680537091,
     })
-    data = fields.Dict(keys=fields.String(), metadata={
-        "description": "Event metadata if available",
-    })
     event = fields.String(required=True, metadata={
         "description": "Event type",
         "example": EventType.PackageUpdated,
     })
-    message = fields.String(metadata={
-        "description": "Event message if available",
-    })
     object_id = fields.String(required=True, metadata={
         "description": "Event object identifier",
         "example": "ahriman",
     })
+    message = fields.String(metadata={
+        "description": "Event message if available",
+    })
+    data = fields.Dict(keys=fields.String(), metadata={
+        "description": "Event metadata if available",
+    })

src/ahriman/web/schemas/event_search_schema.py

@@ -31,14 +31,14 @@ class EventSearchSchema(PaginationSchema):
         "description": "Event type",
         "example": EventType.PackageUpdated,
     })
-    from_date = fields.Integer(metadata={
-        "description": "Minimal creation timestamp, inclusive",
-        "example": 1680537091,
-    })
     object_id = fields.String(metadata={
         "description": "Event object identifier",
         "example": "ahriman",
     })
+    from_date = fields.Integer(metadata={
+        "description": "Minimal creation timestamp, inclusive",
+        "example": 1680537091,
+    })
     to_date = fields.Integer(metadata={
         "description": "Maximal creation timestamp, exclusive",
         "example": 1680537091,

src/ahriman/web/schemas/log_schema.py

@@ -33,10 +33,10 @@ class LogSchema(Schema):
     message = fields.String(required=True, metadata={
         "description": "Log message",
     })
-    process_id = fields.String(metadata={
-        "description": "Process unique identifier",
-    })
     version = fields.String(required=True, metadata={
         "description": "Package version to tag",
         "example": __version__,
     })
+    process_id = fields.String(metadata={
+        "description": "Process unique identifier",
+    })

src/ahriman/web/schemas/login_schema.py

@@ -25,11 +25,11 @@ class LoginSchema(Schema):
     request login schema
     """
 
-    password = fields.String(required=True, metadata={
-        "description": "Login password",
-        "example": "pa55w0rd",
-    })
     username = fields.String(required=True, metadata={
         "description": "Login username",
         "example": "user",
     })
+    password = fields.String(required=True, metadata={
+        "description": "Login password",
+        "example": "pa55w0rd",
+    })

src/ahriman/web/schemas/logs_search_schema.py

@@ -1,39 +0,0 @@
-#
-# Copyright (c) 2021-2025 ahriman team.
-#
-# This file is part of ahriman
-# (see https://github.com/arcan1s/ahriman).
-#
-# This program is free software: you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation, either version 3 of the License, or
-# (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program. If not, see <http://www.gnu.org/licenses/>.
-#
-from ahriman import __version__
-from ahriman.web.apispec import fields
-from ahriman.web.schemas.pagination_schema import PaginationSchema
-
-
-class LogsSearchSchema(PaginationSchema):
-    """
-    request log search schema
-    """
-
-    head = fields.Boolean(metadata={
-        "description": "Return versions only without fetching logs themselves",
-    })
-    process_id = fields.String(metadata={
-        "description": "Process unique identifier to search",
-    })
-    version = fields.String(metadata={
-        "description": "Package version to search",
-        "example": __version__,
-    })

src/ahriman/web/schemas/package_properties_schema.py

@@ -37,14 +37,22 @@ class PackagePropertiesSchema(Schema):
         "description": "Package build timestamp",
         "example": 1680537091,
     })
-    check_depends = fields.List(fields.String(), metadata={
-        "description": "Package test dependencies list",
-        "example": ["python-pytest"],
-    })
     depends = fields.List(fields.String(), metadata={
         "description": "Package dependencies list",
         "example": ["devtools"],
     })
+    make_depends = fields.List(fields.String(), metadata={
+        "description": "Package make dependencies list",
+        "example": ["python-build"],
+    })
+    opt_depends = fields.List(fields.String(), metadata={
+        "description": "Package optional dependencies list",
+        "example": ["python-aiohttp"],
+    })
+    check_depends = fields.List(fields.String(), metadata={
+        "description": "Package test dependencies list",
+        "example": ["python-pytest"],
+    })
     description = fields.String(metadata={
         "description": "Package description",
         "example": "ArcH linux ReposItory MANager",
@@ -65,14 +73,6 @@ class PackagePropertiesSchema(Schema):
         "description": "Package licenses",
         "example": ["GPL3"],
     })
-    make_depends = fields.List(fields.String(), metadata={
-        "description": "Package make dependencies list",
-        "example": ["python-build"],
-    })
-    opt_depends = fields.List(fields.String(), metadata={
-        "description": "Package optional dependencies list",
-        "example": ["python-aiohttp"],
-    })
     provides = fields.List(fields.String(), metadata={
         "description": "Package provides list",
         "example": ["ahriman-git"],

src/ahriman/web/schemas/package_schema.py

@@ -32,18 +32,18 @@ class PackageSchema(Schema):
         "description": "Package base",
         "example": "ahriman",
     })
-    packager = fields.String(metadata={
-        "description": "packager for the last success package build",
-        "example": "ahriman bot <ahriman@example.com>",
+    version = fields.String(required=True, metadata={
+        "description": "Package version",
+        "example": __version__,
+    })
+    remote = fields.Nested(RemoteSchema(), required=True, metadata={
+        "description": "Package remote properties",
     })
     packages = fields.Dict(
         keys=fields.String(), values=fields.Nested(PackagePropertiesSchema()), required=True, metadata={
             "description": "Packages which belong to this base",
         })
-    remote = fields.Nested(RemoteSchema(), required=True, metadata={
-        "description": "Package remote properties",
-    })
-    version = fields.String(required=True, metadata={
-        "description": "Package version",
-        "example": __version__,
+    packager = fields.String(metadata={
+        "description": "packager for the last success package build",
+        "example": "ahriman bot <ahriman@example.com>",
     })

src/ahriman/web/schemas/repository_stats_schema.py

@@ -25,19 +25,19 @@ class RepositoryStatsSchema(Schema):
     response repository stats schema
     """
 
-    archive_size = fields.Int(metadata={
-        "description": "Total archive size of the packages in bytes",
-        "example": 42000,
-    })
     bases = fields.Int(metadata={
         "description": "Amount of unique packages bases",
         "example": 2,
     })
-    installed_size = fields.Int(metadata={
-        "description": "Total installed size of the packages in bytes",
-        "example": 42000000,
-    })
     packages = fields.Int(metadata={
         "description": "Amount of unique packages",
         "example": 4,
     })
+    archive_size = fields.Int(metadata={
+        "description": "Total archive size of the packages in bytes",
+        "example": 42000,
+    })
+    installed_size = fields.Int(metadata={
+        "description": "Total installed size of the packages in bytes",
+        "example": 42000000,
+    })

src/ahriman/web/schemas/search_schema.py

@@ -25,7 +25,7 @@ class SearchSchema(Schema):
     request package search schema
     """
 
-    for_ = fields.List(fields.String(), data_key="for", required=True, metadata={
+    _for = fields.List(fields.String(), data_key="for", required=True, metadata={
         "description": "Keyword for search",
         "example": ["ahriman"],
     })

src/ahriman/web/views/index.py

@@ -22,7 +22,6 @@ import aiohttp_jinja2
 from typing import Any, ClassVar
 
 from ahriman.core.auth.helpers import authorized_userid
-from ahriman.core.utils import pretty_interval
 from ahriman.models.user_access import UserAccess
 from ahriman.web.apispec import aiohttp_apispec
 from ahriman.web.views.base import BaseView
@@ -38,10 +37,6 @@ class IndexView(BaseView):
             * control - HTML to insert for login control, HTML string, required
             * enabled - whether authorization is enabled by configuration or not, boolean, required
             * username - authenticated username if any, string, null means not authenticated
-        * autorefresh_intervals - auto refresh intervals, optional
-            * interval - auto refresh interval in milliseconds, integer, required
-            * is_active - is current interval active or not, boolean, required
-            * text - text representation of the interval (e.g. "30 seconds"), string, required
         * docs_enabled - indicates if api docs is enabled, boolean, required
         * index_url - url to the repository index, string, optional
         * repositories - list of repositories unique identifiers, required
@@ -71,19 +66,8 @@ class IndexView(BaseView):
             "username": auth_username,
         }
 
-        autorefresh_intervals = [
-            {
-                "interval": interval * 1000,  # milliseconds
-                "is_active": index == 0,  # first element is always default
-                "text": pretty_interval(interval),
-            }
-            for index, interval in enumerate(self.configuration.getintlist("web", "autorefresh_intervals", fallback=[]))
-            if interval > 0  # special case if 0 exists and first, refresh will not be turned on by default
-        ]
-
         return {
             "auth": auth,
-            "autorefresh_intervals": sorted(autorefresh_intervals, key=lambda interval: interval["interval"]),
             "docs_enabled": aiohttp_apispec is not None,
             "index_url": self.configuration.get("web", "index_url", fallback=None),
             "repositories": [

src/ahriman/web/views/v1/packages/logs.py

@@ -90,7 +90,7 @@ class LogsView(StatusViewGuard, BaseView):
 
         try:
             _, status = self.service().package_get(package_base)
-            logs = self.service(package_base=package_base).package_logs_get(package_base, None, None, -1, 0)
+            logs = self.service(package_base=package_base).package_logs_get(package_base, -1, 0)
         except UnknownPackageError:
             raise HTTPNotFound(reason=f"Package {package_base} is unknown")
 

src/ahriman/web/views/v1/service/config.py

@@ -1,84 +0,0 @@
-#
-# Copyright (c) 2021-2025 ahriman team.
-#
-# This file is part of ahriman
-# (see https://github.com/arcan1s/ahriman).
-#
-# This program is free software: you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation, either version 3 of the License, or
-# (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program. If not, see <http://www.gnu.org/licenses/>.
-#
-from aiohttp.web import HTTPNoContent, Response, json_response
-from typing import ClassVar
-
-from ahriman.core.formatters import ConfigurationPrinter
-from ahriman.models.user_access import UserAccess
-from ahriman.web.apispec.decorators import apidocs
-from ahriman.web.schemas import ConfigurationSchema
-from ahriman.web.views.base import BaseView
-
-
-class ConfigView(BaseView):
-    """
-    configuration control view
-
-    Attributes:
-        GET_PERMISSION(UserAccess): (class attribute) get permissions of self
-        POST_PERMISSION(UserAccess): (class attribute) post permissions of self
-    """
-
-    GET_PERMISSION = POST_PERMISSION = UserAccess.Full  # type: ClassVar[UserAccess]
-    ROUTES = ["/api/v1/service/config"]
-
-    @apidocs(
-        tags=["Actions"],
-        summary="Get configuration",
-        description="Get current web service configuration as nested dictionary",
-        permission=GET_PERMISSION,
-        schema=ConfigurationSchema(many=True),
-    )
-    async def get(self) -> Response:
-        """
-        get current web service configuration
-
-        Returns:
-            Response: current web service configuration as nested dictionary
-        """
-        dump = self.configuration.dump()
-
-        response = [
-            {
-                "section": section,
-                "key": key,
-                "value": value,
-            } for section, values in dump.items()
-            for key, value in values.items()
-            if key not in ConfigurationPrinter.HIDE_KEYS
-        ]
-        return json_response(response)
-
-    @apidocs(
-        tags=["Actions"],
-        summary="Reload configuration",
-        description="Reload configuration from current files",
-        permission=POST_PERMISSION,
-    )
-    async def post(self) -> None:
-        """
-        reload web service configuration
-
-        Raises:
-            HTTPNoContent: on success response
-        """
-        self.configuration.reload()
-
-        raise HTTPNoContent

src/ahriman/web/views/v1/service/search.py

@@ -70,11 +70,7 @@ class SearchView(BaseView):
         if not packages:
             raise HTTPNotFound(reason=f"No packages found for terms: {search}")
 
-        comparator: Callable[[AURPackage], tuple[bool, bool, str]] = lambda item: (
-            item.package_base not in search,  # inverted because False < True
-            not any(item.package_base.startswith(term) for term in search),  # same as above
-            item.package_base,
-        )
+        comparator: Callable[[AURPackage], str] = lambda item: str(item.package_base)
         response = [
             {
                 "package": package.package_base,

src/ahriman/web/views/v2/packages/logs.py

@@ -17,22 +17,18 @@
 # You should have received a copy of the GNU General Public License
 # along with this program. If not, see <http://www.gnu.org/licenses/>.
 #
-import itertools
-
 from aiohttp.web import Response, json_response
-from dataclasses import replace
 from typing import ClassVar
 
 from ahriman.models.user_access import UserAccess
 from ahriman.web.apispec.decorators import apidocs
-from ahriman.web.schemas import LogSchema, LogsSearchSchema, PackageNameSchema
+from ahriman.web.schemas import LogSchema, PackageNameSchema, PaginationSchema
 from ahriman.web.views.base import BaseView
 from ahriman.web.views.status_view_guard import StatusViewGuard
 
 
 class LogsView(StatusViewGuard, BaseView):
-    """        else:
-
+    """
     package logs web view
 
     Attributes:
@@ -51,7 +47,7 @@ class LogsView(StatusViewGuard, BaseView):
         error_404_description="Package base and/or repository are unknown",
         schema=LogSchema(many=True),
         match_schema=PackageNameSchema,
-        query_schema=LogsSearchSchema,
+        query_schema=PaginationSchema,
     )
     async def get(self) -> Response:
         """
@@ -65,19 +61,8 @@ class LogsView(StatusViewGuard, BaseView):
         """
         package_base = self.request.match_info["package"]
         limit, offset = self.page()
-        version = self.request.query.get("version", None)
-        process = self.request.query.get("process_id", None)
 
-        logs = self.service(package_base=package_base).package_logs_get(package_base, version, process, limit, offset)
-
-        head = self.request.query.get("head", "false")
-        # pylint: disable=protected-access
-        if self.configuration._convert_to_boolean(head):  # type: ignore[attr-defined]
-            # logs should be sorted already
-            logs = [
-                replace(next(log_records), message="")  # remove messages
-                for _, log_records in itertools.groupby(logs, lambda log_record: log_record.log_record_id)
-            ]
+        logs = self.service(package_base=package_base).package_logs_get(package_base, limit, offset)
 
         response = [log_record.view() for log_record in logs]
         return json_response(response)

src/ahriman/web/web.py

@@ -166,16 +166,11 @@ def setup_server(configuration: Configuration, spawner: Spawn, repositories: lis
     # package cache
     if not repositories:
         raise InitializeError("No repositories configured, exiting")
+    database = SQLite.load(configuration)
     watchers: dict[RepositoryId, Watcher] = {}
-    configuration_path, _ = configuration.check_loaded()
     for repository_id in repositories:
         application.logger.info("load repository %s", repository_id)
-        # load settings explicitly for architecture if any
-        repository_configuration = Configuration.from_path(configuration_path, repository_id)
-        # load database instance, because it holds identifier
-        database = SQLite.load(repository_configuration)
-        # explicitly load local client
-        client = Client.load(repository_id, repository_configuration, database, report=False)
+        client = Client.load(repository_id, configuration, database, report=False)  # explicitly load local client
         watchers[repository_id] = Watcher(client)
     application[WatcherKey] = watchers
     # workers cache
@@ -184,7 +179,6 @@ def setup_server(configuration: Configuration, spawner: Spawn, repositories: lis
     application[SpawnKey] = spawner
 
     application.logger.info("setup authorization")
-    database = SQLite.load(configuration)
     validator = application[AuthKey] = Auth.load(configuration, database)
     if validator.enabled:
         from ahriman.web.middlewares.auth_handler import setup_auth

tests/ahriman/application/application/test_application_packages.py

@@ -141,7 +141,7 @@ def test_add_remote_missing(application_packages: ApplicationPackages, mocker: M
     """
     must raise UnknownPackageError if remote package wasn't found
     """
-    mocker.patch("requests.get", side_effect=Exception)
+    mocker.patch("requests.get", side_effect=Exception())
     with pytest.raises(UnknownPackageError):
         application_packages._add_remote("url")
 

tests/ahriman/application/application/test_application_repository.py

@@ -135,7 +135,7 @@ def test_unknown_no_aur(application_repository: ApplicationRepository, package_a
     must return empty list in case if there is locally stored PKGBUILD
     """
     mocker.patch("ahriman.core.repository.repository.Repository.packages", return_value=[package_ahriman])
-    mocker.patch("ahriman.models.package.Package.from_aur", side_effect=Exception)
+    mocker.patch("ahriman.models.package.Package.from_aur", side_effect=Exception())
     mocker.patch("ahriman.models.package.Package.from_build", return_value=package_ahriman)
     mocker.patch("pathlib.Path.is_dir", return_value=True)
     mocker.patch("ahriman.core.build_tools.sources.Sources.has_remotes", return_value=False)
@@ -149,7 +149,7 @@ def test_unknown_no_aur_no_local(application_repository: ApplicationRepository,
     must return list of packages missing in aur and in local storage
     """
     mocker.patch("ahriman.core.repository.repository.Repository.packages", return_value=[package_ahriman])
-    mocker.patch("ahriman.models.package.Package.from_aur", side_effect=Exception)
+    mocker.patch("ahriman.models.package.Package.from_aur", side_effect=Exception())
     mocker.patch("pathlib.Path.is_dir", return_value=False)
 
     packages = application_repository.unknown()

tests/ahriman/application/handlers/test_handler.py

@@ -46,7 +46,7 @@ def test_call_exception(args: argparse.Namespace, configuration: Configuration,
     """
     args.configuration = Path("")
     args.quiet = False
-    mocker.patch("ahriman.core.configuration.Configuration.from_path", side_effect=Exception)
+    mocker.patch("ahriman.core.configuration.Configuration.from_path", side_effect=Exception())
     logging_mock = mocker.patch("logging.Logger.exception")
 
     _, repository_id = configuration.check_loaded()
@@ -60,7 +60,7 @@ def test_call_exit_code(args: argparse.Namespace, configuration: Configuration,
     """
     args.configuration = Path("")
     args.quiet = False
-    mocker.patch("ahriman.core.configuration.Configuration.from_path", side_effect=ExitCode)
+    mocker.patch("ahriman.core.configuration.Configuration.from_path", side_effect=ExitCode())
     logging_mock = mocker.patch("logging.Logger.exception")
 
     _, repository_id = configuration.check_loaded()

tests/ahriman/application/handlers/test_handler_reload.py

@@ -1,27 +0,0 @@
-import argparse
-
-from pytest_mock import MockerFixture
-
-from ahriman.application.handlers.reload import Reload
-from ahriman.core.configuration import Configuration
-from ahriman.core.repository import Repository
-
-
-def test_run(args: argparse.Namespace, configuration: Configuration, repository: Repository,
-             mocker: MockerFixture) -> None:
-    """
-    must run command
-    """
-    mocker.patch("ahriman.core.repository.Repository.load", return_value=repository)
-    run_mock = mocker.patch("ahriman.core.status.local_client.LocalClient.configuration_reload")
-
-    _, repository_id = configuration.check_loaded()
-    Reload.run(args, repository_id, configuration, report=False)
-    run_mock.assert_called_once_with()
-
-
-def test_disallow_multi_architecture_run() -> None:
-    """
-    must not allow multi architecture run
-    """
-    assert not Reload.ALLOW_MULTI_ARCHITECTURE_RUN

tests/ahriman/application/handlers/test_handler_setup.py

@@ -58,11 +58,9 @@ def test_run(args: argparse.Namespace, configuration: Configuration, repository:
     sudo_configuration_mock = mocker.patch("ahriman.application.handlers.setup.Setup.configuration_create_sudo")
     executable_mock = mocker.patch("ahriman.application.handlers.setup.Setup.executable_create")
     init_mock = mocker.patch("ahriman.core.alpm.repo.Repo.init")
-    owner_guard_mock = mocker.patch("ahriman.models.repository_paths.RepositoryPaths.preserve_owner")
 
     _, repository_id = configuration.check_loaded()
     Setup.run(args, repository_id, configuration, report=False)
-    owner_guard_mock.assert_called_once_with()
     ahriman_configuration_mock.assert_called_once_with(args, repository_id, configuration)
     devtools_configuration_mock.assert_called_once_with(
         repository_id, args.from_configuration, args.mirror, args.multilib, f"file://{repository_paths.repository}")
@@ -270,11 +268,13 @@ def test_executable_create(configuration: Configuration, repository_paths: Repos
     """
     must create executable
     """
+    chown_mock = mocker.patch("ahriman.models.repository_paths.RepositoryPaths.chown")
     symlink_mock = mocker.patch("pathlib.Path.symlink_to")
     unlink_mock = mocker.patch("pathlib.Path.unlink")
 
     _, repository_id = configuration.check_loaded()
     Setup.executable_create(repository_paths, repository_id)
+    chown_mock.assert_called_once_with(Setup.build_command(repository_paths.root, repository_id))
     symlink_mock.assert_called_once_with(Setup.ARCHBUILD_COMMAND_PATH)
     unlink_mock.assert_called_once_with(missing_ok=True)
 

tests/ahriman/application/test_ahriman.py

@@ -1563,35 +1563,6 @@ def test_subparsers_web_option_repository(parser: argparse.ArgumentParser) -> No
     assert args.repository == ""
 
 
-def test_subparsers_web_reload(parser: argparse.ArgumentParser) -> None:
-    """
-    web-reload command must imply architecture, lock, quiet, report, repository and unsafe
-    """
-    args = parser.parse_args(["web-reload"])
-    assert args.architecture == ""
-    assert args.lock is None
-    assert args.quiet
-    assert not args.report
-    assert args.repository == ""
-    assert args.unsafe
-
-
-def test_subparsers_web_reload_option_architecture(parser: argparse.ArgumentParser) -> None:
-    """
-    web-reload command must correctly parse architecture list
-    """
-    args = parser.parse_args(["-a", "x86_64", "web-reload"])
-    assert args.architecture == ""
-
-
-def test_subparsers_web_reload_option_repository(parser: argparse.ArgumentParser) -> None:
-    """
-    web-reload command must correctly parse repository list
-    """
-    args = parser.parse_args(["-r", "repo", "web-reload"])
-    assert args.repository == ""
-
-
 def test_run(args: argparse.Namespace, configuration: Configuration, mocker: MockerFixture) -> None:
     """
     application must be run

tests/ahriman/application/test_lock.py

@@ -230,7 +230,7 @@ def test_clear_close_exception(lock: Lock) -> None:
     must suppress IO exception on file closure
     """
     close_mock = lock._pid_file = MagicMock()
-    close_mock.close.side_effect = IOError
+    close_mock.close.side_effect = IOError()
     lock.clear()
 
 

tests/ahriman/core/alpm/remote/test_aur.py

@@ -108,7 +108,7 @@ def test_aur_request_failed(aur: AUR, mocker: MockerFixture) -> None:
     """
     must reraise generic exception
     """
-    mocker.patch("requests.Session.request", side_effect=Exception)
+    mocker.patch("requests.Session.request", side_effect=Exception())
     with pytest.raises(Exception):
         aur.aur_request("info", "ahriman")
 
@@ -116,7 +116,7 @@ def test_aur_request_failed(aur: AUR, mocker: MockerFixture) -> None:
 def test_aur_request_failed_http_error(aur: AUR, mocker: MockerFixture) -> None:
     """    must reraise http exception
     """
-    mocker.patch("requests.Session.request", side_effect=requests.HTTPError)
+    mocker.patch("requests.Session.request", side_effect=requests.HTTPError())
     with pytest.raises(requests.HTTPError):
         aur.aur_request("info", "ahriman")
 

tests/ahriman/core/alpm/remote/test_official.py

@@ -80,7 +80,7 @@ def test_arch_request_failed(official: Official, mocker: MockerFixture) -> None:
     """
     must reraise generic exception
     """
-    mocker.patch("requests.Session.request", side_effect=Exception)
+    mocker.patch("requests.Session.request", side_effect=Exception())
     with pytest.raises(Exception):
         official.arch_request("akonadi", by="q")
 
@@ -89,7 +89,7 @@ def test_arch_request_failed_http_error(official: Official, mocker: MockerFixtur
     """
     must reraise http exception
     """
-    mocker.patch("requests.Session.request", side_effect=requests.HTTPError)
+    mocker.patch("requests.Session.request", side_effect=requests.HTTPError())
     with pytest.raises(requests.HTTPError):
         official.arch_request("akonadi", by="q")
 

tests/ahriman/core/alpm/remote/test_remote.py

@@ -1,6 +1,5 @@
 import pytest
 
-from dataclasses import replace
 from pytest_mock import MockerFixture
 from unittest.mock import call as MockCall
 
@@ -89,17 +88,6 @@ def test_multisearch_single(aur_package_ahriman: AURPackage, pacman: Pacman, moc
     search_mock.assert_called_once_with("ahriman", pacman=pacman, search_by=None)
 
 
-def test_multisearch_remove_duplicates(aur_package_ahriman: AURPackage, pacman: Pacman, mocker: MockerFixture) -> None:
-    """
-    must remove duplicates from search result
-    """
-    package1 = replace(aur_package_ahriman)
-    package2 = replace(aur_package_ahriman, name="ahriman-triggers")
-    mocker.patch("ahriman.core.alpm.remote.Remote.package_search", return_value=[package1, package2])
-
-    assert Remote.multisearch("ahriman", pacman=pacman) == [package1]
-
-
 def test_remote_git_url(remote: Remote) -> None:
     """
     must raise NotImplemented for missing remote git url

tests/ahriman/core/alpm/test_pacman.py

@@ -62,12 +62,12 @@ def test_database_copy(pacman: Pacman, mocker: MockerFixture) -> None:
     mocker.patch("pathlib.Path.is_file", autospec=True, side_effect=lambda p: p.is_relative_to(path))
     mkdir_mock = mocker.patch("pathlib.Path.mkdir")
     copy_mock = mocker.patch("shutil.copy")
-    owner_guard_mock = mocker.patch("ahriman.models.repository_paths.RepositoryPaths.preserve_owner")
+    chown_mock = mocker.patch("ahriman.models.repository_paths.RepositoryPaths.chown")
 
     pacman.database_copy(pacman.handle, database, path, use_ahriman_cache=True)
     mkdir_mock.assert_called_once_with(mode=0o755, exist_ok=True)
     copy_mock.assert_called_once_with(path / "sync" / "core.db", dst_path)
-    owner_guard_mock.assert_called_once_with(dst_path.parent)
+    chown_mock.assert_called_once_with(dst_path)
 
 
 def test_database_copy_skip(pacman: Pacman, mocker: MockerFixture) -> None:
@@ -242,7 +242,7 @@ def test_files_no_entry(pacman: Pacman, pyalpm_package_ahriman: pyalpm.Package,
     pacman.handle = handle_mock
 
     tar_mock = MagicMock()
-    tar_mock.extractfile.side_effect = KeyError
+    tar_mock.extractfile.side_effect = KeyError()
 
     open_mock = MagicMock()
     open_mock.__enter__.return_value = tar_mock

tests/ahriman/core/alpm/test_pacman_database.py

@@ -131,7 +131,7 @@ def test_sync_exception(pacman_database: PacmanDatabase, mocker: MockerFixture)
     """
     must suppress all exceptions on failure
     """
-    mocker.patch("ahriman.core.alpm.pacman_database.PacmanDatabase.sync_packages", side_effect=Exception)
+    mocker.patch("ahriman.core.alpm.pacman_database.PacmanDatabase.sync_packages", side_effect=Exception())
     pacman_database.sync(force=True)
 
 

tests/ahriman/core/alpm/test_repo.py

@@ -52,7 +52,7 @@ def test_repo_remove_fail_no_file(repo: Repo, mocker: MockerFixture) -> None:
     must fail on missing file
     """
     mocker.patch("pathlib.Path.glob", return_value=[Path("package.pkg.tar.xz")])
-    mocker.patch("pathlib.Path.unlink", side_effect=FileNotFoundError)
+    mocker.patch("pathlib.Path.unlink", side_effect=FileNotFoundError())
 
     with pytest.raises(FileNotFoundError):
         repo.remove("package", Path("package.pkg.tar.xz"))

tests/ahriman/core/auth/test_oauth.py

@@ -90,7 +90,7 @@ async def test_get_oauth_username_exception_1(oauth: OAuth, mocker: MockerFixtur
     """
     must return None in case of OAuth request error (get_access_token)
     """
-    mocker.patch("aioauth_client.GoogleClient.get_access_token", side_effect=Exception)
+    mocker.patch("aioauth_client.GoogleClient.get_access_token", side_effect=Exception())
     user_info_mock = mocker.patch("aioauth_client.GoogleClient.user_info")
 
     email = await oauth.get_oauth_username("code")
@@ -103,7 +103,7 @@ async def test_get_oauth_username_exception_2(oauth: OAuth, mocker: MockerFixtur
     must return None in case of OAuth request error (user_info)
     """
     mocker.patch("aioauth_client.GoogleClient.get_access_token", return_value=("token", ""))
-    mocker.patch("aioauth_client.GoogleClient.user_info", side_effect=Exception)
+    mocker.patch("aioauth_client.GoogleClient.user_info", side_effect=Exception())
 
     email = await oauth.get_oauth_username("code")
     assert email is None

tests/ahriman/core/configuration/test_configuration.py

@@ -133,14 +133,6 @@ def test_dump_architecture_specific(configuration: Configuration) -> None:
     assert dump["build"]["archbuild_flags"] == "hello flag"
 
 
-def test_getintlist(configuration: Configuration) -> None:
-    """
-    must extract list of integers
-    """
-    configuration.set_option("build", "test_int_list", "1 42 3")
-    assert configuration.getintlist("build", "test_int_list") == [1, 42, 3]
-
-
 def test_getlist(configuration: Configuration) -> None:
     """
     must return list of string correctly

tests/ahriman/core/database/migrations/test_m011_repository_name.py

@@ -6,6 +6,7 @@ from unittest.mock import call as MockCall
 
 from ahriman.core.configuration import Configuration
 from ahriman.core.database.migrations.m011_repository_name import migrate_data, migrate_package_repository, steps
+from ahriman.models.repository_id import RepositoryId
 
 
 def test_migration_repository_name() -> None:
@@ -37,3 +38,13 @@ def test_migrate_package_repository(connection: Connection, configuration: Confi
         MockCall(pytest.helpers.anyvar(str, strict=True), {"repository": configuration.repository_id.id}),
         MockCall(pytest.helpers.anyvar(str, strict=True), {"repository": configuration.repository_id.id}),
     ])
+
+
+def test_migrate_package_repository_empty_id(connection: Connection, configuration: Configuration,
+                                             mocker: MockerFixture) -> None:
+    """
+    must do nothing on empty repository id
+    """
+    mocker.patch("ahriman.core.configuration.Configuration.check_loaded", return_value=("", RepositoryId("", "")))
+    migrate_package_repository(connection, configuration)
+    connection.execute.assert_not_called()

tests/ahriman/core/database/migrations/test_migrations_init.py

@@ -42,7 +42,7 @@ def test_apply_migration_exception(migrations: Migrations, mocker: MockerFixture
     must roll back and close cursor on exception during migration
     """
     cursor = MagicMock()
-    mocker.patch("logging.Logger.info", side_effect=Exception)
+    mocker.patch("logging.Logger.info", side_effect=Exception())
     migrations.connection.cursor.return_value = cursor
 
     with pytest.raises(Exception):
@@ -59,7 +59,7 @@ def test_apply_migration_sql_exception(migrations: Migrations) -> None:
     must close cursor on general migration error
     """
     cursor = MagicMock()
-    cursor.execute.side_effect = Exception
+    cursor.execute.side_effect = Exception()
     migrations.connection.cursor.return_value = cursor
 
     with pytest.raises(Exception):

tests/ahriman/core/database/operations/test_logs_operations.py

@@ -71,35 +71,11 @@ def test_logs_insert_get_pagination(database: SQLite, package_ahriman: Package)
     """
     database.logs_insert(LogRecord(LogRecordId(package_ahriman.base, "1"), 42.0, "message 1"))
     database.logs_insert(LogRecord(LogRecordId(package_ahriman.base, "1"), 43.0, "message 2"))
-    assert database.logs_get(package_ahriman.base, None, None, 1, 1) == [
+    assert database.logs_get(package_ahriman.base, 1, 1) == [
         LogRecord(LogRecordId(package_ahriman.base, "1"), 42.0, "message 1"),
     ]
 
 
-def test_logs_insert_get_filter_by_version(database: SQLite, package_ahriman: Package) -> None:
-    """
-    must insert and get package logs with pagination
-    """
-    database.logs_insert(LogRecord(LogRecordId(package_ahriman.base, "1", "p1"), 42.0, "message 1"))
-    database.logs_insert(LogRecord(LogRecordId(package_ahriman.base, "1", "p2"), 43.0, "message 2"))
-    database.logs_insert(LogRecord(LogRecordId(package_ahriman.base, "2", "p1"), 44.0, "message 3"))
-
-    assert database.logs_get(package_ahriman.base, "1", None) == [
-        LogRecord(LogRecordId(package_ahriman.base, "1", "p1"), 42.0, "message 1"),
-        LogRecord(LogRecordId(package_ahriman.base, "1", "p2"), 43.0, "message 2"),
-    ]
-    assert database.logs_get(package_ahriman.base, "2", None) == [
-        LogRecord(LogRecordId(package_ahriman.base, "2", "p1"), 44.0, "message 3"),
-    ]
-    assert database.logs_get(package_ahriman.base, None, "p1") == [
-        LogRecord(LogRecordId(package_ahriman.base, "1", "p1"), 42.0, "message 1"),
-        LogRecord(LogRecordId(package_ahriman.base, "2", "p1"), 44.0, "message 3"),
-    ]
-    assert database.logs_get(package_ahriman.base, "1", "p1") == [
-        LogRecord(LogRecordId(package_ahriman.base, "1", "p1"), 42.0, "message 1"),
-    ]
-
-
 def test_logs_insert_get_multi(database: SQLite, package_ahriman: Package) -> None:
     """
     must insert and get package logs for multiple repositories

tests/ahriman/core/database/test_sqlite.py

@@ -36,17 +36,6 @@ def test_init_skip_migration(database: SQLite, mocker: MockerFixture) -> None:
     migrate_schema_mock.assert_not_called()
 
 
-def test_init_skip_empty_repository(database: SQLite, mocker: MockerFixture) -> None:
-    """
-    must skip migrations if repository identifier is not set
-    """
-    database._repository_id = RepositoryId("", "")
-    migrate_schema_mock = mocker.patch("ahriman.core.database.migrations.Migrations.migrate")
-
-    database.init()
-    migrate_schema_mock.assert_not_called()
-
-
 def test_package_clear(database: SQLite, repository_id: RepositoryId, mocker: MockerFixture) -> None:
     """
     must clear package data

tests/ahriman/core/distributed/test_distributed_system.py

@@ -37,7 +37,7 @@ def test_register_failed(distributed_system: DistributedSystem, mocker: MockerFi
     """
     must suppress any exception happened during worker registration
     """
-    mocker.patch("requests.Session.request", side_effect=Exception)
+    mocker.patch("requests.Session.request", side_effect=Exception())
     distributed_system.register()
 
 
@@ -45,7 +45,7 @@ def test_register_failed_http_error(distributed_system: DistributedSystem, mocke
     """
     must suppress HTTP exception happened during worker registration
     """
-    mocker.patch("requests.Session.request", side_effect=requests.HTTPError)
+    mocker.patch("requests.Session.request", side_effect=requests.HTTPError())
     distributed_system.register()
 
 
@@ -70,7 +70,7 @@ def test_workers_failed(distributed_system: DistributedSystem, mocker: MockerFix
     """
     must suppress any exception happened during worker extraction
     """
-    mocker.patch("requests.Session.request", side_effect=Exception)
+    mocker.patch("requests.Session.request", side_effect=Exception())
     distributed_system.workers()
 
 
@@ -78,5 +78,5 @@ def test_workers_failed_http_error(distributed_system: DistributedSystem, mocker
     """
     must suppress HTTP exception happened during worker extraction
     """
-    mocker.patch("requests.Session.request", side_effect=requests.HTTPError)
+    mocker.patch("requests.Session.request", side_effect=requests.HTTPError())
     distributed_system.workers()

tests/ahriman/core/gitremote/test_remote_pull.py

@@ -85,7 +85,7 @@ def test_run_failed(configuration: Configuration, mocker: MockerFixture) -> None
     """
     must reraise exception on error occurred
     """
-    mocker.patch("ahriman.core.gitremote.remote_pull.RemotePull.repo_clone", side_effect=Exception)
+    mocker.patch("ahriman.core.gitremote.remote_pull.RemotePull.repo_clone", side_effect=Exception())
     _, repository_id = configuration.check_loaded()
     runner = RemotePull(repository_id, configuration, "gitremote")
 

tests/ahriman/core/gitremote/test_remote_push.py

@@ -82,7 +82,7 @@ def test_run_failed(local_client: Client, configuration: Configuration, result:
     """
     must reraise exception on error occurred
     """
-    mocker.patch("ahriman.core.build_tools.sources.Sources.fetch", side_effect=Exception)
+    mocker.patch("ahriman.core.build_tools.sources.Sources.fetch", side_effect=Exception())
     runner = RemotePush(local_client, configuration, "gitremote")
 
     with pytest.raises(GitRemoteError):

tests/ahriman/core/housekeeping/conftest.py

@@ -1,19 +0,0 @@
-import pytest
-
-from ahriman.core.configuration import Configuration
-from ahriman.core.housekeeping import LogsRotationTrigger
-
-
-@pytest.fixture
-def logs_rotation_trigger(configuration: Configuration) -> LogsRotationTrigger:
-    """
-    logs roration trigger fixture
-
-    Args:
-        configuration(Configuration): configuration fixture
-
-    Returns:
-        LogsRotationTrigger: logs rotation trigger test instance
-    """
-    _, repository_id = configuration.check_loaded()
-    return LogsRotationTrigger(repository_id, configuration)

tests/ahriman/core/housekeeping/test_logs_rotation_trigger.py

@@ -1,26 +0,0 @@
-from pytest_mock import MockerFixture
-from unittest.mock import MagicMock
-
-from ahriman.core.configuration import Configuration
-from ahriman.core.housekeeping import LogsRotationTrigger
-from ahriman.core.status import Client
-from ahriman.models.result import Result
-
-
-def test_configuration_sections(configuration: Configuration) -> None:
-    """
-    must correctly parse target list
-    """
-    assert LogsRotationTrigger.configuration_sections(configuration) == ["logs-rotation"]
-
-
-def test_rotate(logs_rotation_trigger: LogsRotationTrigger, mocker: MockerFixture) -> None:
-    """
-    must rotate logs
-    """
-    client_mock = MagicMock()
-    context_mock = mocker.patch("ahriman.core._Context.get", return_value=client_mock)
-
-    logs_rotation_trigger.on_result(Result(), [])
-    context_mock.assert_called_once_with(Client)
-    client_mock.logs_rotate.assert_called_once_with(logs_rotation_trigger.keep_last_records)

tests/ahriman/core/http/test_sync_ahriman_client.py

@@ -51,7 +51,7 @@ def test_login_failed(ahriman_client: SyncAhrimanClient, user: User, mocker: Moc
     must suppress any exception happened during login
     """
     ahriman_client.user = user
-    mocker.patch("requests.Session.request", side_effect=Exception)
+    mocker.patch("requests.Session.request", side_effect=Exception())
     ahriman_client._login(requests.Session())
 
 
@@ -60,7 +60,7 @@ def test_login_failed_http_error(ahriman_client: SyncAhrimanClient, user: User,
     must suppress HTTP exception happened during login
     """
     ahriman_client.user = user
-    mocker.patch("requests.Session.request", side_effect=requests.HTTPError)
+    mocker.patch("requests.Session.request", side_effect=requests.HTTPError())
     ahriman_client._login(requests.Session())
 
 

tests/ahriman/core/http/test_sync_http_client.py

@@ -124,7 +124,7 @@ def test_make_request_failed(mocker: MockerFixture) -> None:
     """
     must process request errors
     """
-    mocker.patch("requests.Session.request", side_effect=Exception)
+    mocker.patch("requests.Session.request", side_effect=Exception())
     logging_mock = mocker.patch("logging.Logger.exception")
 
     with pytest.raises(Exception):
@@ -136,7 +136,7 @@ def test_make_request_suppress_errors(mocker: MockerFixture) -> None:
     """
     must suppress request errors correctly
     """
-    mocker.patch("requests.Session.request", side_effect=Exception)
+    mocker.patch("requests.Session.request", side_effect=Exception())
     logging_mock = mocker.patch("logging.Logger.exception")
 
     with pytest.raises(Exception):

tests/ahriman/core/log/test_http_log_handler.py

@@ -20,12 +20,14 @@ def test_load(configuration: Configuration, mocker: MockerFixture) -> None:
 
     add_mock = mocker.patch("logging.Logger.addHandler")
     load_mock = mocker.patch("ahriman.core.status.Client.load")
+    atexit_mock = mocker.patch("atexit.register")
 
     _, repository_id = configuration.check_loaded()
     handler = HttpLogHandler.load(repository_id, configuration, report=False)
     assert handler
     add_mock.assert_called_once_with(handler)
     load_mock.assert_called_once_with(repository_id, configuration, report=False)
+    atexit_mock.assert_called_once_with(handler.rotate)
 
 
 def test_load_exist(configuration: Configuration) -> None:
@@ -59,7 +61,7 @@ def test_emit_failed(configuration: Configuration, log_record: logging.LogRecord
     must call handle error on exception
     """
     log_record.package_id = LogRecordId(package_ahriman.base, package_ahriman.version)
-    mocker.patch("ahriman.core.status.Client.package_logs_add", side_effect=Exception)
+    mocker.patch("ahriman.core.status.Client.package_logs_add", side_effect=Exception())
     handle_error_mock = mocker.patch("logging.Handler.handleError")
     _, repository_id = configuration.check_loaded()
     handler = HttpLogHandler(repository_id, configuration, report=False, suppress_errors=False)
@@ -74,7 +76,7 @@ def test_emit_suppress_failed(configuration: Configuration, log_record: logging.
     must not call handle error on exception if suppress flag is set
     """
     log_record.package_id = LogRecordId(package_ahriman.base, package_ahriman.version)
-    mocker.patch("ahriman.core.status.Client.package_logs_add", side_effect=Exception)
+    mocker.patch("ahriman.core.status.Client.package_logs_add", side_effect=Exception())
     handle_error_mock = mocker.patch("logging.Handler.handleError")
     _, repository_id = configuration.check_loaded()
     handler = HttpLogHandler(repository_id, configuration, report=False, suppress_errors=True)
@@ -94,3 +96,16 @@ def test_emit_skip(configuration: Configuration, log_record: logging.LogRecord,
 
     handler.emit(log_record)
     log_mock.assert_not_called()
+
+
+def test_rotate(configuration: Configuration, mocker: MockerFixture) -> None:
+    """
+    must rotate logs
+    """
+    rotate_mock = mocker.patch("ahriman.core.status.Client.logs_rotate")
+
+    _, repository_id = configuration.check_loaded()
+    handler = HttpLogHandler(repository_id, configuration, report=False, suppress_errors=False)
+
+    handler.rotate()
+    rotate_mock.assert_called_once_with(handler.keep_last_records)

tests/ahriman/core/log/test_log_loader.py

@@ -61,7 +61,7 @@ def test_load_fallback(configuration: Configuration, mocker: MockerFixture) -> N
     """
     must fall back to stderr without errors
     """
-    mocker.patch("ahriman.core.log.log_loader.fileConfig", side_effect=PermissionError)
+    mocker.patch("ahriman.core.log.log_loader.fileConfig", side_effect=PermissionError())
     _, repository_id = configuration.check_loaded()
     LogLoader.load(repository_id, configuration, LogHandler.Journald, quiet=False, report=False)
 

tests/ahriman/core/report/test_report.py

@@ -13,7 +13,7 @@ def test_report_failure(configuration: Configuration, mocker: MockerFixture) ->
     """
     must raise ReportFailed on errors
     """
-    mocker.patch("ahriman.core.report.html.HTML.generate", side_effect=Exception)
+    mocker.patch("ahriman.core.report.html.HTML.generate", side_effect=Exception())
     _, repository_id = configuration.check_loaded()
 
     with pytest.raises(ReportError):

tests/ahriman/core/report/test_telegram.py

@@ -41,7 +41,7 @@ def test_send_failed(telegram: Telegram, mocker: MockerFixture) -> None:
     """
     must reraise generic exception
     """
-    mocker.patch("requests.Session.request", side_effect=Exception)
+    mocker.patch("requests.Session.request", side_effect=Exception())
     with pytest.raises(Exception):
         telegram._send("a text")
 
@@ -50,7 +50,7 @@ def test_send_failed_http_error(telegram: Telegram, mocker: MockerFixture) -> No
     """
     must reraise http exception
     """
-    mocker.patch("requests.Session.request", side_effect=requests.HTTPError)
+    mocker.patch("requests.Session.request", side_effect=requests.HTTPError())
     with pytest.raises(requests.HTTPError):
         telegram._send("a text")
 

tests/ahriman/core/repository/test_executor.py

@@ -67,7 +67,7 @@ def test_process_build_failure(executor: Executor, package_ahriman: Package, moc
     mocker.patch("ahriman.core.repository.executor.Executor.packages_built")
     mocker.patch("ahriman.core.build_tools.task.Task.build", return_value=[Path(package_ahriman.base)])
     mocker.patch("ahriman.core.build_tools.task.Task.init")
-    mocker.patch("shutil.move", side_effect=Exception)
+    mocker.patch("shutil.move", side_effect=Exception())
     status_client_mock = mocker.patch("ahriman.core.status.Client.set_failed")
 
     executor.process_build([package_ahriman])
@@ -151,7 +151,7 @@ def test_process_remove_failed(executor: Executor, package_ahriman: Package, moc
     must suppress tree clear errors during package base removal
     """
     mocker.patch("ahriman.core.repository.executor.Executor.packages", return_value=[package_ahriman])
-    mocker.patch("ahriman.core.status.local_client.LocalClient.package_remove", side_effect=Exception)
+    mocker.patch("ahriman.core.status.local_client.LocalClient.package_remove", side_effect=Exception())
     executor.process_remove([package_ahriman.base])
 
 
@@ -160,7 +160,7 @@ def test_process_remove_tree_clear_failed(executor: Executor, package_ahriman: P
     must suppress remove errors
     """
     mocker.patch("ahriman.core.repository.executor.Executor.packages", return_value=[package_ahriman])
-    mocker.patch("ahriman.core.alpm.repo.Repo.remove", side_effect=Exception)
+    mocker.patch("ahriman.core.alpm.repo.Repo.remove", side_effect=Exception())
     executor.process_remove([package_ahriman.base])
 
 
@@ -277,7 +277,7 @@ def test_process_update_failed(executor: Executor, package_ahriman: Package, moc
     """
     must process update for failed package
     """
-    mocker.patch("shutil.move", side_effect=Exception)
+    mocker.patch("shutil.move", side_effect=Exception())
     mocker.patch("ahriman.core.repository.executor.Executor.load_archives", return_value=[package_ahriman])
     mocker.patch("ahriman.core.repository.executor.Executor.packages", return_value=[package_ahriman])
     status_client_mock = mocker.patch("ahriman.core.status.Client.set_failed")