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-35526] Update to DependencyDeclarer #10

Merged
merged 2 commits into from
Jun 16, 2016

Conversation

fbelzunc
Copy link
Contributor

Basically, I believe this stacktrace is saying that when the maven-plugin is doing getParent().getPublishers() in Runtime is getting ClassCastException because there is something on the list that is not a Publisher. I believe the problem is in the conditional-buildstep-plugin which is doing a cast to DependecyDeclarer - which is deprecated - and DependencyDeclarer needs to be used instead.

udson.util.HudsonFailedToLoad: org.jvnet.hudson.reactor.ReactorException: java.lang.ClassCastException: org.jenkinsci.plugins.conditionalbuildstep.singlestep.SingleConditionalBuilder cannot be cast to hudson.tasks.Publisher 
at hudson.WebAppMain$3.run(WebAppMain.java:237) 
Caused by: org.jvnet.hudson.reactor.ReactorException: java.lang.ClassCastException: org.jenkinsci.plugins.conditionalbuildstep.singlestep.SingleConditionalBuilder cannot be cast to hudson.tasks.Publisher 
at org.jvnet.hudson.reactor.Reactor.execute(Reactor.java:269) 
at jenkins.InitReactorRunner.run(InitReactorRunner.java:44) 
at jenkins.model.Jenkins.executeReactor(Jenkins.java:935) 
at jenkins.model.Jenkins.<init>(Jenkins.java:816) 
at hudson.model.Hudson.<init>(Hudson.java:83) 
at hudson.model.Hudson.<init>(Hudson.java:79) 
at hudson.WebAppMain$3.run(WebAppMain.java:225) 
Caused by: java.lang.ClassCastException: org.jenkinsci.plugins.conditionalbuildstep.singlestep.SingleConditionalBuilder cannot be cast to hudson.tasks.Publisher 
at hudson.maven.MavenModule.getDependencyRelevancy(MavenModule.java:675) 
at hudson.maven.MavenModule.chooseMoreRelevantModule(MavenModule.java:632) 
at hudson.maven.MavenModule.buildDependencyGraph(MavenModule.java:515) 
at hudson.model.DependencyGraph.build(DependencyGraph.java:95) 
at jenkins.model.Jenkins.rebuildDependencyGraph(Jenkins.java:3816) 
at jenkins.model.Jenkins$20.run(Jenkins.java:2742) 
at org.jvnet.hudson.reactor.TaskGraphBuilder$TaskImpl.run(TaskGraphBuilder.java:169) 
at org.jvnet.hudson.reactor.Reactor.runTask(Reactor.java:282) 
at jenkins.model.Jenkins$8.runTask(Jenkins.java:924) 
at org.jvnet.hudson.reactor.Reactor$2.run(Reactor.java:210) 
at org.jvnet.hudson.reactor.Reactor$Node.run(Reactor.java:117) 
at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source) 
at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source) 
at java.lang.Thread.run(Unknown Source)

@reviewbybees

@oleg-nenashev
Copy link
Member

SingleConditionalBuilder Still implements the DependecyDeclarer

@@ -62,7 +63,7 @@
* @author Anthony Robinson
* @author Dominik Bartholdi (imod)
*/
public class SingleConditionalBuilder extends Builder implements DependecyDeclarer {
public class SingleConditionalBuilder extends Builder implements DependencyDeclarer {
Copy link
Member

Choose a reason for hiding this comment

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

I'll let @imod to decide if he cares about such compatibility breakage

@oleg-nenashev
Copy link
Member

🐝 from me

@imod
Copy link
Member

imod commented Jun 10, 2016

👍 thats fine with me

@fbelzunc
Copy link
Contributor Author

@imod Hello, would you mind to merge this?

@ghost
Copy link

ghost commented Jun 13, 2016

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

@imod imod merged commit cb00336 into jenkinsci:master Jun 16, 2016
@oleg-nenashev
Copy link
Member

@fbelzunc Makes sense to create follow-up bugs to other impacted plugins. https://github.com/search?q=org%3Ajenkinsci+DependecyDeclarer&type=Code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants