Ubuntu
openldap package

[Lucid] backport fix for ITS#6458

Bug #903901 reported by Peter Matulis on 2011-12-13

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	openldap (Ubuntu)	Fix Released	Medium	Unassigned
	Lucid	Fix Released	Medium	Scott Moser

Bug Description

The problem:

Replication fails if an attribute of a multi-value attribute entry is removed if the attribute does not have a equality matching rule defined in the schema.

The problem can be mitigated temporarily by a doing a full re-sync by removing the affected database and restarting the service to trigger a full sync.

This is fixed upstream beginning with the slapd 2.4.25 branch:

openldap-2.4.25/CHANGES: Fixed slapd syncrepl for attributes with no matching rule (ITS#6458)

For Ubuntu, this translates to the fix appearing in Oneiric:

I am requesting the fix be backported at least to Lucid.

Upstream bug:

http://www.openldap.org/its/index.cgi/Software%20Bugs?id=6458;selectid=6458;usearchives=1

SRU Justification:

[Impact]: Serious. Replication, when used, is there for important reasons. It should not break due to a schema, even if poorly designed.
[Development Fix]: Fixed in an upstream release and thus already present since Oneiric
[Stable Fix]: lucid-proposed.debdiff, attached - patches taken from upstream git repository
[Test Case]: Attached as test_case.txt (from upstream bug; for consistency)
[Regression Potential]: looks to be minimal from looking at the patch. There appears to be a comprehensive test suite that continues to pass with this patch applied.

See original description

Tags:

Related branches

lp:ubuntu/lucid-proposed/openldap

Revision history for this message

Robie Basak (racb) wrote on 2011-12-14:

lucid-proposed.debdiff Edit (9.4 KiB, text/plain)

Revision history for this message

Robie Basak (racb) wrote on 2011-12-14:

Peter, could you please complete the SRU justification? Guidelines at https://wiki.ubuntu.com/StableReleaseUpdates#Procedure - thanks!

description:

updated

Revision history for this message

Ubuntu Foundations Team Bug Bot (crichton) wrote on 2011-12-14:

The attachment "lucid-proposed.debdiff" of this bug report has been identified as being a patch in the form of a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-sponsors team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

tags:

added: patch

Revision history for this message

Peter Matulis (petermatulis) wrote on 2011-12-15:

SRU test case Edit (2.1 KiB, text/plain)

description:

updated

Revision history for this message

Peter Matulis (petermatulis) wrote on 2011-12-15:

@Robie

All done.

Scott Moser (smoser) on 2011-12-15

Changed in openldap (Ubuntu):
importance:	Undecided → Medium
status:	New → Fix Released
Changed in openldap (Ubuntu Lucid):
status:	New → Triaged
importance:	Undecided → Medium

Revision history for this message

Scott Moser (smoser) wrote on 2011-12-15:

I started a local build, when it finishes I will publish.
thanks for the well done patch, Robbie and Matulis.

Changed in openldap (Ubuntu Lucid):
assignee:	nobody → Scott Moser (smoser)
status:	Triaged → In Progress

Revision history for this message

Martin Pitt (pitti) wrote on 2011-12-19: Please test proposed package

Hello Peter, or anyone else affected,

Accepted openldap into lucid-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in openldap (Ubuntu Lucid):
status:	In Progress → Fix Committed
tags:	added: verification-needed

Revision history for this message

Robie Basak (racb) wrote on 2012-01-05:

Peter, any news on testing -proposed? This is needed to get this into lucid-updates.

Revision history for this message

Peter Matulis (petermatulis) wrote on 2012-01-11:

No news yet.

Revision history for this message

Peter Matulis (petermatulis) wrote on 2012-02-13:

#10

Verified on my side. Thanks.

Martin Pitt (pitti) on 2012-02-13

tags:

added: verification-done
removed: verification-needed

Revision history for this message

Launchpad Janitor (janitor) wrote on 2012-02-13:

#11

This bug was fixed in the package openldap - 2.4.21-0ubuntu5.7

---------------
openldap (2.4.21-0ubuntu5.7) lucid-proposed; urgency=low

  * Fix replication when attr has no matching rule (LP: #903901):
    - debian/patches/fix-syncrepl-when-attr-has-no-matching-rule.patch:
      backport fix from upstream
    - debian/patches/fix-syncrepl-when-attr-has-no-matching-rule-test.patch:
      backport test from upstream
-- Robie Basak <email address hidden> Wed, 14 Dec 2011 14:05:18 +0000