Bug #11683
closed
Process management: stopped sends multiple reports.
Description
I have tried to stop a service and ensure it's stopped / not started with the Rudder Technique.
Filled fields were:
Service name (as seen in chkconfig or systemctl, or Windows tasks): [ fail2ban ] This process should be: (o) Stopped This service should be set to start on boot: (o) No When stopping this process: (o) Stop gracefully (service xxx stop)
The field Process name (full path to the executable, to be check in 'ps' output) was not filled in because the UI did not complain about it not being set, but if that is the case, there appears to be a reporting error:
fail2ban [Unexpected] Process of fail2ban was stopped [Unexpected] Cannot check the status of fail2ban: the process name is not filed
After filling in the Process name field, I got a 100% green compliance.
The Technique should either require the field to be filled or not send double reports.
(version rudder-agent-4.1.8.release-1.SLES.11)
Updated by
Updated by Benoît PECCATTE over 6 years ago
- Effort required set to Small
- Priority changed from 50 to 66
Updated by Benoît PECCATTE about 6 years ago
- Priority changed from 66 to 64
This happens when the process range is filled, because the number of processes is counted via ps.
I think we should clearly state this in the help.
Moreover the "was stopped" message should not be unexpected
Was the "process range" field filled in your directive ?
Updated by Benoît PECCATTE almost 6 years ago
- Assignee set to Félix DALLIDET
- Priority changed from 64 to 60
Updated by Félix DALLIDET almost 6 years ago
- Target version set to 4.1.13
The technique should not check if the process is active or not if the field name was not filled and should be reporting an error if:
-the name field is empty and "Running" or "Stopped" are selected.
If you do not fill the field name, "Don't change" option should be selected and the technique should report a compliant report.
I will put "Don't change" as the default option.
Updated by Félix DALLIDET almost 6 years ago
- Status changed from New to In progress
Updated by Félix DALLIDET almost 6 years ago
- Status changed from In progress to Pending technical review
- Assignee changed from Félix DALLIDET to Nicolas CHARLES
- Pull Request set to https://github.com/Normation/rudder-techniques/pull/1309
Updated by Félix DALLIDET almost 6 years ago
The technique behaviour is very confusing, changing the sections display may help to understand how it works.
Updated by Félix DALLIDET almost 6 years ago
- Status changed from Pending technical review to Pending release
Applied in changeset rudder-techniques|c90641ca01177e090a1a9be96e0b2bfe4943905e.
Updated by Anonymous almost 6 years ago
Applied in changeset rudder-techniques|e29fe8ce09a2d89c315b20c69111ca6c3e360e92.
Updated by Vincent MEMBRÉ almost 6 years ago
- Status changed from Pending release to Released
- Priority changed from 60 to 58