You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by re...@apache.org on 2015/12/05 19:12:03 UTC

[2/2] git commit: updated refs/heads/master to e37842c

Merge pull request #1108 from rafaelweingartner/master-lrg-cs-hackday-004

Removed the PlannerBase class because it is does not bring contribution to ACS' code.Removed the PlannerBase class because it is does not bring contribution to ACS' code.

We changed com.cloud.deploy.FirstFitPlanner, now it doesnt extends the PlannerBase and implements the com.cloud.deploy.DeploymentPlanner.

We also removed the method com.cloud.deploy.DeploymentPlanner.check(VirtualMachineProfile, DeploymentPlan, DeployDestination, ExcludeList) that was not used anywhere.

Additionally, we removed the _ from FirstFitPlanner's attributes name, in order to have them in a more standard way.

* pr/1108:
  Removed PlannerBase empty class

Signed-off-by: Remi Bergsma <gi...@remi.nl>


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/e37842cb
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/e37842cb
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/e37842cb

Branch: refs/heads/master
Commit: e37842cbd3407c934a6ecc2515b60742b52d1738
Parents: 74e33dc 7453727
Author: Remi Bergsma <gi...@remi.nl>
Authored: Sat Dec 5 19:11:20 2015 +0100
Committer: Remi Bergsma <gi...@remi.nl>
Committed: Sat Dec 5 19:11:21 2015 +0100

----------------------------------------------------------------------
 api/src/com/cloud/deploy/DeploymentPlanner.java |  18 ---
 .../cloud/deploy/ImplicitDedicationPlanner.java |   8 +-
 .../deploy/UserConcentratedPodPlanner.java      |   4 +-
 .../com/cloud/deploy/UserDispersingPlanner.java |  10 +-
 .../baremetal/manager/BareMetalPlanner.java     |   8 +-
 .../src/com/cloud/deploy/FirstFitPlanner.java   | 109 +++++++++----------
 server/src/com/cloud/deploy/PlannerBase.java    |  23 ----
 7 files changed, 64 insertions(+), 116 deletions(-)
----------------------------------------------------------------------