-
Notifications
You must be signed in to change notification settings - Fork 426
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
FR: Dynamic TriggerTemplate Resources #189
Comments
/assign wlynch |
/kind feature I am not sure why it should take a |
Here's a pass at a proposal: https://docs.google.com/document/d/1fTob_Y6qo61vl6UIhraV6tSRM9Vb-DrugCyihB6md_Q/edit# @vdemeester Would particularly like your feedback here, since this has heavy overlap with your catalog URI proposal. |
This seems like it could be satisfied with a |
Stale issues rot after 30d of inactivity. /lifecycle rotten Send feedback to tektoncd/plumbing. |
Issues go stale after 90d of inactivity. /lifecycle stale 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. |
Stale issues rot after 30d of inactivity. /lifecycle rotten Send feedback to tektoncd/plumbing. |
Motivating Use Case
Be able to run/test Tekton Trigger configurations ephemerally without needing to apply it permanently to the cluster. e.g. how do I test a change to a Template/Pipeline config?
Proposed Solution
Add a new type that can be included in TriggerTemplates that will read configuration dynamically from a file.
This type should take a volume generating resource (such as GitResource or StorageResource) as an input.
Example
Where
/path/to/my/config.yaml
references a file containing other TriggerTemplate resources (example)Open Questions
TODO
The text was updated successfully, but these errors were encountered: