Attachment 588089 Details for Bug 826715 – patch for PoC to implement this

[patch] patch for PoC to implement this

bz826715.patch (text/plain), 3.19 KB, created by John Mazzitelli on 2012-05-31 13:24:38 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: John Mazzitelli

Created: 2012-05-31 13:24:38 UTC

Size: 3.19 KB

patch

obsolete

>diff --git a/modules/plugins/jmx/src/main/java/org/rhq/plugins/jmx/JavaUtilLoggingResourceComponent.java b/modules/plugins/jmx/src/main/java/org/rhq/plugins/jmx/JavaUtilLoggingResourceComponent.java
>index 85b6ee6..ac95048 100644
>--- a/modules/plugins/jmx/src/main/java/org/rhq/plugins/jmx/JavaUtilLoggingResourceComponent.java
>+++ b/modules/plugins/jmx/src/main/java/org/rhq/plugins/jmx/JavaUtilLoggingResourceComponent.java
>@@ -35,14 +35,31 @@ import org.rhq.core.domain.configuration.PropertyMap;
> import org.rhq.core.domain.configuration.PropertySimple;
> import org.rhq.core.pluginapi.configuration.ConfigurationUpdateReport;
> import org.rhq.core.pluginapi.inventory.PluginContainerDeployment;
>+import org.rhq.core.pluginapi.operation.OperationResult;
> 
> /**
>  * @author Greg Hinkle
>  */
> public class JavaUtilLoggingResourceComponent extends MBeanResourceComponent {
>+    private Configuration config = null;
>+
>+    @Override
>+    public OperationResult invokeOperation(String name, Configuration parameters) throws Exception {
>+        if ("rhq.reloadResourceConfiguration".equalsIgnoreCase(name)) {
>+            this.config = null;
>+            return new OperationResult("ok");
>+        } else {
>+            return super.invokeOperation(name, parameters);
>+        }
>+    }
>+
>     @Override
>     public Configuration loadResourceConfiguration() {
> 
>+        if (this.config != null) {
>+            return this.config;
>+        }
>+
>         if (getResourceContext().getPluginContainerDeployment() == PluginContainerDeployment.AGENT) {
>             PropertySimple configManagementEnabled = getResourceContext().getPluginConfiguration().getSimple(
>                 "configManagementEnabled");
>@@ -78,6 +95,7 @@ public class JavaUtilLoggingResourceComponent extends MBeanResourceComponent {
>         }
> 
>         configuration.put(list);
>+        this.config = configuration;
>         return configuration;
>     }
> 
>diff --git a/modules/plugins/jmx/src/main/resources/META-INF/rhq-plugin.xml b/modules/plugins/jmx/src/main/resources/META-INF/rhq-plugin.xml
>index 76572a3..378613b 100644
>--- a/modules/plugins/jmx/src/main/resources/META-INF/rhq-plugin.xml
>+++ b/modules/plugins/jmx/src/main/resources/META-INF/rhq-plugin.xml
>@@ -418,9 +418,12 @@
>          <plugin-configuration>
>             <c:simple-property name="objectName" readOnly="true" default="java.util.logging:type=Logging"/>
>             <c:simple-property name="nameTemplate" readOnly="true" default="Logging"/>
>-            <c:simple-property name="configManagementEnabled" type="boolean" default="false" description="When false, configuration reading for this resource will fail."/>
>+            <c:simple-property name="configManagementEnabled" type="boolean" default="true" description="When false, configuration reading for this resource will fail."/>
>          </plugin-configuration>
> 
>+         <operation name="rhq.reloadResourceConfiguration" displayName="Reload Configuration" description="When invoked, this operation prepares the resource to reload its resource configuration the next time it is requested.">
>+         </operation>
>+
>          <resource-configuration>
>             <c:list-property name="AppenderList" readOnly="true">
>                <c:map-property name="Appender">

Actions: View | Diff

Attachments on bug 826715: 588089