Bug 682905 - Review Request: lio-utils - Utilities for Generic Target Infrastructure
Summary: Review Request: lio-utils - Utilities for Generic Target Infrastructure
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 744340
TreeView+ depends on / blocked
 
Reported: 2011-03-07 22:45 UTC by Andy Grover
Modified: 2011-10-07 22:48 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-10-07 22:47:21 UTC
Type: ---


Attachments (Terms of Use)

Description Andy Grover 2011-03-07 22:45:35 UTC
Spec URL: http://fedorapeople.org/~grover/lio-utils.spec
SRPM URL: http://fedorapeople.org/~grover/srpms/lio-utils-3.2-1.src.rpm
Description: This package is used to control the new kernel-based target infrastructure that is present in 2.6.38+. It allows the admin to export scsi block devices (LUNs) via iSCSI, FCoE, and the like.

Comment 1 Bill Nottingham 2011-03-08 16:11:55 UTC
I've probably asked this before, but what's the way forward between this, scsi-target-utils, scst, etc?

Comment 2 Andy Grover 2011-03-08 17:29:44 UTC
LIO was accepted upstream over scst. Long-term, scsi-target-utils will be deprecated in favor of LIO. The exact timing and method of that transition is still TBD.

Comment 3 Terje Røsten 2011-03-08 19:34:01 UTC
Some comments:

 a) no url tag, where is this project living?
 b) you might want to add the sha1 8ad98e3 to the tarball name?
 c) -n %{name}-%{version} in %setup is the default and can be dropped 
   (however, consider b))
 d) mixing of $RPM_BUILD_ROOT and %{buildroot}, pick one.
 e) use macros for /etc, /sbin and /etc/init.d/
 f) target is a very generic name, possible problem
 g) change %defattr(-,root,root) ->  %defattr(-,root,root,-)
 h) the package seems to be free for docs and with no url tag, the user
    will have a hard time be able to use this utility.
 i) Release:  1, please add disttag here.

Comment 4 Terje Røsten 2011-03-08 19:41:47 UTC
 j) it builds in koji, however correct %{optflags} is not used:
   http://koji.fedoraproject.org/koji/getfile?taskID=2895139&name=build.log
   http://koji.fedoraproject.org/koji/taskinfo?taskID=2895138

Comment 5 Andy Grover 2011-03-09 18:08:55 UTC
(In reply to comment #3)
> Some comments:
> 
>  a) no url tag, where is this project living?
>  b) you might want to add the sha1 8ad98e3 to the tarball name?
>  c) -n %{name}-%{version} in %setup is the default and can be dropped 
>    (however, consider b))

I decided to keep the tarball name, since future releases should have proper tarballs.

>  d) mixing of $RPM_BUILD_ROOT and %{buildroot}, pick one.
>  e) use macros for /etc, /sbin and /etc/init.d/
>  f) target is a very generic name, possible problem

True.. it's what upstream is using, so I'm hesitant to diverge.

>  g) change %defattr(-,root,root) ->  %defattr(-,root,root,-)
>  h) the package seems to be free for docs and with no url tag, the user
>     will have a hard time be able to use this utility.

Yeah, I'll be working with upstream to author some manpages.

>  i) Release:  1, please add disttag here.

All the rest are done. Revised versions:

http://fedorapeople.org/~grover/lio-utils.spec(2)
http://fedorapeople.org/~grover/srpms/lio-utils-3.2-1.src.rpm

Comment 6 Andy Grover 2011-03-09 18:37:26 UTC
Whups, added optflags and made disttag conditional.

http://fedorapeople.org/~grover/lio-utils.spec3
http://fedorapeople.org/~grover/srpms/lio-utils-3.2-3.el6.src.rpm

Comment 7 Terje Røsten 2011-03-09 18:41:02 UTC
Would be simpler to follow if you add a real changelog.

Comment 8 Andy Grover 2011-10-07 22:48:51 UTC
This package has been superseded by targetcli, so I'm closing this one.


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