Skip to content

Marvin3#4727

Merged
yadvr merged 115 commits intomasterfrom
marvin3
May 4, 2021
Merged

Marvin3#4727
yadvr merged 115 commits intomasterfrom
marvin3

Conversation

@DaanHoogland
Copy link
Contributor

Description

This PR prepares marvin and tests for python3. it was part of #4479, until nose2 was decided to be abandoned from that PR.

re-PR of #4543 and #3730 to enable cooperation

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)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • Minor

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

How Has This Been Tested?

@DaanHoogland DaanHoogland mentioned this pull request Feb 24, 2021
12 tasks
@yadvr yadvr added this to the 4.16.0.0 milestone Feb 26, 2021
@DaanHoogland DaanHoogland added complexity:epic a few months to a year or more, with more people Severity:Major type:cleanup labels Feb 26, 2021
@apache apache deleted a comment from blueorangutan Feb 26, 2021
@apache apache deleted a comment from blueorangutan Feb 26, 2021
@apache apache deleted a comment from blueorangutan Feb 26, 2021
@apache apache deleted a comment from blueorangutan Feb 26, 2021
@apache apache deleted a comment from blueorangutan Feb 26, 2021
@apache apache deleted a comment from blueorangutan Feb 26, 2021
@DaanHoogland
Copy link
Contributor Author

reference marvin on python2 component test results:

# cat componenttest-summary.txt 
Test completed. 114 look ok, 57 have errors
Only failed tests results shown below:

@DaanHoogland
Copy link
Contributor Author

travis green \o/ cc @GabrielBrascher @rhtyd

@apache apache deleted a comment from blueorangutan Mar 1, 2021
@apache apache deleted a comment from blueorangutan Mar 1, 2021
@apache apache deleted a comment from GabrielBrascher Mar 1, 2021
@apache apache deleted a comment from blueorangutan Mar 1, 2021
@apache apache deleted a comment from blueorangutan Mar 1, 2021
@apache apache deleted a comment from GabrielBrascher Mar 1, 2021
@apache apache deleted a comment from yadvr Mar 1, 2021
@apache apache deleted a comment from GabrielBrascher Mar 1, 2021
@apache apache deleted a comment from blueorangutan Mar 1, 2021
@apache apache deleted a comment from blueorangutan Mar 1, 2021
@apache apache deleted a comment from blueorangutan Mar 1, 2021
@apache apache deleted a comment from blueorangutan Mar 2, 2021
@apache apache deleted a comment from blueorangutan Mar 3, 2021
@apache apache deleted a comment from blueorangutan Mar 3, 2021
@apache apache deleted a comment from blueorangutan Apr 28, 2021
@apache apache deleted a comment from blueorangutan Apr 28, 2021
@apache apache deleted a comment from blueorangutan Apr 28, 2021
@apache apache deleted a comment from blueorangutan Apr 28, 2021
@DaanHoogland
Copy link
Contributor Author

@blueorangutan test matrix

@blueorangutan
Copy link

Trillian test result (tid-554)
Environment: vmware-65u2 (x2), Advanced Networking with Mgmt server 7
Total time taken: 38974 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4727-t554-vmware-65u2.zip
Intermittent failure detected: /marvin/tests/smoke/test_nic.py
Intermittent failure detected: /marvin/tests/smoke/test_scale_vm.py
Intermittent failure detected: /marvin/tests/smoke/test_vm_life_cycle.py
Smoke tests completed. 85 look OK, 3 have error(s)
Only failed tests results shown below:

Test Result Time (s) Test File
test_03_nic_multiple_vmware Error 74.33 test_nic.py
ContextSuite context=TestScaleVm>:setup Error 0.00 test_scale_vm.py
test_01_migrate_VM_and_root_volume Error 0.01 test_vm_life_cycle.py
test_02_migrate_VM_with_two_data_disks Error 0.01 test_vm_life_cycle.py
test_03_migrate_detached_volume Error 0.01 test_vm_life_cycle.py

@blueorangutan
Copy link

Packaging result: ✔️ centos7 ✔️ centos8 ✔️ debian. SL-JID 488

@DaanHoogland
Copy link
Contributor Author

@blueorangutan test matrix

@blueorangutan
Copy link

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

@blueorangutan
Copy link

Trillian test result (tid-558)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 33349 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4727-t558-kvm-centos7.zip
Smoke tests completed. 88 look OK, 0 have error(s)
Only failed tests results shown below:

Test Result Time (s) Test File

@blueorangutan
Copy link

