Skip to content

cks: assorted fixes, test refactoring#4269

Merged
yadvr merged 27 commits intoapache:4.14from
shapeblue:cks-test-refactoring
Sep 29, 2020
Merged

cks: assorted fixes, test refactoring#4269
yadvr merged 27 commits intoapache:4.14from
shapeblue:cks-test-refactoring

Conversation

@shwstppr
Copy link
Copy Markdown
Contributor

@shwstppr shwstppr commented Aug 17, 2020

Description

Fixes #4265
Fixes for:

  • template selection
  • network cleanup
  • capacity check
  • specific patch version during deployment

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)

Screenshots (if appropriate):

How Has This Been Tested?

@yadvr yadvr added this to the 4.15.0.0 milestone Aug 17, 2020
@yadvr yadvr added the Severity:Critical Critical bug label Aug 17, 2020
"minmemory": 2048
}
},
"cks_templates": {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@shwstppr you'll need to sync changes in test_data in trillian specific test_data as well

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Have done that

@yadvr
Copy link
Copy Markdown
Member

yadvr commented Aug 20, 2020

@shwstppr is this ready for testing?

@blueorangutan
Copy link
Copy Markdown

Packaging result: ✔centos7 ✔centos8 ✔debian. JID-1793

@blueorangutan
Copy link
Copy Markdown

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

Test Result Time (s) Test File

@blueorangutan
Copy link
Copy Markdown

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

Test Result Time (s) Test File

@yadvr
Copy link
Copy Markdown
Member

yadvr commented Aug 25, 2020

@blueorangutan package

@blueorangutan
Copy link
Copy Markdown

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

@blueorangutan
Copy link
Copy Markdown

Packaging result: ✔centos7 ✖centos8 ✔debian. JID-1816

@yadvr yadvr closed this Aug 25, 2020
@yadvr yadvr reopened this Aug 25, 2020
@yadvr
Copy link
Copy Markdown
Member

yadvr commented Aug 25, 2020

@blueorangutan package

1 similar comment
@yadvr
Copy link
Copy Markdown
Member

yadvr commented Aug 28, 2020

@blueorangutan package

@blueorangutan
Copy link
Copy Markdown

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

@blueorangutan
Copy link
Copy Markdown

Packaging result: ✔centos7 ✖centos8 ✔debian. JID-1834

@yadvr
Copy link
Copy Markdown
Member

yadvr commented Aug 28, 2020

@blueorangutan test

@blueorangutan
Copy link
Copy Markdown

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

@blueorangutan
Copy link
Copy Markdown

Packaging result: ✔centos7 ✔centos8 ✔debian. JID-1853

@shwstppr shwstppr force-pushed the cks-test-refactoring branch from d79d8af to 72b1691 Compare August 31, 2020 13:56
@blueorangutan
Copy link
Copy Markdown

Packaging result: ✔centos7 ✔centos8 ✔debian. JID-1854

@blueorangutan
Copy link
Copy Markdown

Packaging result: ✔centos7 ✔centos8 ✔debian. JID-1856

@yadvr yadvr closed this Sep 1, 2020
@yadvr yadvr reopened this Sep 1, 2020
@shwstppr
Copy link
Copy Markdown
Contributor Author

shwstppr commented Sep 1, 2020

@blueorangutan test

@blueorangutan
Copy link
Copy Markdown

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

@blueorangutan
Copy link
Copy Markdown

Trillian test result (tid-2633)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 68413 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4269-t2633-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_certauthority_root.py
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Intermittent failure detected: /marvin/tests/smoke/test_router_dhcphosts.py
Intermittent failure detected: /marvin/tests/smoke/test_routers_network_ops.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_router_nics.py
Intermittent failure detected: /marvin/tests/smoke/test_hostha_kvm.py
Smoke tests completed. 81 look OK, 4 have error(s)
Only failed tests results shown below:

Test Result Time (s) Test File
test_provision_certificate Failure 48.28 test_certauthority_root.py
test_01_deploy_kubernetes_cluster Error 0.05 test_kubernetes_clusters.py
test_02_deploy_kubernetes_ha_cluster Error 33.15 test_kubernetes_clusters.py
test_05_deploy_and_upgrade_kubernetes_ha_cluster Error 0.04 test_kubernetes_clusters.py
test_06_deploy_and_invalid_upgrade_kubernetes_cluster Error 0.04 test_kubernetes_clusters.py
test_07_deploy_and_scale_kubernetes_cluster Error 0.04 test_kubernetes_clusters.py
ContextSuite context=TestKubernetesCluster>:teardown Error 41.16 test_kubernetes_clusters.py
test_04_rvpc_network_garbage_collector_nics Error 3836.86 test_vpc_redundant.py
test_hostha_kvm_host_fencing Error 167.92 test_hostha_kvm.py

@blueorangutan
Copy link
Copy Markdown

Packaging result: ✖centos7 ✖centos8 ✖debian. JID-1881

@yadvr
Copy link
Copy Markdown
Member

yadvr commented Sep 2, 2020

@blueorangutan package

@blueorangutan
Copy link
Copy Markdown

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

@shwstppr
Copy link
Copy Markdown
Contributor Author

@blueorangutan test

@blueorangutan
Copy link
Copy Markdown

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

@blueorangutan
Copy link
Copy Markdown

Trillian test result (tid-2751)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 56453 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4269-t2751-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Smoke tests completed. 82 look OK, 1 have error(s)
Only failed tests results shown below:

Test Result Time (s) Test File
test_01_deploy_kubernetes_cluster Error 3613.82 test_kubernetes_clusters.py
test_02_invalid_upgrade_kubernetes_cluster Error 3616.58 test_kubernetes_clusters.py
test_03_deploy_and_upgrade_kubernetes_cluster Error 0.05 test_kubernetes_clusters.py
test_04_deploy_and_scale_kubernetes_cluster Error 0.06 test_kubernetes_clusters.py
test_05_delete_kubernetes_cluster Error 0.05 test_kubernetes_clusters.py
test_07_deploy_kubernetes_ha_cluster Error 0.05 test_kubernetes_clusters.py
test_08_deploy_and_upgrade_kubernetes_ha_cluster Error 0.06 test_kubernetes_clusters.py
test_09_delete_kubernetes_ha_cluster Error 0.06 test_kubernetes_clusters.py
ContextSuite context=TestKubernetesCluster>:teardown Error 130.63 test_kubernetes_clusters.py

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@blueorangutan
Copy link
Copy Markdown

Packaging result: ✔centos7 ✖centos8 ✔debian. JID-2019

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@blueorangutan
Copy link
Copy Markdown

Packaging result: ✔centos7 ✖centos8 ✔debian. JID-2025

@yadvr
Copy link
Copy Markdown
Member

yadvr commented Sep 18, 2020

@blueorangutan test

@blueorangutan
Copy link
Copy Markdown

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

@blueorangutan
Copy link
Copy Markdown

Trillian test result (tid-2777)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 67746 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4269-t2777-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Intermittent failure detected: /marvin/tests/smoke/test_outofbandmanagement.py
Smoke tests completed. 81 look OK, 2 have error(s)
Only failed tests results shown below:

Test Result Time (s) Test File
test_01_deploy_kubernetes_cluster Failure 3625.80 test_kubernetes_clusters.py
test_04_deploy_and_scale_kubernetes_cluster Failure 31.88 test_kubernetes_clusters.py
test_07_deploy_kubernetes_ha_cluster Failure 54.85 test_kubernetes_clusters.py
test_08_deploy_and_upgrade_kubernetes_ha_cluster Failure 0.07 test_kubernetes_clusters.py
test_09_delete_kubernetes_ha_cluster Failure 0.06 test_kubernetes_clusters.py
ContextSuite context=TestKubernetesCluster>:teardown Error 99.27 test_kubernetes_clusters.py
test_oobm_issue_power_cycle Error 15.95 test_outofbandmanagement.py
test_oobm_issue_power_soft Error 16.42 test_outofbandmanagement.py
test_oobm_zchange_password Error 9.84 test_outofbandmanagement.py

@yadvr
Copy link
Copy Markdown
Member

yadvr commented Sep 21, 2020

@shwstppr unfortunately, tests are still intermittently failing

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@blueorangutan
Copy link
Copy Markdown

Packaging result: ✔centos7 ✖centos8 ✔debian. JID-2057

Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@blueorangutan
Copy link
Copy Markdown

Packaging result: ✔centos7 ✖centos8 ✔debian. JID-2067

@blueorangutan
Copy link
Copy Markdown

Trillian test result (tid-2838)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 34704 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4269-t2838-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_pvlan.py
Smoke tests completed. 82 look OK, 1 have error(s)
Only failed tests results shown below:

Test Result Time (s) Test File
test_create_pvlan_network Error 0.04 test_pvlan.py

@yadvr
Copy link
Copy Markdown
Member

yadvr commented Sep 23, 2020

@davidjumani the pvlan test seems to be failing sometimes, can you help debug if it's env related or an intermittent failure?

@davidjumani
Copy link
Copy Markdown
Contributor

davidjumani commented Sep 23, 2020

@rhtyd Looks like it's because the same vlan id is used elsewhere. Changed in #4340

@yadvr
Copy link
Copy Markdown
Member

yadvr commented Sep 24, 2020

@blueorangutan test centos7 vmware-67u3

@blueorangutan
Copy link
Copy Markdown

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) has been kicked to run smoke tests

