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

job/config.xmls are no longer synched after upgrade to 0.0.7.1

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • None
    • - Jenkins: 1.515
      - SCM Synch Configuration plugin: 0.0.7.1
      - Running on Windows 2008 server 64 bit as a service
      - No VM parameters

      It seems that the fix for JENKINS-17545 has stopped any <job_folder>/config.xml files from being synched to SVN.

      What's happening is:

      • I turn on the plugin, all outstanding changes since are successfully added/committed to SVN
      • I change some "master" configuration ... /trunk/hudson.scm.SubversionSCM.xml and /trunk/scm-sync-configuration.xml are committed fine
      • I change a Job configuration via the UI, save, plugin asks for a commit message, I provide it. This job/config.xml does not get synched to SVN
      • No logs anywhere, the config.xml paths are just not being recognised up by the "listener"

      Note: does not occur in 0.0.7 prior to the module directory scan fix

          [JENKINS-18526] job/config.xmls are no longer synched after upgrade to 0.0.7.1

          Looks like this PR would fix the issue

          I'll try to merge it tonight and provide a hpi for test here.
          Will you be ok to test it ?

          Frédéric Camblor added a comment - Looks like this PR would fix the issue I'll try to merge it tonight and provide a hpi for test here. Will you be ok to test it ?

          Provided hpi fixing the issue with implementation from https://github.com/jenkinsci/scm-sync-configuration-plugin/pull/15

          Let me know if this fixes your issue

          Frédéric Camblor added a comment - Provided hpi fixing the issue with implementation from https://github.com/jenkinsci/scm-sync-configuration-plugin/pull/15 Let me know if this fixes your issue

          That seems to have fixed my issue, though I haven't had a chance to test extensively.

          Many Thanks!

          alex parkinson added a comment - That seems to have fixed my issue, though I haven't had a chance to test extensively. Many Thanks!

          Ok I'll package and release a new bugfix version (0.0.7.2) with PR merged, thanks for testing

          Frédéric Camblor added a comment - Ok I'll package and release a new bugfix version (0.0.7.2) with PR merged, thanks for testing

            fcamblor Frédéric Camblor
            ixi alex parkinson
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: