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

JENKINS-54439 Jcasc compatibility #97

Merged
merged 37 commits into from Jun 28, 2019

Conversation

timja
Copy link
Member

@timja timja commented May 27, 2019

Migrates from a "Plugin" to a Descriptor
Removes custom databinding in configure

Updates jelly to use data binding
Update parent pom
Remove findbugs config, it's now in parent pom, (using spotbugs instead)
Upgrades checkstyle to allow using lamdas

@timja
Copy link
Member Author

timja commented May 27, 2019

@rsandell mind taking a look please?

@luispollo
Copy link

@timja Thanks for contributing this! Been looking forward to this feature. @rsandell Any chance you'd get this merged any time soon?

@timja
Copy link
Member Author

timja commented Jun 20, 2019

Weirdly changing from a PluginImpl to a GlobalConfiguration / using the initialiser annotations seems to have some impact on classloading. Currently this branch re-introduces the deserialisation issue (albeit with a different error).

Deep in the jackson code:

getTypeFactory().findClass(subClass)

throws:

java.lang.ClassNotFoundException: com.sonyericsson.jenkins.plugins.bfa.model.indication.BuildLogIndication

On master classloader is:
AntClassLoader$ClassicPluginStrategy

AntClassLoader[/Users/timja/projects/jenkinsci/build-failure-analyzer-plugin/src/main/resources:/Users/timja/projects/jenkinsci/build-failure-analyzer-plugin/target/classes:/Users/timja/.m2/repository/com/fasterxml/jackson/core/jackson-annotations/2.9.3/jackson-annotations-2.9.3.jar:/Users/timja/.m2/repository/com/fasterxml/jackson/core/jackson-databind/2.9.7/jackson-databind-2.9.7.jar:/Users/timja/.m2/repository/com/google/code/findbugs/jsr305/1.3.9/jsr305-1.3.9.jar:/Users/timja/.m2/repository/com/google/guava/guava/11.0.1/guava-11.0.1.jar:/Users/timja/.m2/repository/commons-io/commons-io/2.6/commons-io-2.6.jar:/Users/timja/.m2/repository/commons-net/commons-net/3.6/commons-net-3.6.jar:/Users/timja/.m2/repository/de/undercouch/bson4jackson/2.9.2/bson4jackson-2.9.2.jar:/Users/timja/.m2/repository/javax/persistence/persistence-api/1.0.2/persistence-api-1.0.2.jar:/Users/timja/.m2/repository/org/apache/commons/commons-lang3/3.7/commons-lang3-3.7.jar:/Users/timja/.m2/repository/org/mongodb/bson/3.8.0/bson-3.8.0.jar:/Users/timja/.m2/repository/org/mongodb/mongo-java-driver/3.8.2/mongo-java-driver-3.8.2.jar:/Users/timja/.m2/repository/org/mongodb/mongodb-driver-core/3.8.0/mongodb-driver-core-3.8.0.jar:/Users/timja/.m2/repository/org/mongodb/mongodb-driver/3.8.0/mongodb-driver-3.8.0.jar:/Users/timja/.m2/repository/org/mongojack/mongojack/2.9.4/mongojack-2.9.4.jar]

on this PR:
WebAppClassLoader

WebAppClassLoader=Jenkins v2.60.3@1ad4da42

@timja
Copy link
Member Author

timja commented Jun 20, 2019

Looks like a classloader needs passing, https://jenkins.io/doc/developer/plugin-development/dependencies-and-class-loading/

@timja
Copy link
Member Author

timja commented Jun 21, 2019

All review comments should be fixed / answered, could you please re-review, thanks.

@rsandell rsandell merged commit 41f6f5e into jenkinsci:master Jun 28, 2019
@rsandell
Copy link
Member

I'll release all this as a beta version because I'm going on vacation today and will be gone a month. If people are happy then I can release it as a full version. But I don't think I would dare to release it as a full version now when I will not be available to review and release any bugfixes that might arise.

@timja timja deleted the feature/jcasc-compatibility branch June 28, 2019 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants