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

Config files not being updated

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      Steps to reproduce:

      • Go to http://<jenkins_url>/configfiles/.
      • Click to "Add a new Config" and create a new one (for example a custom file with any content).
      • Once submitted, go and edit it (change the content and click to "Submit").
      • Open edit page again.

      Actual behaviour:

      • The configuration content is the old one, the update did not have any effect.

      Expected behaviour:

      • The configuration content is updated.

      This was not happening in config-file-provider 2.15.3

        Attachments

          Issue Links

            Activity

            Hide
            robinmoss Robin Moss added a comment -

            I've also noticed this with the same version of Jenkins and Config File Plugin..

            Even modifying (what i think is *) the stored config on the file-system doesn't seem to update the config in Jenkins (reloaded config from file)

            • $JENKINS_HOME/org.jenkinsci.plugins.configfiles.GlobalConfigFiles.xml
            Show
            robinmoss Robin Moss added a comment - I've also noticed this with the same version of Jenkins and Config File Plugin.. Even modifying (what i think is *) the stored config on the file-system doesn't seem to update the config in Jenkins (reloaded config from file) $JENKINS_HOME/org.jenkinsci.plugins.configfiles.GlobalConfigFiles.xml
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: imod
            Path:
            src/main/java/org/jenkinsci/plugins/configfiles/GlobalConfigFiles.java
            src/main/java/org/jenkinsci/plugins/configfiles/folder/FolderConfigFileProperty.java
            src/test/java/org/jenkinsci/plugins/configfiles/ConfigFileStoreTest.java
            http://jenkins-ci.org/commit/config-file-provider-plugin/18e445ca64b2391118c938093383b2c0afa20389
            Log:
            [FIXED JENKINS-41161] replace old config object

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: imod Path: src/main/java/org/jenkinsci/plugins/configfiles/GlobalConfigFiles.java src/main/java/org/jenkinsci/plugins/configfiles/folder/FolderConfigFileProperty.java src/test/java/org/jenkinsci/plugins/configfiles/ConfigFileStoreTest.java http://jenkins-ci.org/commit/config-file-provider-plugin/18e445ca64b2391118c938093383b2c0afa20389 Log: [FIXED JENKINS-41161] replace old config object
            Hide
            josergc José Roberto García Chico added a comment -

            This one is also happening to me with the version 2.15.4. So waiting for the new release. Thank you!

            Show
            josergc José Roberto García Chico added a comment - This one is also happening to me with the version 2.15.4. So waiting for the new release. Thank you!
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in jenkins
            User: imod
            Path:
            pom.xml
            src/main/java/org/jenkinsci/plugins/configfiles/GlobalConfigFiles.java
            src/test/java/org/jenkinsci/plugins/configfiles/ConfigFileStoreTest.java
            src/test/java/org/jenkinsci/plugins/configfiles/folder/FolderConfigFileActionTest.java
            src/test/resources/org/jenkinsci/lib/configprovider/SystemConfigFilesManagementTest/testLoadAndMergeOldData/custom-config-files.xml
            src/test/resources/org/jenkinsci/lib/configprovider/SystemConfigFilesManagementTest/testLoadAndMergeOldData/jenkins.mvn.GlobalMavenConfig.xml
            src/test/resources/org/jenkinsci/lib/configprovider/SystemConfigFilesManagementTest/testLoadAndMergeOldData/maven-settings-files.xml
            src/test/resources/org/jenkinsci/lib/configprovider/SystemConfigFilesManagementTest/testLoadAndMergeOldData/org.jenkinsci.plugins.configfiles.json.JsonConfig.xml
            src/test/resources/org/jenkinsci/lib/configprovider/SystemConfigFilesManagementTest/testLoadAndMergeOldData/xml-config-files.xml
            src/test/resources/org/jenkinsci/plugins/configfiles/SystemConfigFilesManagementTest/testLoadAndMergeOldData.zip
            http://jenkins-ci.org/commit/config-file-provider-plugin/8acbe01d96bcfb3109298d36fd38db71895a6f16
            Log:
            JENKINS-41161 add some tests

            Compare: https://github.com/jenkinsci/config-file-provider-plugin/compare/0467185a813a...8acbe01d96bc

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: imod Path: pom.xml src/main/java/org/jenkinsci/plugins/configfiles/GlobalConfigFiles.java src/test/java/org/jenkinsci/plugins/configfiles/ConfigFileStoreTest.java src/test/java/org/jenkinsci/plugins/configfiles/folder/FolderConfigFileActionTest.java src/test/resources/org/jenkinsci/lib/configprovider/SystemConfigFilesManagementTest/testLoadAndMergeOldData/custom-config-files.xml src/test/resources/org/jenkinsci/lib/configprovider/SystemConfigFilesManagementTest/testLoadAndMergeOldData/jenkins.mvn.GlobalMavenConfig.xml src/test/resources/org/jenkinsci/lib/configprovider/SystemConfigFilesManagementTest/testLoadAndMergeOldData/maven-settings-files.xml src/test/resources/org/jenkinsci/lib/configprovider/SystemConfigFilesManagementTest/testLoadAndMergeOldData/org.jenkinsci.plugins.configfiles.json.JsonConfig.xml src/test/resources/org/jenkinsci/lib/configprovider/SystemConfigFilesManagementTest/testLoadAndMergeOldData/xml-config-files.xml src/test/resources/org/jenkinsci/plugins/configfiles/SystemConfigFilesManagementTest/testLoadAndMergeOldData.zip http://jenkins-ci.org/commit/config-file-provider-plugin/8acbe01d96bcfb3109298d36fd38db71895a6f16 Log: JENKINS-41161 add some tests Compare: https://github.com/jenkinsci/config-file-provider-plugin/compare/0467185a813a...8acbe01d96bc
            Hide
            robinmoss Robin Moss added a comment -

            Thanks for the quick fix, can confirm 2.15.5 it's working again

            Show
            robinmoss Robin Moss added a comment - Thanks for the quick fix, can confirm 2.15.5 it's working again

              People

              • Assignee:
                domi Dominik Bartholdi
                Reporter:
                egutierrez Evaristo Gutierrez
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: