elastic/kibana

[Task Manager] Clean up performance tests

Open

#117.300 geöffnet am 3. Nov. 2021

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (19.065 Stars) (8.021 Forks)batch import
Feature:Task ManagerTeam:ResponseOpsestimate:smallgood first issuetechnical debt

Beschreibung

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.

Contributor Guide

Tech Stack
typescriptnodejs
Domain
backendtestingperformance
Issue Type
test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Familiarity with Kibana task managerContext from PR #117294
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65
Research-Richtung
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.