Skip to content

Commit

Permalink
add dockerSockPath to argo config map
Browse files Browse the repository at this point in the history
  • Loading branch information
adrian555 committed Oct 21, 2019
1 parent 4d2939d commit 2846e37
Show file tree
Hide file tree
Showing 6 changed files with 36 additions and 0 deletions.
1 change: 1 addition & 0 deletions argo/base/config-map.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ data:
{
executorImage: $(executorImage),
containerRuntimeExecutor: $(containerRuntimeExecutor),
dockerSockPath: $(dockerSockPath),
artifactRepository:
{
s3: {
Expand Down
7 changes: 7 additions & 0 deletions argo/base/kustomization.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,13 @@ vars:
apiVersion: v1
fieldref:
fieldpath: data.containerRuntimeExecutor
- name: dockerSockPath
objref:
kind: ConfigMap
name: workflow-controller-parameters
apiVersion: v1
fieldref:
fieldpath: data.dockerSockPath
- name: artifactRepositoryBucket
objref:
kind: ConfigMap
Expand Down
1 change: 1 addition & 0 deletions argo/base/params.env
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
namespace=
executorImage=argoproj/argoexec:v2.3.0
containerRuntimeExecutor=docker
dockerSockPath=/var/run/docker.sock
artifactRepositoryBucket=mlpipeline
artifactRepositoryKeyPrefix=artifacts
artifactRepositoryEndpoint=minio-service.kubeflow:9000
Expand Down
9 changes: 9 additions & 0 deletions tests/argo-base_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -135,6 +135,7 @@ data:
{
executorImage: $(executorImage),
containerRuntimeExecutor: $(containerRuntimeExecutor),
dockerSockPath: $(dockerSockPath),
artifactRepository:
{
s3: {
Expand Down Expand Up @@ -329,6 +330,7 @@ varReference:
namespace=
executorImage=argoproj/argoexec:v2.3.0
containerRuntimeExecutor=docker
dockerSockPath=/var/run/docker.sock
artifactRepositoryBucket=mlpipeline
artifactRepositoryKeyPrefix=artifacts
artifactRepositoryEndpoint=minio-service.kubeflow:9000
Expand Down Expand Up @@ -379,6 +381,13 @@ vars:
apiVersion: v1
fieldref:
fieldpath: data.containerRuntimeExecutor
- name: dockerSockPath
objref:
kind: ConfigMap
name: workflow-controller-parameters
apiVersion: v1
fieldref:
fieldpath: data.dockerSockPath
- name: artifactRepositoryBucket
objref:
kind: ConfigMap
Expand Down
9 changes: 9 additions & 0 deletions tests/argo-overlays-application_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -190,6 +190,7 @@ data:
{
executorImage: $(executorImage),
containerRuntimeExecutor: $(containerRuntimeExecutor),
dockerSockPath: $(dockerSockPath),
artifactRepository:
{
s3: {
Expand Down Expand Up @@ -384,6 +385,7 @@ varReference:
namespace=
executorImage=argoproj/argoexec:v2.3.0
containerRuntimeExecutor=docker
dockerSockPath=/var/run/docker.sock
artifactRepositoryBucket=mlpipeline
artifactRepositoryKeyPrefix=artifacts
artifactRepositoryEndpoint=minio-service.kubeflow:9000
Expand Down Expand Up @@ -434,6 +436,13 @@ vars:
apiVersion: v1
fieldref:
fieldpath: data.containerRuntimeExecutor
- name: dockerSockPath
objref:
kind: ConfigMap
name: workflow-controller-parameters
apiVersion: v1
fieldref:
fieldpath: data.dockerSockPath
- name: artifactRepositoryBucket
objref:
kind: ConfigMap
Expand Down
9 changes: 9 additions & 0 deletions tests/argo-overlays-istio_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -172,6 +172,7 @@ data:
{
executorImage: $(executorImage),
containerRuntimeExecutor: $(containerRuntimeExecutor),
dockerSockPath: $(dockerSockPath),
artifactRepository:
{
s3: {
Expand Down Expand Up @@ -366,6 +367,7 @@ varReference:
namespace=
executorImage=argoproj/argoexec:v2.3.0
containerRuntimeExecutor=docker
dockerSockPath=/var/run/docker.sock
artifactRepositoryBucket=mlpipeline
artifactRepositoryKeyPrefix=artifacts
artifactRepositoryEndpoint=minio-service.kubeflow:9000
Expand Down Expand Up @@ -416,6 +418,13 @@ vars:
apiVersion: v1
fieldref:
fieldpath: data.containerRuntimeExecutor
- name: dockerSockPath
objref:
kind: ConfigMap
name: workflow-controller-parameters
apiVersion: v1
fieldref:
fieldpath: data.dockerSockPath
- name: artifactRepositoryBucket
objref:
kind: ConfigMap
Expand Down

0 comments on commit 2846e37

Please sign in to comment.