Bug 583086 - lsb compliancy issues
Summary: lsb compliancy issues
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: iscsi-initiator-utils
Version: 5.5
Hardware: All
OS: Linux
low
medium
Target Milestone: rc
: ---
Assignee: Mike Christie
QA Contact: Red Hat Kernel QE team
URL:
Whiteboard:
Depends On:
Blocks: 584019 596292
TreeView+ depends on / blocked
 
Reported: 2010-04-16 16:01 UTC by michal novacek
Modified: 2011-01-13 22:58 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
When the "stop" operation was invoked, the iscsid init script did not remove the process identifier (PID) file. This update modifies the iscsid script to remove the PID file during the stop operation.
Clone Of:
: 584019 596292 (view as bug list)
Environment:
Last Closed: 2011-01-13 22:58:30 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
proposed fix for /etc/init.d/iscsi (142 bytes, text/plain)
2010-04-16 16:01 UTC, michal novacek
no flags Details
proposed fix for /etc/init.d/iscsid (35 bytes, text/plain)
2010-04-16 16:01 UTC, michal novacek
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2011:0072 0 normal SHIPPED_LIVE iscsi-initiator-utils bug fix update 2011-01-12 17:22:06 UTC

Description michal novacek 2010-04-16 16:01:07 UTC
Created attachment 407139 [details]
proposed fix for /etc/init.d/iscsi

Description of problem:
There are two problems that I encountered, but is they are easy to fix I'm putting them under one bugzilla.

1/ initscripts /etc/init.d/iscsi and /etc/init.d/iscsid are not removing theirs .pid file which results in bad return codes (non-lsb compliant) and "Process dead but pid file exists" message on each starting of the service. (diff to illustrate the proposed fix: etc-init.d-iscsi.diff)

2/ start() in /etc/init.d/iscsi is using RETVAL variable which is used as general return value. It should be either renamed or declared local. ((diff to illustrate the proposed fix: etc-init.d-iscsid.diff)

Version-Release number of selected component (if applicable):
iscsi-initiator-utils-6.2.0.871-0.16.el5

How reproducible: always.

Steps to Reproduce:
1. install iscsi-initiator-utils
2. service iscsi start
3. echo $? (issue 1)
4. service iscsi restart (issue 2)
  
Actual results:


Expected results:


Additional info:
# rpm -q --verify iscsi-initiator-utils
# echo $? 
0

# service iscsi status
iscsid is stopped

# service iscsi start
iscsid is stopped
Starting iSCSI daemon:                                     [  OK  ]
                                                           [  OK  ]
Setting up iSCSI targets: iscsiadm: No records found!
                                                           [  OK  ]
# echo $?															 <<<< issue 1
3

# service iscsi restart
Stopping iSCSI daemon: 
iscsid dead but pid file exists                            [  OK  ]  <<<< issue 2
Starting iSCSI daemon:                                     [  OK  ]
                                                           [  OK  ]
Setting up iSCSI targets: iscsiadm: No records found!
                                                           [  OK  ]

Comment 1 michal novacek 2010-04-16 16:01:57 UTC
Created attachment 407141 [details]
proposed fix for /etc/init.d/iscsid

Comment 2 Mike Christie 2010-04-16 23:06:23 UTC
Thanks Michal. This seems ok to me. Adding devel ack for 5.6.

Comment 4 RHEL Program Management 2010-08-09 19:11:03 UTC
This request was evaluated by Red Hat Product Management for
inclusion in the current release of Red Hat Enterprise Linux.
Because the affected component is not scheduled to be updated in the
current release, Red Hat is unfortunately unable to address this
request at this time. Red Hat invites you to ask your support
representative to propose this request, if appropriate and relevant,
in the next release of Red Hat Enterprise Linux.

Comment 5 Mike Christie 2010-09-25 08:51:13 UTC
This is fixed in 
iscsi-initiator-utils-6.2.0.872-2.el5.
It can be downloaded here:
http://people.redhat.com/mchristi/iscsi/rhel5.6/iscsi-initiator-utils/

Comment 8 Barry Donahue 2010-10-20 19:21:04 UTC
Verified on RHEL5.6-Server-20101014.0

Comment 9 Jaromir Hradilek 2010-12-08 10:56:22 UTC
    Technical note added. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    New Contents:
When the "stop" operation was invoked, the iscsid init script did not remove the process identifier (PID) file. This update modifies the iscsid script to remove the PID file during the stop operation.

Comment 11 errata-xmlrpc 2011-01-13 22:58:30 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHBA-2011-0072.html


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