community icon indicating copy to clipboard operation
community copied to clipboard

Draft TEP: resolver caching

Open brianwcook opened this issue 5 months ago • 2 comments
trafficstars

This is a draft TEP for the work I am doing in tektoncd/pipeline#8825 to add caching to the resolvers, primarily git and bundle resolvers.

brianwcook avatar Jun 16 '25 01:06 brianwcook

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: twoGiants, 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:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

tekton-robot avatar Jul 01 '25 18:07 tekton-robot

... how this could/would interact with non built-in resolver ? (and if it should)

@vdemeester a small API could be provided for custom resolver which want caching. Not sure its really necessary though.

twoGiants avatar Jul 01 '25 18:07 twoGiants

... how this could/would interact with non built-in resolver ? (and if it should)

@vdemeester an small API could be provided for custom resolver which want caching. Not sure its really necessary though.

Yeah, I think it's not necessary now at least, we can go ahead with the proposal and see later if this is something that we need.

vdemeester avatar Jul 02 '25 12:07 vdemeester

I need lgtm label please 🙏

brianwcook avatar Jul 08 '25 12:07 brianwcook

/lgtm

afrittoli avatar Jul 08 '25 15:07 afrittoli

@Antonio1t2 what is Data-monster-6aae0 🤔

waveywaves avatar Aug 29 '25 20:08 waveywaves