@blueorangutan
Copy link
Copy Markdown

Trillian test result (tid-2848)
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 43270 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4269-t2848-vmware-67u3.zip
Intermittent failure detected: /marvin/tests/smoke/test_vpc_router_nics.py
Smoke tests completed. 79 look OK, 0 have error(s)
Only failed tests results shown below:

Test Result Time (s) Test File

@shwstppr shwstppr marked this pull request as ready for review September 25, 2020 11:35
Copy link
Copy Markdown
Member

@yadvr yadvr left a comment

Choose a reason for hiding this comment

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

LGTM, CKS tests passing

@yadvr yadvr requested a review from DaanHoogland September 25, 2020 12:31
@yadvr yadvr modified the milestones: 4.15.0.0, 4.14.1.0 Sep 25, 2020
Copy link
Copy Markdown
Contributor

@Pearl1594 Pearl1594 left a comment

Choose a reason for hiding this comment

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

LGTM
Did some basic tests:

  1. deployed a k8s cluster
  2. destroyed the k8s cluster
  3. Deleted the k8s template and re-registered another one with the same name as defined earlier (and as defined in the global setting) and verified that it picks up the new one and DOES NOT throw an error of being unable to find the template

@yadvr yadvr merged commit 9391fa9 into apache:4.14 Sep 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CKS smoke test failures on master

6 participants