Skip to content
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

Fix migrate vol xen vmware test #2755

Merged
merged 2 commits into from Aug 17, 2018

Conversation

borisstoyanov
Copy link
Contributor

Description

Adds a livemigrate = True to vmware and xenserver test execution, since it was reported in the latest failures
The volume Vol[254|vm=233|DATADISK]is attached to a vm and for migrating it the parameter livemigrate should be specified'

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)
  • Cleanup (Code refactoring and cleanup, that may add test cases)

GitHub Issue/PRs

Screenshots (if appropriate):

NA

How Has This Been Tested?

Trillian smoketests

Checklist:

  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
    Testing
  • I have added tests to cover my changes.
  • All relevant new and existing integration tests have passed.
  • A full integration testsuite with all test that can run on my environment has passed.

@borisstoyanov
Copy link
Contributor Author

@blueorangutan package

@blueorangutan
Copy link

@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

@blueorangutan
Copy link

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2188

@borisstoyanov
Copy link
Contributor Author

@blueorangutan test matrix

@blueorangutan
Copy link

@borisstoyanov a Trillian-Jenkins matrix job (centos6 mgmt + xs71, centos7 mgmt + vmware65, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests

@blueorangutan
Copy link

Trillian test result (tid-2864)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 35273 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2755-t2864-kvm-centos7.zip
Intermitten failure detected: /marvin/tests/smoke/test_certauthority_root.py
Intermitten failure detected: /marvin/tests/smoke/test_deploy_virtio_scsi_vm.py
Intermitten failure detected: /marvin/tests/smoke/test_primary_storage.py
Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermitten failure detected: /marvin/tests/smoke/test_snapshots.py
Intermitten failure detected: /marvin/tests/smoke/test_vm_life_cycle.py
Intermitten failure detected: /marvin/tests/smoke/test_host_maintenance.py
Intermitten failure detected: /marvin/tests/smoke/test_hostha_kvm.py
Smoke tests completed. 61 look OK, 7 have error(s)
Only failed tests results shown below:

Test Result Time (s) Test File
ContextSuite context=TestDeployVirtioSCSIVM>:setup Error 0.00 test_deploy_virtio_scsi_vm.py
test_01_add_primary_storage_disabled_host Error 0.93 test_primary_storage.py
test_01_primary_storage_nfs Error 0.14 test_primary_storage.py
ContextSuite context=TestStorageTags>:setup Error 0.25 test_primary_storage.py
test_03_vpc_privategw_restart_vpc_cleanup Failure 1314.02 test_privategw_acl.py
test_04_rvpc_privategw_static_routes Failure 354.97 test_privategw_acl.py
test_02_list_snapshots_with_removed_data_store Error 1.19 test_snapshots.py
test_01_secure_vm_migration Error 5.19 test_vm_life_cycle.py
test_02_unsecure_vm_migration Error 1.10 test_vm_life_cycle.py
test_03_secured_to_nonsecured_vm_migration Error 1.10 test_vm_life_cycle.py
test_04_nonsecured_to_secured_vm_migration Error 1.10 test_vm_life_cycle.py
ContextSuite context=TestVMLifeCycle>:setup Error 4.25 test_vm_life_cycle.py
test_01_cancel_host_maintenace_with_no_migration_jobs Failure 0.12 test_host_maintenance.py
test_02_cancel_host_maintenace_with_migration_jobs Error 2.34 test_host_maintenance.py
test_hostha_enable_ha_when_host_in_maintenance Error 1.61 test_hostha_kvm.py

@blueorangutan
Copy link

Trillian test result (tid-2863)
Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7
Total time taken: 40095 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2755-t2863-xenserver-71.zip
Intermitten failure detected: /marvin/tests/smoke/test_certauthority_root.py
Intermitten failure detected: /marvin/tests/smoke/test_scale_vm.py
Intermitten failure detected: /marvin/tests/smoke/test_volumes.py
Intermitten failure detected: /marvin/tests/smoke/test_vpc_vpn.py
Smoke tests completed. 66 look OK, 2 have error(s)
Only failed tests results shown below:

Test Result Time (s) Test File
test_01_scale_vm Error 22.64 test_scale_vm.py
test_11_migrate_volume_and_change_offering Error 10.52 test_volumes.py

@borisstoyanov
Copy link
Contributor Author

Current failures are cause around test_11_migrate_volume_and_change_offering by this: #2758 they should pass when the issue is fixed

Copy link
Member

@GabrielBrascher GabrielBrascher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @borisstoyanov. Based on code review, LGTM. Shall we wait for the fix [#2636] be merged to test it?

@blueorangutan
Copy link

Trillian test result (tid-2870)
Environment: vmware-65 (x2), Advanced Networking with Mgmt server 7
Total time taken: 42665 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2755-t2870-vmware-65.zip
Intermitten failure detected: /marvin/tests/smoke/test_certauthority_root.py
Intermitten failure detected: /marvin/tests/smoke/test_deploy_vm_iso.py
Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermitten failure detected: /marvin/tests/smoke/test_volumes.py
Smoke tests completed. 66 look OK, 2 have error(s)
Only failed tests results shown below:

Test Result Time (s) Test File
test_provision_certificate Error 7.24 test_certauthority_root.py
test_11_migrate_volume_and_change_offering Error 16.66 test_volumes.py

@borisstoyanov
Copy link
Contributor Author

yes, thanks @GabrielBrascher

@borisstoyanov
Copy link
Contributor Author

@blueorangutan package

@blueorangutan
Copy link

@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

@DaanHoogland
Copy link
Contributor

@borisstoyanov please rebase instead of merging master back in

@borisstoyanov borisstoyanov force-pushed the fix-migrate-vol-xen-vmware-test branch from 7d0f42b to c9ced96 Compare July 23, 2018 08:01
@rohityadavcloud rohityadavcloud added this to To do in 4.11 LTS via automation Jul 23, 2018
@rohityadavcloud rohityadavcloud added this to To do in Master (4.12.0.0) via automation Jul 23, 2018
@rohityadavcloud rohityadavcloud removed this from To do in 4.11 LTS Jul 23, 2018
@rohityadavcloud rohityadavcloud removed this from To do in Master (4.12.0.0) Jul 23, 2018
@borisstoyanov borisstoyanov force-pushed the fix-migrate-vol-xen-vmware-test branch from c9ced96 to b5f32f5 Compare July 23, 2018 08:09
@borisstoyanov
Copy link
Contributor Author

@blueorangutan package

@blueorangutan
Copy link

@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

@blueorangutan
Copy link

Packaging result: ✔centos6 ✖centos7 ✖debian. JID-2198

@blueorangutan
Copy link

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2200

@borisstoyanov
Copy link
Contributor Author

@blueorangutan test

@blueorangutan
Copy link

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

@blueorangutan
Copy link

Trillian test result (tid-2880)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 27325 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2755-t2880-kvm-centos7.zip
Intermitten failure detected: /marvin/tests/smoke/test_deploy_virtio_scsi_vm.py
Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermitten failure detected: /marvin/tests/smoke/test_routers_network_ops.py
Intermitten failure detected: /marvin/tests/smoke/test_vm_life_cycle.py
Intermitten failure detected: /marvin/tests/smoke/test_vpc_vpn.py
Smoke tests completed. 65 look OK, 3 have error(s)
Only failed tests results shown below:

Test Result Time (s) Test File
ContextSuite context=TestDeployVirtioSCSIVM>:setup Error 0.00 test_deploy_virtio_scsi_vm.py
test_03_vpc_privategw_restart_vpc_cleanup Failure 1117.00 test_privategw_acl.py
test_01_secure_vm_migration Error 131.69 test_vm_life_cycle.py
test_02_unsecure_vm_migration Error 132.73 test_vm_life_cycle.py
test_03_secured_to_nonsecured_vm_migration Error 131.72 test_vm_life_cycle.py
test_04_nonsecured_to_secured_vm_migration Error 132.73 test_vm_life_cycle.py

@borisstoyanov
Copy link
Contributor Author

@blueorangutan test

@blueorangutan
Copy link

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

@borisstoyanov
Copy link
Contributor Author

vm_lifecycle tests are caused by the following issue #2763

@rohityadavcloud
Copy link
Member

@borisstoyanov can you comment if this is ready to be merged?

@borisstoyanov
Copy link
Contributor Author

@rhtyd let me rebase and run tests again to see that we've actually fixed the issue.

@borisstoyanov borisstoyanov force-pushed the fix-migrate-vol-xen-vmware-test branch from b5f32f5 to 777a822 Compare July 26, 2018 07:54
@borisstoyanov
Copy link
Contributor Author

@blueorangutan package

@blueorangutan
Copy link

@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

@blueorangutan
Copy link

Packaging result: ✔centos6 ✖centos7 ✔debian. JID-2210

@borisstoyanov
Copy link
Contributor Author

@blueorangutan package

@blueorangutan
Copy link

@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

@blueorangutan
Copy link

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2211

@borisstoyanov
Copy link
Contributor Author

@blueorangutan test

@blueorangutan
Copy link

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

@rafaelweingartner
Copy link
Member

@borisstoyanov do you know what happened with the latest test run?

@rohityadavcloud
Copy link
Member

@borisstoyanov is this ready to merge, given other PRs have been merged?
@blueorangutan package

@blueorangutan
Copy link

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

@blueorangutan
Copy link

Packaging result: ✖centos6 ✖centos7 ✖debian. JID-2246

@DaanHoogland
Copy link
Contributor

@blueorangutan package

@blueorangutan
Copy link

@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

@blueorangutan
Copy link

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2253

@DaanHoogland
Copy link
Contributor

@blueorangutan test

@blueorangutan
Copy link

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

@blueorangutan
Copy link

Trillian test result (tid-2935)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 26545 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2755-t2935-kvm-centos7.zip
Intermitten failure detected: /marvin/tests/smoke/test_internal_lb.py
Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermitten failure detected: /marvin/tests/smoke/test_vm_life_cycle.py
Intermitten failure detected: /marvin/tests/smoke/test_hostha_kvm.py
Smoke tests completed. 68 look OK, 1 have error(s)
Only failed tests results shown below:

Test Result Time (s) Test File
test_03_vpc_privategw_restart_vpc_cleanup Failure 1167.49 test_privategw_acl.py

@DaanHoogland DaanHoogland merged commit bd7a09b into apache:master Aug 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants