Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-32785

Deprecate listview -> columns -> lastBuildNode() to generate a new config

    XMLWordPrintable

Details

    Description

      listView -> columns -> lastBuildNode() warns about build-node-column-plugin which is deprecated.

      From a discussion on the mailing list: https://groups.google.com/d/topic/job-dsl-plugin/xYnd1p2a7VI/discussion

      I wrote:
      I am using listView -> columns -> lastBuildNode() and receive the following warning when I run my example:

      Warning: (script, line 65) version 0.1 or later of plugin 'build-node-column' needs to be installed

      This plugin however is deprecated and on GitHub they state that the extra-columns-plugin should be used instead.

      I would welcome if the Job DSL plugin would not require deprecated libraries but am unsure how to contribute.

      Can the code be updated to "require" the build-node-column plugin instead? If so, how to find out what version is the minimum?

      Daniel answered:

      [...] The problem is that we use the @RequiresPlugin annotation in the DSL to specify which plugin provides the config and that annotation can not express "plugin A or plugin B". And to keep compatibility we can't just change the existing method to generate the new config only. The best option would be to deprecate the existing lastBuildNode method and to add a new lastBuildNode2 (or whatever) method which will then generate the new config. Please file a JIRA issue [...]

      Attachments

        Issue Links

          Activity

            bherb Benjamin Herbert created issue -
            daspilker Daniel Spilker made changes -
            Field Original Value New Value
            Issue Type Bug [ 1 ] Improvement [ 4 ]
            daspilker Daniel Spilker made changes -
            Link This issue is duplicated by JENKINS-33674 [ JENKINS-33674 ]
            daspilker Daniel Spilker made changes -
            Resolution Fixed [ 1 ]
            Status Open [ 1 ] Resolved [ 5 ]
            daspilker Daniel Spilker made changes -
            Status Resolved [ 5 ] Closed [ 6 ]
            rtyler R. Tyler Croy made changes -
            Workflow JNJira [ 168450 ] JNJira + In-Review [ 209674 ]

            People

              daspilker Daniel Spilker
              bherb Benjamin Herbert
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: