Skip to content

Exclude loopback interface from ethtool#13042

Merged
yzhan289 merged 2 commits into
masterfrom
bonnefoa/network-exclude-lo
Oct 4, 2022
Merged

Exclude loopback interface from ethtool#13042
yzhan289 merged 2 commits into
masterfrom
bonnefoa/network-exclude-lo

Conversation

@bonnefoa

@bonnefoa bonnefoa commented Oct 4, 2022

Copy link
Copy Markdown
Collaborator

What does this PR do?

Loopback interfaces don't support ethtool ioctl call and they are not gonna expose driver specific informations.
Exclude them so that we don't generate error logs.

Motivation

We want to avoid error logs. While they don't have an impact, they certainly are noisy.

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

Loopback interfaces doesn't support this ioctl call and they
are not gonna expose driver specific informations. Exclude them
so that we don't generate error logs.
@github-actions

github-actions Bot commented Oct 4, 2022

Copy link
Copy Markdown
Contributor

Label changelog/Changed was added to this Pull Request, so the next release will bump major version. Please make sure this is a breaking change, or use the changelog/Fixed label.

@codecov

codecov Bot commented Oct 4, 2022

Copy link
Copy Markdown

Codecov Report

Merging #13042 (369d882) into master (a9d2378) will increase coverage by 0.00%.
The diff coverage is 0.00%.

Flag Coverage Δ
network 84.47% <0.00%> (+0.50%) ⬆️

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

@KSerrania KSerrania added changelog/Fixed category/bugfix For use during Agent Release period and removed changelog/Changed labels Oct 4, 2022
@bonnefoa
bonnefoa marked this pull request as ready for review October 4, 2022 12:32
@bonnefoa
bonnefoa requested review from a team as code owners October 4, 2022 12:32
@KSerrania KSerrania changed the title network check: exclude loopback interface from ethtool Exclude loopback interface from ethtool Oct 4, 2022
@yzhan289
yzhan289 merged commit 3895ba2 into master Oct 4, 2022
@yzhan289
yzhan289 deleted the bonnefoa/network-exclude-lo branch October 4, 2022 16:53
yzhan289 pushed a commit that referenced this pull request Oct 4, 2022
* Exclude loopback interfaces lo and lo0 from SIOCETHTOOL

Loopback interfaces doesn't support this ioctl call and they
are not gonna expose driver specific informations. Exclude them
so that we don't generate error logs.

* Update configuration example to mention lo exclusion for ethtool
steveny91 pushed a commit that referenced this pull request Oct 27, 2022
* Exclude loopback interfaces lo and lo0 from SIOCETHTOOL

Loopback interfaces doesn't support this ioctl call and they
are not gonna expose driver specific informations. Exclude them
so that we don't generate error logs.

* Update configuration example to mention lo exclusion for ethtool
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