Improves email configurations descriptions

[stephankruggg]

Description

The alert.email.addresses description is ambiguous and can cause doubts to operators. This description has been altered to avoid confusion. In addition, typos in alert.smtp.useStartTLS and project.smtp.useStartTLS have been fixed.

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)
• Enhancement (improves an existing feature and functionality)
• Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

• Major
• Minor

How Has This Been Tested?

In a local lab I applied this patch and the descriptions were successfully updated.

[boring-cyborg]

Congratulations on your first Pull Request and welcome to the Apache CloudStack community! If you have any issues or are unsure about any anything please check our Contribution Guide (https://github.com/apache/cloudstack/blob/main/CONTRIBUTING.md)
Here are some useful points:

• In case of a new feature add useful documentation (raise doc PR at https://github.com/apache/cloudstack-documentation)
• Be patient and persistent. It might take some time to get a review or get the final approval from the committers.
• Pay attention to the quality of your code, ensure tests are passing and your PR doesn't have conflicts.
• Please follow ASF Code of Conduct for all communication including (but not limited to) comments on Pull Requests, Issues, Mailing list and Slack.
• Be sure to read the CloudStack Coding Conventions.
  Apache CloudStack is a community-driven project and together we are making it better 🚀.
  In case of doubts contact the developers at:
  Mailing List: dev@cloudstack.apache.org (https://cloudstack.apache.org/mailing-lists.html)
  Slack: https://apachecloudstack.slack.com/

[codecov]

Codecov Report

Merging #6806 (55d7fb8) into main (713a236) will decrease coverage by 0.06%.
The diff coverage is n/a.

❗ Current head 55d7fb8 differs from pull request most recent head b6e449b. Consider uploading reports for the commit b6e449b to get more accurate results

@@             Coverage Diff              @@
##               main    #6806      +/-   ##
============================================
- Coverage     10.60%   10.54%   -0.07%     
+ Complexity     6849     6788      -61     
============================================
  Files          2466     2464       -2     
  Lines        244549   244168     -381     
  Branches      38262    38205      -57     
============================================
- Hits          25936    25741     -195     
+ Misses       215331   215178     -153     
+ Partials       3282     3249      -33     

Impacted Files	Coverage Δ
...pi/src/main/java/com/cloud/alert/AlertManager.java	100.00% <ø> (ø)
.../src/main/java/com/cloud/configuration/Config.java	89.23% <ø> (ø)
...c/main/java/com/cloud/projects/ProjectManager.java	0.00% <ø> (ø)
...ne/schema/src/main/java/com/cloud/vm/UserVmVO.java	28.57% <0.00%> (-19.16%)	⬇️
...dstack/network/contrail/model/ModelObjectBase.java	21.15% <0.00%> (-7.70%)	⬇️
...in/java/com/cloud/server/ManagementServerImpl.java	2.47% <0.00%> (-2.73%)	⬇️
...n/java/com/cloud/template/TemplateManagerImpl.java	10.97% <0.00%> (-1.41%)	⬇️
...chema/src/main/java/com/cloud/vm/VMInstanceVO.java	34.52% <0.00%> (-1.20%)	⬇️
...a/com/cloud/network/router/CommandSetupHelper.java	0.12% <0.00%> (-1.14%)	⬇️
.../src/main/java/com/cloud/vm/UserVmManagerImpl.java	5.60% <0.00%> (-1.01%)	⬇️
... and 16 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[JoaoJandre]

@stephankruggg we could use some revision in QuotaConfig.java as well.

[github-actions]

This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

80.0% Coverage
0.0% Duplication

[JoaoJandre]

LGTM

[yadvr]

Given the changes are in doc strings of Config/ConfigKey and Travis has passed, merging based on that.

[boring-cyborg]

Awesome work, congrats on your first merged pull request!