-
Notifications
You must be signed in to change notification settings - Fork 420
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrading to Triggers 0.5 from 0.4 errors if you used type: in a TT's params #583
Comments
/kind bug |
Adding #532 as this was mentioned on Slack and looks to be the culprit |
@a-roberts is the value of
Yeah, I'll edit the release notes. The |
Yeah that's right, https://github.com/tektoncd/experimental/blob/b45e925f437ad85c7ee840fa22d124877169740f/webhooks-extension/base/400-monitor-triggertemplate.yaml - all strings. Initially added here tektoncd/experimental@5e70732, it's not clear why, so I'm cool with updating the notes personally although I wonder if users were using this as an API and expected an upcoming deprecation notice. None of the examples referenced this field though so you could go either way 🤔 |
Stale issues rot after 30d of inactivity. /lifecycle rotten Send feedback to tektoncd/plumbing. |
Rotten issues close after 30d of inactivity. /close Send feedback to tektoncd/plumbing. |
@tekton-robot: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Expected Behavior
I expected the upgrade from Triggers 0.4 to Triggers 0.5 to not require this, or to mention this in the release notes
Actual Behavior
As part of tektoncd/experimental#547 I noticed the problem and the solution is to delete all references to type: for all parameters in my TriggerTemplate, and to delete any existing TriggerTemplates
My guess is, you can have one small TriggerTemplate using type: on Triggers 0.4, then try to apply the same one on Triggers 0.5, and you'd see the error.
Steps to Reproduce the Problem
Additional Info
There's probably a much easier reproduce than this but it's the use case I'm particularly interested in, I'll handle things on the extension side but wondering if we want a 0.5.1 release or to call this out in the release notes?
The text was updated successfully, but these errors were encountered: