Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add gson-api plugin to bom #2723

Merged
merged 1 commit into from
Dec 6, 2023

Conversation

jonesbusy
Copy link
Contributor

Not sure about the order. Should we fist replace dependency by the api plugin or first include it to bom ?

Testing done

PLUGINS=gson-api bash local-test.sh 

Submitter checklist

Copy link
Member

@timja timja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

either works

@timja timja added the enhancement New feature or request label Dec 6, 2023
@jonesbusy jonesbusy marked this pull request as ready for review December 6, 2023 14:02
@jonesbusy jonesbusy requested a review from a team as a code owner December 6, 2023 14:02
@MarkEWaite
Copy link
Contributor

MarkEWaite commented Dec 6, 2023

Thanks for the pull request @jonesbusy . I'm borrowing it to run a larger test as part of the Artifactory brownout . I've added the weekly-test label so that it will run a wider range of tests on the pull request while we're in the brownout period. Brownout will end in about 45 minutes

Copy link
Contributor

@MarkEWaite MarkEWaite left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks very much. Passed the larger test

@MarkEWaite MarkEWaite merged commit 9d9b8ad into jenkinsci:master Dec 6, 2023
165 checks passed
@jonesbusy jonesbusy deleted the feature/gson-api-plugin branch December 6, 2023 16:32
@basil
Copy link
Member

basil commented Dec 7, 2023

@jonesbusy Very nice! Thanks a lot for this contribution. If you're interested, it would be nice to get rid of GSON from these plugins:

./git-client/WEB-INF/lib/gson-2.10.1.jar
./google-compute-engine/WEB-INF/lib/gson-2.9.0.jar
./google-kubernetes-engine/WEB-INF/lib/gson-2.10.1.jar
./google-oauth-plugin/WEB-INF/lib/gson-2.9.1.jar
./google-storage-plugin/WEB-INF/lib/gson-2.10.1.jar
./trilead-api/WEB-INF/lib/gson-2.8.9.jar

If you aren't interested, then that is quite alright and I will file some Jira tickets against those individual plugins so that we can track the long-term effort to consolidate this library.

Thanks again.

@basil
Copy link
Member

basil commented Dec 7, 2023

Unrelated to this PR, but another top offender is Joda Time, which has embedded itself in many places:

aws-java-sdk-minimal/WEB-INF/lib/joda-time-2.8.1.jar
dockerhub-notification/WEB-INF/lib/joda-time-2.10.14.jar
google-metadata-plugin/WEB-INF/lib/joda-time-2.12.5.jar
google-oauth-plugin/WEB-INF/lib/joda-time-2.12.5.jar
google-storage-plugin/WEB-INF/lib/joda-time-2.12.5.jar
handy-uri-templates-2-api/WEB-INF/lib/joda-time-2.10.2.jar
pipeline-model-api/WEB-INF/lib/joda-time-2.10.5.jar
saml/WEB-INF/lib/joda-time-2.11.0.jar

org.json:json would be nice to turn into a library, especially because old versions have a weird license:

configuration-as-code/WEB-INF/lib/json-20231013.jar
jackson2-api/WEB-INF/lib/json-20231013.jar

The pair of json-path/json-smart also crops up now and again:

google-kubernetes-engine/WEB-INF/lib/json-path-2.8.0.jar
google-kubernetes-engine/WEB-INF/lib/json-smart-2.4.10.jar
token-macro/WEB-INF/lib/json-path-2.8.0.jar
token-macro/WEB-INF/lib/json-smart-2.4.10.jar

@jonesbusy
Copy link
Contributor Author

@basil Thanks, yea I will do the replacement step by step starting with existing API plugins in the bom. I started with the 'git-client' for now

I can take the point of org.json:json and joda-time it's ok for me.

@basil
Copy link
Member

basil commented Dec 11, 2023

Very nice!

@basil
Copy link
Member

basil commented Jan 4, 2024

I have filed the following tickets to track the updating of analysis-model-api and warnings-ng:

@basil
Copy link
Member

basil commented Mar 13, 2024

I have filed the following tickets to track the updating of google-compute-engine, google-oauth, and google-storage:

I have filed the following tickets to track the updating of google-metadata, google-oauth, and google-storage:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request weekly-test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants