Ticket 18274

Summary: Adding energy data to jobcomp_common.c
Product: Slurm Reporter: Massimo Benini <massimo.benini>
Component: AccountingAssignee: Danny Auble <da>
Status: RESOLVED FIXED QA Contact:
Severity: C - Contributions    
Priority: ---    
Version: 23.11.0   
Hardware: Linux   
OS: Linux   
Site: CSCS - Swiss National Supercomputing Centre Alineos Sites: ---
Atos/Eviden Sites: --- Confidential Site: ---
Coreweave sites: --- Cray Sites: ---
DS9 clusters: --- HPCnow Sites: ---
HPE Sites: --- IBM Sites: ---
NOAA SIte: --- NoveTech Sites: ---
Nvidia HWinf-CS Sites: --- OCF Sites: ---
Recursion Pharma Sites: --- SFW Sites: ---
SNIC sites: --- Tzag Elita Sites: ---
Linux Distro: --- Machine Name:
CLE Version: Version Fixed: 23.11.1 24.08.0rc1
Target Release: --- DevPrio: ---
Emory-Cloud Sites: ---
Attachments: it is containing the diff with the modifications of 2 files: plugins/common/jobcomp_common.c and slurmctld/job_mgr.c

Description Massimo Benini 2023-11-23 08:09:11 MST 
Created attachment 33444 [details]
it is containing the diff with the modifications of 2 files: plugins/common/jobcomp_common.c and slurmctld/job_mgr.c

Hi,

as spoken with Danny at SC23, at CSCS we have the need to add energy used in jobcomp_common, in order to be sent out through the Kafka or the elastic plugin.
Comment 1 Danny Auble 2023-12-01 13:22:29 MST 
Comment on attachment 33444 [details]
it is containing the diff with the modifications of 2 files: plugins/common/jobcomp_common.c and slurmctld/job_mgr.c

Thanks Massimo! I altered the patch a bit to print out tres_req_raw as well as alloc_raw.  I didn't want to change the current variables to avoid messing others up.
Comment 2 Danny Auble 2023-12-01 13:22:59 MST 
Please reopen if needed.
Comment 3 Danny Auble 2023-12-01 13:29:31 MST 
Sorry, this was fixed in commits 0387d0a7c5..a53d2afa3e.