Bug 52122 - Remove Nagios in UCS 5.0
Remove Nagios in UCS 5.0
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: Monitoring (Prometheus or Nagios)
UCS 5.0
Other Linux
: P5 normal (vote)
: UCS 5.0
Assigned To: Arvid Requate
Florian Best
: interim-1
Depends on: 52991 52992 53008 53009
Blocks:
  Show dependency treegraph
 
Reported: 2020-09-21 11:11 CEST by Florian Best
Modified: 2021-05-25 16:01 CEST (History)
4 users (show)

See Also:
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:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Florian Best univentionstaff 2020-09-21 11:11:48 CEST
There are no nagios packages in debian buster.
We should either remove the integration or provide packages.

For now, we have to remove nagios from the dependencies.
Comment 1 Ingo Steuwer univentionstaff 2020-09-21 13:39:35 CEST
Current status is: there will be no Nagios App in UCS 5.0, features will be merged into the UCS Dashboard. We will need NRPE to do so.
Comment 2 Florian Best univentionstaff 2020-09-22 09:31:49 CEST
So far, the following commits have been done:

100814cf897e | Bug #52122: remove dependency on nagios

univention-demo-configuration (8.0.0-2)
1aa4a4d943bf | Bug #52122: Remove nagios from univention-demo-configuration
Comment 3 Arvid Requate univentionstaff 2021-01-25 21:20:12 CET
4583940e34 Remove univention-nagios-client from Recommends in univention-server
Comment 4 Florian Best univentionstaff 2021-01-25 21:27:51 CET
(In reply to Arvid Requate from comment #3)
> 4583940e34 Remove univention-nagios-client from Recommends in
> univention-server

Why?
I think we have some packages requiring this because of /usr/lib/nagios/plugins/check_univention_replication.
Comment 5 Arvid Requate univentionstaff 2021-01-25 22:05:09 CET
Yeah probably you are right.

007dea45e1 revert
Comment 6 Florian Best univentionstaff 2021-02-05 12:43:27 CET
(In reply to Arvid Requate from comment #5)
> Yeah probably you are right.
> 
> 007dea45e1 revert

It seems only ucs-test needs it.
Comment 7 Florian Best univentionstaff 2021-02-05 12:46:06 CET
Ignore package to exists in test/scenarios/install-testing/master-all-components.cfg:
034a0a70c38d | Bug #52122: ignore univention-nagios-server
Comment 8 Jürn Brodersen univentionstaff 2021-02-22 22:58:14 CET
I removed nagios from the running services check

[5.0-0 9e8ce92fe4] Bug #52122: remove nagios test 01_base/09services_other_running
Comment 9 Philipp Hahn univentionstaff 2021-03-26 16:44:57 CET
[5.0-0] 01ab68eaf6 fix[dvd]: Re-add Nagios checks to DVD
 base/univention-dvd/debian/changelog         |  6 ++++++
 base/univention-dvd/tasks/ucs500/task-ucs500 | 11 +++++------
 2 files changed, 11 insertions(+), 6 deletions(-)

Package: univention-dvd
Version: 5.0.1-2A~5.0.0.202103261642

TODO: Build a new DVD ...
Comment 11 Arvid Requate univentionstaff 2021-04-20 15:08:02 CEST
32b60b0362 | Remove Nagios in UCS 5.0
c95a678dfd | Bug #52122: Remove univention-nagios-server


Package: univention-nagios
Version: 13.0.2-1A~5.0.0.202104201449
Branch: ucs_5.0-0

Package: univention-server
Version: 15.0.4-1A~5.0.0.202104201505
Branch: ucs_5.0-0
Comment 12 Florian Best univentionstaff 2021-04-29 12:25:25 CEST
The check for squid and cups have been removed. They need to be readded:

> -                       univention-directory-manager nagios/service modify "$@" \
> -                               --dn "cn=UNIVENTION_SQUID,cn=nagios,$ldap_base" \
> -                               --append assignedHosts="${ldap_hostdn}" || die

> -                       univention-directory-manager nagios/service modify "$@" \
> -                               --dn "cn=UNIVENTION_CUPS,cn=nagios,$ldap_base" \
> -                               --append assignedHosts="${ldap_hostdn}"


We also need correct removal code, e.g. `--remove assignedHosts="${ldap_hostdn}"` for all checks.

We have to revert Bug #52991 and Bug #52992.
Comment 13 Arvid Requate univentionstaff 2021-05-05 17:22:13 CEST
ad13749bcc | fixup! Bug #52122: Remove Nagios in UCS 5.0
6826ca7d67 | Cleanup univention-nagios-client.postinst
e59d9a6648 | add nagios/service unregister code to univention-printserver
5af3e3fd20 | fix nagios/service unregister code in univention-squid
3cac4d6c32 | debian/changelog univention-printserver
72643c4334 | debian/changelog univention-squid
b2b50dd2c2 | Remove Nagios Server from the manual

Package: univention-squid
Version: 13.0.3-2A~5.0.0.202105051623
Branch: ucs_5.0-0

Package: univention-printserver
Version: 13.0.2-9A~5.0.0.202105051621
Branch: ucs_5.0-0
Comment 14 Florian Best univentionstaff 2021-05-06 18:19:04 CEST
OK: removal of univention-nagios-server
OK: removal from manual
OK: removal from scenarios
OK: changelog entry
Comment 15 Florian Best univentionstaff 2021-05-25 16:01:19 CEST
UCS 5.0 has been released:
 https://docs.software-univention.de/release-notes-5.0-0-en.html
 https://docs.software-univention.de/release-notes-5.0-0-de.html

If this error occurs again, please use "Clone This Bug".