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

[PM-18504] Enable SQL transaction log backup #5433

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Mart124
Copy link
Contributor

@Mart124 Mart124 commented Feb 21, 2025

Hello,

๐ŸŽŸ๏ธ Tracking

bwdata/mssql/data/vault_log.ldf, which is the SQL transaction log file, grows indefinitely.

๐Ÿ“” Objective

This PR then adds the transaction log backup to the full backup, so that it is properly rotated, preventing it to grow.
It then fixes #648, with a better / safer approach than #3140.

๐Ÿฆฎ User guidelines

If one wants to shrink the transaction log file, which may have grown unexpectedly (because not backed-up until now), the following SQL command will be needed (after 2 backups at least so that the end of the transaction log gets unused) :
use vault;
go
DBCC SHRINKFILE (vault_log);
go
It has not been added as a command to the backup script, as it is not considered as a regular maintenance one, but added a a guideline.

Thank you very much ๐Ÿ‘

@bitwarden-bot
Copy link

Thank you for your contribution! We've added this to our internal Community PR board for review.
ID: PM-18504
Link: https://bitwarden.atlassian.net/browse/PM-18504

Details on our contribution process can be found here: https://contributing.bitwarden.com/contributing/pull-requests/community-pr-process.

@bitwarden-bot bitwarden-bot changed the title Enable SQL transaction log backup [PM-18504] Enable SQL transaction log backup Feb 21, 2025
So that it does not grow indefinitely
@Mart124
Copy link
Contributor Author

Mart124 commented Feb 21, 2025

Hello @kspearrin,
As you reviewed previous PR on these file (a few years back now !), I would be glad if you could look at this one :)
Thank you very much ๐Ÿ‘

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

Successfully merging this pull request may close these issues.

Cleaning or Rotating vault_log.ldf
2 participants