It forgets successes here, and forget failures when loading the admin page to run the upgrades. Unless I'm missing something, won't this cause a completely successful migration to report that every user needs to be migrated? We might need to add a config flag to mark this migration as complete to avoid that.