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

Plugin does not start build if there is only one change set

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Component/s: tfs-plugin
    • Labels:
      None
    • Environment:
      Hudson ver. 1.363, both released and current tfs plugin
      tfs 2008 server
      MS TEE Command line client 10.0.0
    • Similar Issues:

      Description

      After checking in changes, TFS log in Hudson shows them consistently (see below). However, no automatic builds are being triggered.

      The problem is the newly released TEE Command line client – it does not output the

      "-------------------------------"

      seperation lines that the parsing algorithm depends upon. We have reverted to using the Microsoft Team System Explorer command line utility and everything seems to be working fine now.

      However, wouldn't it still be possible to change the parser to starting a block with "Changeset:" instead of ending it with "-----------------"?

      If necessary, I could provide a diff...

       
      Started on 28.06.2010 18:18:38
      [workspace] $ C:\javabuildtools\TEE-CLC-10.0.0\tf.cmd history $/Services/Server/DEV/SP/MavenDefs -noprompt -version:D2010-06-28T12:52:47Z~D2010-06-28T16:18:38Z -recursive -format:detailed -server:http://s-tfs:8080/ ********
      
      Changeset: 364235
      User: GRP\ObendorfH
      Date: 28.06.2010 17:25:40
      
      Comment:
        Versionierung für EJB EAR
      
      Items:
        edit $/Services/Server/DEV/SP/MavenDefs/pom.xml
      
      Check-in Notes:
        Code Reviewer: 
      
        Performance Reviewer: 
        Security Reviewer: 
      
      Done. Took 8,6 Sekunden
      No changes
      

        Attachments

          Activity

          Hide
          hartmut hartmut added a comment -

          Setting the polling time to 1, 5, and 10 minutes does not change the plugin's behavior (see attached image).

          Show
          hartmut hartmut added a comment - Setting the polling time to 1, 5, and 10 minutes does not change the plugin's behavior (see attached image).
          Hide
          redsolo redsolo added a comment -

          If there are several changests, are they separated by anything? Apparently the order of the keywords is not specific and could change (http://social.msdn.microsoft.com/Forums/en-US/tfsversioncontrol/thread/c41d1d0c-dfe1-4a00-b128-c5e3f503170e#3ac1216e-c121-4c9d-ab53-7c1c4fb823e6).

          But please attach an output containing two change sets.

          Show
          redsolo redsolo added a comment - If there are several changests, are they separated by anything? Apparently the order of the keywords is not specific and could change ( http://social.msdn.microsoft.com/Forums/en-US/tfsversioncontrol/thread/c41d1d0c-dfe1-4a00-b128-c5e3f503170e#3ac1216e-c121-4c9d-ab53-7c1c4fb823e6 ). But please attach an output containing two change sets.
          Hide
          hartmut hartmut added a comment - - edited

          Differences are smaller than we thought – IF there are several changesets in the output, they are separated by a line of "-". IF the output contains only a single changeset, there is NO separation line. That caused the error.

          MS TF.EXE output - WITH leading ---- - line

          $ /cygdrive/c/Programme/Microsoft\ Visual\ Studio\ 10.0/Common7/IDE/TF.exe history $/TFSJavaBuild -noprompt -version:D2010-05-06T11:00:26Z~D2010-05-06T11:30:17Z -recursive -format:detailed -server:http://v-tfsbuildtest:8080/tfs
          -----------------------------------------------------------------------------------------------------------------------
          Changeset: 24
          User: ObendorfH
          Date: Donnerstag, 6. Mai 2010 13:29:24
          
          Comment:
            Initialer Check-In
          
          Items:
            add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base
            add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.project
            add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.settings
            add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.tpignore
            add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/pom.xml
            add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/src
            add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.settings/org.maven.ide.eclipse.prefs
            add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/src/site
          
          Check-in Notes:
            Code Reviewer:
            Performance Reviewer:
            Security Reviewer:
          
          -----------------------------------------------------------------------------------------------------------------------
          Changeset: 23
          User: ObendorfH
          Date: Donnerstag, 6. Mai 2010 13:28:21
          
          Comment:
          
          Items:
            branch $/TFSJavaBuild/Sources/Maven/Current
          
          Check-in Notes:
            Code Reviewer:
            Performance Reviewer:
            Security Reviewer:
          

          MS TEE TF.cmd - WITHOUT leading ---- - line

          $ /cygdrive/c/Programme/TEE-CLC-10.0.0/tf.cmd history $/TFSJavaBuild -noprompt -version:D2010-05-06T11:00:26Z~D2010-05-06T11:30:17Z -recursive -format:detailed -server:http://v-tfsbuildtest:8080/tfs
          Changeset: 24
          User: GRP\ObendorfH
          Date: 06.05.2010 13:29:24
          
          Comment:
            Initialer Check-In
          
          Items:
            add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base
            add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.project
            add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.settings
            add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.tpignore
            add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/pom.xml
            add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/src
            add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.settings/org.maven.ide.eclipse.prefs
            add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/src/site
          
          Check-in Notes:
            Code Reviewer:
            Performance Reviewer:
            Security Reviewer:
          
          -----------------------------------------------------------------------------------------------------------------------
          Changeset: 23
          User: GRP\ObendorfH
          Date: 06.05.2010 13:28:21
          
          Comment:
          
          
          Items:
            branch $/TFSJavaBuild/Sources/Maven/Current
          
          Check-in Notes:
            Code Reviewer:
            Performance Reviewer:
            Security Reviewer:
          
          
          Show
          hartmut hartmut added a comment - - edited Differences are smaller than we thought – IF there are several changesets in the output, they are separated by a line of "-". IF the output contains only a single changeset, there is NO separation line. That caused the error. MS TF.EXE output - WITH leading ---- - line $ /cygdrive/c/Programme/Microsoft\ Visual\ Studio\ 10.0/Common7/IDE/TF.exe history $/TFSJavaBuild -noprompt -version:D2010-05-06T11:00:26Z~D2010-05-06T11:30:17Z -recursive -format:detailed -server:http://v-tfsbuildtest:8080/tfs ----------------------------------------------------------------------------------------------------------------------- Changeset: 24 User: ObendorfH Date: Donnerstag, 6. Mai 2010 13:29:24 Comment: Initialer Check-In Items: add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.project add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.settings add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.tpignore add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/pom.xml add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/src add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.settings/org.maven.ide.eclipse.prefs add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/src/site Check-in Notes: Code Reviewer: Performance Reviewer: Security Reviewer: ----------------------------------------------------------------------------------------------------------------------- Changeset: 23 User: ObendorfH Date: Donnerstag, 6. Mai 2010 13:28:21 Comment: Items: branch $/TFSJavaBuild/Sources/Maven/Current Check-in Notes: Code Reviewer: Performance Reviewer: Security Reviewer: MS TEE TF.cmd - WITHOUT leading ---- - line $ /cygdrive/c/Programme/TEE-CLC-10.0.0/tf.cmd history $/TFSJavaBuild -noprompt -version:D2010-05-06T11:00:26Z~D2010-05-06T11:30:17Z -recursive -format:detailed -server:http://v-tfsbuildtest:8080/tfs Changeset: 24 User: GRP\ObendorfH Date: 06.05.2010 13:29:24 Comment: Initialer Check-In Items: add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.project add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.settings add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.tpignore add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/pom.xml add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/src add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.settings/org.maven.ide.eclipse.prefs add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/src/site Check-in Notes: Code Reviewer: Performance Reviewer: Security Reviewer: ----------------------------------------------------------------------------------------------------------------------- Changeset: 23 User: GRP\ObendorfH Date: 06.05.2010 13:28:21 Comment: Items: branch $/TFSJavaBuild/Sources/Maven/Current Check-in Notes: Code Reviewer: Performance Reviewer: Security Reviewer:
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in hudson
          User: : redsolo
          Path:
          trunk/hudson/plugins/tfs/src/main/java/hudson/plugins/tfs/commands/DetailedHistoryCommand.java
          trunk/hudson/plugins/tfs/src/test/java/hudson/plugins/tfs/commands/DetailedHistoryCommandTest.java
          trunk/hudson/plugins/tfs/src/test/resources/hudson/plugins/tfs/commands/issue-6870-2.txt
          trunk/hudson/plugins/tfs/src/test/resources/hudson/plugins/tfs/commands/issue-6870.txt
          http://jenkins-ci.org/commit/34965
          Log:
          [FIXED JENKINS-6870] Fixed parsing of history output even if there is no starting separator when using the TEE tf.cmd.

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in hudson User: : redsolo Path: trunk/hudson/plugins/tfs/src/main/java/hudson/plugins/tfs/commands/DetailedHistoryCommand.java trunk/hudson/plugins/tfs/src/test/java/hudson/plugins/tfs/commands/DetailedHistoryCommandTest.java trunk/hudson/plugins/tfs/src/test/resources/hudson/plugins/tfs/commands/issue-6870-2.txt trunk/hudson/plugins/tfs/src/test/resources/hudson/plugins/tfs/commands/issue-6870.txt http://jenkins-ci.org/commit/34965 Log: [FIXED JENKINS-6870] Fixed parsing of history output even if there is no starting separator when using the TEE tf.cmd.
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in hudson
          User: : redsolo
          Path:
          trunk/hudson/plugins/tfs/src/test/java/hudson/plugins/tfs/commands/DetailedHistoryCommandTest.java
          trunk/hudson/plugins/tfs/src/test/resources/hudson/plugins/tfs/commands/issue-6454.txt
          http://jenkins-ci.org/commit/34967
          Log:
          [FIXED JENKINS-6454] Fixed parsing of the MS TEE tf.cmd in issue JENKINS-6870 fixed also this issue.

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in hudson User: : redsolo Path: trunk/hudson/plugins/tfs/src/test/java/hudson/plugins/tfs/commands/DetailedHistoryCommandTest.java trunk/hudson/plugins/tfs/src/test/resources/hudson/plugins/tfs/commands/issue-6454.txt http://jenkins-ci.org/commit/34967 Log: [FIXED JENKINS-6454] Fixed parsing of the MS TEE tf.cmd in issue JENKINS-6870 fixed also this issue.

            People

            • Assignee:
              redsolo redsolo
              Reporter:
              hartmut hartmut
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: