Skip to content

Conversation

@jglick
Copy link
Member

@jglick jglick commented Sep 28, 2021

Fixes #657. Going to see what, if anything, jenkinsci/plugin-compat-tester#276 was actually needed for. Also wondering if jenkinsci/plugin-compat-tester#299 or other recent changes will break things.

@jglick jglick added the chore Reduces future maintenance label Sep 28, 2021
@jglick
Copy link
Member Author

jglick commented Sep 28, 2021

Something weird in https://ci.jenkins.io/job/Tools/job/bom/job/PR-659/1/pipeline-console/?selected-node=263 and something else in jquery3-api; I suspect there is some sort of regression affecting plugins with no src/test/java/ (just InjectedTest).

https://github.com/jenkinsci/bom/pull/659/checks?check_run_id=3733673007 seems to reproduce the need for jenkinsci/plugin-compat-tester#276:

java.lang.NoSuchMethodError: io.jenkins.plugins.casc.ConfigurationAsCode.toYaml(Lio/jenkins/plugins/casc/model/CNode;)Lio/jenkins/plugins/casc/snakeyaml/nodes/Node;
	at io.jenkins.plugins.casc.misc.Util.toYamlString(Util.java:152)
	at hudson.plugins.view.dashboard.ConfigurationAsCodeBackCompatTest.testOldImagePortletUrl(ConfigurationAsCodeBackCompatTest.java:63)

@jetersen
Copy link
Member

if dashboard is the only plugin with this issue why not just fix that plugin instead of creating an hack?

@jglick
Copy link
Member Author

jglick commented Sep 28, 2021

TBD if only dashboard is affected. If the problem is just an old JCasC dep, then fine. But AFAICT it is using something newish.

@jglick
Copy link
Member Author

jglick commented Sep 28, 2021

Hmm, failFast not working any more? (In this case, good.)

@jglick
Copy link
Member Author

jglick commented Sep 28, 2021

850f995 might have broken failFast, TBD.

@jglick
Copy link
Member Author

jglick commented Sep 28, 2021

Well, maybe it does still work, just more slowly.

@timja
Copy link
Member

timja commented Sep 28, 2021

if dashboard is the only plugin with this issue why not just fix that plugin instead of creating an hack?

from what i remember it's not a hack as such, the configuration as code plugin and test harness are not correctly update by PCT, only the plugin is updated and not the test harness.

It creates a nightmare when they aren't updated together.

not such an issue now while there's not many changes but still has been a huge pain before

@jglick
Copy link
Member Author

jglick commented Sep 28, 2021

Perplexed by why https://ci.jenkins.io/job/Tools/job/bom/job/PR-659/2/testReport/hudson.plugins.view.dashboard/ConfigurationAsCodeTest/ would be passing against the latest line yet failing against the oldest line. The Jenkins version should not make any difference, and we are using the same configuration-as-code version in all lines. (And same for snakeyaml-api.)

@jglick
Copy link
Member Author

jglick commented Sep 28, 2021

There is also a PCT regression affecting workflow-scm-step (log):

 [ERROR]     'dependencies.dependency.version' for org.jenkins-ci.plugins.workflow:workflow-step-api:jar is missing. @ line 66, column 17

@jglick
Copy link
Member Author

jglick commented Sep 28, 2021

In sum, there are at least three PCT bugs visible in this project:

  • configuration-as-code test harness issue, as already suspected
  • library wrapper plugins broken
  • something involving workflow-scm-step

@imonteroperez
Copy link
Contributor

In sum, there are at least three PCT bugs visible in this project:

  • configuration-as-code test harness issue, as already suspected
  • library wrapper plugins broken
  • something involving workflow-scm-step

Regarding workflow-scm-step I can confirm it was failing with the same issue before jenkinsci/plugin-compat-tester#299 gets merged

prep.sh Outdated

# TODO find a way to encode this in some POM so that it can be managed by Dependabot
version=0.5.1-rc1066.9cfd30140d30 # TODO https://github.com/jenkinsci/plugin-compat-tester/pull/276
version=0.5.1-rc1106.47640a4e521e
Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Member Author

@jglick jglick Nov 8, 2021

Choose a reason for hiding this comment

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

We still need jenkinsci/plugin-compat-tester#276, or something like it.

Co-authored-by: Ildefonso Montero <[email protected]>
@jglick
Copy link
Member Author

jglick commented Nov 23, 2021

jenkinsci/plugin-compat-tester#320 ought to speed things up a bit.

@jglick jglick mentioned this pull request Apr 21, 2022
@jglick
Copy link
Member Author

jglick commented Jul 28, 2022

#1336

@jglick jglick closed this Jul 28, 2022
@jglick jglick deleted the latest-pct branch July 28, 2022 02:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Reduces future maintenance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use newer PCT

5 participants