Add a repair migration to fix legacy Calendar events without "utcInstanceEpoch"
ClosedPublic
Actions

Authored by epriestley on Apr 10 2017, 11:37 AM.

Details

Reviewers

chad

Maniphest Tasks

T12488: Old Calendar events didn't get a utcInstanceEpoch populated, causing fatals on export

Commits

rPd4d46501ed1b: Add a repair migration to fix legacy Calendar events without "utcInstanceEpoch"

Summary

Fixes T12488. Some events appear to have survived earlier migrations without getting completely fixed. Fix them.

Test Plan

Ran migration locally with bin/storage upgrade (but: I could not reproduce this problem locally).
Ran migration in production and saw ICS import stop fataling.

Diff Detail

Repository

rP Phabricator

Branch

cal1

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 16357
Build 21752: Run Core Tests
Build 21751: arc lint + arc unit

Event Timeline

epriestley created this revision.Apr 10 2017, 11:37 AM

Harbormaster completed remote builds in B16357: Diff 42428.Apr 10 2017, 11:37 AM

Harbormaster completed remote builds in B16357: Diff 42428.

chad accepted this revision.Apr 10 2017, 3:41 PM

This revision is now accepted and ready to land.Apr 10 2017, 3:41 PM

Closed by commit rPd4d46501ed1b: Add a repair migration to fix legacy Calendar events without "utcInstanceEpoch" (authored by epriestley, committed by epriestley). · Explain WhyApr 10 2017, 3:47 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

sql/

autopatches/

20170410.calendar.01.repair.php

42 lines

Diff 42428

View Options

resources/sql/autopatches/20170410.calendar.01.repair.php

Add a repair migration to fix legacy Calendar events without "utcInstanceEpoch"ClosedPublicActions