Merge pull request #59 from volcano-sh/feature/support-controller-log

Support displaying log files

Author: Klaus Ma

.gitignore

@@ -124,4 +124,4 @@ kubernetes.tar.gz
 *.pyc
 
 # e2e log files
-/hack/*.log
+*.log

.travis.yml

@@ -31,3 +31,8 @@ jobs:
         - make cli
         - make docker
         - make e2e-test-kind
+      after_failure:
+        # Echo logs and upload
+        - test -f volcano-admission.log && echo "******<<admission logs>>******" && cat volcano-admission.log
+        - test -f volcano-controller.log && echo "******<<controller logs>>******" && cat volcano-controller.log
+        - test -f volcano-scheduler.log && echo "******<<scheduler logs>>******" && cat volcano-scheduler.log

hack/run-e2e-kind.sh

@@ -74,6 +74,13 @@ function uninstall-volcano {
   helm delete integration --purge --kubeconfig ${KUBECONFIG}
 }
 
+function generate-log {
+    echo "Generating volcano log files"
+    kubectl logs -lapp=volcano-admission -n kube-system > volcano-admission.log
+    kubectl logs -lapp=volcano-controller -n kube-system > volcano-controller.log
+    kubectl logs -lapp=volcano-scheduler -n kube-system > volcano-scheduler.log
+}
+
 # clean up
 function cleanup {
   uninstall-volcano
@@ -118,3 +125,8 @@ install-volcano
 # Run e2e test
 cd ${VK_ROOT}
 KUBECONFIG=${KUBECONFIG} go test ./test/e2e -v -timeout 30m
+
+if [[ $? != 0 ]]; then
+  generate-log
+  exit 1
+fi

installer/chart/volcano/templates/admission.yaml

@@ -40,21 +40,18 @@ apiVersion: apps/v1
 kind: Deployment
 metadata:
   labels:
-    app: admission
-    admission: "true"
+    app: volcano-admission
   name: {{ .Release.Name }}-admission
   namespace: {{ .Release.Namespace }}
 spec:
   replicas: 1
   selector:
     matchLabels:
-      app: admission
-      admission: "true"
+      app: volcano-admission
   template:
     metadata:
       labels:
-        app: admission
-        admission: "true"
+        app: volcano-admission
     spec:
       serviceAccount: {{ .Release.Name }}-admission
       {{ if .Values.basic.image_pull_secret }}
@@ -90,7 +87,7 @@ apiVersion: v1
 kind: Service
 metadata:
   labels:
-    test: admission
+    app: volcano-admission
   name: {{ .Release.Name }}-admission-service
   namespace: {{ .Release.Namespace }}
 spec:
@@ -99,5 +96,5 @@ spec:
       protocol: TCP
       targetPort: 443
   selector:
-    admission: "true"
+    app: volcano-admission
   sessionAffinity: None

installer/chart/volcano/templates/controllers.yaml

@@ -63,15 +63,17 @@ apiVersion: apps/v1
 metadata:
   name: {{ .Release.Name }}-controllers
   namespace: {{ .Release.Namespace }}
+  labels:
+    app: volcano-controller
 spec:
   replicas: 1
   selector:
     matchLabels:
-      vk-controllers: test
+      app: volcano-controller
   template:
     metadata:
       labels:
-        vk-controllers: test
+        app: volcano-controller
     spec:
       serviceAccount: {{ .Release.Name }}-controllers
       {{ if .Values.basic.image_pull_secret }}

installer/chart/volcano/templates/scheduler.yaml

@@ -84,15 +84,17 @@ apiVersion: apps/v1
 metadata:
   name: {{ .Release.Name }}-scheduler
   namespace: {{ .Release.Namespace }}
+  labels:
+    app: volcano-scheduler
 spec:
   replicas: 1
   selector:
     matchLabels:
-      vk-scheduler: test
+      app: volcano-scheduler
   template:
     metadata:
       labels:
-        vk-scheduler: test
+        app: volcano-scheduler
     spec:
       serviceAccount: {{ .Release.Name }}-scheduler
       containers: