[Bug]:1.0.11 AgentScopeJvmShutdownHook conflict with spring

[lfz757077613]

spring工程的SIGTERM信号响应是由spring的ShutdownHook处理的，agentscope又注册了一个AgentScopeJvmShutdownHook。这两个hook之间是并行的，会造成spring容器没有完全关闭时会正常接受请求，但是agent已经关闭了，调用会报错返回Operation interrupted due to system shutting down, please retry。
按照我的理解，在spring工程中agentscope不需要这种hook，生命周期是由spring统一管理。
之前dubbo也有这种问题，提供了一个ShutdownHook和spring的ShutdownHook并发冲突。

[lfz757077613]

jdk12以上没有办法通过反射修改final字段，也就修改不了AgentScopeJvmShutdownHook中的private static final AtomicBoolean REGISTERED属性。
目前看只能在自己工程实现一个相同全路径名的AgentScopeJvmShutdownHook 取代 agentscope自身的这个类解决问题，类加载机制优先加载自己工程内的该类。

package io.agentscope.core.shutdown;
import lombok.extern.slf4j.Slf4j;
@Slf4j
public final class AgentScopeJvmShutdownHook {
    public static void register(GracefulShutdownManager manager) {
        log.info("replace AgentScopeJvmShutdownHook");
    }
}

[uuuyuqi]

按照目前我们推的最佳实践，会在 prestop 完成之前就完成负载均衡侧摘流。因此应用收到 term 信号的情况下，如果仍然有新请求进来时，确实会有你说的问题。

这个地方是需要修复的，我建议是默认关闭 AgentScopeJvmShutdownHook

然后，在非 Spring 环境，或者应用接收 term 信号阶段保证没有新请求进来的情况下，可以手工注册这个 hook。Spring 场景下想做 agent 优雅下线，可以手工 check 这个方法的返回值是否归 0（在途请求已经执行完毕）： io.agentscope.core.shutdown.GracefulShutdownManager#getActiveRequestCount

cc @LearningGp

[Wzy19930507]

@uuuyuqi @LearningGp @uuuyuqi 你们好，能把这个工作分配给我吗？

我准备在 GracefulShutdownConfig 中加一个配置 isRegister 默认 false，

然后在 AgentScopeJvmShutdownHook 中判断是否注册

    public static void register(GracefulShutdownManager manager) {
        if (!manager.getConfig().isRegister()) {
            return;
        }
        ......
    }   

在 GracefulShutdownConfig 中提供兼容向后兼容的构造方法，默认不注册 shutdown hook

    public GracefulShutdownConfig(Duration shutdownTimeout, PartialReasoningPolicy partialReasoningPolicy) {
        this(shutdownTimeout, partialReasoningPolicy, false); 
    }

[uuuyuqi]

@uuuyuqi @LearningGp @uuuyuqi 你们好，能把这个工作分配给我吗？

我准备在 GracefulShutdownConfig 中加一个配置 isRegisterJvmShutdownHook 默认 false，

然后在 AgentScopeJvmShutdownHook 中判断是否注册

    public static void register(GracefulShutdownManager manager) {
        if (!manager.getConfig().isRegisterJvmShutdownHook()) {
            return;
        }
        ......
    }   

在 GracefulShutdownConfig 中提供兼容向后兼容的构造方法，默认不注册 shutdown hook

    public GracefulShutdownConfig(Duration shutdownTimeout, PartialReasoningPolicy partialReasoningPolicy) {
        this(shutdownTimeout, partialReasoningPolicy, false); 
    }

你想表达应该是，isRegister 这个参数直接放到构造方法里面，这样的话应该没啥大问题

[Wzy19930507]

@uuuyuqi @LearningGp @uuuyuqi 你们好，能把这个工作分配给我吗？
我准备在 GracefulShutdownConfig 中加一个配置 isRegisterJvmShutdownHook 默认 false，
然后在 AgentScopeJvmShutdownHook 中判断是否注册

    public static void register(GracefulShutdownManager manager) {
        if (!manager.getConfig().isRegisterJvmShutdownHook()) {
            return;
        }
        ......
    }   

在 GracefulShutdownConfig 中提供兼容向后兼容的构造方法，默认不注册 shutdown hook

    public GracefulShutdownConfig(Duration shutdownTimeout, PartialReasoningPolicy partialReasoningPolicy) {
        this(shutdownTimeout, partialReasoningPolicy, false); 
    }

你想表达应该是，isRegister 这个参数直接放到构造方法里面，这样的话应该没啥大问题

是的，GracefulShutdownConfig 的构造方法里加个参数 isRegister。