tektoncd/pipeline

Add integration tests for /tekton/debug/scripts

Ouverte

#4 426 ouverte le 14 déc. 2021

Voir sur GitHubGuide contributeur
 (5 commentaires) (0 réaction) (0 personne assignée)Go (1 943 forks)auto 404
area/testinghelp wanted

Métriques du dépôt

Stars
 (9 013 étoiles)
Métriques de merge PR
 (Merge moyen 9j 3h) (137 PRs mergées en 30 j)

Description

/area testing

AFAIK the integration tests work by checking if TaskRuns/PipelineRuns have reached a desired state, based on which the tests are marked as success or failure.

To test scripts in /tekton/debug/scripts dir in the TaskRun Pod, first we would need the TaskRun Pod running with the breakpoint(s) in place. Then, we need to run exec commands allowing us to run scripts in the container(s) with breakpoint. Thus helping us assess the state/logs of the TaskRun after, based on the commands we have run.

How can this be achieved ? What are the possible problems we could face while writing these tests ?

Guide contributeur

Direction de recherche
Tout d'abord, comprenez le cadre de test d'intégration existant dans le référentiel Tekton Pipelines. Identifiez comment les états de TaskRun/PipelineRun sont vérifiés. Ensuite, concevez un test qui crée un TaskRun avec des points d'arrêt, utilise `kubectl exec` pour exécuter des scripts de `/tekton/debug/scripts` dans le conteneur et affirme l'état ou les journaux attendus. Considérez des problèmes tels que la disponibilité du conteneur, les autorisations d'exécution des scripts et l'isolation des tests.
Stack technique
go
Domaine
backend
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
GoKubernetesTekton
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30