Skip to content

Issue 6497 - lib389 - Configure replication for multiple suffixes#6498

Merged
droideck merged 1 commit into389ds:mainfrom
droideck:repl-multi-suffixes
Jan 14, 2025
Merged

Issue 6497 - lib389 - Configure replication for multiple suffixes#6498
droideck merged 1 commit into389ds:mainfrom
droideck:repl-multi-suffixes

Conversation

@droideck
Copy link
Copy Markdown
Member

Bug Description: When trying to set up replication across multiple suffixes - particularly if one of those suffixes is a subsuffix - lib389 fails to properly configure the replication agreements, service accounts, and required groups. The references to the replication_managers group and service account naming do not correctly account for non-default additional suffixes.

Fix Description: Ensure replication DNs and credentials are correctly tied to each suffix.
Add a test (test_multi_subsuffix_replication) to verify multi-suffix replication across four suppliers.

Fixes: #6497

Reviewed: ?

@progier389
Copy link
Copy Markdown
Contributor

Change looks incomplete: lots of replication test are now failing.

@droideck droideck force-pushed the repl-multi-suffixes branch 2 times, most recently from 5eccbf5 to f3f3261 Compare January 14, 2025 04:12
Bug Description: When trying to set up replication across multiple suffixes -
particularly if one of those suffixes is a subsuffix - lib389 fails to properly
configure the replication agreements, service accounts, and required groups.
The references to the replication_managers group and service account
naming do not correctly account for non-default additional suffixes.

Fix Description: Ensure replication DNs and credentials are correctly tied to each suffix.
Enable DSLdapObject.present method to compare values as
a normalized DNs if they are DNs.
Add a test (test_multi_subsuffix_replication) to verify multi-suffix
replication across four suppliers.
Fix tests that are related to repl service accounts.

Fixes: 389ds#6497

Reviewed: ?
@droideck droideck force-pushed the repl-multi-suffixes branch from f3f3261 to 425e0ab Compare January 14, 2025 15:09
Copy link
Copy Markdown
Contributor

@progier389 progier389 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Much better! 😀
LGTM

@droideck
Copy link
Copy Markdown
Member Author

Thanks! Yeah, I first fixed the main functionality and posted it for a preliminary review (and went to sleep).
The next day, I continued to fix the tests. My mistake is that I didn't add 'work in progress' :)

@droideck droideck merged commit eed4305 into 389ds:main Jan 14, 2025
droideck added a commit that referenced this pull request Jan 14, 2025
)

Bug Description: When trying to set up replication across multiple suffixes -
particularly if one of those suffixes is a subsuffix - lib389 fails to properly
configure the replication agreements, service accounts, and required groups.
The references to the replication_managers group and service account
naming do not correctly account for non-default additional suffixes.

Fix Description: Ensure replication DNs and credentials are correctly tied to each suffix.
Enable DSLdapObject.present method to compare values as
a normalized DNs if they are DNs.
Add a test (test_multi_subsuffix_replication) to verify multi-suffix
replication across four suppliers.
Fix tests that are related to repl service accounts.

Fixes: #6497

Reviewed: @progier389 (Thanks!)
droideck added a commit that referenced this pull request Jan 14, 2025
)

Bug Description: When trying to set up replication across multiple suffixes -
particularly if one of those suffixes is a subsuffix - lib389 fails to properly
configure the replication agreements, service accounts, and required groups.
The references to the replication_managers group and service account
naming do not correctly account for non-default additional suffixes.

Fix Description: Ensure replication DNs and credentials are correctly tied to each suffix.
Enable DSLdapObject.present method to compare values as
a normalized DNs if they are DNs.
Add a test (test_multi_subsuffix_replication) to verify multi-suffix
replication across four suppliers.
Fix tests that are related to repl service accounts.

Fixes: #6497

Reviewed: @progier389 (Thanks!)
droideck added a commit that referenced this pull request Jan 14, 2025
)

Bug Description: When trying to set up replication across multiple suffixes -
particularly if one of those suffixes is a subsuffix - lib389 fails to properly
configure the replication agreements, service accounts, and required groups.
The references to the replication_managers group and service account
naming do not correctly account for non-default additional suffixes.

Fix Description: Ensure replication DNs and credentials are correctly tied to each suffix.
Enable DSLdapObject.present method to compare values as
a normalized DNs if they are DNs.
Add a test (test_multi_subsuffix_replication) to verify multi-suffix
replication across four suppliers.
Fix tests that are related to repl service accounts.

Fixes: #6497

