Skip to content

chore(deps): remove lz4 and bzip2#14489

Merged
janisz merged 3 commits intomasterfrom
remove_bzip2_lz4
Apr 9, 2025
Merged

chore(deps): remove lz4 and bzip2#14489
janisz merged 3 commits intomasterfrom
remove_bzip2_lz4

Conversation

@janisz
Copy link
Copy Markdown
Contributor

@janisz janisz commented Mar 4, 2025

Description

lz4 and bzip2 were added to support rocksdb since we do not use it anymore we can remove them.


  • CHANGELOG update is not needed
  • Documentation is not needed

Testing

  • inspected CI results

Automated testing

  • modified existing tests
  • contributed no automated tests

How I validated my change

CI

@janisz
Copy link
Copy Markdown
Contributor Author

janisz commented Mar 4, 2025

This change is part of the following stack:

Change managed by git-spice.

@rhacs-bot
Copy link
Copy Markdown
Contributor

rhacs-bot commented Mar 4, 2025

Images are ready for the commit at c8b566d.

To use with deploy scripts, first export MAIN_IMAGE_TAG=4.8.x-257-gc8b566d3ea.

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 4, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 48.89%. Comparing base (0762805) to head (c8b566d).
Report is 179 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #14489      +/-   ##
==========================================
- Coverage   49.17%   48.89%   -0.29%     
==========================================
  Files        2532     2547      +15     
  Lines      185508   186867    +1359     
==========================================
+ Hits        91223    91367     +144     
- Misses      87047    88269    +1222     
+ Partials     7238     7231       -7     
Flag Coverage Δ
go-unit-tests 48.89% <ø> (-0.29%) ⬇️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@janisz janisz force-pushed the remove_bzip2_lz4 branch from 8f7afb7 to a36833b Compare March 18, 2025 12:09
@msugakov msugakov added the ci-all-qa-tests Tells CI to run all API tests (not just BAT). label Mar 24, 2025
Base automatically changed from remove_kill to master March 25, 2025 12:06
This reverts commit 4259821.

Signed-off-by: Tomasz Janiszewski <[email protected]>
@janisz janisz requested a review from msugakov April 1, 2025 13:58
Copy link
Copy Markdown
Contributor

@msugakov msugakov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I unchecked this box for you
image

because I see the CI is running currently.

Please ping me for re-review once you checked the CI status. Thank you.

@janisz
Copy link
Copy Markdown
Contributor Author

janisz commented Apr 3, 2025

/retest

@janisz janisz requested a review from msugakov April 8, 2025 09:50
@janisz janisz merged commit e83aaec into master Apr 9, 2025
98 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/helm ci-all-qa-tests Tells CI to run all API tests (not just BAT).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants