search

cdfoundation / tekton-helm-chart

CDF official helm chart for Tekton
Apache License 2.0
53 stars 49 forks source link

chore: upgrade tekton pipelines to 0.36.1 #40

Closed ankitm123 closed 1 year ago

ankitm123 commented 1 year ago

Signed-off-by: ankitm123 ankitmohapatra123@gmail.com

ankitm123 commented 1 year ago

Running into some issues bumping tekton library in lighthouse, once that is fixed, we can upgrade ...

ankitm123 commented 1 year ago

This PR made quite a few changes to step template: https://github.com/tektoncd/pipeline/pull/4778/files

nicolaerosia commented 1 year ago

@ankitm123 can you please take a look again and merge?

nicolaerosia commented 1 year ago

would be great if we can push even to a newer version

nicolaerosia commented 1 year ago

@ankitm123 kind ping

ankitm123 commented 1 year ago

@nicolaerosia hello, I am currently on a road trip, and will be back on March 10. I will try to fix the issues with lighthouse and merge this PR.

nicolaerosia commented 1 year ago

@ankitm123 thank you!

nicolaerosia commented 1 year ago

Kind reminder

tomhobson commented 1 year ago

going to try to look at the lighthouse issues and get this merged. there's a lot of functionality that this would enable

tomhobson commented 1 year ago

https://github.com/jenkins-x/lighthouse/pull/1538

tomhobson commented 1 year ago

Merging as the other one is merged too

nicolaerosia commented 1 year ago

Thanks! @tomhobson don’t want to be too pushy, but now that you understand the flow, can we go to even a newer version?:)