389-ds-base

#50810 Issue 50806 - Fix minor issues in lib389 health checks
Closed by spichugi. Opened by mreynolds.
mreynolds/389-ds-base issue50809  into  master

Download 50810.patch

Description:

For permissions checks, add a list of permissions that is acceptable instead of single value.

For RI plugin attribute indexing checks, we now check if a container scope is specified. If it is set, we skip all the other backends that are not in the scope. This prevents false positives.

relates: https://pagure.io/389-ds-base/issue/50806

Wouldn't it suffice doing if perms not in ds_file['perms']: here instead of the loop above? Other than that this LGTM.

Wouldn't it suffice doing if perms not in ds_file['perms']: here instead of the loop above? Other than that this LGTM.

Thanks for fixing my terrible python code :-p

rebased onto caca64e7ee89e1f9ea63150f3dbfb5d1e8ea6b1a

Pull-Request has been merged by mreynolds

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 pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/3864

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.

Pull-Request has been closed by spichugi
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.