Update docs about new Services work #8655
Conversation
docs/services.md
Outdated
There was a problem hiding this comment.
That hyphen was technically correct, but whatever.
English is right-associative by default, so without the hyphen it's (virtual-IP) (based bridge).
|
I still see a lot of references to "portal IP". |
docs/services.md
Outdated
|
regarding portal: that hasn't been changed yet. I'll followup Justin's LB changes to see if we can do that reasonably and if so re-update the docs. @justinsb for a read-over to see if I missed anything. |
|
new push is up |
docs/services.md
Outdated
There was a problem hiding this comment.
Quotation problem here (missing closing-double quote)
|
new push with nits fixed |
There was a problem hiding this comment.
Do we have to explain the contents of the ingress field?
There was a problem hiding this comment.
I mentioned it, I don't feel like we need to go into too much depth here right now.
|
New push is up with nits addressed |
|
LGTM outside of existing comments |
|
Do you want this or #8740 to be merged first? |
|
This first On Tue, May 26, 2015 at 1:25 PM, Brian Grant [email protected]
|
|
LGTM |
|
Pushed with a few more "virtual" references removed. |
Update docs about new Services work
@justinsb @vishh @bgrant0607 @jbeda
This should not be committed until Justin's load-balancer rework is in.