-
Notifications
You must be signed in to change notification settings - Fork 1.3k
test: fix test_vm_autoscaling.py which does not work due to userdata improvement #7921
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
test: fix test_vm_autoscaling.py which does not work due to userdata improvement #7921
Conversation
|
@blueorangutan LLpackage |
|
@weizhouapache a [LL] 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 [LL]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 6171 |
|
@blueorangutan LLtest |
Codecov Report
@@ Coverage Diff @@
## main #7921 +/- ##
============================================
- Coverage 14.40% 14.39% -0.01%
Complexity 10108 10108
============================================
Files 2748 2748
Lines 259404 259404
Branches 40391 40391
============================================
- Hits 37358 37354 -4
- Misses 217210 217218 +8
+ Partials 4836 4832 -4 see 2 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
|
@blueorangutan LLtest |
|
@weizhouapache a [LL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
|
[LL]Trillian test result (tid-6730) |
|
@blueorangutan test |
|
@weizhouapache a [SF] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
yadvr
left a comment
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
|
[SF] Trillian test result (tid-7604)
|
…improvement (apache#7921) (cherry picked from commit 78411fd) Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
Description
This PR fixes the smoke test
test_vm_autoscaling.pywhich failed at the setup stage , because the autoscale vm profile uses a userdata which does not have a header.it worked with 4.18, but not with main due to #7918
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?