slurm.conf.example: Filter out more Slurm node state flags #469
Conversation
| Thanks for the patch. Overall looks good! Could be nice if you try to keep a similar structure for the commit message with a title which starts with the part of code being touched. See e500d9b by example :) |
There was a problem hiding this comment.
I hope Slurm supports UTF-8, as it will eventually exhaust us-ascii special characters... ;)
Wondering if you should not have a different strategy, where you filter only keeping the standard characters and remove all the other ones?
There was a problem hiding this comment.
I can't wait for emoji-based node states :)
But more seriously, I think the filtering should ideally not be based on what Slurm states strings could contain. but rather on what is considered acceptable characters in a group name, namely not those:
clustershell/lib/ClusterShell/NodeSet.py
Line 74 in a1875cb
But I'm not sure we can get that illegal characters list in a group configuration file, can we?
kcgthb changed the title 
* filter out the following characters from the node states returned by sinfo: *~#!%$@+^- Slurm 21.08 introduced a new node state flag (^), and we missed a few before. See https://slurm.schedmd.com/sinfo.html#SECTION_NODE-STATE-CODES for a complete list.
| Good point about the commit message, fixed. |
degremont left a comment
degremont left a comment There was a problem hiding this comment.
Based on the discussion, i'm ok with the current patch
3 participants
Slurm 21.08 introduced a new node state flag (^), and we missed a few before.
See https://slurm.schedmd.com/sinfo.html#SECTION_NODE-STATE-CODES for a complete list