Reviewed: @progier389 (Thanks!)
droideck added a commit that referenced this pull request Jan 14, 2025
)

Bug Description: When trying to set up replication across multiple suffixes -
particularly if one of those suffixes is a subsuffix - lib389 fails to properly
configure the replication agreements, service accounts, and required groups.
The references to the replication_managers group and service account
naming do not correctly account for non-default additional suffixes.

Fix Description: Ensure replication DNs and credentials are correctly tied to each suffix.
Enable DSLdapObject.present method to compare values as
a normalized DNs if they are DNs.
Add a test (test_multi_subsuffix_replication) to verify multi-suffix
replication across four suppliers.
Fix tests that are related to repl service accounts.

Fixes: #6497

Reviewed: @progier389 (Thanks!)
droideck added a commit that referenced this pull request Jan 14, 2025
)

Bug Description: When trying to set up replication across multiple suffixes -
particularly if one of those suffixes is a subsuffix - lib389 fails to properly
configure the replication agreements, service accounts, and required groups.
The references to the replication_managers group and service account
naming do not correctly account for non-default additional suffixes.

Fix Description: Ensure replication DNs and credentials are correctly tied to each suffix.
Enable DSLdapObject.present method to compare values as
a normalized DNs if they are DNs.
Add a test (test_multi_subsuffix_replication) to verify multi-suffix
replication across four suppliers.
Fix tests that are related to repl service accounts.

Fixes: #6497

Reviewed: @progier389 (Thanks!)
droideck added a commit that referenced this pull request Jan 14, 2025
)

Bug Description: When trying to set up replication across multiple suffixes -
particularly if one of those suffixes is a subsuffix - lib389 fails to properly
configure the replication agreements, service accounts, and required groups.
The references to the replication_managers group and service account
naming do not correctly account for non-default additional suffixes.

Fix Description: Ensure replication DNs and credentials are correctly tied to each suffix.
Enable DSLdapObject.present method to compare values as
a normalized DNs if they are DNs.
Add a test (test_multi_subsuffix_replication) to verify multi-suffix
replication across four suppliers.
Fix tests that are related to repl service accounts.

Fixes: #6497

Reviewed: @progier389 (Thanks!)
droideck added a commit that referenced this pull request Jan 14, 2025
)

Bug Description: When trying to set up replication across multiple suffixes -
particularly if one of those suffixes is a subsuffix - lib389 fails to properly
configure the replication agreements, service accounts, and required groups.
The references to the replication_managers group and service account
naming do not correctly account for non-default additional suffixes.

Fix Description: Ensure replication DNs and credentials are correctly tied to each suffix.
Enable DSLdapObject.present method to compare values as
a normalized DNs if they are DNs.
Add a test (test_multi_subsuffix_replication) to verify multi-suffix
replication across four suppliers.
Fix tests that are related to repl service accounts.

Fixes: #6497

Reviewed: @progier389 (Thanks!)
droideck added a commit that referenced this pull request Jan 14, 2025
)

Bug Description: When trying to set up replication across multiple suffixes -
particularly if one of those suffixes is a subsuffix - lib389 fails to properly
configure the replication agreements, service accounts, and required groups.
The references to the replication_managers group and service account
naming do not correctly account for non-default additional suffixes.

Fix Description: Ensure replication DNs and credentials are correctly tied to each suffix.
Enable DSLdapObject.present method to compare values as
a normalized DNs if they are DNs.
Add a test (test_multi_subsuffix_replication) to verify multi-suffix
replication across four suppliers.
Fix tests that are related to repl service accounts.

Fixes: #6497

Reviewed: @progier389 (Thanks!)
droideck added a commit that referenced this pull request Jan 14, 2025
)

Bug Description: When trying to set up replication across multiple suffixes -
particularly if one of those suffixes is a subsuffix - lib389 fails to properly
configure the replication agreements, service accounts, and required groups.
The references to the replication_managers group and service account
naming do not correctly account for non-default additional suffixes.

Fix Description: Ensure replication DNs and credentials are correctly tied to each suffix.
Enable DSLdapObject.present method to compare values as
a normalized DNs if they are DNs.
Add a test (test_multi_subsuffix_replication) to verify multi-suffix
replication across four suppliers.
Fix tests that are related to repl service accounts.

Fixes: #6497

Reviewed: @progier389 (Thanks!)
@droideck droideck deleted the repl-multi-suffixes branch January 15, 2026 05:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cannot configure replication for multiple suffixes via ReplicationManager in lib389

2 participants