Ticket 13982 - pedantic: misuse of 'deprecated' in diagnostics
Summary: pedantic: misuse of 'deprecated' in diagnostics
Status: RESOLVED FIXED
Alias: None
Product: Slurm
Classification: Unclassified
Component: User Commands (show other tickets)
Version: 21.08.5
Hardware: Linux Linux
: 4 - Minor Issue
Assignee: Oscar Hernández
QA Contact:
URL:
Depends on:
Blocks:
 
Reported: 2022-05-02 17:07 MDT by Mark Allen
Modified: 2022-08-05 11:08 MDT (History)
1 user (show)

See Also:
Site: University of Oregon
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: 22.05.3, 23.02.0pre1
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 Mark Allen 2022-05-02 17:07:51 MDT 
In several diagnostics, the word 'deprecated' is being used as a synonym for 'removed' or 'superseded'.  It's not.

As a term of art, it means that the feature in question still works, but may cease to work in the near future, might produced warnings, might no longer be well supported, etc.  If the software no longer accepts something, it's not deprecated--the feature has been removed.

For example: "sacct: fatal: AllocGRES is deprecated, please use AllocTRES"

Thanks.
Comment 5 Oscar Hernández 2022-05-06 10:25:36 MDT 
Hi Mark,

thanks for your feedback. We are reviewing our deprecated/removed options and will let you know what we decide to do with them and their corresponding messages.

Regards,
Oscar
Comment 12 Oscar Hernández 2022-08-05 11:08:13 MDT 
Hi Mark,

Let me apologize for taking a while to respond. Taking into account your comments, for the next 22.05 release, some of the messages have been reworded. Here the commits:

1. AllocGRES:
commit 9e40da1ede2e51ed3469e7d53e795b50ed561ea5

    Change AllocGRES/ReqGRES fatal messages wording
    
    Both options are not availabe, so the message should say "removed"

2.Debugflag=task:
commit 907471073cb07fa8f3c5dc7a1f3987a2188c0037

    Change DebugFlag=Task error message from "deprecated" to "removed"

and in master, pointing to next major:

3.CR_ALLOCATE_FULL_SOCKET:
commit 14bbc869f7f7dc28b5a7ff10158d62ad805e8248

    Remove CR_ALLOCATE_FULL_SOCKET references
    
    Flag was removed in 15.08.

I am closing the bug now, but do not hesitate to re-open if you find any other related problem.
 
Thanks a lot for the feedback.
Oscar