Bug 2315606

Summary: Review Request: geopmd - GEOPM daemon
Product: [Fedora] Fedora Reporter: Ali Erdinc Koroglu <aekoroglu>
Component: Package ReviewAssignee: Simone Caronni <negativo17>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: unspecified    
Version: rawhideCC: christopher.m.cantalupo, negativo17, package-review
Target Milestone: ---Flags: negativo17: fedora-review+
Target Release: ---   
Hardware: Unspecified   
OS: Linux   
URL: https://geopm.github.io
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2025-01-12 14:02:50 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 2315604    
Bug Blocks: 2315607    

Description Ali Erdinc Koroglu 2024-09-30 06:40:52 UTC
SPEC Url: https://download.copr.fedorainfracloud.org/results/aekoroglu/geopm/fedora-rawhide-x86_64/08090545-geopmd/geopmd.spec
SRPM Url: https://download.copr.fedorainfracloud.org/results/aekoroglu/geopm/fedora-rawhide-x86_64/08090545-geopmd/geopmd-3.1.0-1.fc42.src.rpm

Description:
The Global Extensible Open Power Manager (GEOPM) provides a framework to
explore power and energy optimizations on platforms with heterogeneous mixes
of computing hardware.

Users can monitor their system's energy and power consumption, and safely
optimize system hardware settings to achieve energy efficiency and/or
performance objectives.

Reproducible: Always

Comment 1 Fedora Review Service 2024-09-30 06:43:13 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/8091490
(failed)

Build log:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2315606-geopmd/fedora-rawhide-x86_64/08091490-geopmd/builder-live.log.gz

Please make sure the package builds successfully at least for Fedora Rawhide.

- If the build failed for unrelated reasons (e.g. temporary network
  unavailability), please ignore it.
- If the build failed because of missing BuildRequires, please make sure they
  are listed in the "Depends On" field


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 2 Simone Caronni 2024-12-01 11:30:36 UTC
SPEC file uses a mix of spaces and tabs (switch occurs in line 12-13).

Line with "Name:" is misaligned with the rest of the other directives, if you want to adjust that as well.

Package approved. Please adjust spaces/tabs before importing.

As the other geopm packages, I think it would make more sense to build all from one single spec file.

Comment 3 Fedora Admin user for bugzilla script actions 2024-12-09 16:33:07 UTC
The Pagure repository was created at https://src.fedoraproject.org/rpms/geopmd