Bug 702691
Summary: | ProcessInfo in resource context never refreshes | ||
---|---|---|---|
Product: | [Other] RHQ Project | Reporter: | Lukas Krejci <lkrejci> |
Component: | Agent | Assignee: | RHQ Project Maintainer <rhq-maint> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Mike Foley <mfoley> |
Severity: | high | Docs Contact: | |
Priority: | high | ||
Version: | 4.0.1 | CC: | hrupp, mazz, skondkar |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | 700461 | Environment: | |
Last Closed: | 2013-09-02 07:25:25 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: | |||
Bug Depends On: | |||
Bug Blocks: | 678340, 694476 |
Description
Lukas Krejci
2011-05-06 15:33:28 UTC
commit 508fae8bb1cb9a3d161754a753a36b7781315ccd Author: Lukas Krejci <lkrejci> Date: Fri May 6 17:35:20 2011 +0200 BZ 702691 - ResourceContext.getNativeProcess() now always reports the current process info or null if no process exists for given resource. We should go over all plugins to see if they dereference the process info without a check for null. The samba one did for example not have the check and thus throw NPEs over the place. Commit in release-4.0.0 branch: f619ef04ec20394cc2bac6809d8b7b3085ac847d Steps to test: 1) In an apache configuration define 2 vhosts, one "plain", one nested inside an IfDefine: Listen 8080 Listen 8081 .... <VirtualHost 127.0.0.1:8080> </VirtualHost> <IfDefine RHQ_TESTING> <VirtualHost 127.0.0.1:8081> </VirtualHost> </IfDefine> 2) start apache server: apachectl start 3) discover the server in RHQ, only the 127.0.0.1:8080 vhost should appear. 4) restart the apache server apachectl stop apachectl -k start -D RHQ_TESTING 5) on agent commandline do "discovery -f" 6) the second vhost should be discovered Verified on build#31 (Version: 4.0.1-SNAPSHOT Build Number: dc9a4b6) Defined 2 vhosts, one "plain" and one nested inside the IfDefine as explained in the steps. Discovered the server in RHQ, the vhost 127.0.0.1:8080 appeared. After restarting with RHQ_TESTING parameter and doing discovery -f at agent prompt, the second vhost 127.0.0.1:8081 is discovered. Marking as verified. Bulk closing of issues that were VERIFIED, had no target release and where the status changed more than a year ago. |