Ticket 437

Summary:	Need tests for new commands
Product:	Slurm	Reporter:	Moe Jette <jette>
Component:	Test Suite	Assignee:	Nathan Yee <nyee32>
Status:	RESOLVED FIXED	QA Contact:
Severity:	5 - Enhancement
Priority:	---	CC:	nyee32
Version:	14.03.x
Hardware:	Linux
OS:	Linux
Site:	SchedMD	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:		Target Release:	---
DevPrio:	---	Emory-Cloud Sites:	---
Attachments:	Test for the qalter and qrerun options

Description Moe Jette 2013-10-01 09:04:25 MDT

I need some tests for two new commands in the contrib/torque directory: qalter and qrerun. Each command has a help package (e.g. "qalter --help") that will describe the options to be tested.

Comment 1 Nathan Yee 2013-10-20 12:50:19 MDT

(In reply to Moe Jette from comment #0)
> I need some tests for two new commands in the contrib/torque directory:
> qalter and qrerun. Each command has a help package (e.g. "qalter --help")
> that will describe the options to be tested.

For these tests do I have to build a torque system? If I do is there documentation or instructions on how to do it that you could point me to?

Thanks

Comment 2 Nathan Yee 2013-10-23 16:31:27 MDT

Did you want a seperate test for all the options or do you want just one test that will test all of them?

Comment 3 Moe Jette 2013-10-24 03:46:29 MDT

Individual tests for each command please. Start with test20.5

Comment 4 Nathan Yee 2013-11-02 08:00:37 MDT

Created attachment 493 [details]
Test for the qalter and qrerun options

In test20.7 the qalter -o option test that changes the stdout path. when changing the path, it seems to only be set in the comment when doing a scontrol show job. I set this test to look at the comment value since the StdOut value is never changed. Not sure if this is standard behavior. If the test needs to be change please let me know.

Comment 5 Moe Jette 2013-11-04 02:38:17 MST

I needed to add the descriptions to README, added a time limit to the jobs (fyi, the format is "qsub -l walltime=1:00 ..."), otherwise just some minor cosmetic changes.

Comment 6 Moe Jette 2013-11-04 02:40:04 MST

One more thing, the commit is here:

https://github.com/SchedMD/slurm/commit/b27f177ea0bda2d411307a9c94c5498f80cea56b

Comment 7 Moe Jette 2013-11-04 09:51:49 MST

One more issue, the test id is always of the form "#.#" not "test#.#".