-
Notifications
You must be signed in to change notification settings - Fork 1.2k
java: fix one typo in many files #11284
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
CheckDataStoreStoragePolicyComplainceCommand -> CheckDataStoreStoragePolicyComplianceCommand Also renamed the file
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #11284 +/- ##
=========================================
Coverage 17.39% 17.39%
- Complexity 15283 15285 +2
=========================================
Files 5889 5889
Lines 526141 526141
Branches 64234 64234
=========================================
Hits 91542 91542
Misses 424265 424265
Partials 10334 10334
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
I think this fix must go in but it is a risky one as a lot of code changes. |
@DaanHoogland 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. |
Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 14344 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clgtm
@blueorangutan test matrix |
@DaanHoogland a [SL] Trillian-Jenkins matrix job (EL8 mgmt + EL8 KVM, Ubuntu22 mgmt + Ubuntu22 KVM, EL8 mgmt + VMware 7.0u3, EL9 mgmt + XCP-ng 8.2 ) has been kicked to run smoke tests |
[SF] Trillian test result (tid-13887)
|
[SF] Trillian test result (tid-13888)
|
[SF] Trillian test result (tid-13890)
|
[SF] Trillian test result (tid-13889)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code LGTM
This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch. |
@blueorangutan package |
@DaanHoogland 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. |
Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ el10 ✔️ debian ✔️ suse15. SL-JID 15002 |
@blueorangutan test matrix |
@DaanHoogland a [SL] Trillian-Jenkins matrix job (EL8 mgmt + EL8 KVM, Ubuntu22 mgmt + Ubuntu22 KVM, EL8 mgmt + VMware 7.0u3, EL9 mgmt + XCP-ng 8.2 ) has been kicked to run smoke tests |
[SF] Trillian Build Failed (tid-14338) |
[SF] Trillian test result (tid-14339)
|
[SF] Trillian test result (tid-14341)
|
@blueorangutan package |
@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. |
Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ el10 ✔️ debian ✔️ suse15. SL-JID 15045 |
[SF] Trillian test result (tid-14358)
|
[SF] Trillian test result (tid-14340)
|
CheckDataStoreStoragePolicyComplainceCommand -> CheckDataStoreStoragePolicyComplianceCommand
isStoragePoolStoragepolicyComplaince -> isStoragePoolStoragepolicyCompliance
Also renamed the file. So the one typo is:
Complaince
is changed toCompliance
and also fixed the all lowercase versionDescription
This PR...
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?
How did you try to break this feature and the system with this change?