Skip to content

fix(test-optimization): cap screenshot upload retries - #10056

Merged
bm1549 merged 1 commit into
masterfrom
BridgeAR/2026-08-28-cypress-screenshot-retry
Aug 28, 2026
Merged

bm1549 merged 1 commit into
masterfrom
BridgeAR/2026-08-28-cypress-screenshot-retry

Conversation

@BridgeAR

Copy link
Copy Markdown
Member

A persistent media 5xx bypassed the ordinary retry cap because screenshot uploads have a finalization deadline. Cypress then waited until the integration process timed out.

Screenshot uploads now keep the deadline for cancellation but stop after the ordinary retry cap. Final flush requests continue to retry until their deadline.

A persistent media 5xx bypassed the ordinary retry cap because the upload had a finalization deadline, so the Cypress runner waited until the integration process timed out.
@dd-octo-sts

dd-octo-sts Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Overall package size

Self size: 8.44 MB
Deduped: 9.1 MB
No deduping: 9.1 MB

Dependency sizes | name | version | self size | total size | |------|---------|-----------|------------| | import-in-the-middle | 3.3.3 | 125.43 kB | 445.14 kB | | opentracing | 0.14.7 | 194.81 kB | 194.81 kB | | dc-polyfill | 0.1.11 | 25.74 kB | 25.74 kB |

🤖 This report was automatically generated by heaviest-objects-in-the-universe

@datadog-official

datadog-official Bot commented Aug 28, 2026 •

Copy link
Copy Markdown

Pipelines  Tests

⚠️ Warnings

❌ Your PR has failed checks. Please review the issues below and take necessary action before merging.

🚦 3 Pipeline jobs failed

All Green | all-green — ❌ 7 tests failed

View more details · View in GitHub Actions

xcrun not installed or not found, coverage.py not found, and no gcov data found.

❌ with @anthropic-ai/sdk >=0.33.0 (0.117.1) anthropic real SDK reader path "before all" hook for "evaluates the request snapshot sent before caller mutation" from anthropic real SDK reader path
require() of ES Module /home/runner/work/dd-trace-js/dd-trace-js/versions/@anthropic-ai/sdk@0/node_modules/@stablelib/base64/lib/base64.js from /home/runner/work/dd-trace-js/dd-trace-js/versions/@anthropic-ai/sdk@0/node_modules/standardwebhooks/dist/index.js not supported.
Instead change the require of base64.js in /home/runner/work/dd-trace-js/dd-trace-js/versions/@anthropic-ai/sdk@0/node_modules/standardwebhooks/dist/index.js to a dynamic import() which is available in all CommonJS modules.

Error [ERR_REQUIRE_ESM]: require() of ES Module /home/runner/work/dd-trace-js/dd-trace-js/versions/@anthropic-ai/sdk@0/node_modules/@stablelib/base64/lib/base64.js from /home/runner/work/dd-trace-js/dd-trace-js/versions/@anthropic-ai/sdk@0/node_modules/standardwebhooks/dist/index.js not supported.
Instead change the require of base64.js in /home/runner/work/dd-trace-js/dd-trace-js/versions/@anthropic-ai/sdk@0/node_modules/standardwebhooks/dist/index.js to a dynamic import() which is available in all CommonJS modules.
    at Object.<anonymous> (versions/@anthropic-ai/sdk@0/node_modules/standardwebhooks/dist/index.js:4:16)
    at Object.<anonymous> (versions/@anthropic-ai/sdk@0/node_modules/@anthropic-ai/sdk/resources/beta/webhooks.js:6:28)
    at Object.<anonymous> (versions/@anthropic-ai/sdk@0/node_modules/@anthropic-ai/sdk/resources/beta/beta.js:19:42)
    at Object.<anonymous> (versions/@anthropic-ai/sdk@0/node_modules/@anthropic-ai/sdk/resources/index.js:7:14)
    at Object.<anonymous> (versions/@anthropic-ai/sdk@0/node_modules/@anthropic-ai/sdk/client.js:24:34)