Trillian test result (tid-557)
Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7
Total time taken: 35552 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4727-t557-xenserver-71.zip
Intermittent failure detected: /marvin/tests/smoke/test_host_maintenance.py
Smoke tests completed. 88 look OK, 0 have error(s)
Only failed tests results shown below:

Test Result Time (s) Test File

@blueorangutan
Copy link

Trillian test result (tid-559)
Environment: vmware-65u2 (x2), Advanced Networking with Mgmt server 7
Total time taken: 39006 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4727-t559-vmware-65u2.zip
Intermittent failure detected: /marvin/tests/smoke/test_scale_vm.py
Smoke tests completed. 87 look OK, 1 have error(s)
Only failed tests results shown below:

Test Result Time (s) Test File
ContextSuite context=TestScaleVm>:setup Error 0.00 test_scale_vm.py

@blueorangutan
Copy link

Packaging result: ✔️ centos7 ✔️ centos8 ✔️ debian. SL-JID 495

@DaanHoogland
Copy link
Contributor Author

@blueorangutan test matrix

@blueorangutan
Copy link

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

@blueorangutan
Copy link

Trillian test result (tid-567)
Environment: xenserver-71 (x2), Advanced Networking with Mgmt server 7
Total time taken: 33696 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4727-t567-xenserver-71.zip
Smoke tests completed. 88 look OK, 0 have error(s)
Only failed tests results shown below:

Test Result Time (s) Test File

@blueorangutan
Copy link

Trillian test result (tid-568)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 33729 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4727-t568-kvm-centos7.zip
Smoke tests completed. 88 look OK, 0 have error(s)
Only failed tests results shown below:

Test Result Time (s) Test File

@blueorangutan
Copy link

Trillian test result (tid-569)
Environment: vmware-65u2 (x2), Advanced Networking with Mgmt server 7
Total time taken: 36686 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4727-t569-vmware-65u2.zip
Intermittent failure detected: /marvin/tests/smoke/test_password_server.py
Smoke tests completed. 88 look OK, 0 have error(s)
Only failed tests results shown below:

Test Result Time (s) Test File

@DaanHoogland
Copy link
Contributor Author

ping @GabrielBrascher @weizhouapache @rhtyd @shwstppr can we merge?

@pdion891
Copy link
Contributor

pdion891 commented May 4, 2021

+1

@yadvr yadvr mentioned this pull request May 4, 2021
@yadvr
Copy link
Member

yadvr commented May 4, 2021

+1 but we should run a test round against ubuntu and CentOS8 as well
@blueorangutan help

@blueorangutan
Copy link

@rhtyd I understand these words: "help", "hello", "thanks", "package", "test"
Test command usage: test [mgmt os] [hypervisor] [keepEnv]
Mgmt OS options: ['centos6', 'centos7', 'centos8', 'ubuntu18', 'ubuntu20']
Hypervisor options: ['kvm-centos6', 'kvm-centos7', 'kvm-centos8', 'kvm-ubuntu18', 'kvm-ubuntu20', 'xenserver-71', 'xenserver-65sp1', 'vmware-70u1', 'vmware-67u3', 'vmware-65u2', 'vmware-60u2', 'vmware-55u3', 'xcpng76', 'xcpng80', 'xcpng81', 'xenserver-74', 'xcpng74']
Note: when keepEnv is passed, you need to specify mgmt server os and hypervisor or use the matrix command.

Blessed contributors for kicking Trillian test jobs: ['rhtyd', 'nvazquez', 'PaulAngus', 'borisstoyanov', 'DaanHoogland', 'shwstppr', 'andrijapanicsb', 'Spaceman1984', 'Pearl1594', 'davidjumani', 'harikrishna-patnala', 'vladimirpetrov', 'sureshanaparti', 'weizhouapache']

@yadvr
Copy link
Member

yadvr commented May 4, 2021

Nevermind - I'll take my comment back, as Trilian uses CentOS7 to run tests so it wouldn't make a difference if we run a smoketests again with centos8/ubuntu etc. Let's merge based on test matrix and Travis! Fantastic effort @DaanHoogland

@DaanHoogland DaanHoogland mentioned this pull request Jun 7, 2021
8 tasks
@DaanHoogland DaanHoogland mentioned this pull request Aug 17, 2021
37 tasks
<exclude>**/*.md</exclude>
<exclude>.java-version</exclude>
<exclude>.python-version</exclude>
<exclude>systemvm/.pythen-version</exclude>
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes,m definitely, but also this file is not in use anymore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

complexity:epic a few months to a year or more, with more people Severity:Major type:cleanup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants