Tommy Li comments

Results 188 comments of


                                            Tommy Li

Param Values container $$ are handled incorrectly

I think there are some test cases for the `spec.steps.*.script`, but not for `params`. https://github.com/tektoncd/pipeline/blob/main/examples/v1alpha1/taskruns/step-script.yaml#L102-L118

Param Values container $$ are handled incorrectly

```yaml taskSpec: steps: - name: main command: - sh - -c - | set -e echo $0 echo $1 echo $2 - $(inputs.params.msg1) - $(inputs.params.msg2) - $(inputs.params.msg3) image: alpine:3.6 ```...

Add request encoder/decoder in Explainer model wrapper

Yes that make sense to me.

[kubeflow 1.3] unable to route requests to the kfserving pod due to auth policy

We should add this `AuthorizationPolicy` to the Kubeflow manifests because Kubeflow 1.4 still has this issue.

Add an example of Lime text explainer from AIX360 toolkit

@C1berwiz I'm not sure do we want to host a big binary file like [rfmodel.pickle](https://github.com/kserve/kserve/pull/2355/files#diff-cecaba4f0caa00d6cbe43637c9043bc9ce59a4bd08c466e9977f85d21a239a31) in the KServe repo. Usually we either provide a simple script to train the model...

Leverage the minimal pipelinerun status from Tekton 0.35 to reduce the yaml size in k8s ETCD

/close

Discussion on how to trim down Tekton/Argo go pkg to resolve go.mod conflicts.

cc @yhwang @rimolive

Discussion on how to trim down Tekton/Argo go pkg to resolve go.mod conflicts.

related: https://github.com/tektoncd/pipeline/issues/6483

Update fluentd example to support new KFP V2 log archive

/help

Ensure Kubeflow meets the CNCF IP Policy

KFP-Tekton is licensed under [Apache 2.0](https://github.com/kubeflow/kfp-tekton/blob/master/LICENSE).