[YUNIKORN-1277] Simplify command line processing code

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.1.0
• Component/s: shim - kubernetes
• Labels: pull-request-available
• Assignee: Peter Bacsko
• Reporter: Peter Bacsko
• Created: 04/Aug/22 19:29
• Updated: 07/Feb/23 22:34
• Resolved: 08/Aug/22 07:10

Description

Currently, we parse flags (cmd line args) like this:

kubeConfig := flag.String("kubeConfig", "",
		"absolute path to the kubeconfig file")

but there's a simpler version without the need of an extra variable:

conf := &SchedulerConf{
  SchedulerName:  constants.SchedulerName,
}

flag.StringVar(&conf.KubeConfig, "kubeConfig", "",
		"absolute path to the kubeconfig file")
...

Issue Links

• links to: GitHub Pull Request #445