-
Notifications
You must be signed in to change notification settings - Fork 38.6k
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
Upgrade examples/guestbook to v1 #9455
Upgrade examples/guestbook to v1 #9455
Conversation
EXPERIMENTAL JENKINS PR BUILDER: e2e build succeeded. |
@caesarxuchao I will take a look tomorrow. |
I will update guest-go after this one has been reviewed |
5db6f2b
to
39382b2
Compare
EXPERIMENTAL JENKINS PR BUILDER: e2e build succeeded. |
|
||
Thus, once created, the service proxy on each minion is configured to set up a proxy on the specified port (in this case port 6379). | ||
|
||
*targetPort* will default to *port* if it is omitted in the configuration. For simplicity's sake, we omit it in the following configurations. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You say "following configurations" here, but you omit it in the previous one as well. Maybe just don't say "following."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nevermind, I misread.
Upgrade examples/guestbook to v1
GCE e2e build/test passed for commit 39382b2. |
Update the example to v1;
Removed some default fields from the configuration files;
Update the description about load balancers and public IPs. (Reviewers: please take special care of this part as it's a quite big change)
ref #9286
@krousey @nikhiljindal @bgrant0607