rhtyd commented on a change in pull request #4978:
URL: https://github.com/apache/cloudstack/pull/4978#discussion_r708903698



##########
File path: debian/control
##########
@@ -56,3 +56,10 @@ Package: cloudstack-integration-tests
 Architecture: all
 Depends: ${misc:Depends}, cloudstack-marvin (= ${source:Version})
 Description: The CloudStack Marvin integration tests
+
+Package: cloudstack-agent-ha-helper

Review comment:
       @GabrielBrascher we'll need similar for CentOS7, 8 and Suse packaging.
   Can you advise why we need a separate process, can't the agent do this or a 
thread in libvirtcomputingresource? And it seems it may create a security 
issue, is the service port (8080) secured and authenticated?




-- 
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