lucas-a-martins opened a new pull request, #8480:
URL: https://github.com/apache/cloudstack/pull/8480

   ### Description
   
   Currently, when starting the Agent, ACS checks if the `cryptsetup` command 
can be used by using the command's `help`; the output of the command is printed 
in the logs. However, this output is too verbose and can be confusing.
   
   This PR addresses this issue by using the utility version to check the 
command, instead of `help`, which is much less verbose.
   
   ### Types of changes
   
   - [ ] Breaking change (fix or feature that would cause existing 
functionality to change)
   - [ ] New feature (non-breaking change which adds functionality)
   - [ ] Bug fix (non-breaking change which fixes an issue)
   - [x] Enhancement (improves an existing feature and functionality)
   - [ ] Cleanup (Code refactoring and cleanup, that may add test cases)
   - [ ] build/CI
   
   ### Feature/Enhancement Scale or Bug Severity
   
   #### Feature/Enhancement Scale
   
   - [ ] Major
   - [x] Minor
   
   #### Bug Severity
   
   - [ ] BLOCKER
   - [ ] Critical
   - [ ] Major
   - [ ] Minor
   - [ ] Trivial
   
   
   ### Screenshots (if appropriate):
   
   -  Before changes:
     
   
![grep_usage_host-1](https://github.com/apache/cloudstack/assets/56271185/260ceae6-fad0-46fb-96c1-ffc6f55294b5)
   
   -  After changes:
   
   
![grep_version_host-1](https://github.com/apache/cloudstack/assets/56271185/90c68e98-3f0b-4daf-805a-483902b57842)
   
   
   ### How Has This Been Tested?
   
   I checked the logs before and after changes. As shown in the screenshots 
above, the logs became more intuitive.
   


-- 
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: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to