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

          Hide
          Adrian Cole added a comment -

          Committed revision 1369707.

          Show
          Adrian Cole 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
          Hide
          Adrian Cole added a comment -

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

          Show
          Adrian Cole added a comment - reworked and tested zookeeper on cloudservers-us (default ubuntu) aws-ec2 (amzn_linux us-east-1/ami-aecd60c7)
          Hide
          Adrian Cole 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 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 .
          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.
          Hide
          Adrian Cole 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 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
          Hide
          Adrian Cole added a comment -

          reworked for trunk

          Show
          Adrian Cole added a comment - reworked for trunk
          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.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development