Ticket 9723

Summary:	sinfo state filtering
Product:	Slurm	Reporter:	Brian Christiansen <brian>
Component:	User Commands	Assignee:	Brian Christiansen <brian>
Status:	RESOLVED FIXED	QA Contact:
Severity:	4 - Minor Issue
Priority:	---
Version:	20.02.3
Hardware:	Linux
OS:	Linux
Site:	SchedMD	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:	20.11.0pre1
Target Release:	---	DevPrio:	---
Emory-Cloud Sites:	---
Attachments:	v1

Description Brian Christiansen 2020-09-02 09:43:07 MDT

2 things.

1. sinfo -t doesn't except the cloud state
2. sinfo -t returns any node in any of the states instead of asking giving back nodes that are in all states (|| vs &&).

For example, I can't filter out IDLE+POWER_SAVE nodes

brian@lappy:~/slurm/20.02/lappy$ sinfo -t idle
PARTITION AVAIL  TIMELIMIT  NODES  STATE NODELIST 
q            up   infinite      1   idle q1 
d            up   infinite      2   idle d[1-2] 
cloud        up   infinite      9  idle~ cloud[2-10] 
debug*       up   infinite     10   idle lappy[1-10] 
debug2       up   infinite     10   idle lappy[1-10] 
gpu          up   infinite      5   idle zpu[1-5] 

brian@lappy:~/slurm/20.02/lappy$ sinfo -t down
PARTITION AVAIL  TIMELIMIT  NODES  STATE NODELIST 
q            up   infinite      1  down* q2 
d            up   infinite      0    n/a  
cloud        up   infinite      1  down~ cloud1 
debug*       up   infinite      0    n/a  
debug2       up   infinite      0    n/a  
gpu          up   infinite      0    n/a  

brian@lappy:~/slurm/20.02/lappy$ sinfo -t power_down
PARTITION AVAIL  TIMELIMIT  NODES  STATE NODELIST 
q            up   infinite      0    n/a  
d            up   infinite      0    n/a  
cloud        up   infinite      9  idle~ cloud[2-10] 
cloud        up   infinite      1  down~ cloud1 
debug*       up   infinite      0    n/a  
debug2       up   infinite      0    n/a  
gpu          up   infinite      0    n/a  

brian@lappy:~/slurm/20.02/lappy$ sinfo -t idle,power_down
PARTITION AVAIL  TIMELIMIT  NODES  STATE NODELIST 
q            up   infinite      1  idle* q1 
d            up   infinite      2   idle d[1-2] 
cloud        up   infinite      9  idle~ cloud[2-10] 
cloud        up   infinite      1  down~ cloud1 
debug*       up   infinite     10   idle lappy[1-10] 
debug2       up   infinite     10   idle lappy[1-10] 
gpu          up   infinite      5   idle zpu[1-5]

Comment 1 Brian Christiansen 2020-09-02 09:54:08 MDT

Created attachment 15695 [details]
v1

Proposed changes:

brian@lappy:~/slurm/20.02/lappy$ sinfo -t cloud
PARTITION AVAIL  TIMELIMIT  NODES  STATE NODELIST 
q            up   infinite      0    n/a  
d            up   infinite      0    n/a  
cloud        up   infinite      9  idle~ cloud[2-10] 
cloud        up   infinite      1  down~ cloud1 
debug*       up   infinite      0    n/a  
debug2       up   infinite      0    n/a  
gpu          up   infinite      0    n/a  

brian@lappy:~/slurm/20.02/lappy$ sinfo -t cloud,idle
PARTITION AVAIL  TIMELIMIT  NODES  STATE NODELIST 
q            up   infinite      0    n/a  
d            up   infinite      2   idle d[1-2] 
cloud        up   infinite      9  idle~ cloud[2-10] 
cloud        up   infinite      1  down~ cloud1 
debug*       up   infinite     10   idle lappy[1-10] 
debug2       up   infinite     10   idle lappy[1-10] 
gpu          up   infinite      5   idle zpu[1-5] 

brian@lappy:~/slurm/20.02/lappy$ sinfo -t "cloud&idle"
PARTITION AVAIL  TIMELIMIT  NODES  STATE NODELIST 
q            up   infinite      0    n/a  
d            up   infinite      0    n/a  
cloud        up   infinite      9  idle~ cloud[2-10] 
debug*       up   infinite      0    n/a  
debug2       up   infinite      0    n/a  
gpu          up   infinite      0    n/a  

