Skip to content

Fix system dashboard#4472

Merged
joshdover merged 4 commits intoelastic:mainfrom
flash1293:system-metric-fix
Oct 19, 2022
Merged

Fix system dashboard#4472
joshdover merged 4 commits intoelastic:mainfrom
flash1293:system-metric-fix

Conversation

@flash1293
Copy link
Copy Markdown
Contributor

What does this PR do?

The network traffic panels on the system overview page don't work as long as there isn't a filter for a specific host set. This issue isn't released yet.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
    - [ ] I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.

@flash1293 flash1293 added the bug Something isn't working, use only for issues label Oct 19, 2022
@flash1293 flash1293 requested review from a team as code owners October 19, 2022 09:56
@flash1293 flash1293 requested review from belimawr and rdner October 19, 2022 09:56
@elasticmachine
Copy link
Copy Markdown

elasticmachine commented Oct 19, 2022

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2022-10-19T10:09:55.605+0000

  • Duration: 14 min 2 sec

Test stats 🧪

Test Results
Failed 0
Passed 246
Skipped 0
Total 246

🤖 GitHub comments

Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@elasticmachine
Copy link
Copy Markdown

🚀 Benchmarks report

Package system 👍(1) 💚(0) 💔(2)

Expand to view
Data stream Previous EPS New EPS Diff (%) Result
auth 2169.2 1552.8 -616.4 (-28.42%) 💔
security 1968.5 821.02 -1147.48 (-58.29%) 💔

To see the full report comment with /test benchmark fullreport

@elasticmachine
Copy link
Copy Markdown

🌐 Coverage report

Name Metrics % (covered/total) Diff
Packages 100.0% (3/3) 💚
Files 100.0% (4/4) 💚 2.712
Classes 100.0% (4/4) 💚 2.712
Methods 60.759% (48/79) 👎 -29.231
Lines 98.793% (2702/2735) 👍 7.229
Conditionals 100.0% (0/0) 💚

@joshdover joshdover merged commit 32f68fe into elastic:main Oct 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working, use only for issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants