Bug 198562 - Review Request: zabbix - Open-source monitoring solution for your IT infrastructure
Review Request: zabbix - Open-source monitoring solution for your IT infrastr...
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Jeffrey C. Ollie
Fedora Package Reviews List
:
: 198561 (view as bug list)
Depends On:
Blocks: FE-ACCEPT
  Show dependency treegraph
 
Reported: 2006-07-11 17:47 EDT by Jarod Wilson
Modified: 2007-12-14 11:31 EST (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2006-11-22 13:55:38 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
kevin: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Jarod Wilson 2006-07-11 17:47:44 EDT
Spec URL: http://wilsonet.com/packages/zabbix/zabbix.spec
SRPM URL: http://wilsonet.com/packages/zabbix/zabbix-1.1-1.fc6.src.rpm
Description: 
ZABBIX is software that monitors numerous parameters of a
network and the health and integrity of servers. ZABBIX
uses a flexible notification mechanism that allows users
to configure e-mail based alerts for virtually any event.
This allows a fast reaction to server problems. ZABBIX
offers excellent reporting and data visualisation features
based on the stored data. This makes ZABBIX ideal for
capacity planning.

ZABBIX supports both polling and trapping. All ZABBIX
reports and statistics, as well as configuration
parameters are accessed through a web-based front end. A
web-based front end ensures that the status of your network
and the health of your servers can be assessed from any
location. Properly configured, ZABBIX can play an important
role in monitoring IT infrastructure. This is equally true
for small organisations with a few servers and for large
companies with a multitude of servers.

Assorted notes:

-rpmlint complains about the non-standard-uid/gid on some directories that are owned by 'zabbix'. This is necessary to run zabbix as non-root. (Similar deal to my ganglia package).

-the zabbix server component is currently set to Requires: mysql-server. Technically, it doesn't, as the mysql server could be on another machine, but I think this is a nice convenience, since the vast majority of people run the zabbix server against a local db.
Comment 1 Jarod Wilson 2006-07-11 17:52:32 EDT
*** Bug 198561 has been marked as a duplicate of this bug. ***
Comment 2 John Mahowald 2006-07-26 22:22:42 EDT
Missing the usual chkconfig and /sbin/service Requires. fedora-qa script says:

* Missing dependancy on service for %preun (package initscripts)
* Missing dependancy on chkconfig for %post (package chkconfig)
* Missing dependancy on chkconfig for %preun (package chkconfig)

for both zabbix and zabbix-agent.

Strange, rpmlint says objdump failed. x86_64 devel.

W: zabbix-agent objdump-failed objdump:
/tmp/zabbix-agent-1.1-1.fc6.x86_64.rpm.23260/usr/bin/zabbix_get: File format not
recognized
W: zabbix-agent objdump-failed objdump:
/tmp/zabbix-agent-1.1-1.fc6.x86_64.rpm.23260/usr/bin/zabbix_sender: File format
not recognized
W: zabbix-agent objdump-failed objdump:
/tmp/zabbix-agent-1.1-1.fc6.x86_64.rpm.23260/usr/bin/zabbix_agent: File format
not recognized
W: zabbix-agent objdump-failed objdump:
/tmp/zabbix-agent-1.1-1.fc6.x86_64.rpm.23260/usr/bin/zabbix_agentd: File format
not recognized
Comment 3 Jarod Wilson 2006-07-27 10:14:24 EDT
Deps on chkconfig and service added, misc other little spec clean-ups.

http://wilsonet.com/packages/zabbix/zabbix.spec
http://wilsonet.com/packages/zabbix/zabbix-1.1-2.fc6.src.rpm

As for those rpmlint objdump failures, you aren't by chance running on an FC5
box, building in an FC6 mock chroot, are you? I just ran into this today myself.
FC5 builds analyzed on FC5 work fine, but not the FC6 ones. If I scp the
mock-built FC6 packages over to an up-to-date rawhide box, rpmlint does its
thing without a problem. I believe this is due to recent glibc hash function
changes that rpmlint/objdump on FC5 doesn't understand.
Comment 4 John Mahowald 2006-08-14 08:52:05 EDT
Correct, I was building FC6 on FC5.

I notice MySQL is enabled, but configure also looked for Postgre. Would this be
something you would want to --enable but not Require?

You'll also want zabbix-web to Require php-mysql

Please substitute the references to /var with %{_localstatedir} in your sed scripts.
Comment 5 Jeffrey C. Ollie 2006-08-15 11:12:21 EDT
Zabbix 1.1.1 has been released:

http://www.zabbix.com/rn1.1.1.php
Comment 6 Jarod Wilson 2006-08-15 11:35:18 EDT
(In reply to comment #4)
> I notice MySQL is enabled, but configure also looked for Postgre. Would this be
> something you would want to --enable but not Require?

We could do that. However, based on the 1.1.1 release notes, it looks like
postgres support is currently broken, fix slated for 1.1.2.

Speaking of which, new build is updated to v1.1.1 (thank you, Jeff).

> You'll also want zabbix-web to Require php-mysql

Okay, I simply changed Requires: php to php-mysql, since php-mysql requires php.

> Please substitute the references to /var with %{_localstatedir} in your sed
scripts.

Done.

Latest build:
http://wilsonet.com/packages/zabbix/zabbix.spec
http://wilsonet.com/packages/zabbix/zabbix-1.1.1-1.fc6.src.rpm
Comment 7 Jeffrey C. Ollie 2006-08-15 16:35:19 EDT
(In reply to comment #4)
> 
> I notice MySQL is enabled, [...]. Would this be
> something you would want to --enable but not Require?

I would concur with this statement.  I would think that on larger installations
a person would want to run the Zabbix server and the MySQL server on separate
systems, and wouldn't want to have mysql-server installed needlessly.  Sure
having as a Require will make things easier for newbie administrators, but I
don't think that that is as important right now.  Plus hopefully the Zabbix devs
will have PostgreSQL support fixed soon and I'm sure that PostgreSQL users
wouldn't want the MySQL server installed on their systems and vice-versa.
Comment 8 Jarod Wilson 2006-08-15 17:02:11 EDT
(In reply to comment #7)
> (In reply to comment #4)
> > 
> > I notice MySQL is enabled, [...]. Would this be
> > something you would want to --enable but not Require?
> 
> I would concur with this statement.

Heh, I think John was only talking about Postgres there, but see my note on this
very issue in the initial description. I'm definitely open to not including
mysql-server as a Requires (I'll make that change to my local copy now).
Comment 9 Jeffrey C. Ollie 2006-08-17 16:36:31 EDT
Probably want to Require: fping so that the icmpping and icmppingsec checks work.
Comment 10 Jeffrey C. Ollie 2006-08-17 16:42:58 EDT
The web interface also needs to Require: php-gd for graphs to work.
Comment 11 Jarod Wilson 2006-08-17 17:09:40 EDT
Requires: on mysql-server removed, Requires: for fping (main package) and php-gd (-web package) 
added. Updated files:

http://wilsonet.com/packages/zabbix/zabbix.spec
http://wilsonet.com/packages/zabbix/zabbix-1.1.1-2.fc6.src.rpm
Comment 12 Dan Horák 2006-10-02 03:06:24 EDT
Zabbix 1.1.2 is out for some time and it adds support for PostgreSQL as the
backend database. Is it possible to parametrize the database support in the spec
file? To have something like the following:

%if !%db
%define db mysql
%endif

BuildRequires: %{db}-devel
Requires: php-%{db}

and when running the configure there will be "--with-%{db}" instead of
--with-mysql/--with-pgsql
Comment 13 Jarod Wilson 2006-11-14 11:00:14 EST
Eep, now Zabbix 1.1.3 is out, and I've been slacking on this package... Its not
entirely pretty to do this conditionally, as only one or the other can be
enabled, but I've set up the default build to use mysql, and if '--with
postgresql' is passed in, it'll build with postgresql instead. Didn't bother
with a '--with mysql' flag, don't want to hassle with what to do if both were
passed in. Both mysql and postgresql versions are building fine for me on
FC6/x86_64.

Updated SRPM:
http://wilsonet.com/packages/zabbix/zabbix-1.1.3-1.fc6.src.rpm

Updated spec:
http://wilsonet.com/packages/zabbix/zabbix.spec

Comment 14 Jeffrey C. Ollie 2006-11-14 12:09:49 EST
The proper Requires for PostgreSQL support in PHP is "php-pgsql" not
"php-postgresql".
Comment 15 Jarod Wilson 2006-11-14 12:17:52 EST
D'oh. That's what I get for only build-testing... Fixed, thank you.

http://wilsonet.com/packages/zabbix/zabbix-1.1.3-2.fc6.src.rpm
http://wilsonet.com/packages/zabbix/zabbix.spec
Comment 16 Colin Stubbs 2006-11-14 20:55:29 EST
Nice spec Jarod, thanks.

There's a few more build/runtime dependencies you probably need to add though.

Build: gnutls-devel

Runtime: net-snmp-libs

The build scripts sometimes can't determine where libgnutls is by default,
gnutls-devel includes libgnutls-config which fixes this. 

Without net-snmp-libs the zabbix_server binary can't load. Usually get's
resolved automatically, but given ucd-snmp is now net-snmp and that's what
Fedora is packaging, it should be OK to explicitly require net-snmp-libs ?
Comment 17 Jarod Wilson 2006-11-14 21:49:15 EST
Added BR: gnutls-devel and R: net-snmp-libs (yeah, both are the correct thing to
do).

http://wilsonet.com/packages/zabbix/zabbix-1.1.3-3.fc6.src.rpm
http://wilsonet.com/packages/zabbix/zabbix.spec
Comment 18 Colin Stubbs 2006-11-15 00:46:47 EST
Spec works well on RHEL4/CentOS4 too, and I've tested it with Zabbix 1.1.4 - no
compile issues, but the same problem I'm seeing below occurred.

Seems to center around 'make install'. Along with the .a libs you've excluded,
it's gone and dumped a bunch of object files in /usr/lib in my build
environment, so I'm getting unpackaged file errors on build.

Checking for unpackaged file(s): /usr/lib/rpm/check-files
/home/cstubbs/rpm/tmp/zabbix-1.1.3-3-root-cstubbs
error: Installed (but unpackaged) file(s) found:
   /usr/lib/stD0Ikh0
   /usr/lib/stIMYNMd/active.o
   /usr/lib/stIMYNMd/autoregister.o
   /usr/lib/stIMYNMd/trapper.o
   /usr/lib/stLeJGOP
   /usr/lib/stRDGpXX/action.o
....
etc

One work around is a few more exlude lines,

%exclude %{_libdir}/st*
%exclude %{_libdir}/st*/*.o
Comment 19 Dan Horák 2006-11-15 04:10:02 EST
(In reply to comment #18)
> Spec works well on RHEL4/CentOS4 too, and I've tested it with Zabbix 1.1.4 - no
> compile issues, but the same problem I'm seeing below occurred.
> 
> Seems to center around 'make install'. Along with the .a libs you've excluded,
> it's gone and dumped a bunch of object files in /usr/lib in my build
> environment, so I'm getting unpackaged file errors on build.
> One work around is a few more exlude lines,
> 
> %exclude %{_libdir}/st*
> %exclude %{_libdir}/st*/*.o

I see the same problems, my workaround was to delete the *.a files manualy
during the %install and not use the %exclude directive at all. When you only
comment out the %exclude line, then the strange files st*/*.o are not created.
Comment 20 Jarod Wilson 2006-11-15 09:53:19 EST
Odd, I remember seeing those files at one point in time, but I don't with the
1.1.3 spec that only %exclude'd .a files... If killing static libs in %install
works around it for cases where its still happening, I have no problem switching
to that instead of %exclude. I've done that in a 1.1.4 package:

http://wilsonet.com/packages/zabbix/zabbix-1.1.4-1.fc6.src.rpm
http://wilsonet.com/packages/zabbix/zabbix.spec
Comment 21 Jeffrey C. Ollie 2006-11-16 12:24:38 EST
John, I hope you don't mind if I reassign this review to myself...
Comment 22 Jeffrey C. Ollie 2006-11-16 12:26:01 EST
* source files match upstream
8e733e41506dd34759daba01deeeefd9  zabbix-1.1.4.tar.gz
8e733e41506dd34759daba01deeeefd9  zabbix-1.1.4.tar.gz.1
* package meets naming and packaging guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* dist tag is present.
* build root is correct.
* license field matches the actual license.
* license is open source-compatible.  License text included in package.
* latest version is being packaged.
* BuildRequires are proper.
* compiler flags are appropriate.
* %clean is present.
* package builds in mock (fc6 i386).
* package installs properly. 
* rpmlint says:

E: zabbix non-standard-uid /var/run/zabbix zabbix
E: zabbix non-standard-gid /var/run/zabbix zabbix
E: zabbix non-standard-uid /var/log/zabbix zabbix
E: zabbix non-standard-gid /var/log/zabbix zabbix
E: zabbix-agent non-standard-uid /var/log/zabbix zabbix
E: zabbix-agent non-standard-gid /var/log/zabbix zabbix
E: zabbix-agent non-standard-uid /var/run/zabbix zabbix
E: zabbix-agent non-standard-gid /var/run/zabbix zabbix

Can be ignored.

E: zabbix-web script-without-shebang /usr/share/zabbix/include/locales/pt_br.inc.php

! fix in %prep with %{__chmod} a-x frontends/php/include/locales/pt_br.inc.php

E: zabbix-web wrong-script-end-of-line-encoding
/usr/share/zabbix/include/locales/pt_br.inc.php

! fix in %prep with %{__sed} -i 's/\r//' frontends/php/include/locales/pt_br.inc.php

* %check is not present; There is no test code in the districution.
* no shared libraries are present
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* scriptlets are OK
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* no headers
* no unversioned .so file
* no pkconfig file
* no libtool .la droppings.

There should be:

Requires(pre): /usr/sbin/useradd

for both the main package and the agent package.
Comment 23 Jarod Wilson 2006-11-16 13:38:09 EST
(In reply to comment #22)
[...]
> E: zabbix-web script-without-shebang
/usr/share/zabbix/include/locales/pt_br.inc.php
> 
> ! fix in %prep with %{__chmod} a-x frontends/php/include/locales/pt_br.inc.php
> 
> E: zabbix-web wrong-script-end-of-line-encoding
> /usr/share/zabbix/include/locales/pt_br.inc.php
> 
> ! fix in %prep with %{__sed} -i 's/\r//'
frontends/php/include/locales/pt_br.inc.php

Ugh. Musta been added to one of the recent builds, rpmlint used to be silent...
That's what I get for not linting new builds... :)

> There should be:
> 
> Requires(pre): /usr/sbin/useradd
> 
> for both the main package and the agent package.

Damn, missed that one...


Okay, new build should fix all of the above.

http://wilsonet.com/packages/zabbix/zabbix-1.1.4-2.fc6.src.rpm
Comment 24 Jeffrey C. Ollie 2006-11-16 13:48:11 EST
APPROVED
Comment 25 Jarod Wilson 2006-11-16 13:53:12 EST
Excellent, thanks much, importing shortly...
Comment 26 John Mahowald 2006-11-16 19:12:54 EST
(In reply to comment #21)
> John, I hope you don't mind if I reassign this review to myself...
> 

Not at all, I was getting behind on the builds. I agree with the review.
Comment 27 Jarod Wilson 2006-11-22 13:55:38 EST
Imported and built for devel, FC6 and FC5.
Comment 28 Dan Horák 2007-09-11 02:24:29 EDT
Package Change Request
======================
Package Name: zabbix
Updated Fedora Owners: jwilson,sharkcz
Updated EPEL Owners: jwilson,sharkcz
Comment 29 Kevin Fenzi 2007-09-11 12:23:28 EDT
This request appears to have already been done via the web interface. 
Feel free to reset the flag if further cvsadmin attention is needed. 
Comment 30 Dan Horák 2007-12-13 16:16:57 EST
Package Change Request
======================
Package Name: zabbix
Updated Fedora Owners: sharkcz,jcollie
Updated EPEL Owners: sharkcz,jcollie
Comment 31 Kevin Fenzi 2007-12-14 11:31:27 EST
again, this looks to have been completed via the pkgdb web interface. 
Let me know if you need anything further here. 

Note You need to log in before you can comment on or make changes to this bug.