Page MenuHomePhabricator

Correct some issues around IMPORTED_PERMANENT in RefEngine
ClosedPublic

Authored by epriestley on Jan 23 2021, 12:38 AM.
Tags
None
Referenced Files
F15477122: D21518.id51212.diff
Mon, Apr 7, 12:40 PM
F15455014: D21518.id51212.diff
Sat, Mar 29, 9:52 PM
F15448931: D21518.id51218.diff
Fri, Mar 28, 7:43 AM
F15445737: D21518.diff
Thu, Mar 27, 3:44 PM
F15440398: D21518.id.diff
Wed, Mar 26, 12:37 PM
F15436295: D21518.diff
Tue, Mar 25, 1:13 PM
F15434274: D21518.id.diff
Tue, Mar 25, 2:50 AM
Unknown Object (File)
Feb 21 2025, 4:03 PM
Subscribers
None

Details

Summary

Ref T13591. Fixes a few issues with the recent updates here discovered in more thorough testing.

Test Plan
  • Stopped the daemons.
  • Created a new copy of Phabricator in Diffusion.
  • Pulled it with bin/repository pull ....
    • Got 17,278 commits on disk with git log --all --format=%H.
  • Set permanent refs to "master".
  • Discovered it with bin/repository discover ....
    • This took 31.5s and inserted 17,278 tasks.
    • Verified that all tasks have priority 4,000 (PRIORITY_IMPORT).
    • Observed that 16,799 commits have IMPORTED_PERMANENT and 479 commits do not.
      • This matches git log master --format=%H exactly.
  • Ran bin/repository refs .... Expected no changes and saw no changes.
  • Ran bin/worker execute --active for a minute or two. It processed all the impermanent changes first (since bin/worker is LIFO and these are supposed to process last).
    • Ran bin/repository refs. Expected no changes and saw no changes.
    • Marked all refs as permanent.
    • Starting state: 16,009 message tasks, all at priority 4000.
    • Ran bin/repository refs, expecting 479 new tasks at priority 4000.
    • Saw count rise to 16,488 as expected.
    • Saw all the new tasks have priority 4000 and all commits now have the IMPORTED_PERMANENT flag.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable