Fix golint warnings

This commit is contained in:
Hang Yan
2018-03-25 16:22:08 +08:00
parent ff97e413f2
commit 505cb8b754
7 changed files with 23 additions and 19 deletions
+2
View File
@@ -581,6 +581,7 @@ func DurationStrToSecondsInt(s string) (*int64, error) {
return &r, nil
}
// GetEnvsFromFile get env vars from env_file
func GetEnvsFromFile(file string, opt kobject.ConvertOptions) (map[string]string, error) {
// Get the correct file context / directory
composeDir, err := transformer.GetComposeFileDir(opt.InputFiles)
@@ -598,6 +599,7 @@ func GetEnvsFromFile(file string, opt kobject.ConvertOptions) (map[string]string
return envLoad, nil
}
// FormatEnvName format env name
func FormatEnvName(name string) string {
envName := strings.Trim(name, "./")
envName = strings.Replace(envName, ".", "-", -1)
+1 -1
View File
@@ -579,7 +579,7 @@ func (k *Kubernetes) ConfigEnvs(name string, service kobject.ServiceConfig, opt
}
// Add configMapKeyRef to each environment variable
for k, _ := range envLoad {
for k := range envLoad {
envs = append(envs, api.EnvVar{
Name: k,
ValueFrom: &api.EnvVarSource{
+2 -2
View File
@@ -5,7 +5,7 @@ import (
"strings"
)
// getImageTag get tag name from image name
// GetImageTag get tag name from image name
// if no tag is specified return 'latest'
func GetImageTag(image string) string {
// format: registry_host:registry_port/repo_name/image_name:image_tag
@@ -31,7 +31,7 @@ func GetImageTag(image string) string {
}
// getAbsBuildContext returns build context relative to project root dir
// GetAbsBuildContext returns build context relative to project root dir
func GetAbsBuildContext(context string) (string, error) {
cmd := exec.Command("git", "rev-parse", "--show-prefix")
cmd.Dir = context
+11 -12
View File
@@ -224,7 +224,7 @@ func BuildDockerImage(service kobject.ServiceConfig, name string, relativePath s
}
// Connect to the Docker client
client, err := docker.DockerClient()
client, err := docker.Client()
if err != nil {
return err
}
@@ -251,20 +251,19 @@ func PushDockerImage(service kobject.ServiceConfig, serviceName string) error {
if service.Image == "" {
log.Warnf("No image name has been passed for service %s, skipping pushing to repository", serviceName)
return nil
} else {
}
// Connect to the Docker client
client, err := docker.DockerClient()
if err != nil {
return err
}
// Connect to the Docker client
client, err := docker.Client()
if err != nil {
return err
}
push := docker.Push{Client: *client}
err = push.PushImage(service.Image)
push := docker.Push{Client: *client}
err = push.PushImage(service.Image)
if err != nil {
return err
}
if err != nil {
return err
}
return nil