weizhouapache commented on code in PR #9840:
URL: https://github.com/apache/cloudstack/pull/9840#discussion_r1830802218


##########
engine/orchestration/src/main/java/com/cloud/agent/manager/AgentManagerImpl.java:
##########
@@ -223,17 +231,20 @@ public boolean configure(final String name, final 
Map<String, Object> params) th
 
         registerForHostEvents(new SetHostParamsListener(), true, true, false);
 
-        _executor = new ThreadPoolExecutor(threads, threads, 60l, 
TimeUnit.SECONDS, new LinkedBlockingQueue<Runnable>(), new 
NamedThreadFactory("AgentTaskPool"));
+        final int agentTaskThreads = DirectAgentLoadSize.value();
+        _executor = new ThreadPoolExecutor(agentTaskThreads, agentTaskThreads, 
60L, TimeUnit.SECONDS, new LinkedBlockingQueue<Runnable>(), new 
NamedThreadFactory("AgentTaskPool"));
 
         _connectExecutor = new ThreadPoolExecutor(100, 500, 60l, 
TimeUnit.SECONDS, new LinkedBlockingQueue<Runnable>(), new 
NamedThreadFactory("AgentConnectTaskPool"));
         // allow core threads to time out even when there are no items in the 
queue
         _connectExecutor.allowCoreThreadTimeOut(true);
 
-        _connection = new NioServer("AgentManager", Port.value(), 
Workers.value() + 10, this, caService);
+        _connection = new NioServer("AgentManager", Port.value(), 
Workers.value() + 10,
+                RemoteAgentSslHandshakeMinWorkers.value(), 
RemoteAgentSslHandshakeMaxWorkers.value(), this,
+                caService, RemoteAgentSslHandshakeTimeout.value());
         logger.info("Listening on {} with {} workers.", Port.value(), 
Workers.value());
 
         // executes all agent commands other than cron and ping
-        _directAgentExecutor = new 
ScheduledThreadPoolExecutor(DirectAgentPoolSize.value(), new 
NamedThreadFactory("DirectAgent"));
+        _directAgentExecutor = new 
ThreadPoolExecutor(Math.max(agentTaskThreads/10, 1), 
DirectAgentPoolSize.value(), 120L, TimeUnit.SECONDS, new 
LinkedBlockingQueue<>(), new NamedThreadFactory("DirectAgent"));

Review Comment:
   thanks for the checking
   resolved



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to