• Icon: New Feature New Feature
    • Resolution: Fixed
    • Icon: Minor Minor
    • p4-plugin
    • None

      It would be good to expose the duration of the p4 actions so i can track how long it took for reconsile,cleanup,sync etc.

      I can then track syncs that take too long etc.

      There is a plugin that can add timestamps to logs but it is too buggy and doesn't always support the Jenkins LTS version

          [JENKINS-24741] Expose duration of plugin steps

          Morne Joubert added a comment -

          This fix provides the 2 times it need.
          How long the prepare(cleanup) took and how long the sync took

          Morne Joubert added a comment - This fix provides the 2 times it need. How long the prepare(cleanup) took and how long the sync took

          Morne Joubert added a comment -

          done git pull request for the fix

          Morne Joubert added a comment - done git pull request for the fix

          Paul Allen added a comment -

          Reposted here...

          I need to understand why the p4-plugin.properties file is necessary?
          For the moment I have commented it out, I'll look at it tomorrow (it 22:00 in the UK).

          I am concerned that an extra file might break other users builds or even appear in their releases!

          Paul Allen added a comment - Reposted here... I need to understand why the p4-plugin.properties file is necessary? For the moment I have commented it out, I'll look at it tomorrow (it 22:00 in the UK). I am concerned that an extra file might break other users builds or even appear in their releases!

          Morne Joubert added a comment -

          Reason was that I couldn't figure out any env variables that the plugin set to find the CL that was synced to. ( not unshelved)

          Morne Joubert added a comment - Reason was that I couldn't figure out any env variables that the plugin set to find the CL that was synced to. ( not unshelved)

          Morne Joubert added a comment -

          BTW what timezones are you in? I work near Cambridge UK

          Morne Joubert added a comment - BTW what timezones are you in? I work near Cambridge UK

          Paul Allen added a comment -

          I work in Wokingham, so only down the road.

          Paul Allen added a comment - I work in Wokingham, so only down the road.

          Paul Allen added a comment -

          You could use TagAction, it was designed for labeling builds, but could be a way to stash the value. I'll have a think, but getting a bit too late for me.

          Paul Allen added a comment - You could use TagAction, it was designed for labeling builds, but could be a way to stash the value. I'll have a think, but getting a bit too late for me.

          Morne Joubert added a comment -

          the latest version now shows duration

          ... duration: 0m 5s
          SCM Task: restoring all missing and changed revisions.
          ... [list] = reconcile [-n, -e, -d, -f, -m]
          ... duration: 2m 0s
          SCM Task: syncing files at change: 74939
          ... sync /var/lib/jenkins/workspace/__l1phy_int_mlb_10/...@74939
          ... force update false
          ... bypass have false
          ... duration: (226ms)
          SCM Task: unshelve review: 61023
          ... sync /var/lib/jenkins/workspace/__l1phy_int_mlb_10/...
          ... unshelve -f -s 61023 /var/lib/jenkins/workspace/__l1phy_int_mlb_10/...
          ... revert -k /var/lib/jenkins/workspace/__l1phy_int_mlb_10/...
          ... duration: 0m 1s

          Morne Joubert added a comment - the latest version now shows duration ... duration: 0m 5s SCM Task: restoring all missing and changed revisions. ... [list] = reconcile [-n, -e, -d, -f, -m] ... duration: 2m 0s SCM Task: syncing files at change: 74939 ... sync /var/lib/jenkins/workspace/__l1phy_int_mlb_10/...@74939 ... force update false ... bypass have false ... duration: (226ms) SCM Task: unshelve review: 61023 ... sync /var/lib/jenkins/workspace/__l1phy_int_mlb_10/... ... unshelve -f -s 61023 /var/lib/jenkins/workspace/__l1phy_int_mlb_10/... ... revert -k /var/lib/jenkins/workspace/__l1phy_int_mlb_10/... ... duration: 0m 1s

            p4paul Paul Allen
            mornejoubert Morne Joubert
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: