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

logstorage: Fix issue with gzip #571

Merged
merged 1 commit into from
Dec 15, 2023
Merged

Conversation

minminlittleshrimp
Copy link
Collaborator

@minminlittleshrimp minminlittleshrimp commented Nov 19, 2023

Current implementation facing issue generating gzip dlt log, hence fix is applied:

  • Fsync for gzip
  • Adapt option parsing of gzip with others
  • Align gzip option status (on,off,default,unknown)
  • Make gzip a dlt proprietary

Current implementation facing issue generating gzip dlt log,
hence fix is applied:
+ Fsync for gzip
+ Adapt option parsing of gzip with others
+ Align gzip otpion status (on,off,default,unknown)
+ Make gzip a dlt proprietary

Signed-off-by: LUU QUANG MINH <Minh.LuuQuang@vn.bosch.com>
@minminlittleshrimp
Copy link
Collaborator Author

Hello @michael-methner @lvklevankhanh
Fix is tested and passed CI/CD, please kindly review.
Thank you

Copy link
Collaborator

@michael-methner michael-methner left a comment

Choose a reason for hiding this comment

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

Looks good to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Not offline logstoreage to new file in continuous power on
2 participants