Skip to content

[Backport v1.27] Fix E2e tests on main#3068

Merged
tbavelier merged 1 commit into
v1.27from
backport-3064-to-v1.27
Jun 2, 2026
Merged

[Backport v1.27] Fix E2e tests on main#3068
tbavelier merged 1 commit into
v1.27from
backport-3064-to-v1.27

Conversation

@dd-octo-sts
Copy link
Copy Markdown

@dd-octo-sts dd-octo-sts Bot commented Jun 2, 2026

Backport 6ef503a from #3064.


What does this PR do?

  • Use ECR registry instead of default registry.datadoghq.com to pull images faster on EC2: eeccc0d
  • Run agent status ... -j with DD_LOG_LEVEL=off in k8s e2e checks so status output is clean JSON.
  • Remove regex-based status JSON extraction and parse agent status -j output directly, failing when the command does not return valid JSON.

Motivation

The e2e assertions were compensating for log lines mixed into agent status -j output by extracting the first JSON-looking object. Disabling Agent logging for status commands makes the command output deterministic and lets the tests validate the actual JSON contract.

Additional Notes

No production code changes.

Minimum Agent Versions

Are there minimum versions of the Datadog Agent and/or Cluster Agent required?

  • Agent: N/A
  • Cluster Agent: N/A

Describe your test plan

  • make lint-e2e

Checklist

  • PR has at least one valid label: bug, enhancement, refactoring, documentation, tooling, and/or dependencies
  • PR has a milestone or the qa/skip-qa label
  • All commits are signed (see: signing commits)

* Use ECR registry instead of default registry.datadoghq.com

* Disable log level when running status command to ensure no polluted output

* Fix lint-e2e by using require.NoError

---------

Co-authored-by: khewonc <39867936+khewonc@users.noreply.github.com>
(cherry picked from commit 6ef503a)
@dd-octo-sts dd-octo-sts Bot added tooling qa/skip-qa backport label added by backport action bot label added by backport bot labels Jun 2, 2026
@dd-octo-sts dd-octo-sts Bot requested a review from a team as a code owner June 2, 2026 06:33
@dd-octo-sts dd-octo-sts Bot added team/container-platform tooling qa/skip-qa backport label added by backport action bot label added by backport bot labels Jun 2, 2026
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jun 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 41.61%. Comparing base (1ad9264) to head (95e5523).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##            v1.27    #3068   +/-   ##
=======================================
  Coverage   41.61%   41.61%           
=======================================
  Files         336      336           
  Lines       28737    28737           
=======================================
  Hits        11958    11958           
  Misses      15984    15984           
  Partials      795      795           
Flag Coverage Δ
unittests 41.61% <ø> (ø)

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


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1ad9264...95e5523. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@datadog-datadog-prod-us1
Copy link
Copy Markdown

Code Coverage

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 41.91% (+0.00%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 95e5523 | Docs | Datadog PR Page | Give us feedback!

@tbavelier tbavelier merged commit ea99112 into v1.27 Jun 2, 2026
87 checks passed
@tbavelier tbavelier deleted the backport-3064-to-v1.27 branch June 2, 2026 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport label added by backport action bot label added by backport bot qa/skip-qa team/container-platform tooling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants