Merge pull request #1364 from mskacelik/JBEAP-26190

[JBEAP-26190] Domain mode has server-group missing Resume operation in Web Console

Author: hpehl

app/src/main/java/org/jboss/hal/client/runtime/group/ServerGroupColumn.java

@@ -162,23 +162,16 @@ public List<ItemAction<ServerGroup>> actions() {
                             .handler(serverGroupActions::restart)
                             .constraints(constraints(item, RESTART_SERVERS))
                             .build());
-                }
-                if (item.getServers(Server::isStarted).size() - item.getServers(Server::isSuspended)
-                        .size() > 0) {
                     actions.add(new ItemAction.Builder<ServerGroup>()
                             .title(resources.constants().suspend())
                             .handler(serverGroupActions::suspend)
                             .constraints(constraints(item, SUSPEND_SERVERS))
                             .build());
-                }
-                if (item.hasServers(Server::isSuspended)) {
                     actions.add(new ItemAction.Builder<ServerGroup>()
                             .title(resources.constants().resume())
                             .handler(serverGroupActions::resume)
                             .constraints(constraints(item, RESUME_SERVERS))
                             .build());
-                }
-                if (item.hasServers(Server::isStarted)) {
                     actions.add(new ItemAction.Builder<ServerGroup>()
                             .title(resources.constants().stop())
                             .handler(serverGroupActions::stop)

core/src/main/java/org/jboss/hal/core/runtime/group/ServerGroupActions.java

@@ -237,7 +237,9 @@ public void onError(Throwable error) {
     }
 
     public void resume(ServerGroup serverGroup) {
-        List<Server> suspendedServers = serverGroup.getServers(Server::isSuspended);
+        // Suspended state is not available (only data from server-config is),
+        // So let's use the started state from server-config, which is good enough here.
+        List<Server> suspendedServers = serverGroup.getServers(Server::isStarted);
         if (!suspendedServers.isEmpty()) {
             prepare(serverGroup, suspendedServers, RESUME);
             Operation operation = new Operation.Builder(serverGroup.getAddress(), RESUME_SERVERS).build();