[PATCH v2] t5520: don't expire reflogs where it matters

From: Thomas Bachem <mail@thomasbachem.com>

"git merge" saves any uncommitted changes with "git stash" before it tries a merge strategy. When the strategy does not handle the merge, it restores them with "git stash apply --index". If some of the changes are staged, that runs "git reset", which writes an entry to the reflog of HEAD. The autostash tests in this script run eight such merges.

An upcoming change makes "git stash apply --index" merge the index in-core, so it no longer runs "git reset" and those entries go away. Another makes the default "merge" backend of "git rebase" run auto maintenance when it finishes. Together, they change when auto maintenance expires all reflogs, which it does once a hundred entries in the reflog of HEAD are due to expire.

With both, the expiry comes at the end of the "git pull --rebase" in the "--rebase with rebased upstream" test. The "git pull --rebase -f" in the next test looks for the fork point in the reflog of refs/remotes/me/copy, but as the test suite dates every reflog entry to 2005, the expiry has emptied that reflog. Pull then finds no fork point, so the rebase also replays copy-orig, the commit "copy" was rewound from, and it conflicts.

Disable reflog expiration in this script, as ea7d894f44 (t34xx: don't expire reflogs where it matters, 2026-02-24) did for the rebase tests, so that the test no longer depends on where the expiry falls.

Reported-by: Junio C Hamano <gitster@pobox.com>
Helped-by: D. Ben Knoble <ben.knoble@gmail.com>
Helped-by: Phillip Wood <phillip.wood@dunelm.org.uk>
Assisted-by: Claude Fable 5.1
Signed-off-by: Thomas Bachem <mail@thomasbachem.com>
---
    t5520: don't expire reflogs where it matters
    
    The t5520 failure Junio saw in 'seen' with Ben Knoble's stash series,
    bisected by Ben to tb/rerere-lock-grace and taken apart in the thread:
    https://lore.kernel.org/git/a59c4225-f093-4001-b77a-2083dfecce6e@gmail.com/
    
    Changes since v1: only the commit message, rewritten along the points
    Phillip raised on Ben's copy of this patch:
    https://lore.kernel.org/git/3547f4aa-649a-4f46-868c-0e50dfa69466@gmail.com/

Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-2243%2Fthomasbachem%2Ft5520-reflog-expire-v2
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-2243/thomasbachem/t5520-reflog-expire-v2
Pull-Request: https://github.com/gitgitgadget/git/pull/2243

Range-diff vs v1:

 1:  1d8d6ed7f1 ! 1:  6699f3782e t5520: don't expire reflogs where it matters
     @@ Metadata
       ## Commit message ##
          t5520: don't expire reflogs where it matters
      
     -    The "--rebase -f with rebased upstream" test computes its fork point
     -    from the reflog of refs/remotes/me/copy, and the entry it needs is
     -    the one that the fetch of the test before it wrote. Like every reflog
     -    entry the suite writes after test_tick, it is dated 2005, so the
     -    first "git reflog expire --all" after that fetch removes it. Pull
     -    then finds no fork point and rebases onto the merge head with the
     -    merge head as the upstream, and the rewound commits come back as a
     -    conflict.
     +    "git merge" saves any uncommitted changes with "git stash" before it
     +    tries a merge strategy. When the strategy does not handle the merge,
     +    it restores them with "git stash apply --index". If some of the
     +    changes are staged, that runs "git reset", which writes an entry to
     +    the reflog of HEAD. The autostash tests in this script run eight such
     +    merges.
      
     -    Since 452b12c2e0 (builtin/maintenance: use "geometric" strategy by
     -    default, 2026-02-24) auto maintenance runs that expiry once the reflog
     -    of HEAD holds a hundred entries it would remove, the default of
     -    maintenance.reflog-expire.auto. Which run crosses the threshold
     -    depends on the entries and maintenance runs before it, so the script
     -    passed by chance: a stash topic that no longer runs "git reset" from
     -    "stash apply --index" and a rebase topic that runs auto maintenance
     -    at the end of "git rebase" together move the expiry between the two
     -    tests.
     +    An upcoming change makes "git stash apply --index" merge the index
     +    in-core, so it no longer runs "git reset" and those entries go away.
     +    Another makes the default "merge" backend of "git rebase" run auto
     +    maintenance when it finishes. Together, they change when auto
     +    maintenance expires all reflogs, which it does once a hundred entries
     +    in the reflog of HEAD are due to expire.
      
     -    Pin the expiry as ea7d894f44 (t34xx: don't expire reflogs where it
     -    matters, 2026-02-24) did for the rebase tests. That covers a "git gc"
     -    as well, which expires reflogs on its own, where turning off the auto
     -    trigger of the reflog-expire task alone would not.
     +    With both, the expiry comes at the end of the "git pull --rebase" in
     +    the "--rebase with rebased upstream" test. The "git pull --rebase -f"
     +    in the next test looks for the fork point in the reflog of
     +    refs/remotes/me/copy, but as the test suite dates every reflog entry
     +    to 2005, the expiry has emptied that reflog. Pull then finds no fork
     +    point, so the rebase also replays copy-orig, the commit "copy" was
     +    rewound from, and it conflicts.
     +
     +    Disable reflog expiration in this script, as ea7d894f44 (t34xx: don't
     +    expire reflogs where it matters, 2026-02-24) did for the rebase tests,
     +    so that the test no longer depends on where the expiry falls.
      
          Reported-by: Junio C Hamano <gitster@pobox.com>
          Helped-by: D. Ben Knoble <ben.knoble@gmail.com>

 t/t5520-pull.sh | 6 ++++++
 1 file changed, 6 insertions(+)