-
Notifications
You must be signed in to change notification settings - Fork 40k
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
update meteor example according to config best practices. #11399
Conversation
GCE e2e build/test passed for commit 3cc0f95e251a481bd4955ce8e5d97767e9422785. |
@@ -14,7 +14,6 @@ | |||
"selector": { | |||
"name": "meteor" | |||
}, | |||
"sessionAffinity": "ClientIP", | |||
"type": "LoadBalancer" |
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.
why do you remove the loadbalancer?
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.
accident, I thought it was a default for a second. I have readded it.
LGTM. Thanks! |
GCE e2e build/test passed for commit 392d4e0708f0a4a35dbfe910415cb59ff219cc0c. |
f6b855b
to
f981857
Compare
LGTM, thanks! |
GCE e2e build/test failed for commit f6b855b231918518e67e214b5fa1445a5e7652cd. |
#11460 will fix the shippable failure here, but it's not merged yet |
GCE e2e build/test passed for commit f981857bf3efccbb7404b5796ebdc440c505e322. |
GCE e2e build/test passed for commit 1133b62. |
update meteor example according to config best practices.
related to #11361