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

Do not save config.xml of Maven modules

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      Seems that JobConfigHistory.isSaveable assumes any AbstractProject should be saved. This makes sense for a MavenModuleSet but rarely makes sense for a MavenModule. I think you should only save a TopLevelItem (at least by default); otherwise you wind up saving tons of config records for things which users rarely if ever save changes to.

      BTW showing config changes to a Maven module does not even work, though they are recorded:

      java.lang.IllegalArgumentException: A job with this name could not be found: test:b
      	at hudson.plugins.jobConfigHistory.JobConfigHistoryBaseAction.getConfigXml(JobConfigHistoryBaseAction.java:163)
      	at hudson.plugins.jobConfigHistory.JobConfigHistoryBaseAction.getDiffFile(JobConfigHistoryBaseAction.java:300)
      	at hudson.plugins.jobConfigHistory.JobConfigHistoryBaseAction.getDiffLines(JobConfigHistoryBaseAction.java:316)
      

      Seems like you fail to use getItemByFullName correctly.

        Attachments

          Activity

          Hide
          mfriedenhagen Mirko Friedenhagen added a comment -

          Kathi

          Show
          mfriedenhagen Mirko Friedenhagen added a comment - Kathi
          Hide
          kstutz Kathi Stutz added a comment - - edited

          Version 2.4 (Apr 25 2013) of the plugin adds a configuration option that lets you choose whether you want to save the configs of MavenModules or not. (Saving is enabled by default.)
          ShowDiffs works for the single modules now as well.

          Show
          kstutz Kathi Stutz added a comment - - edited Version 2.4 (Apr 25 2013) of the plugin adds a configuration option that lets you choose whether you want to save the configs of MavenModules or not. (Saving is enabled by default.) ShowDiffs works for the single modules now as well.

            People

            Assignee:
            kstutz Kathi Stutz
            Reporter:
            jglick Jesse Glick
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: