[SRU] Clustered VGs require monitoring to be turned on

[Ante Karamatić]

[Impact]
This bug impacts all clusters with clustered LVM in 12.04. By default, Ubuntu ships LVM with monitoring disabled (even when it's enabled in lvm.conf). Cause of that, clustered VGs can not be activated. This patch makes sure VG is activated with monitoring enabled.

[Development Fix]
There is no fix in development branch, since it doesn't exist yet. OTOH, this is fixed upstream.

[Stable Fix]
Patch represents two upstream commits that address this issue. These changes are minimal and all done within LVM RA, which is a shell script.

[Test Case]
Test case requires setting up pacemaker cluster and clvm service on top of it. This procedure is too complex to be written within the bug report :/ FWIW, I've tested the patch multiple times.

[Regression Potential]
Patch affects only LVM RA, meaning that it can not produce side effects on any other service. Regression potential regarding LVM within the cluster is minimal, since activating LVM doesn't work at all now.

[Colin Watson (cjwatson)]

Looks good, thanks. I'll apply this to quantal first as 1:3.9.2-5ubuntu5.

[Colin Watson (cjwatson)]

Uploaded to quantal and precise-proposed. Unsubscribing ubuntu-sponsors since it may take a little while before this is SRU-approved but it doesn't need to wait on the sponsorship queue any more.

[Launchpad Janitor (janitor)]

This bug was fixed in the package resource-agents - 1:3.9.2-5ubuntu5

---------------
resource-agents (1:3.9.2-5ubuntu5) quantal; urgency=low

  * debian/patches/vgchange-monitor.patch:
    - apply two upstream commits:
      + 02c72cf9
      + 9dd12940
    - activating clustered VGs requires enabled monitoring
    - (LP: #988984)
 -- Ante Karamatic <email address hidden> Thu, 26 Apr 2012 09:06:57 +0200

[Colin Watson (cjwatson)]

Oh, sigh, this clashes with an existing SRU in precise-proposed. Could you please:

 1) apply the patch currently in precise-proposed to quantal, since it will now not be possible to copy it up automatically
 2) provide a rebased version of this patch against 1:3.9.2-5ubuntu4.1
 3) make it clearer in future when there are multiple things potentially in flight ...

[Bart Verwilst (verwilst)]

Any update on this?

[Ante Karamatić (ivoks)]

Since the version in quantal now contains both fixes, here's an updated version of the patch. This fixes issues 1) and 2) from comment 5. As for 3), I'll try better next time.

I'll work on a test case if no one is able to validate the fix (once it's in -proposed).

[Bart Verwilst (verwilst)]

Anyone cares to backport this to Precise?

[Steve Langasek (vorlon)]

The Precise Pangolin has reached end of life, so this bug will not be fixed for that release