Bug 1328442

Summary: Missing Statistics Enabled configuration property for some resources
Product: [JBoss] JBoss Operations Network Reporter: Filip Brychta <fbrychta>
Component: Plugin -- JBoss EAP 6Assignee: Ruben Vargas Palma <rvargasp>
Status: CLOSED CURRENTRELEASE QA Contact: Filip Brychta <fbrychta>
Severity: medium Docs Contact:
Priority: high    
Version: JON 3.3.5CC: jmartine, loleary, mfoley, spinder
Target Milestone: ER02Keywords: Triaged
Target Release: One-off release   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 1328440 Environment:
Last Closed: 2017-07-13 15:17:20 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 1327633    

Description Filip Brychta 2016-04-19 11:55:39 UTC
+++ This bug was initially created as a clone of Bug #1328440 +++

Description of problem:
It's possible to set Statistics Enabled property on eap side (or via admin console) but it's not possible to do that via JON.
Found following resources which are missing this option:
- datasources, both xa and non-xa
- webservices subsystem

Version-Release number of selected component (if applicable):
JON3.3.5 with eap plugin pack update04 ER01

How reproducible:
Always

Steps to Reproduce:
1. navigate to any of previously mentioned resources
2. go to configuration tab


Actual results:
'Statistics enabled' is not displayed but it should because it's valid option for this resource on eap side.

Expected results:
It should be possible to set Statistics enabled: via JON for mentioned resources.

Additional info:
This property is available in JON for e.g. undertow subsystem.
This problem is valid for eap6 as well.

Comment 1 Simeon Pinder 2017-03-09 17:27:21 UTC
Master commits and move to ON_DEV.

commit 499bdca4d2a2d59cd4e92f1da17f1bbc384d420f
Merge: 3557efb 0ffac5d
Author:     Michael Burman <yak>
AuthorDate: Thu Mar 9 10:34:23 2017 +0200
Commit:     GitHub <noreply>
CommitDate: Thu Mar 9 10:34:23 2017 +0200

    Merge pull request #300 from rubenvp8510/BZ1328442
    
    Bug 1328442 - Missing Statistics Enabled configuration property for s…


commit 0ffac5df1513470c43ea10bb079837b2bfee22d7
Author:     Ruben Vargas <ruben.vp8510>
AuthorDate: Thu Mar 9 00:31:11 2017 -0600
Commit:     Ruben Vargas <ruben.vp8510>
CommitDate: Thu Mar 9 00:31:11 2017 -0600

    Bug 1328442 - Missing Statistics Enabled configuration property for some resources

Comment 2 Josejulio Martínez 2017-03-09 18:44:13 UTC
Moving to POST as the commits are already on master.

Comment 6 Ruben Vargas Palma 2017-04-10 14:34:24 UTC
commit 2fc7c1862189b321fb01c35101b2f9bab14eddd3
Merge: 129ca1c a768893
Author: Josejulio Martínez <finwemartinez>
Date:   Tue Apr 4 12:47:04 2017 -0500

    Merge pull request #302 from rubenvp8510/EAP6-no-stadistics
    
    Bug 1328442  Remove statistics-enabled from webservices subsystem on EAP6

commit a76889367c62dfeaf78664998e4568ef7ed203fc
Author: Ruben Vargas <ruben.vp8510>
Date:   Wed Mar 29 12:45:58 2017 -0600

    Remove statistics-enabled from webservices subsystem on EAP6

Comment 8 Simeon Pinder 2017-04-28 10:06:35 UTC
Moving to ON_QA for verification.

Where:
http://download.eng.bos.redhat.com/brewroot/packages/org.jboss.on-jboss-on-parent/3.3.0.GA/126/maven/org/jboss/on/jon-plugins-patch/3.3.0.GA/jon-plugins-patch-3.3.0.GA.zip

* NOTE: Unlike previous releases where the test binary was attached, this artifact is hosted in brew[1], where the jon-plugins-patch-3.3.0.GA.zip file IS the ER02 build of JON 3.3 EAP plugin pack update 04 and as such will be renamed to jon-plugin-pack-eap-3.3.0.GA-update-05.zip at publication time(unless replaced by a newer build).

[1]https://brewweb.engineering.redhat.com/brew/buildinfo?buildID=554191