...
❌ Plugin with @anthropic-ai/sdk >=0.33.0 (0.117.1) "before all" hook in "with @anthropic-ai/sdk >=0.33.0 (0.117.1)" from with @anthropic-ai/sdk >=0.33.0 (0.117.1)
require() of ES Module /home/runner/work/dd-trace-js/dd-trace-js/versions/@anthropic-ai/sdk@0/node_modules/@stablelib/base64/lib/base64.js from /home/runner/work/dd-trace-js/dd-trace-js/versions/@anthropic-ai/sdk@0/node_modules/standardwebhooks/dist/index.js not supported.
Instead change the require of base64.js in /home/runner/work/dd-trace-js/dd-trace-js/versions/@anthropic-ai/sdk@0/node_modules/standardwebhooks/dist/index.js to a dynamic import() which is available in all CommonJS modules.

Error [ERR_REQUIRE_ESM]: require() of ES Module /home/runner/work/dd-trace-js/dd-trace-js/versions/@anthropic-ai/sdk@0/node_modules/@stablelib/base64/lib/base64.js from /home/runner/work/dd-trace-js/dd-trace-js/versions/@anthropic-ai/sdk@0/node_modules/standardwebhooks/dist/index.js not supported.
Instead change the require of base64.js in /home/runner/work/dd-trace-js/dd-trace-js/versions/@anthropic-ai/sdk@0/node_modules/standardwebhooks/dist/index.js to a dynamic import() which is available in all CommonJS modules.
    at Hook.Module.require (packages/dd-trace/src/ritm.js:121:31)
    at Object.<anonymous> (versions/@anthropic-ai/sdk@0/node_modules/standardwebhooks/dist/index.js:4:16)
    at Module._compile (packages/datadog-instrumentations/src/helpers/rewriter/loader.js:8:18)
    at Hook.Module.require (packages/dd-trace/src/ritm.js:121:31)
    at Object.<anonymous> (versions/@anthropic-ai/sdk@0/node_modules/@anthropic-ai/sdk/resources/beta/webhooks.js:6:28)
...
❌ esm with @anthropic-ai/sdk >=0.33.0 (0.117.1) is instrumented default from with @anthropic-ai/sdk >=0.33.0 (0.117.1)
Process exited with status code 1.

Error: Process exited with status code 1.
    at ChildProcess.<anonymous> (integration-tests/helpers/index.js:266:25)
    at Object.onceWrapper (node:events:632:26)
    at ChildProcess.emit (node:events:517:28)
    at ChildProcess._handle.onexit (node:internal/child_process:292:12)
    at Process.callbackTrampoline (node:internal/async_hooks:128:17)
↳ and 4 more — View all
Instrumentation | instrumentation-anthropic-lifecycle — 🔧 Needs a code fix, caused by this PR

View more details · View in GitHub Actions

Error [ERR_REQUIRE_ESM]: cannot require() an ES Module at /home/runner/work/dd-trace-js/dd-trace-js/versions/@anthropic-ai/sdk@0/node_modules/standardwebhooks/dist/index.js

LLMObs | anthropic — 🔧 Needs a code fix, caused by this PR

View more details · View in GitHub Actions

Error [ERR_REQUIRE_ESM]: require() of ES Module base64.js not supported. Use dynamic import() instead in standardwebhooks/dist/index.js.

📋 Copy fix prompt
CI on my pull request is failing. Help me find and fix the root cause of each failing job below — they were flagged as caused by changes in this PR, so focus on the diff. For each job, explain the failure and propose a fix.

Before you start, set up the Datadog software-delivery tooling so you can
query the CI data yourself:

1. Check whether you already have the Datadog software-delivery MCP tools
   (e.g. a `search_datadog_ci_pipeline_events` tool) and the `unblock-pr` skill.
2. If either is missing, STOP and ask me for permission before installing
   anything. Do not install or run anything until I have said yes.
3. Only with my explicit approval, set up the Datadog software-delivery MCP
   server and skills by following:
     https://docs.datadoghq.com/getting_started/software_delivery_mcp_tools/
   then restart so the skill is picked up.
