Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.8.0
    • Component/s: None
    • Labels:
      None

      Description

      We expect this release at the beginning of January 2012.

      1. WHIRR-456-snapshot-release.patch
        4 kB
        Andrei Savu
      2. WHIRR-456.patch
        8 kB
        Andrei Savu
      3. WHIRR-456.patch
        18 kB
        Andrei Savu

        Issue Links

          Activity

          Hide
          Andrei Savu added a comment -
          Show
          Andrei Savu added a comment - This update is going to include http://code.google.com/p/jclouds/issues/detail?id=746
          Hide
          Andrei Savu added a comment -

          Here is a quick patch with some changes needed to upgrade to 1.3.0. I have been able to run the ZooKeeper integration tests on aws-ec2 using t1.micro (this used to fail with 1.2.1).

          Show
          Andrei Savu added a comment - Here is a quick patch with some changes needed to upgrade to 1.3.0. I have been able to run the ZooKeeper integration tests on aws-ec2 using t1.micro (this used to fail with 1.2.1).
          Hide
          Andrei Savu added a comment -

          The jclouds release is on track for this weekend.

          Show
          Andrei Savu added a comment - The jclouds release is on track for this weekend.
          Hide
          Andrei Savu added a comment -

          Updated patch for trunk. Tested with cassandra on cloudservers-uk & aws-ec2.

          Show
          Andrei Savu added a comment - Updated patch for trunk. Tested with cassandra on cloudservers-uk & aws-ec2.
          Hide
          Andrei Savu added a comment - - edited

          I have just checked that this builds fine with jclouds 1.3.0 from Maven Central. Ready for review!

          Show
          Andrei Savu added a comment - - edited I have just checked that this builds fine with jclouds 1.3.0 from Maven Central. Ready for review!
          Hide
          Adrian Cole (Inactive) added a comment -

          we should convert usage of whirr.login-user to set <provider>.login-user and same for image-id, right? That way, we can rid the custom class

          Show
          Adrian Cole (Inactive) added a comment - we should convert usage of whirr.login-user to set <provider>.login-user and same for image-id, right? That way, we can rid the custom class
          Hide
          Andrei Savu added a comment -

          Or maybe do this internally? Use whirr.bootstrap-user to set the value for <provider>.login-user?

          Show
          Andrei Savu added a comment - Or maybe do this internally? Use whirr.bootstrap-user to set the value for <provider>.login-user?
          Hide
          Adrian Cole (Inactive) added a comment -

          yep, I meant internally (directly should also work, tho)

          Show
          Adrian Cole (Inactive) added a comment - yep, I meant internally (directly should also work, tho)
          Hide
          Andrei Savu added a comment -

          I think WHIRR-475 should go in before making this change here. What do you think?

          Show
          Andrei Savu added a comment - I think WHIRR-475 should go in before making this change here. What do you think?
          Hide
          Adrian Cole (Inactive) added a comment -

          +1

          Show
          Adrian Cole (Inactive) added a comment - +1
          Hide
          Andrei Savu added a comment -

          Update patch to make use of <provider>.image.login-user now available in jclouds. Adrian let me know what you think.

          Here is the diff on Github (easier to review):
          https://github.com/andreisavu/whirr/compare/whirr-456

          Show
          Andrei Savu added a comment - Update patch to make use of <provider>.image.login-user now available in jclouds. Adrian let me know what you think. Here is the diff on Github (easier to review): https://github.com/andreisavu/whirr/compare/whirr-456
          Hide
          Adrian Cole (Inactive) added a comment -

          +1 great to see all the lines removed

          Show
          Adrian Cole (Inactive) added a comment - +1 great to see all the lines removed
          Hide
          Andrei Savu added a comment -

          I have just checked that the bootstrap process fails with a wrong user:password pair specified as whirr.bootstrap-user.

          Show
          Andrei Savu added a comment - I have just checked that the bootstrap process fails with a wrong user:password pair specified as whirr.bootstrap-user.
          Hide
          Andrei Savu added a comment -

          Committed to trunk. Thanks Adrian for reviewing.

          Show
          Andrei Savu added a comment - Committed to trunk. Thanks Adrian for reviewing.

            People

            • Assignee:
              Andrei Savu
              Reporter:
              Andrei Savu
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development