The movePrimary command does a check that the host name of the shard exists in the config.shards collection. But we got the host name by looking up the shard identifier, which means the shard must exist. This check is left over from when movePrimary took the target shard by hostname instead of shard name. Now it is redundant of the other checks in place for shard existence.