insights-core icon indicating copy to clipboard operation
insights-core copied to clipboard

chore: do not collect ls_dev in core collection

Open xiangce opened this issue 6 months ago • 3 comments

  • as the size of ls_dev on some host is extraordinary large, which lead the engine to OOM, it's replace with some new datasources (#4440)
  • this PR stop colleting it in insights archive, but its parser is still avaiable for sos report
  • RHINENG-17123

All Pull Requests:

Check all that apply:

  • [ ] Have you followed the guidelines in our Contributing document, including the instructions about commit messages?
  • [ ] No Sensitive Data in this change?
  • [ ] Is this PR to correct an issue?
  • [ ] Is this PR an enhancement?

Complete Description of Additions/Changes:

Add your description here

xiangce avatar May 12 '25 07:05 xiangce

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 77.64%. Comparing base (fce95f0) to head (56abded).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4453      +/-   ##
==========================================
- Coverage   77.64%   77.64%   -0.01%     
==========================================
  Files         745      745              
  Lines       41669    41668       -1     
  Branches     6696     6696              
==========================================
- Hits        32354    32353       -1     
  Misses       8285     8285              
  Partials     1030     1030              
Flag Coverage Δ
unittests 77.63% <ø> (-0.01%) :arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

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

codecov-commenter avatar May 12 '25 08:05 codecov-commenter

New datasource to replace the ls_dev is https://github.com/RedHatInsights/insights-core/pull/4456

xiangce avatar May 27 '25 05:05 xiangce

test me

zhangqianqian avatar May 28 '25 02:05 zhangqianqian