4. If I decline, skip all of the above and work from the context below alone.

Then run /unblock-pr — it will pull the CI data itself. The job context below is what we already know.

If /unblock-pr is not available — because I declined the setup above, or it did not install — work from the context below instead.

Datadog has already classified this failure as caused by changes in this PR.
Take that as given and work the fix:

1. Locate the change. Diff this branch against its base and find the change
   that produces this error. Explain the mechanism, don't just name a file:
     git fetch origin && git diff $(git merge-base origin/master HEAD)...HEAD
2. Reproduce it locally. Run the failing job's command or test before
   proposing anything.
3. Propose the smallest fix that addresses the root cause — not a workaround,
   not a broadened assertion, not a disabled or skipped test.
4. Re-run the same command to confirm, and say exactly what you ran.
5. If the failure turns out to be intermittent rather than deterministic, say
   so plainly instead of "fixing" it — that is a flaky test, and patching it
   hides the problem.

If the right move is to re-run the job rather than change code, use the job
link in the context below. For GitHub Actions: `gh run rerun <run-id> --failed`,
where the run ID is the number after `/runs/` in that URL (not the trailing
number, which is the job ID).

Branch: BridgeAR/2026-08-28-cypress-screenshot-retry

Instrumentation | instrumentation-anthropic-lifecycle
Commit: 7d0e74e14582426449d0669d1e7aa13ccea5dbcf
Error (code / build):
Error [ERR_REQUIRE_ESM]: cannot require() an ES Module at /home/runner/work/dd-trace-js/dd-trace-js/versions/@anthropic-ai/sdk@0/node_modules/standardwebhooks/dist/index.js
CI job: https://github.com/DataDog/dd-trace-js/actions/runs/33201896498/job/98954637626

LLMObs | anthropic
Commit: 7d0e74e14582426449d0669d1e7aa13ccea5dbcf
Error (code / build):
Error [ERR_REQUIRE_ESM]: require() of ES Module base64.js not supported. Use dynamic import() instead in standardwebhooks/dist/index.js.
CI job: https://github.com/DataDog/dd-trace-js/actions/runs/33201896489/job/98953135905

ℹ️ Info

No other issues found (see more)

❄️ No new flaky tests detected

🎯 Code Coverage (details)
• Patch Coverage: 100.00%
• Overall Coverage: 97.18% (+0.29%)

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 5aed405 | Docs | View more details | Give us feedback!

@pr-commenter

pr-commenter Bot commented Aug 28, 2026

Copy link
Copy Markdown

Benchmarks

Benchmark execution time: 2026-08-28 19:09:20

Comparing candidate commit 5aed405 in PR branch BridgeAR/2026-08-28-cypress-screenshot-retry with baseline commit 23bcda4 in branch master.

📊 Benchmarking dashboard

Found 1 performance improvements and 0 performance regressions! Performance is the same for 2295 metrics, 14 unstable metrics.

Explanation

This is an A/B test comparing a candidate commit's performance against that of a baseline commit. Performance changes are noted in the tables below as:

  • 🟩 = significantly better candidate vs. baseline
  • 🟥 = significantly worse candidate vs. baseline

We compute a confidence interval (CI) over the relative difference of means between metrics from the candidate and baseline commits, considering the baseline as the reference.

If the CI is entirely outside the configured SIGNIFICANT_IMPACT_THRESHOLD (or the deprecated UNCONFIDENCE_THRESHOLD), the change is considered significant.

Feel free to reach out to #apm-benchmarking-platform on Slack if you have any questions.

More details about the CI and significant changes

You can imagine this CI as a range of values that is likely to contain the true difference of means between the candidate and baseline commits.

CIs of the difference of means are often centered around 0%, because often changes are not that big:

---------------------------------(------|---^--------)-------------------------------->
                              -0.6%    0%  0.3%     +1.2%
                                 |          |        |
         lower bound of the CI --'          |        |
