Bug 511148

Summary: Review Request: matahari - qmf agent for host management
Product: [Fedora] Fedora Reporter: Arjun Roy <arroy>
Component: Package ReviewAssignee: David Lutterkort <lutter>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, hbrock, lutter, notting
Target Milestone: ---Flags: lutter: fedora-review+
j: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-07-22 15:21:17 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Arjun Roy 2009-07-13 21:44:13 UTC
Spec URL: http://arjunroy.fedorapeople.org/matahari.spec
SRPM URL: http://arjunroy.fedorapeople.org/matahari-0.0.4-3.fc11.src.rpm

Description: matahari provides a QMF Agent that can be used to control and manage various pieces of functionality for a host system, using the AMQP protocol.

While this is intended to be used for node management for ovirt (http://ovirt.org) the functionality isn't specific to ovirt, and so it has been moved to a separate project.

This is my first package and I will need a sponsor.

Comment 1 Arjun Roy 2009-07-13 21:46:32 UTC
So far, it seems to build on koji just fine:
https://koji.fedoraproject.org/koji/taskinfo?taskID=1472018

I've verified it works on x86_64, but haven't tried it on other architectures yet. rpmlint doesn't give any errors or warnings for either the src rpm or the built rpm, and there are only two gcc warnings that I have verified are not an issue.

Additional notes: This package is used by ovirt.

Comment 2 Jason Tibbitts 2009-07-13 21:54:22 UTC
Please read through http://fedoraproject.org/wiki/PackageMaintainers/Join if you haven't already.

Comment 3 David Lutterkort 2009-07-15 23:23:02 UTC
  OK - Package name
  OK - License info is accurate
  OK - License tag is correct and licenses are approved
  OK - License files are installed as %doc
  OK - Specfile name
  OK - Specfile is legible
  OK - No prebuilt binaries included

  FIX- BuildRoot value (one of the recommended values)
       See https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag

  OK - PreReq not used
  FIX- Source md5sum matches upstream
       No upstream release - consider setting up a simple webpage and a
       tarball (e.g., on fedorahosted); at the very minimum, tag the release
       in git. Once you have that URL should point to that released tarball,
       not git. If you don't do that, you need to explain in a comment before
       the URL how exactly you created the tarball (include the git commit/tag
       from which it was created)

  OK - No hardcoded pathnames
  OK - Package owns all the files it installs
  OK - 'Requires' create needed unowned directories
  OK - Package builds successfully on i386 and x86_64 (mock)
  OK - BuildRequires sufficient
  OK - File permissions set properly
  OK - Macro usage is consistent
  OK - rpmlint is silent
  OK - Proper debuginfo packages

Please fix the two nits above

Comment 4 Arjun Roy 2009-07-16 15:44:51 UTC
(In reply to comment #3)

>   FIX- BuildRoot value (one of the recommended values)
>        See https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag

Buildroot changed to %{_tmppath}/%{name}-%{version}-%{release}-root.

>   FIX- Source md5sum matches upstream
>        No upstream release - consider setting up a simple webpage and a
>        tarball (e.g., on fedorahosted); at the very minimum, tag the release
>        in git. Once you have that URL should point to that released tarball,
>        not git. If you don't do that, you need to explain in a comment before
>        the URL how exactly you created the tarball (include the git commit/tag
>        from which it was created)

Project URL updated to: http://arjunroy.fedorapeople.org/matahari/index.html
Site contains link to git, and current rpm files and source tarball.

Resubmitting with fixes for consideration:

SRPM: http://arjunroy.fedorapeople.org/matahari/matahari-0.0.4-4.fc11.src.rpm
Specfile: http://arjunroy.fedorapeople.org/matahari/matahari.spec

Comment 5 David Lutterkort 2009-07-17 01:52:26 UTC
Great - one thing I missed yesterday is that Source: should be a URL pointing to the where the tarball for the release can be downloaded, and ideally those tarballs are kept around for a while. Please add that before adding this package to CVS.

APPROVED

Please follow http://fedoraproject.org/wiki/CVSAdminProcedure and import
the package. Close this bug as RAWHIDE once it's been successfully imported
and built.

Comment 6 Itamar Reis Peixoto 2009-07-20 16:20:22 UTC
*** Bug 512748 has been marked as a duplicate of this bug. ***

Comment 7 Arjun Roy 2009-07-20 17:58:03 UTC
New Package CVS Request
=======================
Package Name: matahari
Short Description: host qmf agent used by ovirt
Owners: arjunroy
Branches: F-11 F-12
InitialCC:

Comment 8 Jason Tibbitts 2009-07-21 15:19:57 UTC
As mentioned in the other ticket, it is too early to request F-12 branches.

Otherwise, CVS done.

Comment 9 Arjun Roy 2009-07-22 15:21:17 UTC
Package built in CVS for F11 and devel (F12).

devel:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1492245

F11:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1492292

Closing bug as rawhide.

Comment 10 Fedora Update System 2009-07-22 15:40:27 UTC
matahari-0.0.4-4.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/matahari-0.0.4-4.fc11

Comment 11 Fedora Update System 2009-10-14 01:52:58 UTC
matahari-0.0.4-4.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.