Bug 4518
| Summary: | [PATCH] Pump forgets stuff after first lease renewal. | ||
|---|---|---|---|
| Product: | [Retired] Red Hat Linux | Reporter: | dunham |
| Component: | pump | Assignee: | Erik Troan <ewt> |
| Status: | CLOSED NEXTRELEASE | QA Contact: | |
| Severity: | low | Docs Contact: | |
| Priority: | medium | ||
| Version: | 6.0 | CC: | djschaap |
| Target Milestone: | --- | ||
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 1999-10-28 14:48:05 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: | |||
Have you tried the errata pump release? The latest one I had looked at was 0.6.7, apparently 0.7.0 is in the the updates directory but not on the errate web page. Either way, the problem is still there in 0.7.0. I will update the Debian package to 0.7.0 and add my fix. In addition to looking into this problem, you might want to make sure 0.7.0 shows up on the errata page. page. Thanks for the patch! |
There is a typo in the parseLease function, clearing out everything but the lease option in pump, rather than clearing out just the lease option, i.e. line 885 of pump.c, which reads: intf->set &= INTFINFO_HAS_LEASE; should be: intf->set &= ~INTFINFO_HAS_LEASE; (note the negation.) To reproduce on a machine that is not currently running pump, run "pump" to obtain a lease, then run "pump -s", observe "Domain" and other extra options. Run "pump -R" to force renewal. Run "pump -s" again and see that "Domain" has disappeared.