cloudstack icon indicating copy to clipboard operation
cloudstack copied to clipboard

Fix spelling in docs, logs, exception messages etc

Open jbampton opened this issue 1 year ago • 4 comments

Description

This PR cleans up the codebase by fixing spelling.

Fixed grammar and removed whitespace.

Used the correct case of some terms.

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)
  • [X] Cleanup (Code refactoring and cleanup, that may add test cases)
  • [ ] build/CI

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • [ ] Major
  • [ ] Minor

Bug Severity

  • [ ] BLOCKER
  • [ ] Critical
  • [ ] Major
  • [ ] Minor
  • [ ] Trivial

Screenshots (if appropriate):

How Has This Been Tested?

How did you try to break this feature and the system with this change?

jbampton avatar May 12 '24 23:05 jbampton

Codecov Report

Attention: Patch coverage is 0% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 15.28%. Comparing base (21af134) to head (aa72704).

Files Patch % Lines
...ain/java/com/cloud/network/NetworkServiceImpl.java 0.00% 3 Missing :warning:
Additional details and impacted files
@@             Coverage Diff             @@
##               main    #9076     +/-   ##
===========================================
  Coverage     15.28%   15.28%             
- Complexity    11535    11536      +1     
===========================================
  Files          5425     5425             
  Lines        474138   474138             
  Branches      58984    60484   +1500     
===========================================
+ Hits          72486    72492      +6     
+ Misses       393584   393577      -7     
- Partials       8068     8069      +1     
Flag Coverage Δ
uitests 4.26% <ø> (ø)
unittests 16.02% <0.00%> (+<0.01%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar May 12 '24 23:05 codecov[bot]

@blueorangutan package

jbampton avatar May 13 '24 00:05 jbampton

@jbampton a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

blueorangutan avatar May 13 '24 00:05 blueorangutan

Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 9601

blueorangutan avatar May 13 '24 01:05 blueorangutan

@blueorangutan test

DaanHoogland avatar May 18 '24 20:05 DaanHoogland

@DaanHoogland a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

blueorangutan avatar May 18 '24 20:05 blueorangutan

[SF] Trillian Build Failed (tid-10216)

blueorangutan avatar May 18 '24 20:05 blueorangutan

@blueorangutan test

DaanHoogland avatar May 22 '24 08:05 DaanHoogland

@DaanHoogland a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

blueorangutan avatar May 22 '24 08:05 blueorangutan

[SF] Trillian test result (tid-10246) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 47080 seconds Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr9076-t10246-kvm-centos7.zip Smoke tests completed. 130 look OK, 1 have errors, 0 did not run Only failed and skipped tests results shown below:

Test Result Time (s) Test File
test_01_events_resource Error 417.49 test_events_resource.py

blueorangutan avatar May 22 '24 22:05 blueorangutan