[YUNIKORN-2273] checkLimitMaxApplications fails if there is missing limit in a middle queue

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.5.0
• Component/s: core - common
• Labels: pull-request-available
• Target Version: 1.5.0
• Assignee: PoAn Yang
• Reporter: PoAn Yang
• Created: 15/Dec/23 13:36
• Updated: 20/Mar/24 14:32
• Resolved: 18/Jan/24 05:20

Description

The checkLimitMaxApplications function rejects config which child limit is large than parent limit. However, it doesn't cover all cases like child limit is large than grandparent limit. Following is my test case:

func TestCheckLimitMaxApplicationsGrandLevel(t *testing.T) {
    err := checkLimitMaxApplications(
        QueueConfig{
            Name: "root",
            Limits: []Limit{
                {
                    Limit:           "user1",
                    Users:           []string{"user1"},
                    MaxApplications: 100,
                },
                {
                    Limit:           "user2",
                    Users:           []string{"user2"},
                    MaxApplications: 100,
                },
            },
            Queues: []QueueConfig{
                {
                    Name: "parent",
                    Limits: []Limit{
                        {
                            Limit:           "user1",
                            Users:           []string{"user1"},
                            MaxApplications: 50,
                        },
                    },
                    Queues: []QueueConfig{
                        {
                            Name: "child",
                            Limits: []Limit{
                                {
                                    Limit:           "user1",
                                    Users:           []string{"user1"},
                                    MaxApplications: 10,
                                },
                                {
                                    Limit:           "user2",
                                    Users:           []string{"user2"},
                                    MaxApplications: 150,
                                },
                            },
                        },
                    },
                },
            },
        },
        make(map[string]map[string]uint64),
        make(map[string]map[string]uint64),
        common.Empty,
    )
    assert.Equal(t, err != nil, true, "user2 cpu maxapplications in root.parent.child should not be large than root") // this will fail
}

Issue Links

• links to: GitHub Pull Request #760