Use the new buildpack installer task
Restore max_package_size in the config template

The recent refactor to pull cc templates out of cf-release reverted a
change to support large application packages.
Merge pull request #188 from fraenkel/addbuildpacks

Inject the new buildpack installer task
Remove dead code related to prioritized buildpacks

[#68432850]

Signed-off-by: Matthew Sykes <matthew.sykes@gmail.com>
Need to call destroy to exercise after_destroy

Signed-off-by: Matthew Sykes <matthew.sykes@gmail.com>
Extract BuildpackPositioner

An attempt to reduce the logic in Buildpack and eventually write clearer specs around the positioning

Signed-off-by: Matthew Sykes <matthew.sykes@gmail.com>
Encapsulate shift_to_position in update

- update was the only method using shift_to_position
- there was a second (sorta nested) transaction that was not needed
- [#68432850]

Signed-off-by: Matthew Sykes <matthew.sykes@gmail.com>
Reduce knowledge about the db in positioner

Signed-off-by: Matthew Sykes <matthew.sykes@gmail.com>
Be defensive about hash key style

buildpack_contoller_spec.rb uses string keys but buildpack_installer.rb symbolizes keys, it is not clear which is correct
Positioner no longer know about Buildpack.db
Split responsibilities of calculating position and shifting

In an attempt to marginally improve clarity. They are not unit tested as they are, and are instead exercised via buildpack_spec
Clean up in drain script

Including better logging

Signed-off-by: Aram Price <aprice@pivotallabs.com>
Add more logging and DRY up router deregister

Signed-off-by: Aram Price <aprice@pivotallabs.com>
Hoist timeout values to the top, trim comments

Signed-off-by: Ruth Byers <rbyers@pivotallabs.com>
Merge branch 'pr/196'
Remove unused exceptions, extract the rest to their files

- One class per file makes discoverability earlier
- This is a tiny step that lets us address [#68482028]