search

tektoncd / community

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

TEP-0148: Summary/Aggregation API for Tekton Results #1086

Closed avinal closed 8 months ago

avinal commented 11 months ago

Summary

This TEP proposes a new API endpoint for Tekton results that will allow users to get a summary or aggregation data given a set of filters. This will be useful for getting a quick overview without having to go through all the response and also reduce the amount of data that needs to be transferred. This will also provide a way to get aggregated data for a set of filters.

Co-authored-by: Khurram Baig kbaig@redhat.com Signed-off-by: Avinal Kumar avinal@redhat.com

/kind tep

pritidesai commented 11 months ago

/assign @enarha

tekton-robot commented 11 months ago

@pritidesai: GitHub didn't allow me to assign the following users: enarha.

Note that only tektoncd members, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time. For more information please see the contributor guide

In response to [this](https://github.com/tektoncd/community/pull/1086#issuecomment-1775549485): >/assign @enarha 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 11 months ago

/assign @tektoncd/results-maintainers

vdemeester commented 9 months ago

/assign

avinal commented 8 months ago

I propose gating "implementable" on adding additional non-goals related to aggregation features. Which types of queries will this feature/TEP not support? At which point will we recommend community adopters implement their own data extraction/transform systems?

I will update the TEP to elaborate non-goals.

khrm commented 8 months ago

@alan-ghelardi Can you approve this TEP pr?

vdemeester commented 8 months ago

/lgtm cancel /hold cancel

(@adambkaplan see here why I am "cancelling" the lgtm)

adambkaplan commented 8 months ago

/assign @alan-ghelardi

/assign @dibyom

Assigning to non-Red Hat Results maintainers.

tekton-robot commented 8 months ago

@adambkaplan: GitHub didn't allow me to assign the following users: alan-ghelardi.

Note that only tektoncd members, repo collaborators and people who have commented on this issue/PR can be assigned. Additionally, issues/PRs can only have 10 assignees at the same time. For more information please see the contributor guide

In response to [this](https://github.com/tektoncd/community/pull/1086#issuecomment-1913259141): >/assign @alan-ghelardi > >/assign @dibyom > >Assigning to non-Red Hat Results maintainers. 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.
tekton-robot commented 8 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: adambkaplan, dibyom, vdemeester

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)~~ [dibyom,vdemeester] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
dibyom commented 8 months ago

/lgtm