Skip to content

Commit

Permalink
mark flaky tests
Browse files Browse the repository at this point in the history
  • Loading branch information
emmettbutler committed May 1, 2024
1 parent c8b907b commit 9e6e894
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
2 changes: 2 additions & 0 deletions tests/contrib/celery/test_integration.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
import ddtrace.internal.forksafe as forksafe
from ddtrace.propagation.http import HTTPPropagator
from tests.opentracer.utils import init_tracer
from tests.utils import flaky

from ...utils import override_global_config
from .base import CeleryBaseTestCase
Expand Down Expand Up @@ -209,6 +210,7 @@ def fn_task_parameters(user, force_logout=False):
assert run_span.get_tag("component") == "celery"
assert run_span.get_tag("span.kind") == "consumer"

@flaky(1722529274)
def test_fn_task_delay(self):
# using delay shorthand must preserve arguments
@self.app.task
Expand Down
2 changes: 2 additions & 0 deletions tests/internal/test_tracer_flare.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
from ddtrace.internal.flare import Flare
from ddtrace.internal.flare import FlareSendRequest
from ddtrace.internal.logger import get_logger
from tests.utils import flaky


DEBUG_LEVEL_INT = logging.DEBUG
Expand Down Expand Up @@ -118,6 +119,7 @@ def handle_agent_task():
for p in processes:
p.join()

@flaky(1722529274)
def test_multiple_process_partial_failure(self):
"""
Validte that even if the tracer flare fails for one process, we should
Expand Down

0 comments on commit 9e6e894

Please sign in to comment.