Updated by mfriedrich on 2015-04-27 14:47:42 00:00

• Status changed from New to Assigned
• Assigned to set to dgoetz
• Target Version set to 2.4.0

Updated by dgoetz on 2015-04-28 17:24:52 00:00

• Assigned to changed from dgoetz to mfrosch

Added vim-icinga2 and nano-icinga2 to the spec and changed the corresponding documentation for enabling it.

Build and installation is tested with CentOS 7 and Fedora 20. But all other rpm based distros should be fine, I only had no access to SLES 12 to verify the vim version, but should be 7.4.

lazyfrosch will create the Debian packages (and Ubuntu?). Because of the documentation change we should wait with publishing until he is finish.

Updated by mfriedrich on 2015-05-19 07:57:46 00:00

• Target Version deleted 2.4.0

Updated by mfriedrich on 2015-06-26 09:13:15 00:00

• Target Version set to Backlog

Updated by mfriedrich on 2015-08-27 16:14:56 00:00

Any progress?

Updated by mfrosch on 2016-02-04 10:38:22 00:00

In Debian there is no practice to install nano- packages.

Therefor I created an vim-icinga2 package with the support for vim-addon-manager. The nanorc gets installed with icinga2-common in the correct path.

Should we do the same with RPMs?

I will update the docs here later.

Changes for Debian:
https://anonscm.debian.org/cgit/pkg-nagios/pkg-icinga2.git/commit/?id=522fca2bf7dad4108fa21082bf7a1e53e9a5b90c
https://anonscm.debian.org/cgit/pkg-nagios/pkg-icinga2.git/commit/?id=926142187843c0857facf72f5b8496a8579ee82e
https://anonscm.debian.org/cgit/pkg-nagios/pkg-icinga2.git/commit/?id=b4f5917292441906c52ecce8e2f5eaeb80ac7b6e

Updated by dgoetz on 2016-02-04 11:08:12 00:00

I think we should support the different distributions corresponding their ways how to handle it, so keep the nano subpackage for RPM and just tell the users in the docs would be my preferred way.

Updated by mfrosch on 2016-02-04 11:59:37 00:00

Rebased feature/syntax-8401

Updated by mfrosch on 2016-02-04 15:11:35 00:00

Added my docu bits, please review

Updated by mfrosch on 2016-02-04 15:11:49 00:00

• Assigned to changed from mfrosch to mfriedrich

Updated by mfriedrich on 2016-02-25 00:29:02 00:00

• Status changed from Assigned to New
• Assigned to deleted mfriedrich
• Target Version changed from Backlog to 2.5.0

Updated by mfriedrich on 2016-03-22 20:39:56 00:00

• Relates set to 11432

Updated by mfriedrich on 2016-03-31 10:38:41 00:00

• Status changed from New to Assigned
• Assigned to set to mfriedrich

Updated by mfriedrich on 2016-06-17 12:16:03 00:00

Review

• vim

• Verified the Debian package in a fresh docker container. Installed with the provided documentation, works fine.

• Small suggestion added - ensure that syntax on is set in vimrc. By default it is off.

• Tested the RPM package on a fresh centos docker container. Just installed "vim-icinga2", works. syntax on is already set by default.

• nano

• Debian package icinga2-common installed - does not work, but still requires manual inclusion in nanorc - updated the docs.

• RPM package install plus additional nanorc inclusion as described in the docs - works.

• Minor documentation updates (wording, formatting)

Updated by mfriedrich on 2016-06-17 12:18:17 00:00

• Status changed from Assigned to Resolved
• Done % changed from 0 to 100

Applied in changeset ff0c44d.