Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ipa-healthcheck should include check for services which are important for IPA. #219

Closed
rcritten opened this issue Aug 9, 2021 · 0 comments · Fixed by #271
Closed

ipa-healthcheck should include check for services which are important for IPA. #219

rcritten opened this issue Aug 9, 2021 · 0 comments · Fixed by #271

Comments

@rcritten
Copy link
Collaborator

rcritten commented Aug 9, 2021

Cloned from BZ https://bugzilla.redhat.com/show_bug.cgi?id=1781100

ipa-server-4.8.0-11.module+el8.1.0+4247+9f3fd721.x86_64

How reproducible: Always

Steps to Reproduce:

  1. Install ipa server and ipa-healthcheck tool.
  2. Run ipa-healthcheck --source ipahealthcheck.meta.services

Actual results:
ipa-healthcheck doesn't include checks for the below services, which are important for IPA system to be in health state.

--check chronyd
--check ipa-ods-exporter
--check dbus.service

Expected results: It would be better to include checks for services which are required for proper functioning of IPA or if one of these service get affected they might indirectly affect IPA/its subcomponent.
rcritten added a commit to rcritten/freeipa-healthcheck that referenced this issue Jun 17, 2022
@rcritten
Add more services to check the status, switch to using roles
c24202b 
Some services aren't included in ipaplaform.knownservices
like smb and winbind. It is possible to discover the
service name using the same method used by ipactl, via roles.
Switch some services to the roles  method.

The services added:
* ods_enforcerd
* ipa_ods_exporter
* ipa_dnskeysyncd
* chronyd
* smb
* winbind

Add option to skip disabled services. This is currently only
for chronyd which is not required but we'll check it if its
enabled.

Fixes: freeipa#219
Fixes: freeipa#196

Signed-off-by: Rob Crittenden <rcritten@redhat.com>
@rcritten rcritten mentioned this issue Jun 17, 2022
Merged
rcritten added a commit to rcritten/freeipa-healthcheck that referenced this issue Jun 17, 2022
@rcritten
Add more services to check the status, switch to using roles
a7bb863 
Some services aren't included in ipaplaform.knownservices
like smb and winbind. It is possible to discover the
service name using the same method used by ipactl, via roles.
Switch some services to the roles  method.

The services added:
* ods_enforcerd
* ipa_ods_exporter
* ipa_dnskeysyncd
* chronyd
* smb
* winbind

Add option to skip disabled services. This is currently only
for chronyd which is not required but we'll check it if its
enabled.

Fixes: freeipa#219
Fixes: freeipa#196

Signed-off-by: Rob Crittenden <rcritten@redhat.com>
rcritten added a commit that referenced this issue Mar 28, 2023
@rcritten
Add more services to check the status, switch to using roles
31be12b 
Some services aren't included in ipaplaform.knownservices
like smb and winbind. It is possible to discover the
service name using the same method used by ipactl, via roles.
Switch some services to the roles  method.

The services added:
* ods_enforcerd
* ipa_ods_exporter
* ipa_dnskeysyncd
* chronyd
* smb
* winbind

Add option to skip disabled services. This is currently only
for chronyd which is not required but we'll check it if its
enabled.

Fixes: #219
Fixes: #196

Signed-off-by: Rob Crittenden <rcritten@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add more services to check the status, switch to using roles rcritten/freeipa-healthcheck
1 participant
@rcritten