Skip to content

Commit

Permalink
Merge pull request #1020 from hangyan/fix/env-file-env
Browse files Browse the repository at this point in the history
Fix environment with env_file bug
  • Loading branch information
hangyan committed Jul 10, 2018
2 parents afdc598 + 3b04a91 commit c0360e0
Show file tree
Hide file tree
Showing 6 changed files with 413 additions and 6 deletions.
13 changes: 8 additions & 5 deletions pkg/transformer/kubernetes/kubernetes.go
Expand Up @@ -572,8 +572,11 @@ func (k *Kubernetes) ConfigEnvs(name string, service kobject.ServiceConfig, opt

envs := transformer.EnvSort{}

keysFromEnvFile := make(map[string]bool)

// If there is an env_file, use ConfigMaps and ignore the environment variables
// already specified

if len(service.EnvFile) > 0 {

// Load each env_file
Expand All @@ -600,14 +603,14 @@ func (k *Kubernetes) ConfigEnvs(name string, service kobject.ServiceConfig, opt
Key: k,
}},
})
keysFromEnvFile[k] = true
}

}
}

} else {

// Load up the environment variables
for _, v := range service.Environment {
// Load up the environment variables
for _, v := range service.Environment {
if !keysFromEnvFile[v.Name] {
envs = append(envs, api.EnvVar{
Name: v.Name,
Value: v.Value,
Expand Down
10 changes: 9 additions & 1 deletion script/test/cmd/tests.sh
Expand Up @@ -568,7 +568,6 @@ sed -e "s;%VERSION%;$version;g" -e "s;%CMD%;$cmd;g" $KOMPOSE_ROOT/script/test/f
convert::expect_success "$cmd" "/tmp/output-k8s.json"



# Test unset environment variables are passed correctly
export V3_HOST_ENV_TEST_SET_TO_BAR=BAR
cmd="kompose convert --stdout -j -f $KOMPOSE_ROOT/script/test/fixtures/v3/docker-compose-unset-env.yaml"
Expand All @@ -582,6 +581,15 @@ sed -e "s;%VERSION%;$version;g" -e "s;%CMD%;$cmd;g" $KOMPOSE_ROOT/script/test/f
convert::expect_success "$cmd" "/tmp/output-k8s.json"


# Test key/value env with env_files
cmd="kompose convert --stdout -j -f $KOMPOSE_ROOT/script/test/fixtures/env/docker-compose.yml"
sed -e "s;%VERSION%;$version;g" -e "s;%CMD%;$cmd;g" $KOMPOSE_ROOT/script/test/fixtures/env/output-k8s.json > /tmp/output-k8s.json
convert::expect_success "$cmd" "/tmp/output-k8s.json"

cmd="kompose convert --stdout -j --provider=openshift -f $KOMPOSE_ROOT/script/test/fixtures/env/docker-compose.yml"
sed -e "s;%VERSION%;$version;g" -e "s;%CMD%;$cmd;g" $KOMPOSE_ROOT/script/test/fixtures/env/output-os.json > /tmp/output-os.json
convert::expect_success "$cmd" "/tmp/output-os.json"

# Test that two files that are different versions fail
convert::expect_failure "kompose convert --stdout -j -f $KOMPOSE_ROOT/script/test/fixtures/v3/docker-compose.yaml -f $KOMPOSE_ROOT/script/test/fixtures/etherpad/docker-compose.yaml"

Expand Down
13 changes: 13 additions & 0 deletions script/test/fixtures/env/docker-compose.yml
@@ -0,0 +1,13 @@
version: '3'
services:
namenode:
image: bde2020/hadoop-namenode:2.0.0-hadoop2.7.4-java8
volumes:
- namenode:/hadoop/dfs/name
environment:
- CLUSTER_NAME=test
env_file:
- ./hadoop-hive-namenode.env
ports:
- "50070:50070"
- "8020:8020"
2 changes: 2 additions & 0 deletions script/test/fixtures/env/hadoop-hive-namenode.env
@@ -0,0 +1,2 @@
FOO=BAR
BAR=FOO
165 changes: 165 additions & 0 deletions script/test/fixtures/env/output-k8s.json
@@ -0,0 +1,165 @@
{
"kind": "List",
"apiVersion": "v1",
"metadata": {},
"items": [
{
"kind": "Service",
"apiVersion": "v1",
"metadata": {
"name": "namenode",
"creationTimestamp": null,
"labels": {
"io.kompose.service": "namenode"
},
"annotations": {
"kompose.cmd": "%CMD%",
"kompose.version": "%VERSION%"
}
},
"spec": {
"ports": [
{
"name": "50070",
"port": 50070,
"targetPort": 50070
},
{
"name": "8020",
"port": 8020,
"targetPort": 8020
}
],
"selector": {
"io.kompose.service": "namenode"
}
},
"status": {
"loadBalancer": {}
}
},
{
"kind": "Deployment",
"apiVersion": "extensions/v1beta1",
"metadata": {
"name": "namenode",
"creationTimestamp": null,
"labels": {
"io.kompose.service": "namenode"
},
"annotations": {
"kompose.cmd": "%CMD%",
"kompose.version": "%VERSION%"
}
},
"spec": {
"replicas": 1,
"template": {
"metadata": {
"creationTimestamp": null,
"labels": {
"io.kompose.service": "namenode"
}
},
"spec": {
"volumes": [
{
"name": "namenode",
"persistentVolumeClaim": {
"claimName": "namenode"
}
}
],
"containers": [
{
"name": "namenode",
"image": "bde2020/hadoop-namenode:2.0.0-hadoop2.7.4-java8",
"ports": [
{
"containerPort": 50070
},
{
"containerPort": 8020
}
],
"env": [
{
"name": "BAR",
"valueFrom": {
"configMapKeyRef": {
"name": "namenode-hadoop-hive-namenode-env",
"key": "BAR"
}
}
},
{
"name": "CLUSTER_NAME",
"value": "test"
},
{
"name": "FOO",
"valueFrom": {
"configMapKeyRef": {
"name": "namenode-hadoop-hive-namenode-env",
"key": "FOO"
}
}
}
],
"resources": {},
"volumeMounts": [
{
"name": "namenode",
"mountPath": "/hadoop/dfs/name"
}
]
}
],
"restartPolicy": "Always"
}
},
"strategy": {
"type": "Recreate"
}
},
"status": {}
},
{
"kind": "ConfigMap",
"apiVersion": "v1",
"metadata": {
"name": "namenode-hadoop-hive-namenode-env",
"creationTimestamp": null,
"labels": {
"io.kompose.service": "namenode-hadoop-hive-namenode-env"
}
},
"data": {
"BAR": "FOO",
"FOO": "BAR"
}
},
{
"kind": "PersistentVolumeClaim",
"apiVersion": "v1",
"metadata": {
"name": "namenode",
"creationTimestamp": null,
"labels": {
"io.kompose.service": "namenode"
}
},
"spec": {
"accessModes": [
"ReadWriteOnce"
],
"resources": {
"requests": {
"storage": "100Mi"
}
}
},
"status": {}
}
]
}

0 comments on commit c0360e0

Please sign in to comment.