Whirr
  1. Whirr
  2. WHIRR-640

Recipes and tests using whirr.template should specify minRam there

    Details

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

      Description

      Another gotcha I just uncovered - without ",minRam=2048" on the whirr.template value in the recipes or test properties for CDH, we end up with t1.micros, which doesn't work, unsurprisingly.

      1. WHIRR-640.patch
        4 kB
        Andrew Bayer

        Activity

        Hide
        Andrew Bayer added a comment -

        Fixes those recipes and tests accordingly.

        Show
        Andrew Bayer added a comment - Fixes those recipes and tests accordingly.
        Hide
        Adrian Cole added a comment -

        +1

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

        How about adding a good default value for "whirr.template" to core/src/main/resources/whirr-default.properties?

        whirr.template=osFamily=UBUNTU,osVersionMatches=10.04,os64Bit=true,minRam=2048
        
        Show
        Andrei Savu added a comment - How about adding a good default value for "whirr.template" to core/src/main/resources/whirr-default.properties? whirr.template=osFamily=UBUNTU,osVersionMatches=10.04,os64Bit=true,minRam=2048
        Hide
        Andrew Bayer added a comment -

        Committed to trunk and branch-0.8.

        Show
        Andrew Bayer added a comment - Committed to trunk and branch-0.8.

          People

          • Assignee:
            Andrew Bayer
            Reporter:
            Andrew Bayer
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development