pcp
[Top] [All Lists]

Re: [pcp] pmdaslurm updates

To: Nathan Scott <nathans@xxxxxxxxxx>
Subject: Re: [pcp] pmdaslurm updates
From: Martins Innus <minnus@xxxxxxxxxxx>
Date: Wed, 24 Feb 2016 16:44:16 -0500
Cc: PCP Mailing List <pcp@xxxxxxxxxxx>
Delivered-to: pcp@xxxxxxxxxxx
In-reply-to: <808991005.24516201.1456289077202.JavaMail.zimbra@xxxxxxxxxx>
References: <56CCBFB4.3080901@xxxxxxxxxxx> <808991005.24516201.1456289077202.JavaMail.zimbra@xxxxxxxxxx>
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:38.0) Gecko/20100101 Thunderbird/38.6.0
Nathan,

On 2/23/16 11:44 PM, Nathan Scott wrote:
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!




Yup, that works here.  Thanks.

Martins

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