[YUNIKORN-2482] Failure to set template does not return error

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Resolved
• Fix Version/s: 1.6.0
• Component/s: core - scheduler
• Labels: newbie pull-request-available
• Target Version: 1.6.0
• Assignee: Hsien-Cheng(Ryan) Huang
• Reporter: Wilfred Spiegelenburg
• Created: 13/Mar/24 00:59
• Updated: 22/Apr/24 03:53
• Resolved: 22/Apr/24 03:52

Description

The update of setting a template on a parent could fail if the template is not correct. The error is swallowed and a success is returned but the update of the queue has not finished correctly:
*Queue.applyConf()

if !sq.isLeaf {
    if err = sq.setTemplate(conf.ChildTemplate); err != nil {
       return nil
    }
} 

Need to add tests to make sure we do not regress.

Issue Links

• is caused by: YUNIKORN-193 Add option to allow parent to set unmanaged child queue properties
• links to: GitHub Pull Request #848