-
Notifications
You must be signed in to change notification settings - Fork 136
Conversation
That's an exciting error on OSX :) |
I am going to comment the Python example for now and link to an issue on |
I also created the travis load balancer, so I'm going to close and reopen to make sure Travis is passing. |
41638f7
to
aaa3276
Compare
Ok, this should be RFAL. Tests are now green :) |
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.
Small nits otherwise LGTM.
examples/hellohttp/e2e-test.sh
Outdated
for i in $RANDOM $RANDOM; do | ||
edit "$i" | ||
update | ||
# Don't let K8s slowness cause flakes. |
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.
Nit: k8s
func main() { | ||
http.HandleFunc("/", HelloWorld) | ||
http.ListenAndServe(port, nil) | ||
} |
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.
Nit: add newline at the end of file
@dlorenc If you had anything to add here I'd be happy to follow up, but otherwise I'm going to lighten your load :D |
All the same languages, except C++
Fixes: #29