Page MenuHomePhabricator

Correct a bad Herald migration
ClosedPublic

Authored by epriestley on Sep 4 2015, 9:58 PM.
Tags
None
Referenced Files
F15510198: D14061.id33996.diff
Wed, Apr 16, 6:56 PM
F15510197: D14061.id.diff
Wed, Apr 16, 6:56 PM
F15510196: D14061.diff
Wed, Apr 16, 6:56 PM
F15461011: D14061.id.diff
Tue, Apr 1, 4:19 AM
F15399508: D14061.id33997.diff
Mar 17 2025, 4:56 AM
F15387655: D14061.id33996.diff
Mar 15 2025, 2:44 AM
F15334924: D14061.diff
Mar 8 2025, 1:13 PM
F15334311: D14061.id.diff
Mar 8 2025, 8:12 AM
Subscribers
None
Tokens
"Yellow Medal" token, awarded by meitros.

Details

Summary

One of the migrations in rPa335004a91 (20150730.herald.5.sql) incorrectly swapped "add" and "add blocking" Differential Herald rules.

Swap any rules last modified before this patch was applied back. This is the best we can do without possibly overwriting more recent, intentional data. I'll issue some guidance on this in the changelog.

Test Plan
  • Made a rule, ran patch, no change.
  • Changed rule modified time to a few months ago, ran patch, saw swap from non-blocking to blocking.

Diff Detail

Repository
rP Phabricator
Branch
heraldswap
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 7916
Build 8861: [Placeholder Plan] Wait for 30 Seconds
Build 8860: arc lint + arc unit