Skip to content

Fix controller test#3136

Merged
khewonc merged 3 commits into
mainfrom
khewonc/controller-test-fix
Jun 12, 2026
Merged

Fix controller test#3136
khewonc merged 3 commits into
mainfrom
khewonc/controller-test-fix

Conversation

@khewonc

@khewonc khewonc commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

What does this PR do?

A brief description of the change being made with this pull request.

Motivation

What inspired you to submit this pull request?

Additional Notes

Anything else we should know when reviewing?

Minimum Agent Versions

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

  • Agent: vX.Y.Z
  • Cluster Agent: vX.Y.Z

Describe your test plan

Write there any instructions and details you may have to test your PR.

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)

@khewonc khewonc added this to the v1.28.0 milestone Jun 12, 2026
@khewonc
khewonc requested a review from a team June 12, 2026 16:13
@khewonc khewonc added bug Something isn't working tooling qa/skip-qa labels Jun 12, 2026
@datadog-prod-us1-6

datadog-prod-us1-6 Bot commented Jun 12, 2026

Copy link
Copy Markdown

Pipelines  Code Coverage

Fix all issues with BitsAI

⚠️ Warnings

🚦 1 Pipeline job failed

DataDog/datadog-operator | build_operator_image_fips_arm64   View in Datadog   GitLab

ℹ️ Info

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 43.87%

Useful? React with 👍 / 👎

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

@codecov-commenter

codecov-commenter commented Jun 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 43.59%. Comparing base (f36836a) to head (22248a8).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3136      +/-   ##
==========================================
+ Coverage   43.39%   43.59%   +0.19%     
==========================================
  Files         354      354              
  Lines       30102    30224     +122     
==========================================
+ Hits        13064    13177     +113     
- Misses      16165    16176      +11     
+ Partials      873      871       -2     
Flag Coverage Δ
unittests 43.59% <ø> (+0.19%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 2 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

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

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

@khewonc
khewonc merged commit ef74881 into main Jun 12, 2026
39 of 41 checks passed
@khewonc
khewonc deleted the khewonc/controller-test-fix branch June 12, 2026 18:23
@dd-octo-sts dd-octo-sts Bot mentioned this pull request Jun 12, 2026
3 tasks
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