Juju Charms Collection
hacluster package

Merge lp://staging/~freyes/charms/trusty/hacluster/add-location into lp://staging/~openstack-charmers/charms/trusty/hacluster/next

Proposed by Felipe Reyes on 2015-03-05

Status:

Superseded

Proposed branch:

lp://staging/~freyes/charms/trusty/hacluster/add-location

Merge into:

lp://staging/~openstack-charmers/charms/trusty/hacluster/next

Diff against target:

137 lines (+101/-0)

4 files modified

.bzrignore (+1/-0)
hooks/hooks.py (+10/-0)
setup.cfg (+6/-0)
unit_tests/test_hacluster_hooks.py (+84/-0)

To merge this branch:

bzr merge lp://staging/~freyes/charms/trusty/hacluster/add-location

High

Fix Released

Link a bug report

Reviewer	Review Type	Date Requested	Status
Jorge Niedbalski (community)		2015-03-05	Needs Fixing on 2015-03-05
Review via email: mp+252022@code.staging.launchpad.net

This proposal has been superseded by a proposal from 2015-03-06.

Description of the change

Add 'location' parameter to define location rules[0]. Fixes bug #1428850

[0] http://clusterlabs.org/doc/en-US/Pacemaker/1.1-crmsh/html/Clusters_from_Scratch/_specifying_a_preferred_location.html

Revision history for this message

Jorge Niedbalski (niedbalski) wrote on 2015-03-05:

Felipe,

Thanks for this patch. Mind to add a test for covering your change?

review: Needs Fixing

Revision history for this message

uosci-testing-bot (uosci-testing-bot) wrote on 2015-03-05:

charm_lint_check #2488 hacluster-next for freyes mp252022
LINT OK: passed

Build: http://10.245.162.77:8080/job/charm_lint_check/2488/

Revision history for this message

uosci-testing-bot (uosci-testing-bot) wrote on 2015-03-05:

charm_unit_test #2278 hacluster-next for freyes mp252022
UNIT OK: passed

Build: http://10.245.162.77:8080/job/charm_unit_test/2278/

Revision history for this message

uosci-testing-bot (uosci-testing-bot) wrote on 2015-03-05:

charm_amulet_test #2360 hacluster-next for freyes mp252022
AMULET FAIL: amulet-test missing

AMULET Results (max last 2 lines):
INFO:root:Search string not found in makefile target commands.
ERROR:root:No make target was executed.

Full amulet test output: http://paste.ubuntu.com/10543782/
Build: http://10.245.162.77:8080/job/charm_amulet_test/2360/

lp://staging/~freyes/charms/trusty/hacluster/add-location updated on 2015-03-06

43. By Felipe Reyes on 2015-03-06

Add unit test for configure_principle_cluster_resources()

This is the handler for ha-relation-joined, ha-relation-changed,
hanode-relation-joined and hanode-relation-changed

44. By Felipe Reyes on 2015-03-06

Drop str() when logging parameters

log() already takes care of calling repr() when needed

Unmerged revisions

44. By Felipe Reyes on 2015-03-06

Drop str() when logging parameters

log() already takes care of calling repr() when needed

43. By Felipe Reyes on 2015-03-06

Add unit test for configure_principle_cluster_resources()

This is the handler for ha-relation-joined, ha-relation-changed,
hanode-relation-joined and hanode-relation-changed

42. By Felipe Reyes on 2015-03-05

Add support to define location rules in pcmkr

Fixes bug #1428850

41. By Liam Young on 2015-01-26

Fix charm-helpers.yaml and charmhelper sync

40. By Edward Hope-Morley on 2014-12-17

[hopem,r=niedbalski,tinoco,wolsen]

Don't upgrade pacemaker packages. This restriction
can be removed once 1382842 is fixed but for the
moment we can't allow upgrades due to issues with
pacemaker dependencies.

Closes-Bug: 1382842

39. By Edward Hope-Morley on 2014-12-15

[freyes,r=hopem]

Add debug option to config.yaml and optionally enable
debug logging in corosync.conf

38. By James Page on 2014-12-15

Merge in bundle tests from master charm

37. By Edward Hope-Morley on 2014-12-08

[wolsen,r=hopem]

Specify the vote count for multicast quorum to be the number of nodes
which are configured. Additionally, specify the two_node value in the
quorum section when there are 2 nodes configured.

Closes-Bug: 1394008

36. By Edward Hope-Morley on 2014-12-08