Ticket 18274 - Adding energy data to jobcomp_common.c
Summary: Adding energy data to jobcomp_common.c
Status: RESOLVED FIXED
Alias: None
Product: Slurm
Classification: Unclassified
Component: Accounting (show other tickets)
Version: 23.11.0
Hardware: Linux Linux
: C - Contributions
Assignee: Danny Auble
QA Contact:
URL:
Depends on:
Blocks:
 
Reported: 2023-11-23 08:09 MST by Massimo Benini
Modified: 2023-12-01 13:29 MST (History)
0 users

See Also:
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 (4.00 KB, application/x-tar)
2023-11-23 08:09 MST, Massimo Benini 		Details
Show Obsolete (1) Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this ticket.
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.