elastic/kibana

[Response Ops][Reporting] Use `p-retry` to retry report

Open

#245 437 ouverte le 5 déc. 2025

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)TypeScript (19 065 stars) (8 021 forks)batch import
Feature:Reporting:FrameworkTeam:ResponseOpsgood first issuequick-win

Description

We added retries within the reporting task to handle retries for scheduled reporting in this PR: https://github.com/elastic/kibana/pull/229998

We should consider switching to using the p-retry library to avoid bugs with custom code.

Guide contributeur

Stack technique
typescriptnodejs
Domaine
backend
Type d'issue
refactor
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-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
Node.js async/awaitp retry libraryKibana reporting module
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70
Direction de recherche
Review the existing retry logic in the reporting task, which was added in PR https://github.com/elastic/kibana/pull/229998. Understand how p retry works by reading its documentation. Then, replace the custom retry code with p retry, ensuring that retry behavior (backoff, count) is preserved. Test the change to verify that scheduled reports are retried correctly. Check the comments on this issue for any additional guidance from maintainers.