Bug 850572 - NullPointerException in DriftServerServiceImpl.getDriftDefinitions
Summary: NullPointerException in DriftServerServiceImpl.getDriftDefinitions
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: RHQ Project
Classification: Other
Component: Inventory
Version: 4.4
Hardware: All
OS: All
medium
medium
Target Milestone: ---
: RHQ 4.5.0
Assignee: Jay Shaughnessy
QA Contact: Mike Foley
URL:
Whiteboard:
: 850527 (view as bug list)
Depends On:
Blocks: 852092
TreeView+ depends on / blocked
 
Reported: 2012-08-21 20:48 UTC by Jay Shaughnessy
Modified: 2013-09-01 10:18 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
: 852092 (view as bug list)
Environment:
Last Closed: 2013-09-01 10:18:01 UTC
Embargoed:


Attachments (Terms of Use)

Description Jay Shaughnessy 2012-08-21 20:48:33 UTC
From Heiko:

(2:35:01 PM) pilhuhn: 14:30:21,072 WARN  [DiscoveryBossBean] Could not perform commit synchronization with agent for server [snert.local]
(2:35:01 PM) pilhuhn: java.lang.RuntimeException: java.lang.NullPointerException
(2:35:09 PM) pilhuhn: Caused by: java.lang.NullPointerException
(2:35:10 PM) pilhuhn: 	at org.rhq.enterprise.server.drift.DriftServerServiceImpl.getDriftDefinitions(DriftServerServiceImpl.java:93)
(3:13:18 PM) pilhuhn: saw that flying by when importing servers that were on the Q for some time. It may be that the resource types have changed since - I may have them discovered yesterday or earlier and did a source build this morning. I can't reproduce it on purpose, jshaughn
(3:14:23 PM) jshaughn: had you been doing anything with drift definitions?
(3:16:18 PM) pilhuhn: no, not at all

Comment 1 Jay Shaughnessy 2012-08-21 20:50:20 UTC
master commit 5ffd6e1eb155bf0208690cce526fb18b832ba344

Make drift def sync more efficient, and possibly avoid an NPE that was
seen intermittently.
- DriftServerServiceImpl.getDriftDefinitions() now only generates
  Map entries for resources with drift definitions, avoiding unnecessary
  list creation and subsequent processing.
- Added jdoc to that effect
- Changed private DriftSyncManager.syncConfigs() to syncDriftDefinitions
  for clarity.



Test Notes:
Inventory Sync testing would be appropriate for this.

Comment 2 Heiko W. Rupp 2012-08-27 15:24:11 UTC
*** Bug 850527 has been marked as a duplicate of this bug. ***

Comment 3 Heiko W. Rupp 2013-09-01 10:18:01 UTC
Bulk closing of items that are on_qa and in old RHQ releases, which are out for a long time and where the issue has not been re-opened since.


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