weizhouapache commented on PR #7131:
URL: https://github.com/apache/cloudstack/pull/7131#issuecomment-1692909119

   > 
   > @weizhouapache The tests will run, but if you read the comment I pointed 
out, you will find that the system VMs will stop logging. Please, try accessing 
the system VMs with the change you proposed, and see if there is any logging. 
Last time I checked, it was not working; however, if it is, I'll be happy to 
comply.
   > 
   > Moreover, ACS is working perfectly without this change. The only thing 
that broke is the CI, and only because the CI is "manually" calling the 
EncriptorCLI without passing the proper classpath. This was already discussed 
and solved with @DaanHoogland.
   
   @JoaoJandre 
   this PR should help: https://github.com/apache/cloudstack/pull/7912


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