Skip to content

Commit

Permalink
Merge pull request #1075 from replicatedhq/laverya/dont-parse-empty-s…
Browse files Browse the repository at this point in the history
…tring-as-bool

don't parse empty strings as bool in kurl config
  • Loading branch information
laverya committed Sep 8, 2020
2 parents 87f9dea + f3875ff commit 5a893a8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion kotsadm/pkg/kurl/kurl_nodes.go
Expand Up @@ -96,7 +96,7 @@ func GetNodes(client kubernetes.Interface) (*types.KurlNodes, error) {

toReturn.IsKurlEnabled = true

if val, ok := kurlConf.Data["ha"]; ok {
if val, ok := kurlConf.Data["ha"]; ok && val != "" {
parsedBool, err := strconv.ParseBool(val)
if err != nil {
return nil, errors.Wrapf(err, "parse 'ha' entry in kurl config %q", val)
Expand Down

0 comments on commit 5a893a8

Please sign in to comment.