improved test coverage for migration code.  still ugly though.
remove extra param from DeleteShard()
storage/replica: move to buffered response chan to remove the need for background cleanup to avoid leaks
storage/replica: todo++