Ticket 13982

Summary:	pedantic: misuse of 'deprecated' in diagnostics
Product:	Slurm	Reporter:	Mark Allen <racsadmin>
Component:	User Commands	Assignee:	Oscar Hernández <oscar.hernandez>
Status:	RESOLVED FIXED	QA Contact:
Severity:	4 - Minor Issue
Priority:	---	CC:	albert.gil
Version:	21.08.5
Hardware:	Linux
OS:	Linux
Site:	University of Oregon	Alineos Sites:	---
Atos/Eviden Sites:	---	Confidential Site:	---
Coreweave sites:	---	Cray Sites:	---
DS9 clusters:	---	HPCnow Sites:	---
HPE Sites:	---	IBM Sites:	---
NOAA SIte:	---	OCF Sites:	---
Recursion Pharma Sites:	---	SFW Sites:	---
SNIC sites:	---	Linux Distro:	---
Machine Name:		CLE Version:
Version Fixed:	22.05.3, 23.02.0pre1	Target Release:	---
DevPrio:	---	Emory-Cloud Sites:	---

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