#49985 During MODRDN, memberof may silently fails to update a member
Closed: wontfix 5 years ago by mreynolds. Opened 5 years ago by tbordaz.

Issue Description

Memberof plugin updates group members to add 'memberof' attribute.
It case of schema violation, it adds the 'memberOfAutoAddOC' objectclass to the member and retry. If after adding 'memberOfAutoAddOC' it fails again it propagates the schema violation but at some point the return code is lost and not reported.
As a consequence not all members have 'memberof' attribute

Package Version and Platform

At least since 1.3.6

Steps to reproduce

testcase provided

Actual results

When the update on members is failing, the triggering update of the group succeeds

Expected results

When the update on members is failing, the triggering update of the group should fail


Metadata Update from @tbordaz:
- Issue assigned to tbordaz

5 years ago

I thought this would dhave been corrected by having the autoaddOC always a valid OC? Should we do a schema check on the config validation step so we never get an invalid state, and we require always having an OC (such as nsMemberOf)?

Metadata Update from @firstyear:
- Custom field component adjusted to None
- Custom field origin adjusted to None
- Custom field reviewstatus adjusted to None
- Custom field type adjusted to None
- Custom field version adjusted to None

5 years ago

@firstyear, autoaddOC works just fine. I used it as a fault injector.

Adding an incorrect OC triggers an error and surprisingly addition of memberof fails but the external update (add a member) succeeds.
The problem is that the failure is not propagated.

So ... how do we trigger this then? It sounds like adding an incorrect autoAddOC is the issue here somehow, so is there some validation step we miss? Maybe we need a simple lib389 test so I can understand the problem better. Either way, I think we should fix it ;)

@tbordaz - can you please cherry pick this to 1.3.9 ?

Metadata Update from @mreynolds:
- Issue set to the milestone: 1.3.9

5 years ago

Metadata Update from @mreynolds:
- Issue close_status updated to: fixed
- Issue status updated to: Closed (was: Open)

5 years ago

Metadata Update from @tbordaz:
- Custom field rhbz adjusted to https://bugzilla.redhat.com/show_bug.cgi?id=1710961

4 years ago

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/389ds/389-ds-base/issues/3044

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Metadata Update from @spichugi:
- Issue close_status updated to: wontfix (was: fixed)

3 years ago

Login to comment on this ticket.

Metadata
Attachments 1
Attached 5 years ago View Comment