community icon indicating copy to clipboard operation
community copied to clipboard

Add TEP Test Framework for Tasks

Open jlux98 opened this issue 9 months ago • 10 comments

jlux98 avatar Mar 21 '25 14:03 jlux98

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: To complete the pull request process, please assign dibyom after the PR has been reviewed. You can assign the PR to them by writing /assign @dibyom in a comment when ready.

The full list of commands accepted by this bot can be found 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 Mar 21 '25 14:03 tekton-robot

CLA Signed

The committers listed above are authorized under a signed CLA.

  • :white_check_mark: login: jlux98 (a94657195daebcae950af0ba1d108208aa766607, 1f81b49699cf96dcbb65e65f1c28c4dd89f04cb4, 8c6b05547d04db37f71ce253a0a0fae09d95bf99, 61af91245a217a91de0280ac10ae984909ff4f67, 7e4c3f04baee25bc375f7c062db0f2c4a6272c85, cacc59d96e451b5b6cc6d9d6ea0d3ed8d7c32e4a, 14a927e2baa51714b488d993478d70f9786726ef, a03a3c5e8b0e546d1a37577e70fa48a8d5af8431)

/kind TEP

jlux98 avatar Mar 21 '25 17:03 jlux98

I know, that having a PR with the whole proposal already filled out is not the preferred way to do it and that it's better to wait, until the initial summary and proposal is discussed and merged before adding the design section.

But as I had already put a lot of thought into the design and written out the section I thought it better to publish it quickly and get feedback instead of waiting for feedback on the initial proposal and then having to wait even longer for feedback on the rest.

jlux98 avatar Mar 24 '25 13:03 jlux98

@jlux98 Thank you for your proposal. Did you receive any feedback on this? If not, I apologise for the delayed response. is this still something that you're interested in? If so, I will spend some time reviewing your proposal.

afrittoli avatar May 12 '25 10:05 afrittoli

@afrittoli Thank you for your response.

I did not receive feedback on this proposal yet and it is still something I am interested in, as the feature described in this TEP will be the topic of my bachelor's thesis and ideally I'd like to design and implement it in a way where after I'm done the code can be merged into the official repo with as few changes as possible.

I pushed a small-ish revision today (touching up some of the examples for proposed API objects and rewording some paragraphs to hopefully flow better) and would be really glad, if you could take the time to have a look at the proposal.

jlux98 avatar May 12 '25 22:05 jlux98

@afrittoli did you already have to time to look into the proposal?

I hope me asking doesn't come off as rude and I'm not doing it out of impatience. The reason why I'm asking is because as I wrote in my earlier comment the test framework described here will be the topic of my Bachelor's thesis, but what I didn't write is that I'll have to start implementing the feature in 3-4 weeks. So if you were able to give me the first wave of feedback before then, that would be greatly appreciated - if not then that's still alright, I might be able to still incorporate the feedback into my thesis. And even if the feedback comes too late for that, then I'll compare the final product of the thesis to the feedback and go from there.

jlux98 avatar May 20 '25 18:05 jlux98

/assign

waveywaves avatar May 21 '25 11:05 waveywaves

@jlux98: PR needs rebase.

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.

tekton-robot avatar Jul 08 '25 19:07 tekton-robot

ping @jlux98

vdemeester avatar Oct 07 '25 09:10 vdemeester