[(nobranch,rebasingWFCORE-6503)]: [WFCORE-6503]: Adding warning if YA…

…ML is adding an existing resource without any attribute. Signed-off-by: Emmanuel Hugonnet <[email protected]>
wildfly · Mar 1, 2024 · 5f3fbd9 · 5f3fbd9
1 parent 0ffdfe4
commit 5f3fbd9
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/controller/src/main/java/org/jboss/as/controller/logging/ControllerLogger.java b/controller/src/main/java/org/jboss/as/controller/logging/ControllerLogger.java
@@ -3682,7 +3682,7 @@ OperationFailedRuntimeException capabilityAlreadyRegisteredInContext(String capa
     IllegalArgumentException noResourceForUndefiningAttribute(String attribute, String address);
 
     @LogMessage(level = WARN)
-    @Message(id = 490, value = "You have defined a resource for address %s without any attributes, doing nothing")
+    @Message(id = 490, value = " A YAML resource has been defined for the address %s without any attributes. No actions will be taken.")
     void noAttributeSetForAddress(String address);
 
     @LogMessage(level = WARN)

diff --git a/...er/src/main/java/org/jboss/as/controller/persistence/yaml/YamlConfigurationExtension.java b/...er/src/main/java/org/jboss/as/controller/persistence/yaml/YamlConfigurationExtension.java
@@ -241,6 +241,8 @@ private void processResource(PathAddress parentAddress, Map<String, Object> yaml
                             //we are processing an attribute:
                             MGMT_OP_LOGGER.debugf("We are processing the attribute %s for address %s", name, address.getParent().toCLIStyleString());
                             processAttribute(parentAddress, rootRegistration, name, value, postExtensionOps, xmlOperations);
+                        } else if (value == null) {
+                            MGMT_OP_LOGGER.noAttributeSetForAddress(address.toCLIStyleString());
                         }
                     }
                 } else {