[ 
https://issues.apache.org/jira/browse/YUNIKORN-1832?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yongjun Zhang updated YUNIKORN-1832:
------------------------------------
    Description: 
when user doesn't config instance type node label, the following  code would 
get an empty value, 
{code:java}
 parser.stringVar(&conf.InstanceTypeNodeLabelKey, 
CMSvcNodeInstanceTypeNodeLabelKey){code}
which is called by
{code:java}
// parse values from configmaps
newConf, cmErrors := parseConfig(config, prev) {code}
and it will overwrite the default value DefaultNodeInstanceTypeNodeLabelKey set 
in default config.

Before deciding to overwrite default config value, we should check if the 
config is set or not, and if it's set to a valid value if it's set.

 

 

  was:
when user doesn't config instance type node label, the following  code would 
get an empty value, 
{code:java}
 parser.stringVar(&conf.InstanceTypeNodeLabelKey, 
CMSvcNodeInstanceTypeNodeLabelKey){code}
and it will overwrite the default value DefaultNodeInstanceTypeNodeLabelKey set 
in default config.

Before deciding to overwrite default config value, we should check if the 
config is set or not, and if it's set to a valid value if it's set.

 

 


> Default conf value DefaultNodeInstanceTypeNodeLabelKey is not honored 
> ----------------------------------------------------------------------
>
>                 Key: YUNIKORN-1832
>                 URL: https://issues.apache.org/jira/browse/YUNIKORN-1832
>             Project: Apache YuniKorn
>          Issue Type: Improvement
>          Components: shim - kubernetes
>            Reporter: Yongjun Zhang
>            Assignee: Yongjun Zhang
>            Priority: Major
>
> when user doesn't config instance type node label, the following  code would 
> get an empty value, 
> {code:java}
>  parser.stringVar(&conf.InstanceTypeNodeLabelKey, 
> CMSvcNodeInstanceTypeNodeLabelKey){code}
> which is called by
> {code:java}
> // parse values from configmaps
> newConf, cmErrors := parseConfig(config, prev) {code}
> and it will overwrite the default value DefaultNodeInstanceTypeNodeLabelKey 
> set in default config.
> Before deciding to overwrite default config value, we should check if the 
> config is set or not, and if it's set to a valid value if it's set.
>  
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to