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

      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
      
      1. screenshot.5.png
        9 kB

        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: