kubeflow / kfctl

kfctl is a CLI for deploying and managing Kubeflow
Apache License 2.0
181 stars 137 forks source link

Build fails with error "conversion from int to string yields a string of one rune" #396

Open pinkavaj opened 4 years ago

pinkavaj commented 4 years ago
pkg/kfapp/aws/aws.go:564:74: conversion from int to string yields a string of one rune, not a string of digits (did you mean fmt.Sprint(x)?)

go version go1.15 linux/amd64

issue-label-bot[bot] commented 4 years ago

Issue-Label Bot is automatically applying the labels:

Label Probability
platform/aws 0.81
kind/bug 0.87

Please mark this comment with :thumbsup: or :thumbsdown: to give our bot feedback! Links: app homepage, dashboard and code for this bot.

issue-label-bot[bot] commented 4 years ago

Issue-Label Bot is automatically applying the labels:

Label Probability
area/kfctl 0.65

Please mark this comment with :thumbsup: or :thumbsdown: to give our bot feedback! Links: app homepage, dashboard and code for this bot.

issue-label-bot[bot] commented 4 years ago

Issue-Label Bot is automatically applying the labels:

Label Probability
area/kfctl 0.65

Please mark this comment with :thumbsup: or :thumbsdown: to give our bot feedback! Links: app homepage, dashboard and code for this bot.

issue-label-bot[bot] commented 4 years ago

Issue-Label Bot is automatically applying the labels:

Label Probability
area/kfctl 0.65

Please mark this comment with :thumbsup: or :thumbsdown: to give our bot feedback! Links: app homepage, dashboard and code for this bot.

jtfogarty commented 4 years ago

/priority p1

PatrickXYS commented 4 years ago

I'm not sure if I understand, if we're not using go 1.15, there should be some issues when we upgrade go version in kfctl.

Let's take a step back, what's the reason that we need to upgrade go version for kfctl?