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

journald: wait for journal files fsync()ed on Synchronize() varlink method #32617

Merged
merged 2 commits into from
May 2, 2024

Conversation

yuwata
Copy link
Member

@yuwata yuwata commented May 1, 2024

Closes #32539.

yuwata added 2 commits May 2, 2024 04:15
…ethod

Otherwise, we may not be able to read newer journal entries even after
'journalctl --sync' is called.

Closing systemd#32539.
Copy link

github-actions bot commented May 1, 2024

Important

An -rc1 tag has been created and a release is being prepared, so please note that PRs introducing new features and APIs will be held back until the new version has been released.

@DaanDeMeyer DaanDeMeyer added good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed and removed please-review PR is ready for (re-)review by a maintainer labels May 1, 2024
@DaanDeMeyer
Copy link
Contributor

Thank you!

@YHNdnzj YHNdnzj merged commit 89ad0a7 into systemd:main May 2, 2024
44 of 49 checks passed
@github-actions github-actions bot removed the good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed label May 2, 2024
@yuwata yuwata deleted the journal-sync branch May 2, 2024 05:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

journalctl --sync should guarantee all currently in flight logs are written to the journal
3 participants