Project

General

Profile

Feature #2591

Startd should advertise per-slot memory, not per-machine memory

Added by Burt Holzman almost 8 years ago. Updated over 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Parag Mhashilkar
Category:
-
Target version:
Start date:
03/22/2012
Due date:
% Done:

0%

Estimated time:
Stakeholders:
Duration:

Description

This request came from the OSG AHM.

History

#1 Updated by Igor Sfiligoi almost 8 years ago

The is no way for the startd to know the per-slot memory in the general case;
the information is simply not there.

In the OSG factory, we have been advertising an estimate via
GLIDEIN_MaxMemMBs

and the VO can then use it to define the startd memory with
MEMORY=${GLIDEIN_MaxMemMBs}

GLOW does it here
http://glidein.chtc.wisc.edu/vofrontend/stage/frontend_CHTC-glidein-v1_0/set_default_memory.c21d77

Igor

#2 Updated by Burt Holzman almost 8 years ago

We agreed that we should add an option to do what the GLOW guys are doing if desired.

#3 Updated by Burt Holzman almost 8 years ago

  • Target version set to v2_5_7

#4 Updated by Burt Holzman almost 8 years ago

Also, Brian comments that the request is to enable this option by default in the OSG packaged software, as a separate item. We'll need more input from OSG (Igor?) before we do that, but it sounds reasonable.

#5 Updated by Burt Holzman almost 8 years ago

  • Target version changed from v2_5_7 to v2_7_x

#6 Updated by Parag Mhashilkar almost 8 years ago

Igor do you have any input on this? Also, I wonder if TotalMemory/CPUS is a reasonable estimate?

#7 Updated by Igor Sfiligoi over 7 years ago

It is probably better than nothing, but it is likely not accurate.

You really want to set it to what the allowed limit is.
Thus the GLIDEIN_MaxMemMBs attribute in the OSG gfactory.

PS: No easy way to automatically get the node memory in the factory config.
So not sure how this would help with the Factory packaging.
(the frontends should not care... should just happen)

#8 Updated by Parag Mhashilkar over 7 years ago

branch_v2plus: commit:a76eff1
master: commit:1c7aefa

#9 Updated by Parag Mhashilkar over 7 years ago

  • Status changed from Assigned to Feedback

#10 Updated by Douglas Strain over 7 years ago

  • Status changed from Feedback to Resolved

I have reviewed the code. Looks fine to me. Please merge. Don't forget to also cherry-pick changes over to master branch as well.

#11 Updated by Parag Mhashilkar over 7 years ago

  • Target version changed from v2_7_x to v2_6

#12 Updated by Parag Mhashilkar over 7 years ago

  • Tracker changed from Bug to Feature

#13 Updated by Parag Mhashilkar over 7 years ago

  • Status changed from Resolved to Closed


Also available in: Atom PDF