Bug 772288 - Review Request: infiniband-diags - various InfiniBand fabric test utilities
Summary: Review Request: infiniband-diags - various InfiniBand fabric test utilities
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Albert Strasheim
QA Contact: Fedora Extras Quality Assurance
URL: http://people.redhat.com/dledford/Pac...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-01-06 17:14 UTC by Doug Ledford
Modified: 2012-02-01 19:25 UTC (History)
4 users (show)

Fixed In Version: infiniband-diags-1.5.12-1.fc15
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-02-01 19:23:52 UTC
Type: ---
Embargoed:
fenlason: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Doug Ledford 2012-01-06 17:14:54 UTC
The infiniband-diags package provides a number of utilities for querying the state of an InfiniBand fabric.  It can get the detailed information on any given port on the fabric for diagnostic purposes.

Comment 1 Albert Strasheim 2012-01-09 12:52:44 UTC
I suggest that a devel and devel-static are added for this package:

%description devel
Header files for the infiniband-diags library.

%package devel-static
Summary: Static development files for the infiniband-diags library
Group: System Environment/Libraries
Requires: %{name}-devel = %{version}-%{release}

%description devel-static
Static libraries for the infiniband-diags library.

%files devel
%defattr(-,root,root,-)
%{_libdir}/lib*.so
%{_includedir}/infiniband/*
%{_mandir}/man3/*

%files devel-static
%defattr(-,root,root,-)
%{_libdir}/*.a

The ibnetdisc library that is part of this package is very useful when writing custom admin tools.

Comment 2 Albert Strasheim 2012-01-09 12:57:09 UTC
$ rpmlint infiniband-diags.spec
infiniband-diags.spec:12: W: unversioned-explicit-provides perl(IBswcountlimits)
0 packages and 1 specfiles checked; 0 errors, 1 warnings.

$ rpmlint infiniband-diags-1.5.12-1.fc15.x86_64.rpm
infiniband-diags.x86_64: W: spelling-error %description -l en_US subnet -> subset, sublet, sub net
infiniband-diags.x86_64: W: obsolete-not-provided openib-diags
infiniband-diags.x86_64: W: non-conffile-in-etc /etc/infiniband-diags/error_thresholds
infiniband-diags.x86_64: W: no-manual-page-for-binary check_lft_balance.pl
infiniband-diags.x86_64: W: no-manual-page-for-binary ibdiscover.pl
infiniband-diags.x86_64: W: no-manual-page-for-binary dump_mfts.sh
infiniband-diags.x86_64: W: no-manual-page-for-binary ibqueryerrors.pl
infiniband-diags.x86_64: W: no-manual-page-for-binary dump_lfts.sh
infiniband-diags.x86_64: W: no-manual-page-for-binary ibswportwatch.pl
infiniband-diags.x86_64: W: no-manual-page-for-binary ibfindnodesusing.pl
infiniband-diags.x86_64: W: no-manual-page-for-binary ibprintca.pl
infiniband-diags.x86_64: W: no-manual-page-for-binary ibprintswitch.pl
infiniband-diags.x86_64: W: no-manual-page-for-binary iblinkinfo.pl
infiniband-diags.x86_64: W: no-manual-page-for-binary set_nodedesc.sh
infiniband-diags.x86_64: W: no-manual-page-for-binary ibidsverify.pl
infiniband-diags.x86_64: W: no-manual-page-for-binary ibprintrt.pl
1 packages and 0 specfiles checked; 0 errors, 16 warnings.

Looks like the file in sysconfdir needs a %config(noreplace).

I noticed on the linux-rdma mailing list that these scripts aren't supported anymore:

http://www.spinics.net/lists/linux-rdma/msg10669.html

Do we want to package them?

Comment 3 Doug Ledford 2012-01-09 19:16:51 UTC
New files (although with the same n-v-r) uploaded to my package page listed in the URL above that address the comments.

Comment 4 Albert Strasheim 2012-01-10 12:12:42 UTC
Package looks good. Thanks. Does anything else need to be done to finish the review?

Comment 5 Doug Ledford 2012-01-10 13:03:02 UTC
The reviewer needs to set the fedora-cvs flag I think, and then the rel-eng people will create a cvs (git now, but the flag name hasn't changed) module and create the appropriate branches and then possibly import the srpm into the git repo, or something like that.  I'm a bit fuzzy on the process myself as I don't do it all that often.

Comment 6 Doug Ledford 2012-01-10 17:21:35 UTC
My mistake, now that you've approved it, it's up to me to do the request.

Comment 7 Doug Ledford 2012-01-10 17:23:25 UTC
Except that the way you are supposed to signify that it has passed your review is setting the fedora-review flag to +, then I can proceed with making the request for package addition.

Comment 8 Albert Strasheim 2012-01-10 20:06:00 UTC
It seems I can't set fedora-review flag to +... my only option is ?. I guess I need to get sponsored, which is quite hard, given that I haven't contributed a package.

Comment 9 Doug Ledford 2012-01-10 20:54:20 UTC
Yep, that would be a problem ;-)  I've asked one of my co workers to take a look.

Comment 10 Doug Ledford 2012-01-10 21:30:44 UTC
New Package SCM Request
=======================
Package Name: infiniband-diags
Short Description: Tools to query and diagnose InfiniBand physical networks
Owners: dledford
Branches: f15 f16
InitialCC:

Comment 11 Gwyn Ciesla 2012-01-11 00:58:24 UTC
Git done (by process-git-requests).

Comment 12 Fedora Update System 2012-01-11 19:07:56 UTC
infiniband-diags-1.5.12-1.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/infiniband-diags-1.5.12-1.fc16

Comment 13 Fedora Update System 2012-01-11 19:08:04 UTC
infiniband-diags-1.5.12-1.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/infiniband-diags-1.5.12-1.fc15

Comment 14 Fedora Update System 2012-01-15 20:08:27 UTC
infiniband-diags-1.5.12-1.fc15 has been pushed to the Fedora 15 testing repository.

Comment 15 Fedora Update System 2012-02-01 19:23:52 UTC
infiniband-diags-1.5.12-1.fc16 has been pushed to the Fedora 16 stable repository.

Comment 16 Fedora Update System 2012-02-01 19:25:25 UTC
infiniband-diags-1.5.12-1.fc15 has been pushed to the Fedora 15 stable repository.


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