Skip to content

[AGENT-9912] Improve debug logs for ena and ethtool network metrics#15447

Merged
vivek-datadog merged 1 commit into
masterfrom
vivek-datadog/AGENT-9912
Aug 3, 2023
Merged

[AGENT-9912] Improve debug logs for ena and ethtool network metrics#15447
vivek-datadog merged 1 commit into
masterfrom
vivek-datadog/AGENT-9912

Conversation

@vivek-datadog

@vivek-datadog vivek-datadog commented Aug 1, 2023 •
edited by atlassian Bot
Loading

Copy link
Copy Markdown
Contributor

What does this PR do?

Enhance debug logs for ena and ethtool network metric collection

Motivation

AGENT-9912
Support ticket

Additional Notes

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • PR title must be written as a CHANGELOG entry (see why)
  • Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
  • PR must have changelog/ and integration/ labels attached
  • If the PR doesn't need to be tested during QA, please add a qa/skip-qa label.

@vivek-datadog
vivek-datadog requested review from a team as code owners August 1, 2023 16:45
@vivek-datadog vivek-datadog self-assigned this Aug 1, 2023
@codecov

codecov Bot commented Aug 1, 2023 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

Merging #15447 (bba8f7e) into master (e5a0c7b) will increase coverage by 0.00%.
Report is 13 commits behind head on master.
The diff coverage is 72.72%.

Flag Coverage Δ
network 93.63% <72.72%> (+0.73%) ⬆️

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

@github-actions

github-actions Bot commented Aug 1, 2023

Copy link
Copy Markdown
Contributor

Test Results

    6 files      6 suites   12s ⏱️
  45 tests   45 ✔️   0 💤 0 ❌
182 runs  138 ✔️ 44 💤 0 ❌

Results for commit bba8f7e.

@FlorentClarret

FlorentClarret commented Aug 2, 2023

Copy link
Copy Markdown
Member

Switched from no-changelog to fixed since this is shipped with the agent

@vivek-datadog
vivek-datadog merged commit f1d670b into master Aug 3, 2023
@vivek-datadog
vivek-datadog deleted the vivek-datadog/AGENT-9912 branch August 3, 2023 08:14
@vivek-datadog vivek-datadog added the qa/skip-qa Automatically skip this PR for the next QA label Aug 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@FlorentClarret FlorentClarret FlorentClarret approved these changes
@clarkb7 clarkb7 clarkb7 approved these changes

Assignees

@vivek-datadog vivek-datadog

Labels

integration/network qa/skip-qa Automatically skip this PR for the next QA

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@vivek-datadog @FlorentClarret @clarkb7