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
Allow autoscale group and name override #8324
Allow autoscale group and name override #8324
Conversation
@rg9975 |
We create an extension of AutoScaleManagerImpl.java, override these 2 methods, and override the spring-server-core-managers-context.xml to set the value of the manager to our extension class:
This allows us to deploy local changes that may not make sense for Cloudstack submission. In this case, very specific naming size and format requirements. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8324 +/- ##
============================================
+ Coverage 29.13% 29.15% +0.02%
- Complexity 31026 31269 +243
============================================
Files 5193 5251 +58
Lines 366232 368802 +2570
Branches 53546 53767 +221
============================================
+ Hits 106692 107524 +832
- Misses 244918 246564 +1646
- Partials 14622 14714 +92
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@rg9975 |
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.
LGTM
@blueorangutan package |
@shwstppr 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]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 7966 |
@blueorangutan test |
@DaanHoogland a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
[SF] Trillian test result (tid-8505)
|
Awesome work, congrats on your first merged pull request! |
Updates AutoScaleManager/AutoScaleManagerImpl so that getNextVmHostName and checkAutoScaleVmGroupName can be overridden in derivative implementations to allow for custom naming conditions and restrictions. If possible, would like to include this in 4.19 since it is a trivial change. This can be used to create an extension of AutoScaleManagerImpl.java, overriding these 2 methods
… conventions
Description
Updates AutoScaleManager/AutoScaleManagerImpl so that getNextVmHostName and checkAutoScaleVmGroupName can be overridden in derivative implementations to allow for custom naming conditions and restrictions. If possible, would like to include this in 4.19 since it is a trivial change.
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?
Extended implementation class with different VM names and tested for shorter auto-scale names.
How did you try to break this feature and the system with this change?
Use group names longer than the extended code allowed.