ShardRegistry::createConnection is used during addShard to create a detached Shard object corresponding to the servers we are in the process of trying to add. We use it only to get the RemoteCommandTargeter off it. We should instead just make a RemoteCommandTargeter directly, and no longer be in the business of having Shard objects for servers that aren't yet actually shards