sample mean (center of the CI) -------------'        |
         upper bound of the CI ----------------------'

As described above, a change is considered significant if the CI is entirely outside the configured SIGNIFICANT_IMPACT_THRESHOLD (or the deprecated UNCONFIDENCE_THRESHOLD).

For instance, for an execution time metric, this confidence interval indicates a significantly worse performance:

----------------------------------------|---------|---(---------^---------)---------->
                                       0%        1%  1.3%      2.2%      3.1%
                                                  |   |         |         |
       significant impact threshold --------------'   |         |         |
                      lower bound of CI --------------'         |         |
       sample mean (center of the CI) --------------------------'         |
                      upper bound of CI ----------------------------------'

scenario:encoders-0.4-events-native-20

  • 🟩 max_rss_usage [-10.666MB; -3.744MB] or [-7.500%; -2.633%]

Unstable benchmarks

These benchmarks have a confidence interval too wide to call a change; treat them as noise rather than signal.

scenario:appsec-appsec-enabled-20

  • unstable max_rss_usage [-22.894MB; +48.143MB] or [-12.672%; +26.647%]

scenario:appsec-iast-no-vulnerability-iast-enabled-default-config-20

  • unstable max_rss_usage [-11.778MB; +22.272MB] or [-5.217%; +9.865%]

scenario:appsec-iast-with-vulnerability-iast-enabled-default-config-20

  • unstable max_rss_usage [-25.501MB; +11.817MB] or [-6.896%; +3.196%]

scenario:debugger-line-probe-with-snapshot-minimal-24

  • unstable max_rss_usage [-9.222MB; +16.853MB] or [-3.724%; +6.806%]

scenario:debugger-line-probe-without-snapshot-26

  • unstable max_rss_usage [-16.366MB; +18.836MB] or [-6.094%; +7.014%]

scenario:dogstatsd-aggregated-26

  • unstable execution_time [-63.216ms; +114.155ms] or [-3.810%; +6.881%]

scenario:dogstatsd-with-tags-20

  • unstable cpu_user_time [-201.724ms; +509.001ms] or [-4.057%; +10.238%]
  • unstable execution_time [-207.446ms; +512.554ms] or [-4.106%; +10.145%]
  • unstable throughput [-173294.103op/s; +75051.241op/s] or [-10.419%; +4.512%]

scenario:plugin-graphql-long-with-depth-and-collapse-off-20

  • unstable max_rss_usage [-24.948MB; +38.402MB] or [-6.466%; +9.953%]

scenario:plugin-mongodb-core-plain-find-26

  • unstable execution_time [-137.421ms; +173.905ms] or [-6.687%; +8.463%]
  • unstable throughput [-259120.254op/s; +223143.605op/s] or [-6.434%; +5.540%]

scenario:plugin-pg-service-20

  • unstable execution_time [-73.191ms; +111.912ms] or [-4.499%; +6.879%]

scenario:test-optimization-large-suite-20

  • unstable max_rss_usage [-7.547MB; +4.247MB] or [-9.157%; +5.153%]

@BridgeAR
BridgeAR marked this pull request as ready for review August 28, 2026 19:10
@BridgeAR
BridgeAR requested review from a team as code owners August 28, 2026 19:10
@codecov

codecov Bot commented Aug 28, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.18%. Comparing base (23bcda4) to head (5aed405).

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #10056      +/-   ##
==========================================
+ Coverage   96.89%   97.18%   +0.29%     
==========================================
  Files         992      992              
  Lines      149803   149804       +1     
  Branches    12166    12518     +352     
