Red Hat Bugzilla – Full Text Bug Listing
|Summary:||[as7] Cannot unset resource configuration properties|
|Product:||[Other] RHQ Project||Reporter:||Stefan Negrea <snegrea>|
|Component:||Plugins||Assignee:||Stefan Negrea <snegrea>|
|Status:||CLOSED CURRENTRELEASE||QA Contact:||Mike Foley <mfoley>|
|Target Release:||RHQ 4.5.0|
|Fixed In Version:||Doc Type:||Bug Fix|
|Doc Text:||Story Points:||---|
|:||852662 (view as bug list)||Environment:|
|Last Closed:||2013-09-01 06:09:21 EDT||Type:||Bug|
|oVirt Team:||---||RHEL 7.3 requirements from Atomic Host:|
|Bug Depends On:|
|Bug Blocks:||707223, 852662, 1023564, 1031181|
Description Stefan Negrea 2012-08-28 18:25:25 EDT
Description of problem: For various AS7 subresources, cannot unset resource configuration properties. The properties are updated to any value that is non-empty. The properties however, do not get updated if either the unset checkbox is clicked or if the value from the field is empty. This only applies to non-required properties. For required properties the option to unset the property is not available, and the save button is disable until the field has a non-empty value (UI Validation). How reproducible: Alsways Steps to Reproduce: 1. Discover and import an AS7 standalone server 2. Find any resource with non-required properties that currently set. 3. Either click on the unset checkbox or remove all the contents of the textbox 4. Save the configuration Actual results: The save operation succeeds but the property values is still set to the old value (before unset) Expected results: The save operation succeeds but the property is now null or the default value if a default value is available for the property. Additional info: This bug does not apply to resource creation. If a field is set to unset in the resource creation page, the field will get the default AS7 value upon resource creation since the AS7 plugin will not send a property configuration request to AS7 for the respective property.
Comment 1 Stefan Negrea 2012-08-29 04:35:07 EDT
The following resources had to be updated because of this change. There is no functional update to any of these resources. All the changes are done to prevent regressions due to the plugin sending all the properties back to the server. 1) Transport (*/subsystem=infinispan/cache-container=*/transport=TRANSPORT): Removed Transport properties that were no longer in the resource definition. 2) Mod-cluster Configuration (/subsystem=modcluster,mod-cluster-config=configuration): Replaced a property name referenced by alias with the actual name. The alias is no longer working correctly. 3) Distributed Cache (*/subsystem=infinispan/cache-container=*/distributed-cache=*): Created a separate resource for distributed-cache because it has three extra properties when compared to invalidation-cache and replicated-cache. 4) Socket-Binding Group: Update the resource component to avoid sending port-offset configuration to the server for AS7 host controllers by removing the property completely from the definition and update report. Before this change, the property was not sent anyway because it was null, and null values were not sent to the server. 5) Transactions (*/subsystem=transactions): Added special validation rules for properties. 6) Data-source and XA Data-source: Updated the way configuration updates are sent for 9 properties. While these properties look undefined in the server, they cannot undefined or set to null at any point in time. The application server makes use of the default value at all times.
Comment 2 Stefan Negrea 2012-08-29 04:41:54 EDT
master branch commits: http://git.fedorahosted.org/cgit/rhq/rhq.git/commit/?id=c1655d4d576311b9034a31b505e75981b1d6e720 http://git.fedorahosted.org/cgit/rhq/rhq.git/commit/?id=9f82d413e980967ea3fae7ffe2579c44cc29cbb7 http://git.fedorahosted.org/cgit/rhq/rhq.git/commit/?id=9320bfa050c332a5f89334fcf5606395e5ef9098 http://git.fedorahosted.org/cgit/rhq/rhq.git/commit/?id=91aaaf71e585022ca85f62e371a7d63ae554ff6d http://git.fedorahosted.org/cgit/rhq/rhq.git/commit/?id=a31197fcd63548c6e49606525b9938c123e5ffb2 http://git.fedorahosted.org/cgit/rhq/rhq.git/commit/?id=b8d890aaf85b9d3f74eb60d1bad32af02d030759 http://git.fedorahosted.org/cgit/rhq/rhq.git/commit/?id=96602db100b087e36016b1aa6a2e39780b583898 http://git.fedorahosted.org/cgit/rhq/rhq.git/commit/?id=74aa0f7f9b0f2dd1044450d924f61b3faee78c27
Comment 3 Heiko W. Rupp 2013-09-01 06:09:21 EDT
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.