if a group is moved into a defined memberof scope the memberof attr of its members are not added
all
will attach CI
test case <img alt="ticket49161_test.py" src="/389-ds-base/issue/raw/files/e6e232fdb80de41c2488734ceecfa8f99753dc525d24db856441bc9e82dacca7-ticket49161_test.py" />
Metadata Update from @lkrispen: - Custom field type adjusted to defect
potential fix
<img alt="0001-fix-for-ticket-49161.patch" src="/389-ds-base/issue/raw/files/d3e4d156c3c40743957388be3e5eb8142d3fed7a82f5f10dce67f2e516480fac-0001-fix-for-ticket-49161.patch" />
Looks good, ack
Metadata Update from @mreynolds: - Custom field reviewstatus adjusted to ack
Metadata Update from @mreynolds: - Issue set to the milestone: 1.3.6.0
Ludwig, just a reminder that I acked this fix.
You can push this Ludwig :-)
Metadata Update from @mreynolds: - Issue set to the milestone: 1.3.7.0 (was: 1.3.6.0)
and I didn't, and now it is missing in the conflict tests :-(
too bad, it is not a blocker, but we should not forget it and include in the first batch update, need Bbz as well
Metadata Update from @lkrispen: - Custom field component adjusted to None - Custom field origin adjusted to None - Custom field version adjusted to None
I'll clone this to bugzilla later today. FYI the current patch is full of tabs, just make sure that gets cleaned up before pushing. Thank you!!!
<img alt="0001-Ticket-49161-memberof-fails-if-group-is-moved-into-s.patch" src="/389-ds-base/issue/raw/files/3cae0955ea1fa65d8a120c9fb680cc64cae9f159f5a131fbeb8b467da48679c8-0001-Ticket-49161-memberof-fails-if-group-is-moved-into-s.patch" />
rebased patch, committed to master
To ssh://pagure.io/389-ds-base.git c6d8726..09f4003 master -> master
Can you also please commit this to 1.3.7? Thanks!
To ssh://pagure.io/389-ds-base.git 4fdae3c..abdde67 389-ds-base-1.3.7 -> 389-ds-base-1.3.7
Metadata Update from @mreynolds: - Custom field rhbz adjusted to https://bugzilla.redhat.com/show_bug.cgi?id=1551071
Issue linked to Bugzilla: Bug 1551071
Metadata Update from @mreynolds: - Issue close_status updated to: fixed - Issue status updated to: Closed (was: Open)
@spichugi Please review https://pagure.io/389-ds-base/pull-request/49799 for the CI test case.
@vashirov @spichugi please review https://pagure.io/389-ds-base/pull-request/49852 for CI test.
New one - https://pagure.io/389-ds-base/c/98ad13daa0c2df24244fdcdc1d054c5bd3459031 @spichugi
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/2220
If you want to receive further updates on the issue, please navigate to the github issue and click on subscribe button.
subscribe
Thank you for understanding. We apologize for all inconvenience.
Metadata Update from @spichugi: - Issue close_status updated to: wontfix (was: fixed)
Log in to comment on this ticket.