pcp
[Top] [All Lists]

Re: [pcp] pmdaslurm updates

To: Martins Innus <minnus@xxxxxxxxxxx>
Subject: Re: [pcp] pmdaslurm updates
From: Nathan Scott <nathans@xxxxxxxxxx>
Date: Tue, 23 Feb 2016 23:44:37 -0500 (EST)
Cc: PCP Mailing List <pcp@xxxxxxxxxxx>
Delivered-to: pcp@xxxxxxxxxxx
In-reply-to: <56CCBFB4.3080901@xxxxxxxxxxx>
References: <56CCBFB4.3080901@xxxxxxxxxxx>
Reply-to: Nathan Scott <nathans@xxxxxxxxxx>
Thread-index: iWopoVrJCCAJtKoWPNyk9hjAhDJXKw==
Thread-topic: pmdaslurm updates
Hi Martins,

----- Original Message -----
> Hi,
>       Here is an update to the Slurm PMDA that adds a metric for the cores
> that have been allocated to the job by the scheduler:
> 
> 
> https://github.com/ubccr/pcp/tree/pmdaslurm
> 
> We have been running this for a while but haven't had a chance to push
> upstream.
> 

I ran into this minor issue on merging ...

qa$ ./check 1031
1031 3s ... - output mismatch (see 1031.out.bad)
1a2,3
> Bareword "SHOW_DETAIL" not allowed while "strict subs" in use at pmdaslurm.pl 
> line 104.
> Execution of pmdaslurm.pl aborted due to compilation errors.
3,34c5,6
[...]

I think this is just a QA test issue, and updated accordingly (but needed to
update pmdaslurm itself too - namespace issue?) ... could you double check my
tweak is OK?  Thanks!

cheers.

--
Nathan

<Prev in Thread] Current Thread [Next in Thread>