pipeline-model-definition-plugin icon indicating copy to clipboard operation
pipeline-model-definition-plugin copied to clipboard

[JENKINS-26097] Add auto-completion and validation of label expressions.

Open Zastai opened this issue 5 years ago • 6 comments
trafficstars

  • JENKINS issue(s):
  • Description:
    • Add auto-completion and validation of label expressions.
  • Documentation changes:
    • Only affects declarative generator; probably no doc needed

Zastai avatar Jun 12 '20 21:06 Zastai

Given I only touched UX parts (validation and auto-completion), I don't see how they trigger these test failures; I guess they must be compabilitiy issues with the more recent Jenkins main.

Zastai avatar Jun 13 '20 12:06 Zastai

Probably this:

  20.515 [id=7429]	WARNING	h.i.i.InstallUncaughtExceptionHandler#handleException: Caught unhandled exception with ID 242d9287-d857-4257-ab7b-66246f9b0344
org.apache.commons.jelly.JellyTagException: jar:file:/home/jenkins/workspace/e-model-definition-plugin_PR-394%40tmp/m2repo/org/jenkins-ci/main/jenkins-core/2.241-rc30038.0428cd872667/jenkins-core-2.241-rc30038.0428cd872667.jar!/hudson/model/Run/console.jelly:66:27: <j:whitespace> com/trilead/ssh2/crypto/Base64

Likely a plugin needs to be updated to one that has a direct dependency on trilead-api

timja avatar Jun 13 '20 21:06 timja

I’ll try to dig into it tomorrow.

Zastai avatar Jun 13 '20 22:06 Zastai

Interestingly, but annoyingly, I can't reproduce these myself when running the tests in an IDE. But I'll push a commit that adds a test dependency on trilead-ssh2 in the hopes that it resolves the test errors during the CI build.

Zastai avatar Jun 14 '20 22:06 Zastai

It’s likely the ssh-credentials version that needs bumping.

But if you introduce the plugin bom https://github.com/jenkinsci/bom it should fix these issues

timja avatar Jun 15 '20 05:06 timja

Ah I assumed the BOM would be pulled in by the plugin parent. Will look into it.

Zastai avatar Jun 15 '20 10:06 Zastai

Given that the plugin is now on a much higher Jenkins version, this should hopefully work fine now.

Zastai avatar Jan 19 '24 23:01 Zastai