Bug 565361

Summary: [raw-config]Rollback failing with Null pointer exception.
Product: [Other] RHQ Project Reporter: Preethi Thomas <pthomas>
Component: ConfigurationAssignee: RHQ Project Maintainer <rhq-maint>
Status: CLOSED CURRENTRELEASE QA Contact: Rajan Timaniya <rtimaniy>
Severity: medium Docs Contact:
Priority: medium    
Version: unspecifiedCC: jsanda, whayutin
Target Milestone: ---Keywords: SubBug
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: 2.4 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 557490 Environment:
Last Closed: 2010-08-12 16:57:58 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: 565635    

Description Preethi Thomas 2010-02-15 02:15:00 UTC
Description of problem:
Rollback failing with Null pointer exception.


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
  . 
    1. Edit & save config to generate more than one revision
    2. go to config->History
    3. Select an older version
    4. Click on Rollback to selected buttong
  

  
Actual results:
Rollback failing with Null pointer exception

Expected results:

Rollback successful 

Additional info:

Comment 1 John Sanda 2010-02-16 16:13:40 UTC
The rollback was actually happening, but an exception was getting thrown in the JSF action method after the rollback was performed which is why the status message indicated a NPE. ListConfigurationUpdateUIBean was calling ExistingResourceConfigurationUIBean to clear out its copy of the current configuration. This triggered the exception because ExistingResourceConfigurationUIBean is no longer used and has been replaced by the seam components, ResourceConfigurationViewer and ResourceConfigurationEditor.

Simply removing the call to ExistingResourceConfigurationUIBean resolved the issue.

Comment 2 wes hayutin 2010-02-16 16:57:34 UTC
Temporarily adding the keyword "SubBug" so we can be sure we have accounted for all the bugs.

keyword:
new = Tracking + FutureFeature + SubBug

Comment 3 wes hayutin 2010-02-16 17:02:33 UTC
making sure we're not missing any bugs in rhq_triage

Comment 4 Preethi Thomas 2010-02-16 17:42:26 UTC
looks like this has been fixed. moving to on_qa

Comment 5 Corey Welton 2010-03-24 03:00:52 UTC
qa -> rtimaniy

Comment 6 Rajan Timaniya 2010-03-26 10:52:25 UTC
Verified on master (build #133) GIT Revision: bf2ad7ab6e9532e4869f4373712fcbc0c04f8618

Modified configuration values many time and created multiple version under history. Select older version and rollback it.

Rollback is working fine, its not giving Null Pointer Exception.

Comment 7 Corey Welton 2010-08-12 16:57:58 UTC
Mass-closure of verified bugs against JON.