Skip to content

[fix][broker] Check operstate when get physical NICs - #18883

Closed
mattisonchao wants to merge 3 commits into
apache:masterfrom
mattisonchao:check_operstate
Closed

[fix][broker] Check operstate when get physical NICs#18883
mattisonchao wants to merge 3 commits into
apache:masterfrom
mattisonchao:check_operstate

Conversation

@mattisonchao

@mattisonchao mattisonchao commented Dec 12, 2022

Copy link
Copy Markdown
Member

Fixes #18881

Master Issue: #18881

Motivation

See #18881

Modifications

  • Check operstate when get physical NICs

Verifying this change

  • Make sure that the change passes the CI checks.

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

  • Dependencies (add or upgrade a dependency)
  • The public API
  • The schema
  • The default values of configurations
  • The threading model
  • The binary protocol
  • The REST endpoints
  • The admin CLI options
  • Anything that affects deployment

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

@mattisonchao mattisonchao added this to the 2.12.0 milestone Dec 12, 2022
@mattisonchao mattisonchao self-assigned this Dec 12, 2022
@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Dec 12, 2022
@mattisonchao mattisonchao reopened this Dec 12, 2022
@mattisonchao mattisonchao mentioned this pull request Dec 12, 2022
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@mattisonchao mattisonchao

Labels

doc-not-needed Your PR changes do not impact docs ready-to-test release/2.9.4 release/2.10.3 release/2.11.1

Projects

None yet

Milestone

3.0.0

Development

Successfully merging this pull request may close these issues.

[Bug] incorrect Nic usage collected by pulsar

1 participant

@mattisonchao