-
Notifications
You must be signed in to change notification settings - Fork 1.3k
CLOUDSTACK-9893 / CLOUDSTACK-8239 fix test_deploy_virtio_scsi_vm.py #2066
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
CLOUDSTACK-9893 / CLOUDSTACK-8239 fix test_deploy_virtio_scsi_vm.py #2066
Conversation
|
test results on kvm: |
|
@blueorangutan package |
|
@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
code LGTM, i'll be kicking tests on vmware to make sure ;) |
|
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-677 |
|
@blueorangutan test centos7 vmware-55u3 |
|
@DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + vmware-55u3) has been kicked to run smoke tests |
|
code LGTM. will merge once the tests are ok. |
|
LGTM. @nathanejohnson does this also affect 4.9, if so please change the base branch to 4.9 thanks. |
|
@rhtyd The original PR was only pulled into master (4.10) as far as I'm aware. |
|
Thanks @kiwiflyer |
|
Trillian test result (tid-1039)
|
c5cc989 to
ccf0f61
Compare
|
@DaanHoogland could you kick this off one more time? I should have the issue with the class teardown resolved with the latest rebase |
…m.py Fix issue where unsupported hypervisors were not skipping tests Fix issue where the template referenced no longer existed on openvm.eu
ccf0f61 to
91e237c
Compare
|
@blueorangutan package |
|
@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
|
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-681 |
|
@blueorangutan test centos7 vmware-55u3 |
|
@rhtyd a Trillian-Jenkins test job (centos7 mgmt + vmware-55u3) has been kicked to run smoke tests |
|
Trillian test result (tid-1043)
|
|
thanks @rhtyd and @DaanHoogland @karuturi looks like kvm tests succeed and other hypervisors should now properly skip the test. |
|
LGTM. |
|
@karuturi can you merge this? |
Fix issue where unsupported hypervisors were not skipping tests
Fix issue where the template referenced no longer existed on openvm.eu
@karuturi @DaanHoogland @serg38 @kiwiflyer