Whirr
  1. Whirr
  2. WHIRR-528

Add a retry loop around apt-get and yum commands to overcome transient errors

    Details

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

      Description

      We've done something like this already in WHIRR-517 for 0.7.1 (hotfix release). We need to do the same for trunk but also refactor the code to avoid code duplication.

      1. WHIRR-528.patch
        35 kB
        Adrian Cole
      2. WHIRR-528.patch
        35 kB
        Adrian Cole

        Issue Links

          Activity

          Adrian Cole (Inactive) made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          Adrian Cole (Inactive) added a comment -

          Committed revision 1369707.

          Show
          Adrian Cole (Inactive) added a comment - Committed revision 1369707.
          Hide
          Andrew Bayer added a comment -

          Looks good - running full tests to be sure, but I'd say go for it now. +1

          Show
          Andrew Bayer added a comment - Looks good - running full tests to be sure, but I'd say go for it now. +1
          Adrian Cole (Inactive) made changes -
          Attachment WHIRR-528.patch [ 12539216 ]
          Hide
          Adrian Cole (Inactive) added a comment -

          reworked and tested zookeeper on
          cloudservers-us (default ubuntu)
          aws-ec2 (amzn_linux us-east-1/ami-aecd60c7)

          Show
          Adrian Cole (Inactive) added a comment - reworked and tested zookeeper on cloudservers-us (default ubuntu) aws-ec2 (amzn_linux us-east-1/ami-aecd60c7)
          Hide
          Adrian Cole (Inactive) added a comment -

          recutting patch since WHIRR-613 is now in. As CDH is being handled in a separate issue, I suggest we get this passing on zookeeper and then address any CDH-related stuff in WHIRR-612.

          Show
          Adrian Cole (Inactive) added a comment - recutting patch since WHIRR-613 is now in. As CDH is being handled in a separate issue, I suggest we get this passing on zookeeper and then address any CDH-related stuff in WHIRR-612 .
          Adrian Cole (Inactive) made changes -
          Link This issue blocks WHIRR-580 [ WHIRR-580 ]
          Adrian Cole (Inactive) made changes -
          Link This issue is blocked by WHIRR-613 [ WHIRR-613 ]
          Hide
          Andrew Bayer added a comment -

          The Yarn tests are busted, yeah. Some annoying Hadoop configuration. Otherwise, +1.

          Show
          Andrew Bayer added a comment - The Yarn tests are busted, yeah. Some annoying Hadoop configuration. Otherwise, +1.
          Adrian Cole (Inactive) made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Affects Version/s 0.8.0 [ 12321649 ]
          Assignee Adrian Cole [ adrian@jclouds.org ]
          Fix Version/s 0.8.0 [ 12321649 ]
          Fix Version/s 0.8.0-alpha.1 [ 12318880 ]
          Hide
          Adrian Cole (Inactive) added a comment -

          CdhYarnServiceTest still doesn't pass on aws-ec2, but this patch includes the pieces relevant from WHIRR-517.

          P.S. even after putting WHIRR-580 onto this, we still get JDK errors, as those errors are related to sun jdk
          ex.
          /usr/lib/hadoop-hdfs/bin/hdfs: line 158: /usr/lib/jvm/java-6-sun/bin/java: No such file or directory

          Show
          Adrian Cole (Inactive) added a comment - CdhYarnServiceTest still doesn't pass on aws-ec2, but this patch includes the pieces relevant from WHIRR-517 . P.S. even after putting WHIRR-580 onto this, we still get JDK errors, as those errors are related to sun jdk ex. /usr/lib/hadoop-hdfs/bin/hdfs: line 158: /usr/lib/jvm/java-6-sun/bin/java: No such file or directory
          Adrian Cole (Inactive) made changes -
          Attachment WHIRR-528.patch [ 12539212 ]
          Hide
          Adrian Cole (Inactive) added a comment -

          reworked for trunk

          Show
          Adrian Cole (Inactive) added a comment - reworked for trunk
          Adrian Cole (Inactive) made changes -
          Link This issue blocks WHIRR-580 [ WHIRR-580 ]
          Adrian Cole (Inactive) made changes -
          Link This issue blocks WHIRR-612 [ WHIRR-612 ]
          Hide
          Andrei Savu added a comment -

          Note: there are a few other changes that I've made in WHIRR-517 that we need to make sure get in trunk.

          Show
          Andrei Savu added a comment - Note: there are a few other changes that I've made in WHIRR-517 that we need to make sure get in trunk.
          Andrei Savu made changes -
          Field Original Value New Value
          Link This issue is related to WHIRR-517 [ WHIRR-517 ]
          Andrei Savu created issue -

            People

            • Assignee:
              Adrian Cole (Inactive)
              Reporter:
              Andrei Savu
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development