Attempt to change http-echo image

[ashishb-solo]

Description

This is just a test to see if we can change the image used for our tests from kennship/http-echo to jmalloc/echo-server. See changelog for details.

CI passes successfully, so it doesn't seem like this change causes any harm. I was hoping that CI runs would be faster since this image is smaller than the previous one, but it does not appear to make any difference - compare to the times for CI to run on this pull request as an example and you can see that the times to run do not change much at all.

However, this change does still provide an improvement in that the new image supports HTTP/2 while the other previous image does not:

ashish@laptop5 ~/go/src/github.com/solo-io/gloo/test/kubernetes/e2e/features/tracing/testdata $ kubectl -n curl exec -it curl -- curl --http2 http-echo.http-echo.svc.cluster.local:3000
curl: (52) Empty reply from server
command terminated with exit code 52

ashish@laptop5 ~/go/src/github.com/solo-io/gloo/test/kubernetes/e2e/features/tracing/testdata $ kubectl -n curl exec -it curl -- curl http-echo.http-echo.svc.cluster.local:3000
{"path":"/","headers":{"host":"http-echo.http-echo.svc.cluster.local:3000","user-agent":"curl/7.83.1-DEV","accept":"*/*"},"method":"GET","body":{},"fresh":false,"hostname":"http-echo.http-echo.svc.cluster.local","ip":"::ffff:10.244.0.230","ips":[],"protocol":"http","query":{},"subdomains":["svc","http-echo","http-echo"],"xhr":false}

API changes

Code changes

CI changes

Change http-echo image to use jmalloc/echo-server instead

Docs changes

Context

Interesting decisions

Testing steps

Notes for reviewers

Checklist:

• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• I have added tests that prove my fix is effective or that my feature works

[nfuden]

Perhaps we should also look at which tests coul use docker.io/istio/tcp-echo-server at some point. That being said this seems like a nice incremental bump

[sam-heilbron]

I like the change! There are a number of other references to this image that we are removing still in the codebase. How should we handle removing those from the codebase?

In our docs we refer to hashicorp/http-echo, and in some of our code (

gloo/test/kube2e/util.go

Line 41 in c3337f7

httpEchoImage = "gcr.io/solo-test-236622/http-echo"

) we refer to some other images.

I'm good with this change as is, just wanted to raise the potential for additional improvements in this area