Bug 447307
Summary: | ypbind init does not always respect NISTIMEOUT | ||||||
---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Ian Collier <imc> | ||||
Component: | ypbind | Assignee: | Vitezslav Crhonek <vcrhonek> | ||||
Status: | CLOSED CURRENTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||
Severity: | medium | Docs Contact: | |||||
Priority: | low | ||||||
Version: | 9 | ||||||
Target Milestone: | --- | ||||||
Target Release: | --- | ||||||
Hardware: | All | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | 1.20.4-5.fc9 | Doc Type: | Bug Fix | ||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2008-06-11 04:33:51 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: | |||||||
Attachments: |
|
Description
Ian Collier
2008-05-19 13:31:07 UTC
Created attachment 305944 [details] Suggested fix for ypbind init script issues This could be fixed by renaming "failure" to something more descriptive (e.g., "firsttime"), setting it to 1 before the loop instead of 0, and not setting it in the "wait for ypbind to get started" section. In addition, something real-time based (as suggested in bug 430489) should be used instead of the current hack. Since the script currently runs in /bin/bash this is as simple as setting SECONDS=0 before the loop and then looping on [ $SECONDS -lt $timeout ]. The attached (lightly tested) patch implements the above suggestions and also fixes a few incorrectly (IMHO) indented lines. Thanks for report and patch. Patch seems fine to me, I have no remarks to it. ypbind-1.20.4-5.fc9 has been submitted as an update for Fedora 9 ypbind-1.20.4-5.fc9 has been pushed to the Fedora 9 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update ypbind'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F9/FEDORA-2008-4631 ypbind-1.20.4-5.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report. |