search

tektoncd / community

Community documentation for the Tekton project
https://tekton.dev
Apache License 2.0
363 stars 221 forks source link

TEP-0155 to store PipelineRun/TaskRun Events to DB #1148

Closed khrm closed 6 months ago

khrm commented 7 months ago

Tekton Results stores so far PipelineRuns, TaskRuns and Logs. This TEP proposes to store Pipeline Events as well to have all historical data from a PipelineRun in Tekton Results.

Added design to the original PR at https://github.com/tektoncd/community/pull/1118

khrm commented 7 months ago

/assign @dibyom

chitrangpatel commented 7 months ago

/assign

khrm commented 7 months ago

/test pull-community-teps-lint

tekton-robot commented 7 months ago

@khrm: No presubmit jobs available for tektoncd/community@main

In response to [this](https://github.com/tektoncd/community/pull/1148#issuecomment-2070072453): >/test pull-community-teps-lint > Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
khrm commented 6 months ago

/assign @enarha @adambkaplan This is rewrite of https://github.com/tektoncd/community/pull/1118 with more details. You approved that TEP earlier.

khrm commented 6 months ago

/assign @enarha @adambkaplan

chitrangpatel commented 6 months ago

/approve Since the other PR which this was forked from had both an approval and an lgtm.

adambkaplan commented 6 months ago

@khrm please copy over the original pull request description, with a back-link to the first PR so we have a clear chain of authorship.

khrm commented 6 months ago

@adambkaplan Sure.

khrm commented 6 months ago

I have added the description and PR in the description. Added that description to the commit Subject also. Commit was already showing in GH as "manuelwallrapp" authored and "khrm" committed.

tekton-robot commented 6 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: adambkaplan, avinal, chitrangpatel

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[teps/OWNERS](https://github.com/tektoncd/community/blob/main/teps/OWNERS)~~ [chitrangpatel] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment