[YUNIKORN-1120] Allow unit specifiers in core configs

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.0.0
• Component/s: core - common, core - scheduler
• Labels: pull-request-available
• Target Version: 1.0.0
• Assignee: Craig Condit
• Reporter: Craig Condit
• Created: 14/Mar/22 23:07
• Updated: 04/May/22 23:52
• Resolved: 17/Mar/22 14:36

Description

Currently, only absolute values for resources are allowed in configurations read by the core. We should allow unit specifiers as well (M, Mi, G, Gi, etc.).

From YUNIKORN-165:

Internally we must move to integers for the representation, just to fix the issue described in YUNIKORN-1105. We should be able to handle about 9 exa byte as the maximum size of a cluster using a standard int64 or 18 exa byte if we use an unsigned integer. Not sure if we will overflow that in any resources quickly.

What I meant with the staying closer to what K8s does is to allow specifying the resources with a quantity suffix specifically in the configuration. This is what we currently require:

resources:
    max:
        vcore: 21000
        memory: 610000 

That specifies 21 cpu's (base is milli vcores) and 610GB for memory (base is 1MB). Not that simple to interpret. We should allow 610G, and like with the shim side we need to convert that to B. Same for the vcore, 21 should be converted to 21000 m cores etc.

I would like to see  us support a configuration like this:

resources:
     guaranteed:
        vcore: 2500m
        memory: 600G
     max:
        vcore: 10
        memory: 1T

This is independent of how we represent the data in the REST specification. REST should align with the base suffix: milli for vcores (base == 1e-3) and bytes / undefined for others (base == 1). So never a quantity suffix in REST, vcores always as milli vcores and all other resources as bytes or undefined.

Issue Links

• is a parent of: YUNIKORN-1130 Documentation updates for units on resources
• is related to: YUNIKORN-165 API response fields can be returned as proper data structure rather than free text
• links to: GitHub Pull Request #386