Uploaded image for project: 'Infrastructure'
  1. Infrastructure
  2. INFRA-1666

UC excerpt field wrong for incrementalified plugins

    Details

    • Similar Issues:
    • Epic Link:

      Description

      Karl Shultz noticed an anomaly in the Plugin Manager. Looking into https://updates.jenkins-ci.org/update-center.actual.json I found that most Pipeline plugins specified excerpt="", despite having an index.jelly, because they have no <description> in the POM; but the recently released incrementalified workflow-api has

      "excerpt": "The Jenkins Plugins Parent POM Project"
      

      This is because the workflow-api.hpi!/META-INF/maven/org.jenkins-ci.plugins.workflow/workflow-api/pom.xml specified

      <description>The Jenkins Plugins Parent POM Project</description>
      

      due to it using flatten-maven-plugin. Now backend-update-center2 reads pom.xml#/project/description and sets that to the excerpt, which is wrong because it ignores the parent POM, though we did not notice that before; and because lots of plugins just omit a POM description in favor of the index file. At runtime, index.jelly is preferred by PluginManager/installed.jelly, but PluginManager/table.jelly used by the Updates tab only checks the excerpt.

        Attachments

          Activity

          jglick Jesse Glick created issue -
          jglick Jesse Glick made changes -
          Field Original Value New Value
          Epic Link JENKINS-50686 [ 189770 ]
          jglick Jesse Glick made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          jglick Jesse Glick made changes -
          Remote Link This issue links to "PR 204 (Web Link)" [ 20845 ]
          jglick Jesse Glick made changes -
          Status In Progress [ 3 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          jglick Jesse Glick made changes -
          Remote Link This issue links to "archetypes PR 38 (Web Link)" [ 21117 ]
          Hide
          jglick Jesse Glick added a comment -

          Was reverted, need to address the reported problem and reapply.

          Show
          jglick Jesse Glick added a comment - Was reverted, need to address the reported problem and reapply.
          jglick Jesse Glick made changes -
          Resolution Fixed [ 1 ]
          Status Resolved [ 5 ] Reopened [ 4 ]
          jglick Jesse Glick made changes -
          Status Reopened [ 4 ] In Progress [ 3 ]
          jglick Jesse Glick made changes -
          Remote Link This issue links to "PR 217 (Web Link)" [ 21209 ]
          jglick Jesse Glick made changes -
          Remote Link This issue links to "jenkins.io PR 1665 (Web Link)" [ 21214 ]
          Hide
          jglick Jesse Glick added a comment -

          Resurrected version merged into other changes and expected to land this week.

          Show
          jglick Jesse Glick added a comment - Resurrected version merged into other changes and expected to land this week.
          Hide
          danielbeck Daniel Beck added a comment -

          Fix merged.

          Show
          danielbeck Daniel Beck added a comment - Fix merged.
          danielbeck Daniel Beck made changes -
          Status In Progress [ 3 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]

            People

            • Assignee:
              jglick Jesse Glick
              Reporter:
              jglick Jesse Glick
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: