brooklyn-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From aledsage <>
Subject [GitHub] incubator-brooklyn pull request: BROOKLYN-212 fix
Date Mon, 11 Jan 2016 11:59:14 GMT
GitHub user aledsage opened a pull request:

    BROOKLYN-212 fix


You can merge this pull request into a Git repository by running:

    $ git pull BROOKLYN-212

Alternatively you can review and apply these changes as the patch at:

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1136
commit 810dd8d15078ebd197645e75ce32a64cef6de782
Author: Aled Sage <>
Date:   2016-01-08T21:27:23Z

    Asserts: expectedFailureOfType checks caused-by as well

commit f437e2ea962ea0228697ea41e8339202c67cd19d
Author: Aled Sage <>
Date:   2016-01-08T21:33:25Z

    BROOKLYN-212: AutoScaling doesn’t retry if InsufficientCapacity
    - Adds Resizable.InsufficientCapacityException, thrown by
      Resizable.resize() if could not grow at all.
    - DynamicCluster catches NoMachinesAvailableException, and rethrows
      as InsufficientCapacityException in resize().
    - AutoScalerPolicy catches InsufficientCapacityException, and sets
      insufficientCapacityHighWaterMark to record the max size it can get
      to. Does not try again to resize above that, unless the highWaterMark
      is explicitly cleared by reconfiguring that config value.
    - Tests:
      - Changes TestCluster to include history of sizes and desiredSizes
      - Changes TestCluster, so can throw InsufficientCapacityException
        when gets to a particular size.
      - Test for DynamicCluster throwing InsufficientCapacityException
      - Test for AutoScalerPolicyMetricTest, to not resize above the failure
        level again.
      - Test for AutoScalerPolicyNoMoreMachinesTest, for when BYON location
        has run out of machines in a DynamicCluster.

commit 230edaaab9d9bab8df8cc0a760d79d329071a93d
Author: Aled Sage <>
Date:   2016-01-11T11:39:51Z

    AutoScaler test: remove unnecessary timeout config
    succeedsEventually calls can just use the default timeout (unless it’s 
    a performance test where it must complete in a faster time!)


If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at or file a JIRA ticket
with INFRA.

View raw message