brian@lappy:~/slurm/20.02/lappy$ sinfo -t "cloud&down"
PARTITION AVAIL  TIMELIMIT  NODES  STATE NODELIST 
q            up   infinite      0    n/a  
d            up   infinite      0    n/a  
cloud        up   infinite      1  down~ cloud1 
debug*       up   infinite      0    n/a  
debug2       up   infinite      0    n/a  
gpu          up   infinite      0    n/a

Comment 3 Brian Christiansen 2020-09-21 22:21:59 MDT

*   commit 72d87faa37193ea10f8d20f78734da87c145e52a (HEAD -> master, origin/master, origin/HEAD)
|\  Merge: acd0fea570 811302c229
| | Author:     Brian Christiansen <brian@schedmd.com>
| | AuthorDate: Mon Sep 21 22:20:29 2020 -0600
| | Commit:     Brian Christiansen <brian@schedmd.com>
| | CommitDate: Mon Sep 21 22:20:29 2020 -0600
| | 
| |     Merge branch 'bug9723'
| | 
| * commit 811302c2292f80b580334eb559ab8c73517e7899
| | Author:     Brian Christiansen <brian@schedmd.com>
| | AuthorDate: Mon Sep 21 21:56:38 2020 -0600
| | Commit:     Brian Christiansen <brian@schedmd.com>
| | CommitDate: Mon Sep 21 22:13:11 2020 -0600
| | 
| |     Testsuite - test4.5 test || and && states
| |     
| |     Bug 9723
| | 
| * commit e2339a92de04af67413713979e30a5caf3dd97b3
| | Author:     Brian Christiansen <brian@schedmd.com>
| | AuthorDate: Mon Sep 21 21:40:05 2020 -0600
| | Commit:     Brian Christiansen <brian@schedmd.com>
| | CommitDate: Mon Sep 21 22:13:11 2020 -0600
| | 
| |     Testsuite - test4.5 explicity check for filtered nodes and states
| |     
| |     Bug 9723
| | 
| * commit a0ca7a33a62a2d254b5e50dc9ad312c4b59fc489
| | Author:     Brian Christiansen <brian@schedmd.com>
| | AuthorDate: Mon Sep 21 21:40:05 2020 -0600
| | Commit:     Brian Christiansen <brian@schedmd.com>
| | CommitDate: Mon Sep 21 22:13:11 2020 -0600
| | 
| |     Testsuite - test4.5 Fix test filtering
| |     
| |     Was just filtering on the nodes that it expected in the given state,
| |     which doesn't really validate that --state is filtering out the other
| |     nodes not in the state.
| |     
| |     Bug 9723
| | 
| * commit abdc10ab01af250797c2beb87aae24487582a6c2
| | Author:     Brian Christiansen <brian@schedmd.com>
| | AuthorDate: Mon Sep 21 16:03:47 2020 -0600
| | Commit:     Brian Christiansen <brian@schedmd.com>
| | CommitDate: Mon Sep 21 22:13:11 2020 -0600
| | 
| |     Testsuite - test4.5 Fail test quicker
| |     
| |     Bug 9723
| | 
| * commit 34d1dc082676e6ee3ffdd7e83c026335d405ba69
| | Author:     Brian Christiansen <brian@schedmd.com>
| | AuthorDate: Wed Sep 2 08:57:59 2020 -0600
| | Commit:     Brian Christiansen <brian@schedmd.com>
| | CommitDate: Mon Sep 21 22:13:11 2020 -0600
| | 
| |     Enable and'ing sinfo --state filtering
| |     
| |     Bug 9723
| | 
| * commit 0677f134fdda84cd303f4d6b1ae4539dc4637aaf
| | Author:     Brian Christiansen <brian@schedmd.com>
| | AuthorDate: Wed Sep 2 08:57:59 2020 -0600
| | Commit:     Brian Christiansen <brian@schedmd.com>
| | CommitDate: Mon Sep 21 22:13:11 2020 -0600
| | 
| |     Remove broken sinfo -t idle check
| |     
| |     Bug 9723
| | 
| * commit c7f31a94fa619863fb142824be0d5698eca5b06d
|/  Author:     Brian Christiansen <brian@schedmd.com>
|   AuthorDate: Wed Sep 2 08:57:33 2020 -0600
|   Commit:     Brian Christiansen <brian@schedmd.com>
|   CommitDate: Mon Sep 21 22:13:11 2020 -0600
|   
|       Enable filtering of cloud state in sinfo -t
|       
|       Bug 9723