Whirr
  1. Whirr
  2. WHIRR-123

Cassandra integration tests hang if whirr's scripts bucket is missing

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 0.2.0
    • Fix Version/s: 0.3.0
    • Component/s: core, service/cassandra
    • Labels:
      None

      Description

      I don't know if this is specific to cassandra or just a general issue.

      I ran the integration tests as part of building the 0.2.0 release candidate. The tests failed while verifying cassandra. Looking on the ec2 host I see the following in the stderr (nothing in stdout). Note - I had not created the 0.2.0-incubating bucket on s3, so while this test would never pass, it seems unusual that the test hangs forever rather than tear down the test and report an error (I had to ctrl-c to stop the test, then manually cleanup ec2).

      --------- stderr ---------------

      -2010-11-02 05:51:45-
      http://whirr.s3.amazonaws.com/0.2.0-incubating/sun/java/install
      Resolving whirr.s3.amazonaws.com... 72.21.211.174
      Connecting to whirr.s3.amazonaws.com|72.21.211.174|:80... connected.
      HTTP request sent, awaiting response... 404 Not Found
      2010-11-02 05:51:45 ERROR 404: Not Found.

      runurl: wget failed: 8
      -2010-11-02 05:51:45-
      http://whirr.s3.amazonaws.com/0.2.0-incubating/apache/cassandra/install
      Resolving whirr.s3.amazonaws.com... 72.21.211.174
      Connecting to whirr.s3.amazonaws.com|72.21.211.174|:80... connected.
      HTTP request sent, awaiting response... 404 Not Found
      2010-11-02 05:51:45 ERROR 404: Not Found.

      runurl: wget failed: 8

      1. WHIRR-123.patch
        14 kB
        Tom White
      2. WHIRR-123.patch
        7 kB
        Tom White

        Activity

        Hide
        Patrick Hunt added a comment -

        ps. the test doesn't cleanup properly in this case (ctrl-c after the hang) so be sure to check your ec2 instances and terminate any zombies assoc with the test.

        Show
        Patrick Hunt added a comment - ps. the test doesn't cleanup properly in this case (ctrl-c after the hang) so be sure to check your ec2 instances and terminate any zombies assoc with the test.
        Hide
        Tom White added a comment -

        Here's a patch that checks if the runurl exists (using an HTTP HEAD request).

        Show
        Tom White added a comment - Here's a patch that checks if the runurl exists (using an HTTP HEAD request).
        Hide
        Adrian Cole added a comment -

        patch looks fine and integration tests pass.

        Thanks for putting this together.

        Show
        Adrian Cole added a comment - patch looks fine and integration tests pass. Thanks for putting this together.
        Hide
        Tom White added a comment -

        I've just committed this.

        Show
        Tom White added a comment - I've just committed this.
        Hide
        Tom White added a comment -

        This is causing test failures for me. I've reverted it while I investigate.

        Show
        Tom White added a comment - This is causing test failures for me. I've reverted it while I investigate.
        Hide
        Tom White added a comment -

        The problem with the previous patch was that arguments to the runurl script were being treated as a part of the URL, which caused a problem when the URL was being checked. Here's a new patch which fixes that.

        Show
        Tom White added a comment - The problem with the previous patch was that arguments to the runurl script were being treated as a part of the URL, which caused a problem when the URL was being checked. Here's a new patch which fixes that.
        Hide
        Tom White added a comment -

        This passes integration tests for Cassandra, Hadoop, and ZooKeeper for me, so I'll commit it soon unless there are further comments.

        Show
        Tom White added a comment - This passes integration tests for Cassandra, Hadoop, and ZooKeeper for me, so I'll commit it soon unless there are further comments.
        Hide
        Tom White added a comment -

        I've just committed this.

        Show
        Tom White added a comment - I've just committed this.

          People

          • Assignee:
            Tom White
            Reporter:
            Patrick Hunt
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development