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

JL-16: TimeoutChecker - triggered if using "Run with timeout" in build descript

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Minor
    • Resolution: Fixed
    • Component/s: jenkinslint-plugin
    • Labels:
    • Environment:
      Jenkins 2.57 running using JDK1.8_141 on RH6 Linux, no container
      jenkinslint 0.10.1
      build timeout plugin 1.18
    • Similar Issues:

      Description

      Hello,

       

      Build timeout plugin supply two kind of timeout steps:

      Build environment => "Abort if the build is stuck" 

      If the job it is configured with this step, the TimeoutChecker flag gets correctly green 

      Build Step => "Run with timeout"

      If the job it is configured with this step, the TimeoutChecker flag gets red: also this step should be detected as a correctly configured job.

       

      Best Regards,

      Roberto

       

      P.S. is there in generale more detailed documentation on jenkinslint plugin, with some example on how to fix the jobs?

        Attachments

          Activity

          Hide
          v2v Victor Martinez added a comment -

          Thanks,

          Build Step => "Run with timeout" is not supported atm, just the build wrapper*: '*Abort if the build is stuck" ** 

          You can find those details under:

           

          I'll analyse it further to support it in the upcoming release.

           

          Thanks

          Show
          v2v Victor Martinez added a comment - Thanks, Build Step => "Run with timeout" is not supported atm, just the build wrapper*: '* Abort if the build is stuck"  **  You can find those details under: https://github.com/jenkinsci/jenkinslint-plugin/blob/master/src/main/java/org/jenkins/ci/plugins/jenkinslint/check/TimeoutChecker.java#L30   I'll analyse it further to support it in the upcoming release.   Thanks
          Hide
          v2v Victor Martinez added a comment -

          It's indeed part of the latest release:

          Version 1.18 (Nov 13, 2016)
          Introduced "Run with timeout" build step (PR61)

           

          That's the reason it is not supported. Good spot. Thanks again

           

          Show
          v2v Victor Martinez added a comment - It's indeed part of the latest release: Version 1.18 (Nov 13, 2016) Introduced "Run with timeout" build step (PR61)   That's the reason it is not supported. Good spot. Thanks again  
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Victor Martinez
          Path:
          pom.xml
          src/main/java/org/jenkins/ci/plugins/jenkinslint/check/TimeoutChecker.java
          src/test/java/org/jenkins/ci/plugins/jenkinslint/check/TimeoutCheckerTestCase.java
          http://jenkins-ci.org/commit/jenkinslint-plugin/09badf14452747ed9a727bfc9fcb8808f890219c
          Log:
          JENKINS-45938 Initial approach

          Change-Id: Id3af67394a7a4876ea048b6f588d68c2ba2f25b7

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Victor Martinez Path: pom.xml src/main/java/org/jenkins/ci/plugins/jenkinslint/check/TimeoutChecker.java src/test/java/org/jenkins/ci/plugins/jenkinslint/check/TimeoutCheckerTestCase.java http://jenkins-ci.org/commit/jenkinslint-plugin/09badf14452747ed9a727bfc9fcb8808f890219c Log: JENKINS-45938 Initial approach Change-Id: Id3af67394a7a4876ea048b6f588d68c2ba2f25b7
          Hide
          robrizlav Roberto Rizzetto added a comment -

          > That's the reason it is not supported. Good spot. Thanks again

          sorry I have note realized. thanks to you!

          Show
          robrizlav Roberto Rizzetto added a comment - > That's the reason it is not supported. Good spot. Thanks again sorry I have note realized. thanks to you!
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Victor Martinez
          Path:
          pom.xml
          src/main/java/org/jenkins/ci/plugins/jenkinslint/check/TimeoutChecker.java
          src/test/java/org/jenkins/ci/plugins/jenkinslint/check/TimeoutCheckerTestCase.java
          http://jenkins-ci.org/commit/jenkinslint-plugin/b6adad6b72479ec07815d9b70a83776487b35b6e
          Log:
          JENKINS-45938 Supporting Run with timeout build (#24)

          Change-Id: Id3af67394a7a4876ea048b6f588d68c2ba2f25b7

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Victor Martinez Path: pom.xml src/main/java/org/jenkins/ci/plugins/jenkinslint/check/TimeoutChecker.java src/test/java/org/jenkins/ci/plugins/jenkinslint/check/TimeoutCheckerTestCase.java http://jenkins-ci.org/commit/jenkinslint-plugin/b6adad6b72479ec07815d9b70a83776487b35b6e Log: JENKINS-45938 Supporting Run with timeout build (#24) Change-Id: Id3af67394a7a4876ea048b6f588d68c2ba2f25b7
          Hide
          v2v Victor Martinez added a comment -

          Already released in the version 0.11.0

          Show
          v2v Victor Martinez added a comment - Already released in the version 0.11.0

            People

            • Assignee:
              v2v Victor Martinez
              Reporter:
              robrizlav Roberto Rizzetto
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: