Age | Commit message (Collapse) | Author |
|
* Deprecate Mean and MeanData
* Suppress deprecation warnings.
* Suppress warning InconsistentOverloads
* CopyAnnotations for AutoValue Mean and MeanData
|
|
This change allows the Checker Framework to check the classes annotated with
@AutoValue, even though their generated subclasses are skipped. This commit
also fixes some previously suppressed warnings in the ViewData class.
|
|
* Auto detect GCE and GKE Stackdriver MonitoredResources.
* Use MetadataConfig to set project id for default resources.
|
|
marccarre/issues/1066-more-robust-jaeger-integration-test
More robust integration test for Jaeger exporter.
|
|
|
|
The default instructions should rely on default settings. There's a section further below that provides instructions on how to configure auth + project.
|
|
unexpected connection reset socket exceptions.
Jaeger seems to accept connections even though it is not yet ready to handle HTTP requests.
Without this change, the integration test fails with:
```
io.opencensus.exporter.trace.jaeger.JaegerExporterHandlerIntegrationTest > exportToJaeger FAILED
java.net.SocketException: Connection reset
at java.net.SocketInputStream.read(SocketInputStream.java:197)
at java.net.SocketInputStream.read(SocketInputStream.java:122)
at java.io.BufferedInputStream.fill(BufferedInputStream.java:235)
at java.io.BufferedInputStream.read1(BufferedInputStream.java:275)
at java.io.BufferedInputStream.read(BufferedInputStream.java:334)
at sun.net.www.http.HttpClient.parseHTTPHeader(HttpClient.java:721)
at sun.net.www.http.HttpClient.parseHTTP(HttpClient.java:664)
at sun.net.www.http.HttpClient.parseHTTP(HttpClient.java:692)
at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1389)
at java.net.HttpURLConnection.getResponseCode(HttpURLConnection.java:468)
at com.google.api.client.http.javanet.NetHttpResponse.<init>(NetHttpResponse.java:37)
at com.google.api.client.http.javanet.NetHttpRequest.execute(NetHttpRequest.java:105)
at com.google.api.client.http.HttpRequest.execute(HttpRequest.java:981)
at io.opencensus.exporter.trace.jaeger.JaegerExporterHandlerIntegrationTest.waitForJaegerToStart(JaegerExporterHandlerIntegrationTest.java:239)
at io.opencensus.exporter.trace.jaeger.JaegerExporterHandlerIntegrationTest.exportToJaeger(JaegerExporterHandlerIntegrationTest.java:82)
```
With this change, it logs the above for information/debugging, but
carries on, and eventually, suceeds.
|
|
unexpected EOF socket exceptions.
Jaeger seems to accept connections even though it is not yet ready to handle HTTP requests.
Without this change, the integration test fails with:
```
java.net.SocketException: Unexpected end of file from server
at sun.net.www.http.HttpClient.parseHTTPHeader(HttpClient.java:851)
at sun.net.www.http.HttpClient.parseHTTP(HttpClient.java:678)
at sun.net.www.http.HttpClient.parseHTTPHeader(HttpClient.java:848)
at sun.net.www.http.HttpClient.parseHTTP(HttpClient.java:678)
at sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1587)
at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1492)
at java.net.HttpURLConnection.getResponseCode(HttpURLConnection.java:480)
at com.google.api.client.http.javanet.NetHttpResponse.<init>(NetHttpResponse.java:37)
at com.google.api.client.http.javanet.NetHttpRequest.execute(NetHttpRequest.java:105)
at com.google.api.client.http.HttpRequest.execute(HttpRequest.java:981)
at io.opencensus.exporter.trace.jaeger.JaegerExporterHandlerIntegrationTest.waitForJaegerToStart(JaegerExporterHandlerIntegrationTest.java:239)
at io.opencensus.exporter.trace.jaeger.JaegerExporterHandlerIntegrationTest.exportToJaeger(JaegerExporterHandlerIntegrationTest.java:82)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
at org.junit.internal.runners.statements.FailOnTimeout$CallableStatement.call(FailOnTimeout.java:298)
at org.junit.internal.runners.statements.FailOnTimeout$CallableStatement.call(FailOnTimeout.java:292)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at java.lang.Thread.run(Thread.java:748)
```
With this change, it logs the above for information/debugging, but
carries on, and eventually, suceeds.
|
|
This should make the integration test less sensitive to time, and fix timing errors like cases where CI might be overwhelmed, and Jaeger not ready yet, but we don't wait long enough.
|
|
|
|
This should make what we are working with more explicit.
|
|
Enable FindBugs in tests.
|
|
* Set unit '1' when the aggregation type is Count.
* Add a test for MetricDescriptor creation with aggregation Count.
|
|
This would have caught the unused variable that was fixed by #1082.
Changes in this commit:
- Enable FindBugs in tests in build.gradle.
- Fix some existing minor FindBugs issues in tests.
- Suppress some warnings in all test classes that are less important in test
code, such as requiring nested classes to be static whenever possible.
- Enable nullness-related FindBugs warnings in tests, since we don't run the
Checker Framework on test code.
|
|
Locking the version makes the build more stable for now.
Various to this alternatives are offered at: https://github.com/census-instrumentation/opencensus-java/issues/1066#issuecomment-375554352
|
|
(#1083)
Running `docker -v`, as done before this change, succeeds even when the Docker daemon is down, which means integration tests are attempted even though we cannot start Jaeger's container.
Running `docker version`, as done after this change, checks both the version of the CLI and the daemon, which fails when the daemon is down, and is what we really want here.
|
|
Though this executes another process, this is hopefully more reliable and faster than scanning the `PATH` for
a `docker` executable, and therefore fixes AppVeyor's CI builds.
|
|
If the trace/span ID is `0L` (resp. `1L`), then Jaeger simply returns `"0"` (resp. `"1"`) via its REST API.
Looking at `TraceId#generateRandomId`/`TraceId#INVALID` and `SpanId#generateRandomId`/`SpanId#INVALID`, it seems `0L` is a forbidden value which is actively filtered out, but anything else has the potential to happen.
Therefore, regexes for trace IDs and span IDs should be `{1,32}` and `{1,16}` respectively.
|
|
|
|
* Add git-ignores.
* Add traces exporter to Jaeger (+tests).
* Add string tag with events' description to Jaeger exporter (+tests).
* Add integration test for Jaeger exporter (manually run).
|
|
|
|
* Deprecate Window and WindowType in Stats APIs
* Window and WindowData will continue to be NonNull
* Suppress warnings for impl, exporter and zpages.
* NoopStats should continue to use old method.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
* Implement Prometheus Stats exporter
* Add PromethuesStatsExporter README to releasing update list
* Made some special strings as constants.
* Simplify registration of Prometheus collector
* Add javadoc for data model conversions. Add @since.
|
|
|
|
(closes #949) (#990)
|
|
|
|
|
|
|
|
|
|
|
|
(except examples). (#960)
|
|
|
|
|
|
readmes and build files. (fixes #941) (#946)
|
|
* Rename trace exporters [Logging|Stackdriver|Zipkin]Exporter to [Logging|Stackdriver|Zipkin]TraceExporter.
* Add old [Logging|Stackdriver|Zipkin]Exporter back and deprecate them to maintain compatibility.
* Implement old trace exporters using renamed ones.
|
|
|
|
implementation. (#933)
|
|
|