Skip to content

Commit 8217217

Browse files
authored
chore: retry flares cleanup on tests to remove flakiness on CI (#15198)
## Description <!-- Provide an overview of the change and motivation for the change --> ## Testing <!-- Describe your testing strategy or note what tests are included --> ## Risks <!-- Note any risks associated with this change, or "None" if no risks --> ## Additional Notes <!-- Any other information that would be helpful for reviewers --> Signed-off-by: Juanjo Alvarez <juanjo.alvarezmartinez@datadoghq.com>
1 parent d1b2ce2 commit 8217217

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

tests/internal/test_tracer_flare.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@
2323
from ddtrace.internal.flare.handler import _handle_tracer_flare
2424
from ddtrace.internal.logger import get_logger
2525
from ddtrace.internal.remoteconfig._connectors import PublisherSubscriberConnector
26+
from ddtrace.internal.utils.retry import fibonacci_backoff_with_jitter
2627
from tests.utils import remote_config_build_payload as build_payload
2728

2829

@@ -214,6 +215,7 @@ def test_json_logs(self):
214215
self.flare.clean_up_files()
215216
self.flare.revert_configs()
216217

218+
@fibonacci_backoff_with_jitter(attempts=5, initial_wait=0.1)
217219
def confirm_cleanup(self):
218220
assert not self.flare.flare_dir.exists(), f"The directory {self.flare.flare_dir} still exists"
219221
# Only check for file handler cleanup if prepare() was called

0 commit comments

Comments
 (0)