Skip to main content
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [ptp-dev] Support for SLURM 2.1 committed
It is still being used.  So it shouldn't be commented out.

R^2
--
Randy M. Roberts
work: (505)665-4285

"In my many years I have come to a conclusion
 that one useless man is a shame, two is a law firm,
 and three or more is a congress."
                      -- John Adams

On May 20, 2010, at 7:53 PM, JiangJie wrote:

Hi Greg and Randy,
 
ProcessAttributes.getStatusAttributeDefinition() was commented out in head,
but I restore it back when I submitted new slurm proxy code last night.
 
The defined process state can not reflect the real status of process/job in SLURM.
So I use process status  attribute to report process state provided by SLURM 
(such as job_failed, job_timeout, job_node_fail). These extra statuses are used to 
display corresponding state icons in UI.
 
I'm not very clear why this method got commented out.
Keep it if possible.
 
Jie
 

 
From: g.watson@xxxxxxxxxxxx
Subject: Re: [ptp-dev] Support for SLURM 2.1 committed
Date: Thu, 20 May 2010 16:02:34 -0400
To: ptp-dev@xxxxxxxxxxx

It was commented out in my code also, but now it appears to be back. Weird.

Greg

On May 20, 2010, at 3:46 PM, Randy Roberts wrote:

Unfortunately I'm seeing an error in SLURMRuntimeModelPresentation which is still trying to use ProcessAttributes.getStatusAttributeDefinition(). This method no longer exists, so you might want to check that you've updated to the latest head.

Greg,

This method still exists.  Remember, you use it in calls to the process' parent IPJob.

The definition methods that were removed from ProcessAttributes are:

// public static IntegerAttributeDefinition getPIDAttributeDefinition() {
// return pidAttrDef;
// }

and
// public static IntegerAttributeDefinition getIndexAttributeDefinition() {
// return indexAttrDef;
// }

Regards,
R^2
--
Randy M. Roberts
work: (505)665-4285

"This is how Liberty dies - with thunderous applause."
                            -- Senator Amidala (Star Wars: Episode III)

On May 20, 2010, at 1:25 PM, Randy Roberts wrote:

Randy, I notice that you've just commented out code in ProcessAttributes. Would you please either deprecate the interfaces or remove them completely. Ditto for anywhere else this has been done.

Greg,

I guess I was just a bit uncertain about it.  Now that you've commented on it,
I will remove the commented code.

Thanks,
R^2
--
Randy M. Roberts
work: (505)665-4285

"They that can give up essential liberty to obtain a little temporary
safety deserve neither liberty nor safety."  -- Benjamin Franklin

On May 20, 2010, at 11:38 AM, Greg Watson wrote:

Great! Thanks for the update.

Unfortunately I'm seeing an error in SLURMRuntimeModelPresentation which is still trying to use ProcessAttributes.getStatusAttributeDefinition(). This method no longer exists, so you might want to check that you've updated to the latest head.

Randy, I notice that you've just commented out code in ProcessAttributes. Would you please either deprecate the interfaces or remove them completely. Ditto for anywhere else this has been done.

Thanks,
Greg

On May 20, 2010, at 11:50 AM, JiangJie wrote:


Hi all,

I have committed new ptp_slurm_proxy and related codes to support SLURM_2.1/2.x. 

Major improvements include:
(1) Reimplemented proxy based on SLURM API completely.
(2) Support SLURM-2.1 (and later, if API keeps compatible). 
       Previous versions of SLURM-1.x are not supported anymore.
(3) The topology information of parallel processes is obtained from  job step layout,
       not from "srun" ELF image anymore.
(4) Update job/process/node state in separate threads.
(5) Comply with PTP-4.0 rm proxy protocol.

Any feedback would be appreciated.

Regards,
Jie 
搜索本应是彩色的,快来体验新一代搜索引擎-必应,精美图片每天换哦! 立即试用！
_______________________________________________
ptp-dev mailing list
ptp-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/ptp-dev

_______________________________________________
ptp-dev mailing list
ptp-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/ptp-dev

_______________________________________________
ptp-dev mailing list
ptp-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/ptp-dev

_______________________________________________
ptp-dev mailing list
ptp-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/ptp-dev


使用新一代 Windows Live Messenger 轻松交流和共享！ 立刻下载！ _______________________________________________
ptp-dev mailing list
ptp-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/ptp-dev

Back to the top