Update doc about an error of CLI command

[zepouet]

Just correct a minor error about CLI.

Thanks

[yosifkit]

This needs to go in content.md; README.md is generated.

[tianon]

cc @jsternberg

[jsternberg]

The commands that are already there work for me. It also appears like using --net=container:influxdb and using localhost also works. I intentionally didn't use --link because the documentation indicated that --link was an old feature and wasn't the preferred method of doing things.

Unless there's a strong reason to use the old method rather than the new one and unless I see that the current documentation results in an error, I'm not sure why this change should be used instead.

[tianon]

It's worth noting that --link, while discouraged (given that it doesn't work cross-host) is not officially deprecated yet.

I personally like --link over --net container:xxx because it shows a more general pattern that's easier to adjust for other needs (such as creating a second interactive container in an existing cross-host docker network which is on a separate physical host from the initial container, thus making --net container:xxx impossible to use), and makes it clear that "localhost" connections aren't treated as special in some way (like MySQL can be configured to do, for example).

[jsternberg]

Ok. Whatever you guys think would be the clearer documentation on the matter. The reason why I used --net=container:xxx was because I thought that was the preferred method and I also show examples of setting up a network later on. Adding an example of using --link sounds fine to me in that case.

[tianon]

Thanks @jsternberg (and @zepouet)!