elastic/kibana

[Task Manager] Clean up performance tests

Open

#117 300 ouverte le 3 nov. 2021

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)TypeScript (19 065 stars) (8 021 forks)batch import
Feature:Task ManagerTeam:ResponseOpsestimate:smallgood first issuetechnical debt

Description

With https://github.com/elastic/kibana/pull/117294, we are removing the perf_hooks in task manager due to possible memory leakage. There is also a set of skipped functional tests in https://github.com/elastic/kibana/blob/main/x-pack/test/plugin_api_perf/test_suites/task_manager/index.ts that we should make sure to remove once we've confirmed that 🤞 removing the perf_hooks fixes the memory issue.

Guide contributeur

Stack technique
typescriptnodejs
Domaine
backendtestingperformance
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.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Familiarity with Kibana task managerContext from PR #117294
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65
Direction de recherche
First, verify that PR #117294 has resolved the memory leak issue and is merged. Then, locate the file x pack/test/plugin api perf/test suites/task manager/index.ts in the Kibana repository. Remove the functional tests that are currently skipped (likely marked with `skip` or `xdescribe`). Ensure the tests are no longer needed and that no other parts of the codebase depend on them. Finally, submit a pull request to remove the skipped tests and any references to the old perf hooks usage if not already cleaned up.