[Bug 949073] New: Review Request: nagios-check_mk - A new general purpose Nagios-plugin for retrieving data - package-review

[Bug 949073] New: Review Request: nagios-check_mk - A new general purpose Nagios-plugin for retrieving data

[Bug 980934] New: Review Request:...

[Bug 957238] New: Review Request:...

Red Hat Bugzilla

Friday, 5 April 2013 Fri, 5 Apr '13

2:47 p.m.

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=949073 Bug ID: 949073 Summary: Review Request: nagios-check_mk - A new general purpose Nagios-plugin for retrieving data Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nobody(a)fedoraproject.org Reporter: andrea.veri(a)gmail.com QA Contact: extras-qa(a)fedoraproject.org CC: notting(a)redhat.com, package-review(a)lists.fedoraproject.org Category: --- Spec URL: http://averi.fedorapeople.org/RPMs/check-mk/nagios-check_mk.spec SRPM URL: http://averi.fedorapeople.org/RPMs/check-mk/nagios-check_mk-1.2.0p4-1.fc1... Description: check_mk is a general purpose Nagios-plugin for retrieving data. It adopts a new approach for collecting data from operating systems and network components. It obsoletes NRPE, check_by_ssh, NSClient, and check_snmp and it has many benefits, the most important are a significant reduction of CPU usage on the Nagios host and an automatic inventory of items to be checked on hosts. Fedora Account System Username: averi -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=I3C5MC6oTr&a=cc_unsubscribe

Show replies by date

Red Hat Bugzilla

Monday, 8 April Mon, 8 Apr

9:55 a.m.

New subject: [Bug 949073] Review Request: nagios-check_mk - A new general purpose Nagios-plugin for retrieving data

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=949073 Colin Walters <walters(a)redhat.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |walters(a)redhat.com --- Comment #1 from Colin Walters <walters(a)redhat.com> --- I'm not finding many other standalone nagios plugins in Fedora. All that I see is the "nagios-plugins" katamari and a few subpackages of larger modules. The first thing that struck me as odd is:

...

tar xf agents.tar.gz

It has tarballs in tarballs? I don't know whether this is normal for nagios agents or not. Now, the install section. At a high level, have you considered trying to do any of this upstream? Does it not have "make install DESTDIR=" support? If not, adding it there would reduce maintenance pain later. Not a blocker though. cat > %{buildroot}%{_datadir}/check_mk/modules/defaults <<'EOF' Hmm...some of this seems like it should be configurable by the system administrator of the package. But maybe that's already possible. Those are just some quick comments; this is a hard package for me to review because of my unfamiliarity with nagios and the lack of another similar package as a comparison base. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=EZXj2zCpUs&a=cc_unsubscribe

Red Hat Bugzilla

12:36 p.m.

New subject: [Bug 949073] Review Request: nagios-check_mk - A new general purpose Nagios-plugin for retrieving data

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=949073 --- Comment #2 from Andrea Veri <andrea.veri(a)gmail.com> --- The check-mk upstream doesn't provide any autotool / autoconf files to properly configure and build the package. The upstream tarball is just a collection of tar.gz files that are then extracted and installed through the setup.sh file. What we do in the packaging is different, we completely ignore the setup.sh file, we extract each of the tarballs and we install the files on the relevant directories. Everything is then editable by the system administrator, the default configuration is what we provide, then there are many variables and tweaks to customize the installation. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=YnWdPUlsN0&a=cc_unsubscribe

Red Hat Bugzilla

3:55 p.m.

New subject: [Bug 949073] Review Request: nagios-check_mk - A new general purpose Nagios-plugin for retrieving data

Red Hat Bugzilla

4:23 p.m.

New subject: [Bug 949073] Review Request: nagios-check_mk - A new general purpose Nagios-plugin for retrieving data

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=949073 --- Comment #4 from Gianluca Sforna <giallu(a)gmail.com> --- The license does not look correct. At least, the top level COPYING is GPLv2; GPL is also mentioned on their web site. Additionally, if I grep for license in the installed tree it seems each check has its own license. Most of them are marked as GPL, a few GPLv2 and one GPLv3. Guess we need to add all of them to the license tag -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Vrl7WdFVOV&a=cc_unsubscribe

