-
Notifications
You must be signed in to change notification settings - Fork 49
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Docker commands mentioning "affinity" should co-mention "locality" #96
Comments
@heyawhite There are also a number of hits in placement.md:
In Docker-land (as brian mentioned on the originating ticket) it is called "affinity". I think it would be helpful/good to add something like |
I've got some changes in a PR ready to go, per your review: #97 If there's a better way to go about this process, let me know. |
@trentm poking again for a review! Thanks. |
I have submitted a change review on Gerrit: https://cr.joyent.us/#/c/1047/ |
This issue originated in the documentation repo: https://github.com/joyent/documentation/issues/902
"We refer to locality hints in our doc base for the ability to specify near/far on provision (to allow users to hint where they want containers to go).
However, on the docker side we call this "affinity", which means when someone searches the doc base for "locality" they don't find this.
We should note this, so at the very least this page comes up when they search."
As far as I can tell, an update need to happen in these two places:
https://github.com/joyent/sdc-docker/blob/master/docs/api/commands/create.md
https://github.com/joyent/sdc-docker/blob/master/docs/api/commands/run.md
The text was updated successfully, but these errors were encountered: