Merge pull request opendatahub-io#25 from kubeflow/master

[pull] master from kubeflow:master

Author: openshift-merge-robot

.kfp-rebase-version

@@ -1 +1 @@
-1.8.1
+1.8.4

Makefile

@@ -16,7 +16,7 @@
 #  - The help target was derived from https://stackoverflow.com/a/35730328/5601796
 
 VENV ?= .venv
-KFP_TEKTON_RELEASE ?= v1.3.0
+KFP_TEKTON_RELEASE ?= v1.3.1
 export VIRTUAL_ENV := $(abspath ${VENV})
 export PATH := ${VIRTUAL_ENV}/bin:${PATH}
 DOCKER_REGISTRY ?= aipipeline

VERSION

@@ -1 +1 @@
-1.3.0
+1.3.1

backend/src/cache/server/watcher.go

@@ -4,7 +4,6 @@ import (
 	"context"
 	"encoding/json"
 	"fmt"
-	"log"
 	"reflect"
 	"strconv"
 	"time"
@@ -47,12 +46,16 @@ func WatchPods(ctx context.Context, namespaceToWatch string, clientManager Clien
 		}
 
 		for event := range watcher.ResultChan() {
-			pod := reflect.ValueOf(event.Object).Interface().(*corev1.Pod)
+			pod, ok := reflect.ValueOf(event.Object).Interface().(*corev1.Pod)
+			if !ok {
+				logger.Warnf("Not processing WatchPods as event Object is not a Pod, event type: %s", event.Type)
+				continue
+			}
+
 			if event.Type == watch.Error {
 				logger.Errorf("Watcher error in loop: %v", event.Type)
 				continue
 			}
-			log.Printf((*pod).GetName())
 
 			if !isPodCompletedAndSucceeded(pod) {
 				logger.Warnf("Pod %s is not completed or not in successful status, skip the loop.", pod.ObjectMeta.Name)

guides/kfp_tekton_install.md

@@ -53,15 +53,15 @@ To install the standalone Kubeflow Pipelines with Tekton, run the following step
          -p '{"data":{"enable-custom-tasks": "true"}}'
    ```
 
-3. Install Kubeflow Pipelines with Tekton backend (`kfp-tekton`) `v1.3.0` [custom resource definitions](https://kubernetes.io/docs/concepts/extend-kubernetes/api-extension/custom-resources/)(CRDs).
+3. Install Kubeflow Pipelines with Tekton backend (`kfp-tekton`) `v1.3.1` [custom resource definitions](https://kubernetes.io/docs/concepts/extend-kubernetes/api-extension/custom-resources/)(CRDs).
    > Note: You can ignore the error `no matches for kind "Application" in version "app.k8s.io/v1beta1"` since it's a warning saying `application` CRD is not yet ready.
     ```shell
-    kubectl apply --selector kubeflow/crd-install=true -f install/v1.3.0/kfp-tekton.yaml
+    kubectl apply --selector kubeflow/crd-install=true -f install/v1.3.1/kfp-tekton.yaml
     ```
 
-4. Install Kubeflow Pipelines with Tekton backend (`kfp-tekton`) `v1.3.0` deployment
+4. Install Kubeflow Pipelines with Tekton backend (`kfp-tekton`) `v1.3.1` deployment
     ```shell
-    kubectl apply -f install/v1.3.0/kfp-tekton.yaml
+    kubectl apply -f install/v1.3.1/kfp-tekton.yaml
     ```
 
 5. Then, if you want to expose the Kubeflow Pipelines endpoint outside the cluster, run the following commands: