Latest GEOPM Update #1774

jcsiadal · 2023-04-03T19:42:44Z

Updates GEOPM to version 2.0.1. There are two issues that will need to be addressed:

GEOPM uses separate service and policy components. These are both easily built in one RPM spec to generate two RPMs. The service RPM is installed directly into root fs (/) as a systemd service and the policy RPM is built using a specific toolchain. The issue is that the service RPM is rebuilt for each toolchain (it's a very fast build). Our other option is to split into two specfiles or implement OBS-checking logic and only build under one toolchain.
I've added a python-dasbus component under distro-packages. The el8 and el9 (not SUSE) distro packages are too old to support GEOPM 2. Dasbus is a python3 module required by the GEOPM service at runtime.

Signed-off-by: jcsiadal <[email protected]>

github-actions · 2023-04-03T19:53:30Z

Test Results

  4 files ±0   4 suites ±0 0s ⏱️ ±0s
17 tests ±0 17 ✔️ ±0 0 💤 ±0 0 ❌ ±0
18 runs ±0 18 ✔️ ±0 0 💤 ±0 0 ❌ ±0

Results for commit 1f9d37a. ± Comparison against base commit 5e38a37.

♻️ This comment has been updated with latest results.

Signed-off-by: jcsiadal <[email protected]>

martin-g · 2023-04-04T07:52:09Z

components/perf-tools/geopm/SPECS/geopm.spec

-%if 0%{?suse_version}
-Buildrequires: libelf1
+%if 0%{?suse_version} || 0%{?sle_version}
+Buildrequires: libelf-devel


Suggested change

Buildrequires: libelf-devel

BuildRequires: libelf-devel

github-actions · 2024-07-31T00:27:06Z

A friendly reminder that this PR had no activity for 30 days.

Add dasbus update

c00f3d9

Signed-off-by: jcsiadal <[email protected]>

jcsiadal force-pushed the geopm branch from e15a742 to 1b31d5b Compare April 3, 2023 20:06

Update GEOPM to latest

1f9d37a

Signed-off-by: jcsiadal <[email protected]>

jcsiadal force-pushed the geopm branch 2 times, most recently from b082721 to 1f9d37a Compare April 3, 2023 22:54

martin-g reviewed Apr 4, 2023

View reviewed changes

github-actions bot added the stale-pr label Jul 31, 2024