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

Selenium Plugin: Timeout too small

    Details

    • Type: Patch
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Component/s: other
    • Labels:
      None
    • Environment:
      Platform: All, OS: All
    • Similar Issues:
      Show 5 results

      Description

      The Selenium Plugin tests the Reachablility of a client machine using the Method
      hudson.model.Computer#getHostname().

      While the algorithm is pretty cool, in our network setup (virtual windows server
      on a xen cluster) it fails because of a too small timeout in the
      InetAddress.isReachable() - Method. Due to that, the Selenium Plugin does not
      start any Remote Controls quitting with "Unable to determine the host name.
      Skipping Selenium execution."

      An increased (I'd suggest 3000ms) value in the InetAddress.isReachable() Method
      makes the plugin work in our network setup.

      hudson.model.Computer#getHostname()

      Line 562 (old): if(!ia.isReachable(500)) {
      Line 562 (new): if(!ia.isReachable(3000)) {

      Could you increase the timeout?

        Attachments

          Activity

          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in hudson
          User: : kohsuke
          Path:
          trunk/hudson/main/core/src/main/java/hudson/model/Computer.java
          trunk/www/changelog.html
          http://fisheye4.cenqua.com/changelog/hudson/?cs=19436
          Log:
          [FIXED JENKINS-3981] Increased the timeout to 3secs as suggested. Added caching so that successive look-ups go faster. This fix will be in 1.315.

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in hudson User: : kohsuke Path: trunk/hudson/main/core/src/main/java/hudson/model/Computer.java trunk/www/changelog.html http://fisheye4.cenqua.com/changelog/hudson/?cs=19436 Log: [FIXED JENKINS-3981] Increased the timeout to 3secs as suggested. Added caching so that successive look-ups go faster. This fix will be in 1.315.

            People

            • Assignee:
              Unassigned
              Reporter:
              tboeghk tboeghk
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: