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

p4-plugin 1.10.0 does not sync to label correctly

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Blocker
    • Resolution: Fixed
    • Component/s: p4-plugin
    • Labels:
    • Environment:
      Jenkins 2.179
      p4 plugin 1.10.0
    • Similar Issues:

      Description

      If you try to sync to a label, the p4-plugin syncs to latest changelist instead.

      This does not happen in p4plugin 1.9.7

      Output from same client, different version of plugin

      p4-plugin 1.10.0
      P4 Task: syncing files at change: 230943
      p4 sync -q /auto/wksp/test_client/...@230943
      duration: 0m 26s

      p4-plugin 1.9.7
      p4 label -o REL_XXX +
      P4 Task: syncing files at client/label: REL_XXX
      p4 sync -q /auto/wksp/test_client/...@REL_XXX +
      00:00:39 duration: (280ms)

      Note that there is no p4 label -o REL_XXX output for p4-plugin 1.10.0

        Attachments

          Activity

          Hide
          p4paul Paul Allen added a comment -

          I'll look into the polling logic and see if it can manage static labels. 

          Polling will need to poll against the old label name (e.g. static1), so changes will only get found if a new label (e.g. static2) is used and has newer revisions.  Updating the original label (e.g. static1) would have no effect as the comparison is using the label's name and not its content.

          Show
          p4paul Paul Allen added a comment - I'll look into the polling logic and see if it can manage static labels.  Polling will need to poll against the old label name (e.g. static1), so changes will only get found if a new label (e.g. static2) is used and has newer revisions.  Updating the original label (e.g. static1) would have no effect as the comparison is using the label's name and not its content.
          Hide
          p4paul Paul Allen added a comment -

          Ready for release.

          Show
          p4paul Paul Allen added a comment - Ready for release.
          Hide
          em_boehm Eric Boehm added a comment -

          Is there a way to download so that I can test it?

          Show
          em_boehm Eric Boehm added a comment - Is there a way to download so that I can test it?
          Hide
          em_boehm Eric Boehm added a comment -

          I downloaded and built it and it syncs correctly with a static label.

          Show
          em_boehm Eric Boehm added a comment - I downloaded and built it and it syncs correctly with a static label.
          Hide
          p4paul Paul Allen added a comment -

          Released in 1.10.4

          Show
          p4paul Paul Allen added a comment - Released in 1.10.4

            People

            • Assignee:
              p4paul Paul Allen
              Reporter:
              em_boehm Eric Boehm
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: