Skip to content
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

Modify loadbalancer docs for external source ip preservation feature #1062

Merged
merged 1 commit into from
Aug 24, 2016

Conversation

girishkalele
Copy link

@girishkalele girishkalele commented Aug 19, 2016

WIP - here is the first crack at explaining the new annotation and feature gate.


This change is Reviewable

"type": "LoadBalancer"
}
}

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you should explain what this does to achieve the result, including only routing locally, the fact that nodeports are not handled yet, and the basic healthchecks stuff

@@ -37,15 +43,15 @@ Your configuration file might look like:
"type": "LoadBalancer"
}
}

```
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need a blank line after tick blocks or GH may render it badly

Also note this feature is not currently implemented for all cloudproviders/environments.

### Caveats and Limitations when preserving source IPs

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still want a note that this does not yet work on nodePorts, so "proxy style" balancers can not use this yet.

@thockin
Copy link
Member

thockin commented Aug 22, 2016

One nit then docs LGTM

@girishkalele
Copy link
Author

Fixed nit...marking LGTM.

@girishkalele
Copy link
Author

Rebased.

@girishkalele
Copy link
Author

Fixes #1058

@thockin
Copy link
Member

thockin commented Aug 24, 2016

needs a rebase

@girishkalele
Copy link
Author

I tried the "Update branch" button but it does a pull instead of a rebase.

Rebased manually instead.

@girishkalele girishkalele merged commit e802d51 into kubernetes:master Aug 24, 2016
@girishkalele
Copy link
Author

cc @kubernetes/docs

mikutas pushed a commit to mikutas/k8s-website that referenced this pull request Sep 22, 2022
Okabe-Junya pushed a commit to Okabe-Junya/website that referenced this pull request Dec 4, 2023
…s/rojopolis/spellcheck-github-actions-0.25.0

chore(deps): bump rojopolis/spellcheck-github-actions from 0.24.0 to 0.25.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants