Skip to content

AOF Persistence Updates#266

Open
UdeshyaDhungana wants to merge 3 commits intomainfrom
aof-persistence-updates
Open

AOF Persistence Updates#266
UdeshyaDhungana wants to merge 3 commits intomainfrom
aof-persistence-updates

Conversation

@UdeshyaDhungana
Copy link
Copy Markdown
Contributor

@UdeshyaDhungana UdeshyaDhungana commented Apr 7, 2026

Note

Low Risk
Low risk because changes are confined to the tester harness and fixtures, with only minor process lifecycle guarding in RedisExecutable.Kill().

Overview
AOF tests now validate appendfsync. Default AOF config assertions include CONFIG GET appendfsync (expects everysec), and flag-based tests pass --appendfsync and verify it via CONFIG GET.

Filesystem expectations around AOF directories are tightened. Adds DirDoesNotExistAssertion and expands the AOF directory creation test to also assert the default appendonlydir is not created when appendonly is off, including updated fixtures.

Improves executable lifecycle handling in tests. RedisExecutable now tracks isRunning to make Kill() idempotent, enabling mid-test terminate/restart patterns without spurious kill attempts.

Reviewed by Cursor Bugbot for commit 22b2a52. Bugbot is set up for automated code reviews on this repo. Configure here.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant