Ticket 366 - sh5util doesn't handle jobs in which all steps to not have profile data.
Summary: sh5util doesn't handle jobs in which all steps to not have profile data.
Status: RESOLVED FIXED
Alias: None
Product: Slurm
Classification: Unclassified
Component: Other (show other tickets)
Version: 2.6.x
Hardware: Linux Linux
: 5 - Enhancement
Assignee: Moe Jette
QA Contact:
URL:
Depends on:
Blocks:
 
Reported: 2013-07-12 01:35 MDT by Rod Schultz
Modified: 2013-07-12 07:24 MDT (History)
1 user (show)

See Also:
Site: Atos/Eviden Sites
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:
Target Release: ---
DevPrio: ---
Emory-Cloud Sites: ---

Attachments
Fixes merge when all steps are not profiles. (3.32 KB, application/octet-stream)
2013-07-12 01:35 MDT, Rod Schultz 		Details
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 Rod Schultz 2013-07-12 01:35:00 MDT 
Created attachment 336 [details]
Fixes merge when all steps are not profiles.

If all steps don't have profile data, sh5util will not merge data from all the steps. It currently stops on the first stepno that does not have data.

So if step 0 is a setup step. it will not merge subsequent steps.

Or if step 0 has data, step 1 does not, but step 2 does, it will only merge step 0.

The attached patch fixes the issue.
Comment 1 Moe Jette 2013-07-12 07:24:55 MDT 
Committed to slurm-2.6 branch:

commit c12aa93f025b2ea04f4a14dfb45a47f700a1fd9e
Author: Morris Jette <jette@schedmd.com>
Date:   Fri Jul 12 12:23:29 2013 -0700

    Minor formatting changes, no change to logic

commit 262d754f57242eafc17b5eaa8cfcf3623af5df50
Author: Rod Schultz <Rod.Schultz@bull.com>
Date:   Fri Jul 12 12:22:14 2013 -0700

    Fix merge when all steps are not profiled
    
    Bug 366