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

No popup window when committing changes to job config for putting in commit message

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      I have recently upgraded to Jenkins version 1.559, and with this upgrade I have also upgraded the scm-sync-configuration plugin to version 0.0.7.4.

      After doing the upgrade when I have made a change to a job configuration and hit save button, I dont get a pop-up window to enter the commit message. However I have noticed that the changes do get commited without a message.

      This problem does not occur when saving the global jenkins config.

      When looking through defects list I found https://issues.jenkins-ci.org/browse/JENKINS-14582 which sounded similar to the behaviour I am observing but it is claimed that the defect was fixed in version 0.0.7.

      It is very important from tracking point of view that there are commit messages along with each commit.

      I couldn't find any logs that have any information about the error

        Attachments

          Activity

          Hide
          vladichko Vlad Aginsky added a comment -

          I have the same issue with Jenkins ver. 1.577

          Show
          vladichko Vlad Aginsky added a comment - I have the same issue with Jenkins ver. 1.577
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Tom
          Path:
          src/main/java/hudson/plugins/scm_sync_configuration/strategies/impl/JobConfigScmSyncStrategy.java
          http://jenkins-ci.org/commit/scm-sync-configuration-plugin/83a1bb83ae758f6da31e0570d101983ddeb1f9d1
          Log:
          JENKINS-22666: Fix regexp

          Since PR #19 at GitHub (commit 141cc2b5), we don't get a commit message
          dialog anymore for changes to job configurations.

          The regexp looks broken with the initial slashes and only a single
          [^/]. With the Cloudbees Folders plugin jobs can be not only at

          jobs/Some_Job

          but also at

          jobs/Folder1/jobs/Folder2/jobs/Some_Job

          This commit simplifies the regexp and fixes it.

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Tom Path: src/main/java/hudson/plugins/scm_sync_configuration/strategies/impl/JobConfigScmSyncStrategy.java http://jenkins-ci.org/commit/scm-sync-configuration-plugin/83a1bb83ae758f6da31e0570d101983ddeb1f9d1 Log: JENKINS-22666 : Fix regexp Since PR #19 at GitHub (commit 141cc2b5), we don't get a commit message dialog anymore for changes to job configurations. The regexp looks broken with the initial slashes and only a single [^/] . With the Cloudbees Folders plugin jobs can be not only at jobs/Some_Job but also at jobs/Folder1/jobs/Folder2/jobs/Some_Job This commit simplifies the regexp and fixes it.
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Tom
          Path:
          src/main/java/hudson/plugins/scm_sync_configuration/strategies/impl/JobConfigScmSyncStrategy.java
          http://jenkins-ci.org/commit/scm-sync-configuration-plugin/e995bd405450c7c5b947c0c30871781ffc56b2ad
          Log:
          JENKINS-22666 Fix regexp

          Since PR #19 at GitHub (commit 141cc2b5), we don't get a commit message
          dialog anymore for changes to job configurations.

          The regexp looks broken with the initial slashes and only a single
          [^/]. With the Cloudbees Folders plugin jobs can be not only at

          jobs/Some_Job

          but also at

          jobs/Folder1/jobs/Folder2/jobs/Some_Job

          This commit simplifies the regexp and fixes it.

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Tom Path: src/main/java/hudson/plugins/scm_sync_configuration/strategies/impl/JobConfigScmSyncStrategy.java http://jenkins-ci.org/commit/scm-sync-configuration-plugin/e995bd405450c7c5b947c0c30871781ffc56b2ad Log: JENKINS-22666 Fix regexp Since PR #19 at GitHub (commit 141cc2b5), we don't get a commit message dialog anymore for changes to job configurations. The regexp looks broken with the initial slashes and only a single [^/] . With the Cloudbees Folders plugin jobs can be not only at jobs/Some_Job but also at jobs/Folder1/jobs/Folder2/jobs/Some_Job This commit simplifies the regexp and fixes it.
          Hide
          rodrigc Craig Rodrigues added a comment -

          Fixed in 0.0.9

          Show
          rodrigc Craig Rodrigues added a comment - Fixed in 0.0.9
          Hide
          colonelcoder David Cullen added a comment - - edited

          Hello, Craig Rodrigues,
          I still have this problem with Jenkins 1.642.1 and scm-sync-configuration-plugin 0.0.9.
          UPDATE: I was able to get this to work by

          1. Deleting the GIT repository
          2. Uninstalling the SCM Sync Configuration Plugin
          3. Creating the GIT repository
          4. Reinstalling the SCM Sync Configuration Plugin

          I think part of the problem was the need for the SSH key to be manually accepted. I recommend that anyone installing this plugin SSH from the Jenkins account to the machine with the GIT repository before installation.

          Show
          colonelcoder David Cullen added a comment - - edited Hello, Craig Rodrigues , I still have this problem with Jenkins 1.642.1 and scm-sync-configuration-plugin 0.0.9. UPDATE: I was able to get this to work by Deleting the GIT repository Uninstalling the SCM Sync Configuration Plugin Creating the GIT repository Reinstalling the SCM Sync Configuration Plugin I think part of the problem was the need for the SSH key to be manually accepted. I recommend that anyone installing this plugin SSH from the Jenkins account to the machine with the GIT repository before installation.

            People

            Assignee:
            rodrigc Craig Rodrigues
            Reporter:
            gauravnphadke Gaurav Phadke
            Votes:
            8 Vote for this issue
            Watchers:
            16 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: