Merge lp://qastaging/~chad.smith/charms/precise/block-storage-broker/bsb-use-charmhelpers-to-set-openstack-origin into lp://qastaging/charms/block-storage-broker

Proposed by Chad Smith
Status: Work in progress
Proposed branch: lp://qastaging/~chad.smith/charms/precise/block-storage-broker/bsb-use-charmhelpers-to-set-openstack-origin
Merge into: lp://qastaging/charms/block-storage-broker
Diff against target: 3846 lines (+3261/-146)
23 files modified
Makefile (+10/-7)
charm-helpers.yaml (+2/-0)
config.yaml (+15/-0)
hooks/charmhelpers/contrib/openstack/alternatives.py (+17/-0)
hooks/charmhelpers/contrib/openstack/amulet/deployment.py (+61/-0)
hooks/charmhelpers/contrib/openstack/amulet/utils.py (+275/-0)
hooks/charmhelpers/contrib/openstack/context.py (+789/-0)
hooks/charmhelpers/contrib/openstack/ip.py (+79/-0)
hooks/charmhelpers/contrib/openstack/neutron.py (+201/-0)
hooks/charmhelpers/contrib/openstack/templates/__init__.py (+2/-0)
hooks/charmhelpers/contrib/openstack/templating.py (+279/-0)
hooks/charmhelpers/contrib/openstack/utils.py (+459/-0)
hooks/charmhelpers/contrib/storage/linux/ceph.py (+387/-0)
hooks/charmhelpers/contrib/storage/linux/loopback.py (+62/-0)
hooks/charmhelpers/contrib/storage/linux/lvm.py (+88/-0)
hooks/charmhelpers/contrib/storage/linux/utils.py (+53/-0)
hooks/charmhelpers/core/hookenv.py (+129/-6)
hooks/charmhelpers/core/host.py (+81/-12)
hooks/charmhelpers/fetch/__init__.py (+191/-74)
hooks/charmhelpers/fetch/archiveurl.py (+56/-1)
hooks/charmhelpers/fetch/bzrurl.py (+2/-1)
hooks/hooks.py (+4/-4)
hooks/test_hooks.py (+19/-41)
To merge this branch: bzr merge lp://qastaging/~chad.smith/charms/precise/block-storage-broker/bsb-use-charmhelpers-to-set-openstack-origin
Reviewer Review Type Date Requested Status
David Britton (community) Needs Fixing
Review via email: mp+231594@code.qastaging.launchpad.net

Description of the change

This branch avoids making a static call to charmhelpers' fetch.add_source("cloud-archive:havana") to add cloud archives in favor the a more flexible openstack charms approach as the former approach breaks on newer distribution series (such as trusty).

 This branch is quite sizeable because of pulling in the charmhelpers.contrib.openstack module. The changes to the block-storage-broker are as follows:
  1. sync new charmhelpers dependencies
      - minor Makefile sync target added for simplified charmhelper updates
      - charm-helpers.yaml to define new charmhelpers dependencies contrib.openstack and contrib.storage
      - new files sync'd under charmhelpers (not authored in this branch)

  2. config.yaml has a new openstack-origin parameter that will default the cloud archive repository to the supported distro default but will allow a user to set their own custom cloud archive repository if needed

  3. hooks/hooks.py drop use of fetch.add_source("cloud:havana") instead use charmhelpers.openstack.utils.configure_installation_source()

  4. fix unit tests

The relevant changes that exclude the charmhelpers.contrib directory sync are linked here for quick reference.
http://pastebin.ubuntu.com/8099398/

To post a comment you must log in.
Revision history for this message
David Britton (dpb) wrote :

Hi Chad -- Thanks for this MP!

I don't see any reason why this would be controversial, please clear the merge conflict with trunk and I'll review and commit this straightaway.

review: Needs Fixing
62. By Chad Smith

merge block-storage-broker trunk resolve conflicts and fix unit tests to avoid mocker use

Unmerged revisions

62. By Chad Smith

merge block-storage-broker trunk resolve conflicts and fix unit tests to avoid mocker use

61. By Chad Smith

correct yaml indent in config.yaml

60. By Chad Smith

update unit tests to validate use of charmhelpers config_installation_source

59. By Chad Smith

update charmhelpers sync functionality

58. By Chad Smith

add openstack-origin to config.yaml options and use charmhelpers configure_installation_source to pull appropriate deb packages for a given ubuntu series

57. By Chad Smith

sync added contrib.(storage|openstack) files

56. By Chad Smith

add contrib.openstack and it's dependency contrib.storage to charm-helpers.yaml file

55. By Chad Smith

sync existing charmhelpers dependencies

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: