Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Duplicate
    • Affects Version/s: None
    • Fix Version/s: 0.8.0
    • Component/s: core
    • Labels:
      None

      Description

      yum install is called without the -y flag.
      We might want to use retry_yum, too.

      1. WHIRR-580.patch
        0.7 kB
        Tom White
      2. WHIRR-580.patch
        0.7 kB
        Karel Vervaeke

        Activity

        Karel Vervaeke created issue -
        Hide
        Karel Vervaeke added a comment -

        Patch uses yum install -y, installs the right package (openjdk-1.6.0-devel, the regular package contains only jre), correctly sets JAVA_HOME

        Show
        Karel Vervaeke added a comment - Patch uses yum install -y, installs the right package (openjdk-1.6.0-devel, the regular package contains only jre), correctly sets JAVA_HOME
        Karel Vervaeke made changes -
        Field Original Value New Value
        Attachment WHIRR-580.patch [ 12521303 ]
        Hide
        Tom White added a comment -

        +1 Looks good - I assume you have tested?

        Show
        Tom White added a comment - +1 Looks good - I assume you have tested?
        Hide
        Tom White added a comment -

        Here's a small update to use retry_yum. I successfully tested by manually starting a cluster.

        Show
        Tom White added a comment - Here's a small update to use retry_yum. I successfully tested by manually starting a cluster.
        Tom White made changes -
        Attachment WHIRR-580.patch [ 12521386 ]
        Hide
        Karel Vervaeke added a comment -

        Uh yes, that was tested. I didn't know if I could trust retry_yum yet, but if it works it works.

        Show
        Karel Vervaeke added a comment - Uh yes, that was tested. I didn't know if I could trust retry_yum yet, but if it works it works.
        Hide
        Ioannis Canellos added a comment - - edited

        I've tried the scripts individually (not from inside whirr) on ubuntu/centos created on aws-ec2 and cloudservers-us and they seem to work fine.

        Show
        Ioannis Canellos added a comment - - edited I've tried the scripts individually (not from inside whirr) on ubuntu/centos created on aws-ec2 and cloudservers-us and they seem to work fine.
        Hide
        Andrew Bayer added a comment -

        Seems reasonable. I've got some pending changes to the java home detection for supporting Ubuntu 12.04 and CentOS 6, but that should definitely wait for this.

        Show
        Andrew Bayer added a comment - Seems reasonable. I've got some pending changes to the java home detection for supporting Ubuntu 12.04 and CentOS 6, but that should definitely wait for this.
        Adrian Cole (Inactive) made changes -
        Link This issue blocks WHIRR-612 [ WHIRR-612 ]
        Adrian Cole (Inactive) made changes -
        Link This issue blocks WHIRR-612 [ WHIRR-612 ]
        Adrian Cole (Inactive) made changes -
        Link This issue is blocked by WHIRR-528 [ WHIRR-528 ]
        Hide
        Adrian Cole (Inactive) added a comment -

        this patch depends on an unmerged change

        Show
        Adrian Cole (Inactive) added a comment - this patch depends on an unmerged change
        Hide
        Adrian Cole (Inactive) added a comment -

        this is more straight forward to resolve as a part of WHIRR-528, as the point of WHIRR-528 is to change things to retry. The subtle openjdk needs -devl was resolved in WHIRR-613

        Show
        Adrian Cole (Inactive) added a comment - this is more straight forward to resolve as a part of WHIRR-528 , as the point of WHIRR-528 is to change things to retry. The subtle openjdk needs -devl was resolved in WHIRR-613
        Adrian Cole (Inactive) made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 0.8.0 [ 12321649 ]
        Resolution Duplicate [ 3 ]
        Adrian Cole (Inactive) made changes -
        Link This issue is blocked by WHIRR-528 [ WHIRR-528 ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Karel Vervaeke
          • Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 5m
              5m
              Remaining:
              Remaining Estimate - 5m
              5m
              Logged:
              Time Spent - Not Specified
              Not Specified

                Development