Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=543917

David Nalley <da...@gnsa.us> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+

--- Comment #5 from David Nalley <da...@gnsa.us> 2010-02-28 20:55:33 EST ---
Upstream source matches 
[ke4...@nalleyx60 SOURCES]$ md5sum check_postgres-2.14.2.tar.gz*
c632f1f8f1d228d5dc2f88bdc52437a9  check_postgres-2.14.2.tar.gz
c632f1f8f1d228d5dc2f88bdc52437a9  check_postgres-2.14.2.tar.gz.1

You fixed the permissions issues noted above 


One other comment - that isn't guidelines related you require
postgresql-server. Yet this nagios plugin can be run against a remote server,
so that only postgresql would be required (at least that's my very hasty
reading of it.). Requiring postgtresql-server seems a bit heavy (particularly
if you wanted a central nagios system to monitor multiple pg boxes rather than
installing this plugin on each box and calling it from the remote box. Again -
it isn't a blocker, and not really guidelines related - just thought I'd toss
it out there, based on how I'd plan to use it. 


Approved

Thanks Devrim!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to