Commit Graph

8515 Commits

Author SHA1 Message Date
Peter Findeisen 598ee82659
6929 - fixing unit for runtime.jvm.gc.count (#6930) 2022-10-20 17:43:01 -07:00
Trask Stalnaker 034cac3f76
Fix comment (#6926)
First I tried to update to `20`, but looks like there are no ea releases
available, I think I "mispoke" in that comment, so fixed it.
2022-10-20 14:01:00 -07:00
Trask Stalnaker 222cbb9c30
Add api diffs for 1.19.0 (#6921)
Following post-release instructions at
https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/RELEASING.md#update-release-versions-in-documentations

Will automate later once I understand this better.
2022-10-20 13:07:57 +03:00
Trask Stalnaker c18bea3b79
Revert "Temporary fix for testLatestDeps (#6904)" (#6922)
This reverts commit ceb6975d17.

Looks like this got sorted.
2022-10-20 13:06:58 +03:00
Trask Stalnaker 10ea7b0d9b
Merge 1.19.1 change log back to main (#6924) 2022-10-20 13:06:27 +03:00
Trask Stalnaker bac8b0c8b0
Don't assign PRs to component owners (#6918)
It will still assign @theletterf as a reviewer
2022-10-19 13:42:22 -07:00
Trask Stalnaker f488d94403
Capture net.host.name for netty (#6892)
This may be a regression in 1.19.0 because you can no longer reconstruct
the original url for netty server spans (previously `http.host` was
captured which could be used).
2022-10-19 09:21:50 -07:00
Trask Stalnaker 7ec441bb3a
Update releasing doc (#6911)
Something I learned today...
2022-10-19 13:56:35 +03:00
Etienne Dysli Metref 6fb1f00241
Refactor `io.opentelemetry.instrumentation.resources.ContainerResource` to avoid using null (#6889)
While I was looking at issues
open-telemetry/opentelemetry-java-instrumentation#6694 and
open-telemetry/opentelemetry-java#2337, I saw that the code in
`io.opentelemetry.instrumentation.resources.ContainerResource` used
`null` several times as return value which isn't safe. Nowadays,
`Optional` is better suited to signal the absence of a result, so I
refactored `ContainerResource` to use `Optional`s instead of null.

On the way, I also refactored this class's unit tests into parameterised
tests to reduce test code duplication. These improvements should help
implementing a solution to
open-telemetry/opentelemetry-java-instrumentation#6694.

Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
2022-10-19 03:00:43 +00:00
Trask Stalnaker 465d4d3f9b
Fix patch release process (#6893)
Same as
https://github.com/open-telemetry/opentelemetry-java-contrib/pull/538
2022-10-18 12:02:02 -07:00
Trask Stalnaker ceb6975d17
Temporary fix for testLatestDeps (#6904)
Resolves #6903
2022-10-18 16:28:30 +02:00
dependabot[bot] 29ef29626e
Bump mockito-inline from 4.8.0 to 4.8.1 in /dependencyManagement (#6897) 2022-10-18 02:00:13 +00:00
Trask Stalnaker 895880b121
Fix akka testLatestDeps (#6900)
Resolves #6901
2022-10-17 17:45:05 -07:00
dependabot[bot] cafab8e657
Bump groovyVersion from 4.0.5 to 4.0.6 in /dependencyManagement (#6896) 2022-10-17 23:44:00 +00:00
dependabot[bot] bf56414bc6
Bump mockito-junit-jupiter from 4.8.0 to 4.8.1 in /dependencyManagement (#6898) 2022-10-17 23:13:47 +00:00
dependabot[bot] c211e2faaf
Bump com.gradle.common-custom-user-data-gradle-plugin from 1.8.1 to 1.8.2 (#6895) 2022-10-17 23:07:31 +00:00
Mateusz Rzeszutek b6ded1f9a0
End reactor-netty HTTP client span properly on `Mono#timeout()` (#6891)
Calling `Mono#timeout()` with a timeout value smaller than the HTTP
client timeout caused the on request/response end callbacks to be simply
discarded; and the HTTP span was never finished.
2022-10-17 13:29:02 -07:00
Trask Stalnaker b25283d2ca
Fix release contributors script (#6879)
Similar to
https://github.com/open-telemetry/opentelemetry-java-contrib/pull/530.

Also, I fixed the contributor list in the release notes which was
incorrect due to this bug (I believe only in this latest release due to
introduction of dependabot).
2022-10-14 08:54:45 -07:00
Trask Stalnaker 461420110a
Small release doc update (#6878) 2022-10-14 08:54:01 -07:00
Trask Stalnaker 3601a84b3d
Update repo settings doc (#6877)
Added new "Restrict pushes that create matching branches: UNCHECKED",
which I discovered is needed during contrib release.

Removed the old `v*` branch protections since we don't need to make any
more patch releases from those old branch names anymore.
2022-10-14 08:53:42 -07:00
Trask Stalnaker c1cdadab5d
Fix testLatestDeps (#6880) 2022-10-14 12:03:48 +03:00
dependabot[bot] 2c423b44b2
Bump jackson-bom from 2.13.4.20221012 to 2.13.4.20221013 in /dependencyManagement (#6873) 2022-10-13 21:02:19 -07:00
Trask Stalnaker 9621ea08cf
Update repo settings (#6862)
it seems that github recently tightened up the branch protection
restrictions (in a way that makes more sense than the prior behavior)
2022-10-13 20:07:36 -07:00
Trask Stalnaker 63302ccd3e
Split out license check as separate github action (#6863)
I've been having a bit of trouble with the license check in our distro
repo, and I think it's helpful for it to be a separate github action
(also for visibility).
2022-10-13 19:09:22 -07:00
Trask Stalnaker b1bc39db59
Fix website PR (#6869)
Need to manually run the action after merging.
2022-10-13 07:32:30 -07:00
Trask Stalnaker 7104290294
Fix markdown link check (#6868)
https://kotlinlang.org/docs/coroutines-overview.html appears to be
bot-aware, I can go to it in my browser, but can't curl it
2022-10-13 12:21:17 +03:00
dependabot[bot] 0a88ade490
Bump jackson-bom from 2.13.4 to 2.13.4.20221012 in /dependencyManagement (#6865) 2022-10-12 18:21:53 -07:00
dependabot[bot] a8b0c33fcb
Bump com.gradle.enterprise from 3.11.1 to 3.11.2 (#6864) 2022-10-12 18:19:44 -07:00
OpenTelemetry Bot 8b216a10ac
Copy change log updates from release/v1.19.x (#6867)
Copy log updates from `release/v1.19.x`.
2022-10-12 17:46:48 -07:00
OpenTelemetry Bot e30054eb32
Update version to 1.20.0-SNAPSHOT (#6861)
Update version to `1.20.0-SNAPSHOT`.
2022-10-12 22:01:43 +00:00
Trask Stalnaker c40d3235c0
Update change log for 1.19.0 release (#6859) 2022-10-12 13:11:09 -07:00
Lauri Tulmin 279ab34ba9
Fix regression in webflux http client concurrency test (#6858)
Resolves
https://github.com/open-telemetry/opentelemetry-java-instrumentation/issues/6857
Resolves
https://github.com/open-telemetry/opentelemetry-java-instrumentation/issues/6856
Resolves
https://github.com/open-telemetry/opentelemetry-java-instrumentation/issues/6854
Looks like a regression from
https://github.com/open-telemetry/opentelemetry-java-instrumentation/pull/6836
2022-10-12 09:21:49 -07:00
jack-berg aeac361816
Upgrade to otel java 1.19.0 (#6757)
Working PR to capture all the changes required to update to otel java
1.19.0. The new log API force allows
`:instrumentation-appender-api-internal` and
`:instrumentation-appender-sdk-internal`, but necessitates a decent
amount of refactoring as a result.

The PR points at the `1.19.0-SNAPSHOT`, which I'll update upon
publication.

Co-authored-by: Mateusz Rzeszutek <mrzeszutek@splunk.com>
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
Co-authored-by: Lauri Tulmin <ltulmin@splunk.com>
2022-10-12 09:19:37 -07:00
Trask Stalnaker c5cd8d67a3
Fix create website pull request (#6852) 2022-10-11 18:07:56 -07:00
dependabot[bot] d58965457c
Bump gradle-nullaway-plugin from 1.3.0 to 1.4.0 in /conventions (#6839) 2022-10-11 21:34:22 +00:00
dependabot[bot] 3806e4f295
Bump error_prone_annotations from 2.15.0 to 2.16 in /dependencyManagement (#6847) 2022-10-11 14:20:34 -07:00
dependabot[bot] 59940699c7
Bump error_prone_test_helpers from 2.15.0 to 2.16 in /dependencyManagement (#6840) 2022-10-11 20:56:08 +00:00
jack-berg fcd5876e1a
Add HTTP client and server tests (#6836)
Resolves #6835.
2022-10-11 20:12:48 +00:00
dependabot[bot] 56fa7eedee
Bump gradle-errorprone-plugin from 2.0.2 to 3.0.1 in /conventions (#6841) 2022-10-11 12:55:30 -07:00
dependabot[bot] 6c23d98387
Bump error_prone_core from 2.15.0 to 2.16 in /dependencyManagement (#6849) 2022-10-11 12:52:42 -07:00
Trask Stalnaker 45f95efc81
Update byte buddy from 1.12.17 to 1.12.18 (#6851)
I will rebase the dependabot PRs after this is merged, so that
dependabot won't mark these versions internally as ignored
2022-10-11 12:41:13 -07:00
jack-berg d5aadbab04
Split out :instrumentation:netty:netty-4.1 library (#6820)
Resolves #6734. Builds on #6811.
2022-10-10 16:01:14 -07:00
Mateusz Rzeszutek 77035fc88c
Extract `net.peer.{name,port}` on start for CLIENT spans (#6828)
The [HTTP
spec](https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/trace/semantic_conventions/http.md#http-client)
says these two attributes must be provided at span creation time - I
think it makes sense to extend it over to all `net`-related
instrumentations, cause these are supposed to be the logical peer
name/port, which are supposed to be known before the connection is
started/exchange is made.
2022-10-10 16:00:19 -07:00
Lauri Tulmin 418c414de9
Remove vaadin latest deps restriction (#6829)
Resolves
https://github.com/open-telemetry/opentelemetry-java-instrumentation/issues/6551
2022-10-07 12:14:06 -07:00
Mateusz Rzeszutek 709ab76dd2
Add @theletterf as docs codeowner (#6830)
cc @theletterf
2022-10-07 12:08:34 -07:00
Malte 7117f4226d
ISSUE-2674 Strip sensitive data from the url (#6417)
Fixes #2674 by replacing basic auth information as part of the URL with
`username:password`.

Co-authored-by: Malte <MALPI@users.noreply.github.com>
Co-authored-by: Mateusz Rzeszutek <mrzeszutek@splunk.com>
Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
2022-10-07 13:18:23 +02:00
Trask Stalnaker 5184a523b6
Add test for null httpcontext (#6823)
follow-up to #6792
2022-10-07 12:39:50 +03:00
Lauri Tulmin 5daf453e1d
Remove latest dep restriction from vertx-http-client-4.0 (#6827)
Should have been part of
https://github.com/open-telemetry/opentelemetry-java-instrumentation/pull/6809,
forgot to add this file.
2022-10-07 11:31:57 +02:00
dependabot[bot] d0782b54da
Bump auto-value-annotations from 1.9 to 1.10 in /dependencyManagement (#6822) 2022-10-06 23:34:40 +00:00
jason plumb d9b25d34df
Guard against null HttpContext (#6792)
Resolves #6787

Co-authored-by: Trask Stalnaker <trask.stalnaker@gmail.com>
2022-10-06 16:03:03 -07:00