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

Make the promoted-builds-plugin DSL extension for job-dsl-plugin extensible: Automatically Generated DSL

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      The current DSL extension is static and needs to be modified/improved whenever a new feature is added to promoted-builds-plugin.
      By the extensible nature of this plugin (it provides the PromotionCondition ExtensionPoint for third parties to implement), this makes the DSL extension requirement to be dynamic depending on available plugins ; this doesn't scale well with the current static DSL extension.
      job-dsl-plugin 1.46 introduced the Automatically Generated DSL feature to address this issue on job-dsl-plugin itself by the use of javaposse.jobdsl.dsl.ExtensibleContext / javaposse.jobdsl.dsl.ContextType at the DSL level (model introspection done by helpers such as javaposse.jobdsl.plugin.structs.DescribableContext).

      The promoted-builds-plugin DSL extension should make use of those DSL authoring tool to automatically discover available PromotionCondition.

        Attachments

          Issue Links

            Activity

            fchuong Frédéric Chuong created issue -
            fchuong Frédéric Chuong made changes -
            Field Original Value New Value
            Link This issue is related to JENKINS-34872 [ JENKINS-34872 ]
            fchuong Frédéric Chuong made changes -
            Link This issue is related to JENKINS-34982 [ JENKINS-34982 ]
            fchuong Frédéric Chuong made changes -
            Component/s job-dsl-plugin [ 16720 ]
            fchuong Frédéric Chuong made changes -
            Assignee Daniel Spilker [ daspilker ] Oleg Nenashev [ oleg_nenashev ]
            jbochenski Jakub Bochenski made changes -
            Link This issue relates to JENKINS-40022 [ JENKINS-40022 ]
            Hide
            warandpeace Andrew Ferguson added a comment -

            I'd really love to see this; The inability to use publishers in promoted plugin DSL is causing headaches left and right for my team :/

            Show
            warandpeace Andrew Ferguson added a comment - I'd really love to see this; The inability to use publishers in promoted plugin DSL is causing headaches left and right for my team :/
            oleg_nenashev Oleg Nenashev made changes -
            Epic Link JENKINS-44837 [ 182897 ]
            Hide
            oleg_nenashev Oleg Nenashev added a comment -

            No plans to work on it in the middle-term, feel free to take the issue. IMHO right now it is blocked by  JENKINS-34982, as well as other JobDSL support stories.

            Show
            oleg_nenashev Oleg Nenashev added a comment - No plans to work on it in the middle-term, feel free to take the issue. IMHO right now it is blocked by  JENKINS-34982 , as well as other JobDSL support stories.
            oleg_nenashev Oleg Nenashev made changes -
            Assignee Oleg Nenashev [ oleg_nenashev ]
            Hide
            oleg_nenashev Oleg Nenashev added a comment -

            There was an attempt to land it in https://github.com/jenkinsci/promoted-builds-plugin/pull/96 , but it has not been finished. Anyone interested in JobDSL integration, you are welcome to adopt and recover this pull request or to suggest your own implementation.

            Show
            oleg_nenashev Oleg Nenashev added a comment - There was an attempt to land it in  https://github.com/jenkinsci/promoted-builds-plugin/pull/96  , but it has not been finished. Anyone interested in JobDSL integration, you are welcome to adopt and recover this pull request or to suggest your own implementation.
            oleg_nenashev Oleg Nenashev made changes -
            Labels job-dsl-plugin promoted-builds job-dsl-plugin promoted-builds stalled-pr

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              fchuong Frédéric Chuong
              Votes:
              8 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated: