Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compress CI artifacts #2424

Merged
merged 7 commits into from Apr 13, 2023
Merged

Compress CI artifacts #2424

merged 7 commits into from Apr 13, 2023

Conversation

noisersup
Copy link
Member

@noisersup noisersup commented Apr 13, 2023

Description

Closes #2086.

Without compression:

https://github.com/FerretDB/FerretDB/actions/runs/4689866514 linux-logs | 254 KB
https://github.com/FerretDB/FerretDB/actions/runs/4689866515 ferretdb.log | 1.2 KB

With compression:

https://github.com/FerretDB/FerretDB/actions/runs/4689627132 linux-logs | 45 KB
https://github.com/FerretDB/FerretDB/actions/runs/4689627157 ferretdb-log | 806 Bytes

Readiness checklist

  • I added/updated unit tests.
  • I added/updated integration/compatibility tests.
  • I added/updated comments and checked rendering.
  • I made spot refactorings.
  • I updated user documentation.
  • I ran task all, and it passed.
  • I ensured that PR title is good enough for the changelog.
  • (for maintainers only) I set Reviewers (@FerretDB/core), Assignee, Labels, Project and project's Sprint fields.
  • I marked all done items in this checklist.

@noisersup noisersup added the code/chore Code maintenance improvements label Apr 13, 2023
@noisersup noisersup self-assigned this Apr 13, 2023
@codecov
Copy link

codecov bot commented Apr 13, 2023

Codecov Report

Merging #2424 (e7044d6) into main (669baef) will increase coverage by 37.39%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main    #2424       +/-   ##
===========================================
+ Coverage   26.90%   64.29%   +37.39%     
===========================================
  Files         392      392               
  Lines       19234    19234               
===========================================
+ Hits         5175    12367     +7192     
+ Misses      13505     5966     -7539     
- Partials      554      901      +347     

see 125 files with indirect coverage changes

Flag Coverage Δ
integration 57.33% <ø> (+52.60%) ⬆️
mongodb 4.73% <ø> (ø)
pg 57.26% <ø> (?)
unit 25.53% <ø> (-0.03%) ⬇️

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

@noisersup noisersup marked this pull request as ready for review April 13, 2023 14:02
@noisersup noisersup requested a review from AlekSi as a code owner April 13, 2023 14:02
@noisersup noisersup enabled auto-merge (squash) April 13, 2023 14:02
@noisersup noisersup requested review from a team, w84thesun, rumyantseva and b1ron April 13, 2023 14:03
@AlekSi AlekSi added this to the v1.1.0 milestone Apr 13, 2023
@AlekSi AlekSi requested review from a team and chilagrow April 13, 2023 16:19
Copy link
Member

@rumyantseva rumyantseva left a comment

Choose a reason for hiding this comment

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

Cool!

@noisersup noisersup merged commit 5adf466 into FerretDB:main Apr 13, 2023
21 of 22 checks passed
@noisersup noisersup deleted the compress-ci-2086 branch April 13, 2023 16:31
fenogentov pushed a commit to fenogentov/FerretDB that referenced this pull request Apr 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code/chore Code maintenance improvements
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Compress CI artifacts
3 participants