Red Hat Bugzilla

Tuesday, 9 April Tue, 9 Apr

11:23 a.m.

New subject: [Bug 949073] Review Request: nagios-check_mk - A new general purpose Nagios-plugin for retrieving data

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=949073 --- Comment #5 from Andrea Veri <andrea.veri(a)gmail.com> --- Thanks for the comments Gianluca, I've just finished preparing the new package which uses setup.sh to build and install the sources. (apparently the livestatus build was fixed in this very latest 1.2.2 release) The new urls: SPEC: http://averi.fedorapeople.org/RPMs/check-mk/check-mk.spec SRC.RPM: http://averi.fedorapeople.org/RPMs/check-mk/check-mk-1.2.2-1.fc18.src.rpm I've also split the files in 4 subpackages, respectively docs, livestatus, agent and multisite. In addition to that I've cleaned up some leftovers and made a little trick for having the libraries correctly packaged on x86_64 builders. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=sCllqYDEJM&a=cc_unsubscribe

Red Hat Bugzilla

2:43 p.m.

New subject: [Bug 949073] Review Request: check_mk - A new general purpose Nagios-plugin for retrieving data

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=949073 Andrea Veri <andrea.veri(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Review Request: |Review Request: check_mk - |nagios-check_mk - A new |A new general purpose |general purpose |Nagios-plugin for |Nagios-plugin for |retrieving data |retrieving data | -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=KGP23gHNeZ&a=cc_unsubscribe

Red Hat Bugzilla

5:37 p.m.

New subject: [Bug 949073] Review Request: check_mk - A new general purpose Nagios-plugin for retrieving data

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=949073 Gianluca Sforna <giallu(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|nobody(a)fedoraproject.org |giallu(a)gmail.com Flags| |fedora-review? --- Comment #6 from Gianluca Sforna <giallu(a)gmail.com> --- taking the review -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=FTCm56HvgU&a=cc_unsubscribe

Red Hat Bugzilla

6:24 p.m.

New subject: [Bug 949073] Review Request: check_mk - A new general purpose Nagios-plugin for retrieving data

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=949073 Gianluca Sforna <giallu(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-review? | Flags| |fedora-review+ --- Comment #7 from Gianluca Sforna <giallu(a)gmail.com> --- ==== REVIEW CHECKLIST ==== - package named according to package naming guidelines - package licensed with allowed license - license matches actual license - license file included in %doc - spec written in American english - spec legible - sources match upstream sha256sum - successfully builds in mock for rawhide x86_64 - no locales - no shared libraries - package owns all directories it creates - macro usage is consistent - package contains code - large documentation moved to subpackage - not a GUI app needing a .desktop file - rpmlint has to say: rpmlint /var/lib/mock/fedora-18-x86_64/result/*.rpm check-mk.src: W: spelling-error %description -l en_US snmp -> snip, snap, sump check-mk.src:151: E: hardcoded-library-path in %{_prefix}/lib/check_mk/livecheck check-mk.src:152: E: hardcoded-library-path in %{_prefix}/lib/check_mk/livestatus.o check-mk.src:153: E: hardcoded-library-path in %{_prefix}/lib/check_mk check-mk.src:197: E: hardcoded-library-path in %{_prefix}/lib/check_mk check-mk.x86_64: W: spelling-error %description -l en_US snmp -> snip, snap, sump check-mk.x86_64: E: no-binary check-mk.x86_64: W: non-conffile-in-etc /etc/check_mk/conf.d/README check-mk.x86_64: W: non-conffile-in-etc /etc/check_mk/First-Installation.txt check-mk.x86_64: W: non-standard-uid /var/lib/check_mk/counters nagios check-mk.x86_64: W: non-standard-gid /var/lib/check_mk/counters nagios check-mk.x86_64: W: non-standard-uid /var/lib/check_mk/autochecks nagios check-mk.x86_64: W: non-standard-gid /var/lib/check_mk/autochecks nagios check-mk.x86_64: W: non-standard-uid /var/lib/check_mk/wato nagios check-mk.x86_64: W: non-standard-gid /var/lib/check_mk/wato nagios check-mk.x86_64: W: non-standard-uid /var/lib/check_mk/logwatch nagios check-mk.x86_64: W: non-standard-gid /var/lib/check_mk/logwatch nagios check-mk.x86_64: W: non-standard-uid /var/lib/check_mk/precompiled nagios check-mk.x86_64: W: non-standard-gid /var/lib/check_mk/precompiled nagios check-mk.x86_64: W: non-standard-uid /var/lib/check_mk/web nagios check-mk.x86_64: W: non-standard-gid /var/lib/check_mk/web nagios check-mk.x86_64: E: incorrect-fsf-address /usr/share/doc/check-mk-1.2.2/COPYING check-mk.x86_64: W: non-standard-uid /var/lib/check_mk/cache nagios check-mk.x86_64: W: non-standard-gid /var/lib/check_mk/cache nagios check-mk.x86_64: W: no-manual-page-for-binary check_mk check-mk.x86_64: W: no-manual-page-for-binary cmk check-mk.x86_64: W: no-manual-page-for-binary mkp check-mk-agent.x86_64: E: incorrect-fsf-address /usr/share/doc/check-mk-agent-1.2.2/COPYING check-mk-agent.x86_64: W: no-manual-page-for-binary check-mk-agent check-mk-docs.noarch: W: wrong-file-end-of-line-encoding /usr/share/doc/check_mk/agents/README.OpenVMS check-mk-docs.noarch: W: file-not-utf8 /usr/share/doc/check_mk/treasures/fsc-celsius-m470-sel-1.2.tar.gz check-mk-docs.noarch: E: incorrect-fsf-address /usr/share/doc/check_mk/COPYING check-mk-docs.noarch: W: class-path-in-manifest /usr/share/doc/check_mk/jasperreports/libs/com-jaspersoft-ireport.jar check-mk-livestatus.x86_64: W: no-documentation check-mk-livestatus.x86_64: W: no-manual-page-for-binary unixcat check-mk-multisite.noarch: W: no-documentation 7 packages and 0 specfiles checked; 8 errors, 28 warnings. that I consider harmless. Feel free to tell upstream about the license text address issue and maybe check if it is correct it's still shipping a .tar.gz file No other blockers spotted so this package is APPROVED -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=hwtn9LHd4B&a=cc_unsubscribe

Red Hat Bugzilla

6:26 p.m.

New subject: [Bug 949073] Review Request: check_mk - A new general purpose Nagios-plugin for retrieving data

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=949073 Andrea Veri <andrea.veri(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags| |fedora-cvs? --- Comment #8 from Andrea Veri <andrea.veri(a)gmail.com> --- New Package SCM Request ======================= Package Name: check-mk Short Description: A new general purpose Nagios plugin for retrieving data Owners: averi Branches: f18 f19 el6 InitialCC: averi -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=zz6z7fz9Lo&a=cc_unsubscribe

Red Hat Bugzilla

Wednesday, 10 April Wed, 10 Apr

5:47 a.m.

New subject: [Bug 949073] Review Request: check_mk - A new general purpose Nagios-plugin for retrieving data

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=949073 Jon Ciesla <limburgher(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|fedora-cvs? | Flags| |fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=AQTUUDWrjt&a=cc_unsubscribe

Red Hat Bugzilla

5:47 a.m.

New subject: [Bug 949073] Review Request: check_mk - A new general purpose Nagios-plugin for retrieving data

Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=949073 --- Comment #9 from Jon Ciesla <limburgher(a)gmail.com> --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=AwbgLvudSi&a=cc_unsubscribe

Red Hat Bugzilla

5:48 a.m.