opentelemetry-plugin icon indicating copy to clipboard operation
opentelemetry-plugin copied to clipboard

Stop masking Guava classes (req Jenkins 2.321)

Open basil opened this issue 4 years ago • 3 comments

Bumps the core baseline to 2.321 (2021-11-16), which has updated version of Guava and Guice, which removes the need for <maskClasses>com.google.common.</maskClasses>. This simplifies this plugin's build and deployment. Note that 2.321 is a weekly (not LTS) release, so you may want to wait to merge this until the Guava update is available in an LTS release.

basil avatar Nov 26 '21 22:11 basil

Thanks @basil , it will be a great simplification.

We just bumped our Jenkins Core requirement to v2.277.1. We look at the breakdown by Jenkins Core version of the adoption of the Jenkins Otel Plugin to understand when we can bump. We will see when we can bump and simplify the Guava setup. See https://stats.jenkins.io/pluginversions/opentelemetry.html 2021/11/29 image

cyrille-leclerc avatar Nov 29 '21 10:11 cyrille-leclerc

Refreshed for 2.332.1 LTS

basil avatar Mar 10 '22 01:03 basil

Thanks @basil to follow up on this. As 85% of the installations that can upgrade (ie >= v2.277) are older than v2.321, we are going to wait to bump our requirement.

image

cyrille-leclerc avatar Mar 10 '22 08:03 cyrille-leclerc

Shipped in https://github.com/jenkinsci/opentelemetry-plugin/commit/721c762db006e301280cbc53b4fa0beed78c79ef

cyrille-leclerc avatar Oct 14 '22 21:10 cyrille-leclerc