Bug 50599
| Summary: | Add plugin to umcm-diagnostic to check if memberOf is activated | ||
|---|---|---|---|
| Product: | UCS | Reporter: | Felix Botner <botner> |
| Component: | General | Assignee: | Felix Botner <botner> |
| Status: | CLOSED FIXED | QA Contact: | Erik Damrose <damrose> |
| Severity: | normal | ||
| Priority: | P5 | CC: | best, damrose |
| Version: | UCS 4.4 | ||
| Target Milestone: | UCS 4.4-3-errata | ||
| Hardware: | Other | ||
| OS: | Linux | ||
| What kind of report is it?: | Development Internal | What type of bug is this?: | --- |
| Who will be affected by this bug?: | --- | How will those affected feel about the bug?: | --- |
| User Pain: | Enterprise Customer affected?: | ||
| School Customer affected?: | ISV affected?: | ||
| Waiting Support: | Flags outvoted (downgraded) after PO Review: | ||
| Ticket number: | Bug group (optional): | ||
| Max CVSS v3 score: | |||
|
Description
Felix Botner
A system check in the UMC recognizes if the memberOf overlay is disabled and links to the SDB article in which the activation is documented. The text refers to the fact that there are services that use this overlay, which is the default for new installations. added umc/python/diagnostic/plugins/58_check_memberOf.py OK: Check if UCR ldap/overlay/memberof is true. That does not check if the referenced article https://help.univention.com/t/6439 has been completed (i.e. if univention-update-memberof has actually been run), but it is a good indicator for this check. OK: yaml Reopen: When clicking the 'Test again' button, the UCR instance is not reloaded, so the current state is not checked and reflected in the frontend. Proposed patch: @@ -48,6 +48,7 @@ links = [{ def run(_umc_instance): if util.is_service_active('LDAP'): + ucr.load() if not ucr.is_true('ldap/overlay/memberof'): MODULE.error(warning_message) raise Warning(description=warning_message) (In reply to Erik Damrose from comment #3) > OK: Check if UCR ldap/overlay/memberof is true. That does not check if the > referenced article https://help.univention.com/t/6439 has been completed > (i.e. if univention-update-memberof has actually been run), but it is a good > indicator for this check. > OK: yaml > > Reopen: When clicking the 'Test again' button, the UCR instance is not > reloaded, so the current state is not checked and reflected in the frontend. > Proposed patch: > > @@ -48,6 +48,7 @@ links = [{ > > def run(_umc_instance): > if util.is_service_active('LDAP'): > + ucr.load() > if not ucr.is_true('ldap/overlay/memberof'): > MODULE.error(warning_message) > raise Warning(description=warning_message) OK, fixed OK: "Test again" button in UMC works. OK: yaml Verified |