Add zipkin tracing support for Knative tests about test-infra HOT 10 CLOSED

/assign @dushyanthsc

from test-infra.

@adrcunha: GitHub didn't allow me to assign the following users: dushyanthsc.

Note that only knative members and repo collaborators can be assigned.
For more information please see the contributor guide

from test-infra.

/assign @dushyanthsc

from test-infra.

Commit adds Zipkin tracing support to test infrastructure. However, fundamental requirement of this feature is the availability of Monitoring components installed on the KNative cluster. Currently, monitoring is not enabled by default for e2e tests due to this which is blocked on Issue: #647

Till #647 is addressed, developers can enable Zipkin tracing for clusters setup with Monitoring using following instructions:

1. Setup KNative cluster with Logging and Monitoring enabled.
2. In the test-setup call Zipkin initializer.
3. As recommended for e2e tests, use SpoofingClient to communicate with your KNative cluster.
4. For the HTTP request that you would like to inspect Zipkin trace, call LogZipkinTrace()
5. In the test-cleanup call Zipkin cleanup

As an example a sample commit enabling zipkin tracing is available in this repository.

from test-infra.

@dushyanthsc, can this issue be closed now?

from test-infra.

After talking to Adriano, we have decided to enable atleast Zipkin for E2E tests. I will work on this to enable Zipkin tracing. ETA at the moment is 02/22/2019

from test-infra.

@dushyanthsc isn't this one done already?

from test-infra.

I think this was picked up by @yt3liu and was completed.

from test-infra.

The previous change only enabled zipkin for performance test. Related issue: #539

That's done in this PR knative/serving#3594. Similar change can be done for all tests to enable zipkin.

from test-infra.

Thanks, closing.

from test-infra.