This is an automated email from the ASF dual-hosted git repository.

dockerzhang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-inlong.git


The following commit(s) were added to refs/heads/master by this push:
     new f582b1f  [INLONG-3246][Manager] Add config_failed status after 
suspend/restart/delete failed for inlong group (#3247)
f582b1f is described below

commit f582b1f2943c2eab7c10c4bd1553e8efaaf802dd
Author: kipshi <[email protected]>
AuthorDate: Sun Mar 20 12:56:13 2022 +0800

    [INLONG-3246][Manager] Add config_failed status after 
suspend/restart/delete failed for inlong group (#3247)
---
 .../java/org/apache/inlong/manager/common/enums/GroupState.java     | 6 +++---
 .../service/source/listener/AbstractSourceOperateListener.java      | 1 +
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git 
a/inlong-manager/manager-common/src/main/java/org/apache/inlong/manager/common/enums/GroupState.java
 
b/inlong-manager/manager-common/src/main/java/org/apache/inlong/manager/common/enums/GroupState.java
index 14bf6c9..9ccb97a 100644
--- 
a/inlong-manager/manager-common/src/main/java/org/apache/inlong/manager/common/enums/GroupState.java
+++ 
b/inlong-manager/manager-common/src/main/java/org/apache/inlong/manager/common/enums/GroupState.java
@@ -71,13 +71,13 @@ public enum GroupState {
         GROUP_STATE_AUTOMATON.put(CONFIG_SUCCESSFUL,
                 Sets.newHashSet(CONFIG_SUCCESSFUL, TO_BE_APPROVAL, CONFIG_ING, 
SUSPENDING, DELETING, FINISH));
 
-        GROUP_STATE_AUTOMATON.put(SUSPENDING, Sets.newHashSet(SUSPENDING, 
SUSPENDED));
+        GROUP_STATE_AUTOMATON.put(SUSPENDING, Sets.newHashSet(SUSPENDING, 
SUSPENDED, CONFIG_FAILED));
         GROUP_STATE_AUTOMATON.put(SUSPENDED, Sets.newHashSet(SUSPENDED, 
RESTARTING, DELETING));
 
-        GROUP_STATE_AUTOMATON.put(RESTARTING, Sets.newHashSet(RESTARTING, 
RESTARTED));
+        GROUP_STATE_AUTOMATON.put(RESTARTING, Sets.newHashSet(RESTARTING, 
RESTARTED, CONFIG_FAILED));
         GROUP_STATE_AUTOMATON.put(RESTARTED, Sets.newHashSet(RESTARTED, 
SUSPENDING, TO_BE_APPROVAL, DELETING));
 
-        GROUP_STATE_AUTOMATON.put(DELETING, Sets.newHashSet(DELETING, 
DELETED));
+        GROUP_STATE_AUTOMATON.put(DELETING, Sets.newHashSet(DELETING, DELETED, 
CONFIG_FAILED));
         GROUP_STATE_AUTOMATON.put(DELETED, Sets.newHashSet(DELETED));
 
         GROUP_STATE_AUTOMATON.put(FINISH, Sets.newHashSet(FINISH, DELETING));
diff --git 
a/inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/source/listener/AbstractSourceOperateListener.java
 
b/inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/source/listener/AbstractSourceOperateListener.java
index 4eab60f..ac02145 100644
--- 
a/inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/source/listener/AbstractSourceOperateListener.java
+++ 
b/inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/source/listener/AbstractSourceOperateListener.java
@@ -108,6 +108,7 @@ public abstract class AbstractSourceOperateListener 
implements DataSourceOperate
             } else if (sourceState == SourceState.SOURCE_FAILED || sourceState 
== SourceState.SOURCE_DISABLE) {
                 return false;
             } else {
+                log.warn("StreamSource={} cannot be operated for state={}", 
sourceResponse, sourceState);
                 TimeUnit.SECONDS.sleep(5);
                 sourceResponse = 
streamSourceService.get(sourceResponse.getId(), sourceResponse.getSourceType());
             }

Reply via email to