Merge lp:~andreserl/charms/quantal/hacluster/ceph-ra-installation into lp:~openstack-charmers/charms/quantal/hacluster/trunk

[Andres Rodriguez]

Always install ceph-resource-agents package. Otherwise there will be times on which the cluster will go to unmanaged mode because the resource agents will be installed after extra cluster nodes have been connected to the cluster. This is because it won't find the RA, making the cluster to go to unmanaged mode.

[James Page (james-page)]

The only other approach I could thing of was to install the required resource agents in the principle charm prior to sending data to the hacluster subordinate.

I see the issue and this approach would resolve it; we would then not need to change the hacluster charm to support new resource agents either.

review: Approve

[James Page (james-page)]

review: Abstain

[James Page (james-page)]

Sorry first comment should have been 'Comment' not approve.

I think this is probably what I saw when trying to use the openstack-resource-agents as well.

Unmerged revisions

14. By Andres Rodriguez on 2013-02-19

Update revision

13. By Andres Rodriguez on 2013-02-19

Always install ceph-resource-agents

=== modified file 'hooks/hooks.py'
--- hooks/hooks.py	2013-02-13 23:44:37 +0000
+++ hooks/hooks.py	2013-02-19 21:10:29 +0000
@@ -18,7 +18,7 @@
 def install():
     utils.juju_log('INFO', 'Begin install hook.')
     utils.configure_source()
-    utils.install('corosync', 'pacemaker', 'python-netaddr', 'ipmitool')
+    utils.install('corosync', 'pacemaker', 'python-netaddr', 'ipmitool', 'ceph-resource-agents')
     utils.juju_log('INFO', 'End install hook.')
 
 
@@ -200,9 +200,6 @@
     if True in [ra.startswith('ocf:openstack')
                 for ra in resources.itervalues()]:
         utils.install('openstack-resource-agents')
-    if True in [ra.startswith('ocf:ceph')
-                for ra in resources.itervalues()]:
-        utils.install('ceph-resource-agents')
 
     utils.juju_log('INFO', 'Configuring and restarting corosync')
     emit_corosync_conf()

=== modified file 'revision'
--- revision	2013-02-13 23:44:37 +0000
+++ revision	2013-02-19 21:10:29 +0000
@@ -1,1 +1,1 @@
-61
+62