Ticket 11752 - Curious why acct_gather_energy/cray_aries was renamed to acct_gather_energy/pm_counters in 20.11
Summary: Curious why acct_gather_energy/cray_aries was renamed to acct_gather_energy/p...
Status: RESOLVED INFOGIVEN
Alias: None
Product: Slurm
Classification: Unclassified
Component: Configuration (show other tickets)
Version: 20.11.7
Hardware: Linux Linux
: 4 - Minor Issue
Assignee: Jason Booth
QA Contact:
URL:
Depends on:
Blocks:
 
Reported: 2021-06-02 15:42 MDT by Chris Samuel (NERSC)
Modified: 2021-06-09 14:03 MDT (History)
3 users (show)

See Also:
Site: NERSC
Slinky Site: ---
Alineos Sites: ---
Atos/Eviden Sites: ---
Confidential Site: ---
Coreweave sites: ---
Cray Sites: ---
DS9 clusters: ---
Google sites: ---
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:
Target Release: ---
DevPrio: ---
Emory-Cloud Sites: ---

Attachments
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 Chris Samuel (NERSC) 2021-06-02 15:42:01 MDT 
Hey folks - we just discovered:

commit 9fa00eec067372150427631b1a21d405905e4f61
Author: Tim Wickberg <tim@schedmd.com>
Date:   Thu Aug 20 15:24:42 2020 -0600

    Rename acct_gather_energy/cray_aries to acct_gather_energy/pm_counters.
    
    Bug 7575.

but that bug is private - is it possible to find out the thinking behind this change please?

Thanks!
Chris
Comment 1 Jason Booth 2021-06-03 10:07:31 MDT 
I have added Tim to CC in the event he has more to say on this subject.

This was done to accommodate changes for HPE Shasta. The existing pm_counters interface is the best source for energy-accounting data at present, and the "cray_aries" plugin already supports this format. For this reason we've renamed the acct_gather_energy/cray_aries plugin to acct_gather_energy/pm_counters going forward with Slurm 20.11.
Comment 2 Chris Samuel (NERSC) 2021-06-03 10:20:16 MDT 
Hi Jason,

Oh that's awesome, so we could potentially enable this now on our existing Shasta systems?

Thanks!
Chris
Comment 3 Jason Booth 2021-06-09 13:54:34 MDT 
>Oh that's awesome, so we could potentially enable this now on our existing Shasta systems?

Yes, you can use it now.
Comment 4 Chris Samuel (NERSC) 2021-06-09 14:03:07 MDT 
(In reply to Jason Booth from comment #3)

> >Oh that's awesome, so we could potentially enable this now on our existing Shasta systems?
> 
> Yes, you can use it now.

Oh brilliant, thank you!