Merging pull request 240

Signed-off-by: Lukáš Doktor <ldoktor@redhat.com>

* github.com:distributed-system-analysis/run-perf:
  contrib: Move contrib to runperf/contrib
  .workflows: Remove code-coverage workflow
  Port from setup.py to pyproject.toml

Author: ldoktor

.github/workflows/ci.yml

@@ -15,42 +15,14 @@ jobs:
         python-version: [3.12]
       fail-fast: false
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
         with:
-          fetch-depth: 500
-      - name: Fetch tags required for run-perf version
-        run: git fetch --tags --depth 500
-      - uses: actions/setup-python@v2
+          fetch-depth: 1
+          fetch-tags: true
+      - uses: actions/setup-python@v5
         with:
           python-version: ${{ matrix.python-version }}
       - name: Install missing aexpect dependency (remove when https://github.com/avocado-framework/aexpect/pull/81 is merged)
         run: pip install six numpy
-      - run: pip install -r requirements.txt
+      - run: pip install -e .[dev]
       - run: make check
-
-  code_coverage:
-    name: CodeCoverage
-    runs-on: ubuntu-latest
-    strategy:
-      fail-fast: false
-    steps:
-      - uses: actions/checkout@v2
-        with:
-          fetch-depth: 500
-      - name: Fetch tags required for run-perf version
-        run: git fetch --tags --depth 500
-      - uses: actions/setup-python@v2
-        with:
-          python-version: ${{ matrix.python-version }}
-      - name: Install missing aexpect dependency (remove when https://github.com/avocado-framework/aexpect/pull/81 is merged)
-        run: pip install six
-      - run: pip install -r requirements.txt
-      - run: |
-          pip install coverage
-          curl -L https://codeclimate.com/downloads/test-reporter/test-reporter-latest-linux-amd64 > ./cc-test-reporter
-          chmod +x ./cc-test-reporter
-          ./cc-test-reporter before-build
-          make develop && ./selftests/run_coverage
-          ./cc-test-reporter after-build
-        env:
-          CC_TEST_REPORTER_ID: 009b4c0bfafb850daeb66460df98eded574477a064df0b4a75f65752b18b1d01

Makefile

@@ -6,7 +6,7 @@ all:
 	@echo
 	@echo "Development related targets:"
 	@echo "check:    Executes selftests"
-	@echo "develop:  Runs 'python setup.py --develop' on this tree alone"
+	@echo "develop:  Deploy run-perf editable from this directory"
 	@echo "clean:    Get rid of scratch, byte files and removes the links to other subprojects"
 	@echo "docs:     Build html docs in docs/build/html/ dir"
 	@echo "html_result:  Refresh the docs/source/_static/html_result.html from selftests/.assets/results"
@@ -25,7 +25,7 @@ coverage: clean develop
 	./selftests/run_coverage
 
 develop:
-	$(PYTHON) -m pip install -e .
+	$(PYTHON) -m pip install -e ".[dev]"
 
 clean:
 	$(PYTHON) -m pip uninstall -y runperf
@@ -36,8 +36,8 @@ clean:
 docs: develop
 	make -C docs html
 
-pypi: clean develop
-	RUNPERF_RELEASE=yes $(PYTHON) setup.py sdist bdist_wheel
+pypi: clean
+	$(PYTHON) -m build
 	@echo
 	@echo
 	@echo "Use 'python3 -m twine upload dist/*'"

docs/source/conf.py

@@ -17,7 +17,7 @@
 import sys
 ROOT_PATH = os.path.abspath(os.path.join(os.path.pardir, os.path.pardir))
 sys.path.insert(0, ROOT_PATH)
-from setup import _get_git_version  # pylint: disable=C0413
+from runperf.version import __version__  # pylint: disable=C0413
 
 # -- Project information -----------------------------------------------------
 
@@ -26,15 +26,7 @@
 author = 'Lukáš Doktor'
 
 # The full version, including alpha/beta/rc tags
-release = _get_git_version()
-if release == '0.0':
-    # Probably in shallow-cloned git, fetch the latest tag
-    try:
-        subprocess.call([shutil.which("git"), "fetch",  # nosec
-                         "--depth=500"])
-        release = _get_git_version()
-    except subprocess.SubprocessError:
-        pass
+release = __version__
 
 # -- API docs ----------------------------------------------------------------
 API_SOURCE_DIR = os.path.join(ROOT_PATH, 'runperf')

pyproject.toml

@@ -0,0 +1,91 @@
+[build-system]
+requires = ["setuptools>=64", "setuptools-scm>=8"]
+build-backend = "setuptools.build_meta"
+
+[project]
+name = "runperf"
+description = "Helper to execute perf-beaker-tasks locally or in VM"
+readme = "README.rst"
+license = "GPL-2.0-or-later"
+authors = [
+    {name = "Lukas Doktor", email = "ldoktor@redhat.com"}
+]
+classifiers = [
+    "Development Status :: 4 - Beta",
+    "Intended Audience :: Developers",
+    "Natural Language :: English",
+    "Operating System :: POSIX",
+    "Topic :: Software Development :: Quality Assurance",
+    "Topic :: Software Development :: Testing",
+    "Programming Language :: Python :: 3",
+]
+requires-python = ">=3.9"
+dependencies = [
+    "aexpect>=1.5.1",
+    "PyYAML",
+    "numpy",
+]
+dynamic = ["version"]
+
+[project.optional-dependencies]
+dev = [
+    "inspektor",
+    "Sphinx",
+    "jinja2",
+    "build",
+]
+
+[project.urls]
+Homepage = "https://github.com/distributed-system-analysis/run-perf"
+
+[project.entry-points."runperf.profiles"]
+"50Localhost" = "runperf.profiles:Localhost"
+"50DefaultLibvirt" = "runperf.profiles:DefaultLibvirt"
+"50DefaultLibvirtMulti" = "runperf.profiles:DefaultLibvirtMulti"
+"50Overcommit1_5" = "runperf.profiles:Overcommit1p5"
+"50TunedLibvirt" = "runperf.profiles:TunedLibvirt"
+
+[project.entry-points."runperf.tests"]
+"50DummyTest" = "runperf.tests:DummyTest"
+"50PBenchFio" = "runperf.tests:PBenchFio"
+"50Linpack" = "runperf.tests:Linpack"
+"50UPerf" = "runperf.tests:UPerf"
+"50PBenchNBD" = "runperf.tests:PBenchNBD"
+"50PBenchLibblkio" = "runperf.tests:PBenchLibblkio"
+
+[project.entry-points."runperf.machine.distro_info"]
+"50get_distro_info" = "runperf.machine:get_distro_info"
+
+[project.entry-points."runperf.utils.cloud_image_providers"]
+"50Fedora" = "runperf.utils.cloud_image_providers:Fedora"
+
+[project.entry-points."runperf.provisioners"]
+"50Beaker" = "runperf.provisioners:Beaker"
+
+[project.entry-points."runperf.utils.pbench"]
+"50Dnf" = "runperf.utils.pbench:Dnf"
+
+[tool.setuptools.packages.find]
+exclude = ["selftests*"]
+
+[project.scripts]
+runperf-contrib-path = "runperf.contrib:main"
+
+[tool.setuptools.package-data]
+"*" = ["assets/**/*", "contrib/**/*"]
+
+[tool.setuptools]
+script-files = [
+    "scripts/run-perf",
+    "scripts/analyze-perf",
+    "scripts/compare-perf",
+    "scripts/diff-perf",
+    "scripts/strip-run-perf",
+]
+
+[tool.setuptools_scm]
+# Version format:
+# - Tagged release: X.Y.Z (e.g., 1.2)
+# - Development: X.Y.devN+gHASH (e.g., 1.2.dev86+g62d6338)
+# - Dirty: X.Y.devN+gHASH.dirty (e.g., 1.2.dev86+g62d6338.dirty)
+local_scheme = "dirty-tag"

requirements.txt

@@ -0,0 +1,42 @@
+#!/bin/env python3
+# 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 2 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 LICENSE for more details.
+#
+# Copyright: Red Hat Inc. 2024
+# Author: Lukas Doktor <ldoktor@redhat.com>
+
+"""Helper to locate contrib scripts directory"""
+
+import os
+import sys
+
+
+def get_contrib_path():
+    """
+    Return the path to the contrib scripts directory.
+
+    Works for both editable installs and regular pip installs.
+    """
+    return os.path.join(os.path.dirname(__file__), 'contrib')
+
+
+def main():
+    """Print the path to contrib scripts directory"""
+    path = get_contrib_path()
+    if not os.path.isdir(path):
+        sys.stderr.write(f"Warning: contrib directory not found at {path}\n")
+        return 1
+    print(path)
+    return 0
+
+
+if __name__ == '__main__':
+    sys.exit(main())