Skip to content

Conversation

@fanny-jiang
Copy link
Contributor

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

@fanny-jiang fanny-jiang force-pushed the fanny/AGENTONB-2559/mapper-plugin branch from 48c3616 to ba5f91f Compare October 1, 2025 15:57
@dd-octo-sts
Copy link

dd-octo-sts bot commented Oct 23, 2025

This pull request has been automatically marked as stale because it has not had activity in the past 15 days.

It will be closed in 30 days if no further activity occurs. If this pull request is still relevant, adding a comment or pushing new commits will keep it open. Also, you can always reopen the pull request if you missed the window.

Thank you for your contributions!

@dd-octo-sts dd-octo-sts bot added the stale label Oct 23, 2025
@fanny-jiang fanny-jiang force-pushed the fanny/AGENTONB-2559/mapper-plugin branch from fa7a77a to f437868 Compare October 28, 2025 21:53
@codecov-commenter
Copy link

codecov-commenter commented Oct 28, 2025

Codecov Report

❌ Patch coverage is 15.08152% with 625 lines in your changes missing coverage. Please review.
✅ Project coverage is 37.27%. Comparing base (c350dbf) to head (f437868).

Files with missing lines Patch % Lines
cmd/yaml-mapper/mapper/mapper.go 0.00% 172 Missing ⚠️
cmd/yaml-mapper/mapper/cmd.go 0.00% 95 Missing ⚠️
cmd/kubectl-datadog/helm2dda/helm2dda.go 0.00% 78 Missing ⚠️
cmd/yaml-mapper/utils/maps.go 0.00% 60 Missing ⚠️
cmd/yaml-mapper/utils/charts_remote.go 0.00% 55 Missing ⚠️
cmd/yaml-mapper/utils/deprecations.go 0.00% 48 Missing ⚠️
cmd/yaml-mapper/utils/utils.go 0.00% 47 Missing ⚠️
cmd/yaml-mapper/mapper/map_processors.go 73.50% 24 Missing and 16 partials ⚠️
cmd/yaml-mapper/main.go 0.00% 29 Missing ⚠️
cmd/kubectl-datadog/datadog/datadog.go 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2215      +/-   ##
==========================================
- Coverage   38.05%   37.27%   -0.78%     
==========================================
  Files         255      264       +9     
  Lines       20949    21685     +736     
==========================================
+ Hits         7972     8083     +111     
- Misses      12359    12968     +609     
- Partials      618      634      +16     
Flag Coverage Δ
unittests 37.27% <15.08%> (-0.78%) ⬇️

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

Files with missing lines Coverage Δ
cmd/kubectl-datadog/datadog/datadog.go 0.00% <0.00%> (ø)
cmd/yaml-mapper/main.go 0.00% <0.00%> (ø)
cmd/yaml-mapper/mapper/map_processors.go 73.50% <73.50%> (ø)
cmd/yaml-mapper/utils/utils.go 0.00% <0.00%> (ø)
cmd/yaml-mapper/utils/deprecations.go 0.00% <0.00%> (ø)
cmd/yaml-mapper/utils/charts_remote.go 0.00% <0.00%> (ø)
cmd/yaml-mapper/utils/maps.go 0.00% <0.00%> (ø)
cmd/kubectl-datadog/helm2dda/helm2dda.go 0.00% <0.00%> (ø)
cmd/yaml-mapper/mapper/cmd.go 0.00% <0.00%> (ø)
cmd/yaml-mapper/mapper/mapper.go 0.00% <0.00%> (ø)

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 c350dbf...f437868. Read the comment docs.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants