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

V0.9.7 cherrypicks #908

Merged
merged 10 commits into from
Oct 4, 2018
Merged

Conversation

skriss
Copy link
Contributor

@skriss skriss commented Oct 4, 2018

replaces #906

timoreimann and others added 10 commits October 4, 2018 12:14
This is to better reflect the intent of the user when node ports are
specified explicitly (as opposed to being assigned by Kubernetes). The
`last-applied-configuration` annotation added by `kubectl apply` is one
such indicator we are now leveraging.

We still default to omitting the node ports when the annotation is
missing.

Signed-off-by: Timo Reimann <[email protected]>
Signed-off-by: Wayne Witzel III <[email protected]>
@skriss skriss added this to the v0.9.7 milestone Oct 4, 2018
@skriss skriss requested a review from wwitzel3 October 4, 2018 18:30
@wwitzel3 wwitzel3 merged commit caa962f into vmware-tanzu:release-0.9 Oct 4, 2018
@skriss skriss deleted the v0.9.7-cherrypicks branch October 5, 2018 13:22
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.

6 participants