==========================================
+ Hits       145146   145586     +440     
+ Misses       4657     4218     -439     
Flag Coverage Δ
aiguard 63.39% <ø> (-0.03%) ⬇️
aiguard-integration 59.15% <ø> (ø)
apm-bucket-0 62.74% <ø> (-0.03%) ⬇️
apm-bucket-1 68.24% <ø> (-0.03%) ⬇️
apm-bucket-2 67.82% <ø> (-0.03%) ⬇️
apm-bucket-3 64.47% <ø> (?)
apm-capabilities-tracing 62.60% <100.00%> (+<0.01%) ⬆️
apm-integrations-aerospike 60.72% <ø> (-0.03%) ⬇️
apm-integrations-confluentinc-kafka-javascript 66.91% <ø> (-0.07%) ⬇️
apm-integrations-couchbase 61.24% <ø> (-0.03%) ⬇️
apm-integrations-http 66.49% <ø> (-0.03%) ⬇️
apm-integrations-kafkajs 67.53% <ø> (-0.03%) ⬇️
apm-integrations-next 64.09% <ø> (-0.03%) ⬇️
apm-integrations-prisma ?
apm-integrations-restify_integrations-sharedb ?
apm-integrations-tedious 61.59% <ø> (?)
appsec 77.12% <ø> (-0.02%) ⬇️
appsec-express_fastify_graphql 73.53% <ø> (-0.02%) ⬇️
appsec-integration 49.47% <ø> (-0.04%) ⬇️
appsec-kafka_ldapjs_lodash 67.72% <ø> (-0.03%) ⬇️
appsec-mongodb-core_mongoose_mysql 70.92% <ø> (-0.02%) ⬇️
appsec-next 57.23% <ø> (-0.02%) ⬇️
appsec-node-serialize_passport_postgres 70.38% <ø> (-0.02%) ⬇️
appsec-sourcing_stripe_template 68.80% <ø> (-0.02%) ⬇️
debugger 69.38% <ø> (-0.05%) ⬇️
instrumentations-bucket-0 56.01% <ø> (-0.03%) ⬇️
instrumentations-bucket-1 64.06% <ø> (-0.03%) ⬇️
instrumentations-bucket-10 65.61% <ø> (-0.03%) ⬇️
instrumentations-bucket-11 66.27% <ø> (-0.03%) ⬇️
instrumentations-bucket-12 56.08% <ø> (-0.03%) ⬇️
instrumentations-bucket-13 56.16% <ø> (-0.03%) ⬇️
instrumentations-bucket-14 55.94% <ø> (-0.03%) ⬇️
instrumentations-bucket-2 57.31% <ø> (-0.03%) ⬇️
instrumentations-bucket-3 57.79% <ø> (-0.03%) ⬇️
instrumentations-bucket-4 63.93% <ø> (-0.03%) ⬇️
instrumentations-bucket-5 50.99% <ø> (-0.02%) ⬇️
instrumentations-bucket-6 66.12% <ø> (-0.03%) ⬇️
instrumentations-bucket-7 63.25% <ø> (-0.03%) ⬇️
instrumentations-bucket-8 64.05% <ø> (-0.02%) ⬇️
instrumentations-bucket-9 58.82% <ø> (-0.03%) ⬇️
instrumentations-instrumentation-couchbase 54.80% <ø> (-0.03%) ⬇️
instrumentations-integration-esbuild 34.42% <ø> (ø)
llmobs-ai_bedrock_claude-agent-sdk 67.73% <ø> (-0.02%) ⬇️
llmobs-google-genai_langchain_openai-agents 65.13% <ø> (-0.02%) ⬇️
llmobs-openai 66.96% <ø> (-0.03%) ⬇️
llmobs-sdk 76.49% <ø> (-0.03%) ⬇️
llmobs-vertex-ai 63.52% <ø> (-0.01%) ⬇️
master-coverage 97.18% <100.00%> (?)
openfeature 59.77% <ø> (ø)
openfeature-unit 58.44% <ø> (-0.03%) ⬇️
platform-core_esbuild_instrumentations-misc 41.00% <ø> (-0.02%) ⬇️
platform-integration 64.99% <ø> (ø)
platform-shimmer_unit-guardrails_webpack 38.81% <ø> (-0.02%) ⬇️
plugins-browser-bunyan_bullmq_cassandra 66.52% <ø> (-0.03%) ⬇️
plugins-bucket-0 61.11% <ø> (-0.03%) ⬇️
plugins-bucket-1 58.31% <ø> (ø)
plugins-bucket-11 66.99% <ø> (+0.60%) ⬆️
plugins-bucket-14 ?
plugins-bucket-17 66.74% <ø> (-0.25%) ⬇️
plugins-bucket-18 66.95% <ø> (+1.71%) ⬆️
plugins-bucket-19 66.74% <ø> (-0.32%) ⬇️
plugins-bucket-20 69.47% <ø> (?)
plugins-bucket-4 61.30% <ø> (-0.03%) ⬇️
plugins-cookie_cookie-parser_crypto 55.80% <ø> (-0.03%) ⬇️
plugins-fastify_fetch_fs 65.60% <ø> (-0.03%) ⬇️
plugins-generic-pool_google-cloud-pubsub_grpc 69.25% <ø> (-0.03%) ⬇️
plugins-handlebars_hapi_hono 63.84% <ø> (-0.03%) ⬇️
plugins-ioredis_knex_langgraph 61.82% <ø> (?)
plugins-ioredis_langgraph_ldapjs ?
plugins-ldapjs_light-my-request_limitd-client 63.56% <ø> (?)
plugins-light-my-request_limitd-client_lodash ?
plugins-lodash_mariadb_memcached 64.04% <ø> (?)
plugins-mariadb_memcached_mercurius ?
plugins-moleculer_mongodb-core_mongoose 66.02% <ø> (?)
plugins-mongodb-core_mongoose_multer ?
plugins-multer_mysql_mysql2 63.93% <ø> (?)
plugins-mysql_mysql2_nats ?
plugins-nats_node-serialize_opensearch 65.83% <ø> (?)
plugins-passport-http_pino_postgres 63.64% <ø> (?)
plugins-pino_postgres_process ?
plugins-process_pug_redis 62.43% <ø> (?)
plugins-pug_redis_router ?
plugins-test-and-upstream-rhea_undici_url ?
plugins-undici_url_valkey 62.67% <ø> (?)
plugins-valkey_vm_winston ?
plugins-vm_winston_ws 64.98% <ø> (?)
plugins-ws ?
profiling 66.41% <ø> (-0.03%) ⬇️
serverless-aws-sdk-aws-sdk 55.50% <ø> (-0.02%) ⬇️
serverless-aws-sdk-base-inject-field 55.36% <ø> (-0.03%) ⬇️
serverless-aws-sdk-bedrockruntime 58.40% <ø> (-0.03%) ⬇️
serverless-aws-sdk-client 60.22% <ø> (-0.03%) ⬇️
serverless-aws-sdk-dynamodb 59.29% <ø> (-0.02%) ⬇️
serverless-aws-sdk-eventbridge 57.74% <ø> (-0.02%) ⬇️
serverless-aws-sdk-kinesis 63.32% <ø> (-0.03%) ⬇️
serverless-aws-sdk-lambda 61.25% <ø> (-0.03%) ⬇️
serverless-aws-sdk-s3 59.33% <ø> (-0.03%) ⬇️
serverless-aws-sdk-serverless-peer-service 63.90% <ø> (-0.03%) ⬇️
serverless-aws-sdk-sns 64.22% <ø> (-0.03%) ⬇️
serverless-aws-sdk-sqs 64.68% <ø> (-0.03%) ⬇️
serverless-aws-sdk-stepfunctions 59.19% <ø> (-0.03%) ⬇️
serverless-aws-sdk-util 56.00% <ø> (-0.03%) ⬇️
serverless-bucket-0 57.94% <ø> (ø)
serverless-bucket-1 63.92% <ø> (-0.03%) ⬇️
test-optimization-cucumber 70.77% <75.00%> (+0.02%) ⬆️
test-optimization-cypress 64.57% <100.00%> (+0.95%) ⬆️
test-optimization-jest 72.30% <75.00%> (+<0.01%) ⬆️
test-optimization-mocha 72.22% <75.00%> (+0.04%) ⬆️
test-optimization-playwright-playwright-atr 59.92% <75.00%> (-0.07%) ⬇️
test-optimization-playwright-playwright-efd 60.56% <100.00%> (-0.06%) ⬇️
test-optimization-playwright-playwright-final-status 60.17% <75.00%> (-0.06%) ⬇️
test-optimization-playwright-playwright-impacted-tests 60.28% <75.00%> (+0.08%) ⬆️
test-optimization-playwright-playwright-reporting 61.23% <100.00%> (?)
test-optimization-playwright-playwright-test-management 60.60% <75.00%> (-0.07%) ⬇️
test-optimization-playwright-playwright-test-span 59.96% <75.00%> (-0.01%) ⬇️
test-optimization-selenium 58.97% <75.00%> (-0.09%) ⬇️
test-optimization-vitest 72.91% <75.00%> (+0.05%) ⬆️
test-optimization-vitest-browser 58.80% <75.00%> (-0.06%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@datadog-official datadog-official Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Datadog Autotest: PASS

More details

Screenshot uploads use the normal retry limit when a deadline exists. Final flush requests continue to retry until the deadline.

Was this helpful? React 👍 or 👎

Open Bits AI session

🤖 Datadog Autotest · Commit 5aed405 · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest

@bm1549

bm1549 commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator

/merge --reason "master is currently red. This PR fixes one of the two issues. This is bypassing 1) CI checks due to the second failure and 2) CODEOWNER review for the owning team"

@gh-worker-devflow-routing-ef8351

gh-worker-devflow-routing-ef8351 Bot commented Aug 28, 2026 •

Copy link
Copy Markdown

View all feedbacks in Devflow UI.

2026-08-28 19:31:51 UTC ℹ️ Start processing command /merge --reason "master is currently red. This PR fixes one of the two issues. This is bypassing 1) CI checks due to the second failure and 2) CODEOWNER review for the owning team"


2026-08-28 19:31:54 UTC ❌ MergeQueue

You can't provide a reason for skipping checks if you don't skip checks

@bm1549

bm1549 commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator

/merge -f --reason "master is currently red. This PR fixes one of the two issues. This is bypassing 1) CI checks due to the second failure and 2) CODEOWNER review for the owning team"

@gh-worker-devflow-routing-ef8351

Copy link
Copy Markdown

View all feedbacks in Devflow UI.

2026-08-28 19:33:32 UTC ℹ️ Start processing command /merge -f --reason "master is currently red. This PR fixes one of the two issues. This is bypassing 1) CI checks due to the second failure and 2) CODEOWNER review for the owning team"
If you need support, contact us on Slack #devflow!


2026-08-28 19:33:33 UTC ❌ Devflow: /merge -f --reason "master is currently red. This PR fixes one of the two issues. This is bypassing 1) CI checks due to the second failure and 2) CODEOWNER review for the owning team"

Arguments errors:

If you need support, contact us on Slack #devflow with those details!

@bm1549
bm1549 disabled auto-merge August 28, 2026 19:38
@bm1549
bm1549 merged commit 4e03a95 into master Aug 28, 2026
746 of 750 checks passed
@bm1549
bm1549 deleted the BridgeAR/2026-08-28-cypress-screenshot-retry branch August 28, 2026 19:38
This was referenced Aug 31, 2026
juan-fernandez added a commit that referenced this pull request Aug 31, 2026
Flush worker-exported telemetry before app-closing telemetry snapshots metrics, while leaving retry timers unreferenced so failed endpoints cannot hold the process open. Reuse the merged screenshot retry cap from #10056 because this stacked branch does not yet contain it.
BridgeAR added a commit that referenced this pull request Sep 4, 2026
A persistent media 5xx bypassed the ordinary retry cap because the upload had a finalization deadline, so the Cypress runner waited until the integration process timed out.
BridgeAR added a commit that referenced this pull request Sep 4, 2026
A persistent media 5xx bypassed the ordinary retry cap because the upload had a finalization deadline, so the Cypress runner waited until the integration process timed out.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants