[JIRA] (JENKINS-58362) CommandLauncher2Test.requireApproval failure due to two copies of SystemCommandLanguage

0 views
Skip to first unread message

jglick@cloudbees.com (JIRA)

unread,
Jul 5, 2019, 10:22:01 AM7/5/19
to jenkinsc...@googlegroups.com
Jesse Glick updated Bug JENKINS-58362
 
Jenkins / Bug JENKINS-58362
CommandLauncher2Test.requireApproval failure due to two copies of SystemCommandLanguage
Change By: Jesse Glick
Status: In Progress Review
Add Comment Add Comment
 
This message was sent by Atlassian Jira (v7.11.2#711002-sha1:fdc329d)

jglick@cloudbees.com (JIRA)

unread,
Jul 5, 2019, 10:22:01 AM7/5/19
to jenkinsc...@googlegroups.com
Jesse Glick started work on Bug JENKINS-58362
 
Change By: Jesse Glick
Status: Open In Progress

jglick@cloudbees.com (JIRA)

unread,
Jul 5, 2019, 10:22:02 AM7/5/19
to jenkinsc...@googlegroups.com
Jesse Glick created an issue
Issue Type: Bug Bug
Assignee: Jesse Glick
Components: command-launcher-plugin, core, jenkins-test-harness
Created: 2019-07-05 14:21
Labels: plugin-compat-tester regression lts-candidate
Priority: Minor Minor
Reporter: Jesse Glick

JENKINS-57528 included a bump to the detached script-security which should have been paired with a bump to the detached command-launcher. To reproduce, run

mvn -f command-launcher-plugin test -Djenkins.version=2.176.1 -Dtest=CommandLauncher2Test\#requireApproval

which passes; now bump its parent to 3.47 and include

-Djenkins.version=2.176.2-rc28314.ce9ac38e8574 -Pconsume-incrementals

which pulls in this backport by Oliver Gondža and you will get a failure:

java.lang.AssertionError: expected:<org.jenkinsci.plugins.scriptsecurity.scripts.languages.SystemCommandLanguage@...> but was:<org.jenkinsci.plugins.command_launcher.SystemCommandLanguage@...>
	at org.junit.Assert.fail(Assert.java:88)
	at org.junit.Assert.failNotEquals(Assert.java:834)
	at org.junit.Assert.assertEquals(Assert.java:118)
	at org.junit.Assert.assertEquals(Assert.java:144)
	at hudson.slaves.CommandLauncher2Test$1.evaluate(CommandLauncher2Test.java:119)

jglick@cloudbees.com (JIRA)

unread,
Jul 5, 2019, 10:24:01 AM7/5/19
to jenkinsc...@googlegroups.com
Jesse Glick commented on Bug JENKINS-58362
 
Re: CommandLauncher2Test.requireApproval failure due to two copies of SystemCommandLanguage

While there are three associated pull requests, only jenkins #4099 is important to backport to the stable-2.176 branch.

jglick@cloudbees.com (JIRA)

unread,
Jul 5, 2019, 10:25:02 AM7/5/19
to jenkinsc...@googlegroups.com

jglick@cloudbees.com (JIRA)

unread,
Jul 11, 2019, 10:52:01 AM7/11/19
to jenkinsc...@googlegroups.com
Change By: Jesse Glick
Status: In Review Fixed but Unreleased
Resolution: Fixed
Released As: 2.185

o.v.nenashev@gmail.com (JIRA)

unread,
Jul 15, 2019, 6:29:02 AM7/15/19
to jenkinsc...@googlegroups.com

adrien.lecharpentier@gmail.com (JIRA)

unread,
Aug 8, 2019, 4:22:02 AM8/8/19
to jenkinsc...@googlegroups.com

ogondza@gmail.com (JIRA)

unread,
Aug 8, 2019, 6:26:02 AM8/8/19
to jenkinsc...@googlegroups.com

Adrien Lecharpentier, 2.176.3 is already complete and is currently being tested without this fix in it. So it will have to wait for the next line.

dbeck@cloudbees.com (JIRA)

unread,
Sep 5, 2019, 5:54:18 AM9/5/19
to jenkinsc...@googlegroups.com
Daniel Beck updated an issue
 
Change By: Daniel Beck
Labels: lts-candidate plugin-compat-tester regression
Reply all
Reply to author
Forward
0 new messages