-
Notifications
You must be signed in to change notification settings - Fork 1.2k
CLOUDSTACK-10163: Component tests sanity #2344
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-10163: Component tests sanity #2344
Conversation
Merge branch 'master' of https://github.com/shapeblue/cloudstack into component-tests-sanity
… component-tests-sanity
…calations_templates.py, test_persistent_networks.py, test_vpc_vm_life_cycle.py, test_vpn_service.py
…ooking for a VM that was never defined in test
…dary storage count to sync
@blueorangutan package |
@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1319 |
@blueorangutan help |
@borisstoyanov I understand these words: "help", "hello", "thanks", "package", "test" Blessed contributors for kicking Trillian test jobs: ['rhtyd', 'nvazquez', 'PaulAngus', 'borisstoyanov', 'DaanHoogland'] |
…udstack into component-tests-sanity
Trillian test result (tid-1710)
|
@borisstoyanov can you fix conflicts, let's re-review/test and merge this? |
@blueorangutan package |
@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1435 |
@blueorangutan test centos7 kvm-centos7 component/test_*.py |
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
Trillian test result (tid-1838)
Additional tests completed. 1 look ok, 0 have error(s)
|
Looks like wildcard passing of component tests did not work, this only ran smoke tests. Maybe use jenkins directly to kick component tests @borisstoyanov? |
@blueorangutan test centos7 kvm-centos7 component/test_lb_secondary_ip.py component/test_acl_listvm.py component/test_snapshots.py component/test_ldap_auto_import.py component/test_host_maintenance.py component/test_assign_vm.py component/test_invalid_gw_nm.py component/test_cpu_limits.py component/test_storage_motion.py component/test_host_ha.py component/test_escalations_routers.py component/test_secsr_mount.py component/test_escalations_volumes.py component/test_recurring_snapshots.py component/test_browse_volumes.py component/test_stopped_vm.py component/test_allocation_states.py component/test_snapshot_gc.py component/test_mm_max_limits.py component/test_interop_xd_ccp.py component/test_acl_listvolume.py component/test_tags.py component/test_affinity_groups_projects.py component/test_resource_limits.py component/test_security_groups.py component/test_escalations_ipaddresses.py component/test_cpu_project_limits.py component/test_mm_domain_limits.py component/test_cpu_max_limits.py component/test_ps_max_limits.py component/test_vpc_network_internal_lbrules.py component/test_ps_resize_volume.py component/test_escalations_templates.py component/test_vpc_network.py component/test_eip_elb.py component/test_deploy_vgpu_vm.py component/test_vm_passwdenabled.py component/test_browse_templates2.py component/test_separate_backup_from_snapshot.py component/test_escalations_vpncustomergateways.py component/test_project_resources.py component/test_multiple_ips_per_nic.py component/test_template_from_snapshot_with_template_details.py component/test_add_remove_network.py component/test_ncc_integration_dedicated.py component/test_acl_isolatednetwork_delete.py component/test_templates.py component/test_overcommit.py component/test_reset_ssh_keypair.py component/test_ps_resource_limits_volume.py component/test_escalations_vmware.py component/test_mm_project_limits.py component/test_simultaneous_volume_attach.py component/test_acl_listsnapshot.py component/test_netscaler_nw_off.py component/test_escalations_snapshots.py component/test_volumes.py component/test_regions.py component/test_ss_project_limits.py component/test_routers.py component/test_vpn_users.py component/test_host.py component/test_dhcp_dns_offload.py component/test_escalations_instances.py component/test_redundant_router_cleanups.py component/test_cpu_domain_limits.py component/test_persistent_networks.py component/test_usage.py component/test_ss_max_limits.py component/test_netscaler_lb_algo.py component/test_accounts.py component/test_projects.py component/test_vpc_vm_life_cycle.py component/test_ss_domain_limits.py component/test_haproxy.py component/test_vpc.py component/test_blocker_bugs.py component/test_rootvolume_resize.py component/test_netscaler_lb.py component/test_asa1000v_fw.py component/test_netscaler_lb_sticky.py component/test_organization_states.py component/test_project_usage.py component/test_regions_accounts.py component/test_ncc_integration_shared.py component/test_memory_limits.py component/test_base_image_updation.py component/test_deploy_vm_userdata_reg.py component/test_project_limits.py component/test_baremetal.py component/test_acl_isolatednetwork.py component/test_brocade_vcs.py component/test_bigswitch_bcf.py component/test_concurrent_create_volume_from_snapshot.py component/test_escalations_networks.py component/test_custom_hostname.py component/test_network_offering.py component/test_browse_templates.py component/test_project_configs.py component/test_VirtualRouter_alerts.py component/test_vpc_vms_deployment.py component/test_vpc_offerings.py component/test_acl_sharednetwork_deployVM-impersonation.py component/test_vpn_service.py component/test_implicit_planner.py component/test_deploy_vm_userdata_multi_nic.py component/test_affinity_groups.py component/test_redundant_router_upgrades.py component/test_advancedsg_networks.py component/test_vpc_network_pfrules.py component/test_ldap.py component/test_netscaler_configs.py component/test_update_vm.py component/test_egress_fw_rules.py component/test_multiple_ip_ranges.py component/test_ps_limits.py component/test_ip_reservation.py component/test_ps_domain_limits.py component/test_region_vpc.py component/test_dynamic_compute_offering.py component/test_ss_limits.py component/test_ps_project_limits.py component/test_escalation_listTemplateDomainAdmin.py component/test_multiple_public_interfaces.py component/test_escalations_securitygroups.py component/test_vpc_distributed_routing_offering.py component/test_non_contiguous_vlan.py component/test_shared_networks.py component/test_portable_ip.py component/test_escalations_isos.py component/test_list_nics.py component/test_snapshots_improvement.py component/test_vpc_network_lbrules.py component/test_egress_rules.py component/test_vpc_routers.py component/test_vmware_drs.py component/test_acl_sharednetwork.py component/test_concurrent_snapshots_limit.py component/test_vpc_network_staticnatrule.py component/test_list_pod.py component/test_migrate_vol_to_maintained_pool.py component/test_snapshot_limits.py component/test_redundant_router_services.py |
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
Trillian test result (tid-1858)
Additional tests completed. 87 look ok, 45 have error(s)
|
…ooking for a VM that was never defined in test
…dary storage count to sync
Also fixes regression from rebase Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>
d49d6d1
to
a927d4f
Compare
Rebased and fixed conflicts @borisstoyanov |
@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. |
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1475 |
@blueorangutan test centos7 kvm-centos7 component/test_lb_secondary_ip.py component/test_acl_listvm.py component/test_snapshots.py component/test_ldap_auto_import.py component/test_host_maintenance.py component/test_assign_vm.py component/test_invalid_gw_nm.py component/test_cpu_limits.py component/test_storage_motion.py component/test_host_ha.py component/test_escalations_routers.py component/test_secsr_mount.py component/test_escalations_volumes.py component/test_recurring_snapshots.py component/test_browse_volumes.py component/test_stopped_vm.py component/test_allocation_states.py component/test_snapshot_gc.py component/test_mm_max_limits.py component/test_interop_xd_ccp.py component/test_acl_listvolume.py component/test_tags.py component/test_affinity_groups_projects.py component/test_resource_limits.py component/test_security_groups.py component/test_escalations_ipaddresses.py component/test_cpu_project_limits.py component/test_mm_domain_limits.py component/test_cpu_max_limits.py component/test_ps_max_limits.py component/test_vpc_network_internal_lbrules.py component/test_ps_resize_volume.py component/test_escalations_templates.py component/test_vpc_network.py component/test_eip_elb.py component/test_deploy_vgpu_vm.py component/test_vm_passwdenabled.py component/test_browse_templates2.py component/test_separate_backup_from_snapshot.py component/test_escalations_vpncustomergateways.py component/test_project_resources.py component/test_multiple_ips_per_nic.py component/test_template_from_snapshot_with_template_details.py component/test_add_remove_network.py component/test_ncc_integration_dedicated.py component/test_acl_isolatednetwork_delete.py component/test_templates.py component/test_overcommit.py component/test_reset_ssh_keypair.py component/test_ps_resource_limits_volume.py component/test_escalations_vmware.py component/test_mm_project_limits.py component/test_simultaneous_volume_attach.py component/test_acl_listsnapshot.py component/test_netscaler_nw_off.py component/test_escalations_snapshots.py component/test_volumes.py component/test_regions.py component/test_ss_project_limits.py component/test_routers.py component/test_vpn_users.py component/test_host.py component/test_dhcp_dns_offload.py component/test_escalations_instances.py component/test_redundant_router_cleanups.py component/test_cpu_domain_limits.py component/test_persistent_networks.py component/test_usage.py component/test_ss_max_limits.py component/test_netscaler_lb_algo.py component/test_accounts.py component/test_projects.py component/test_vpc_vm_life_cycle.py component/test_ss_domain_limits.py component/test_haproxy.py component/test_vpc.py component/test_blocker_bugs.py component/test_rootvolume_resize.py component/test_netscaler_lb.py component/test_asa1000v_fw.py component/test_netscaler_lb_sticky.py component/test_organization_states.py component/test_project_usage.py component/test_regions_accounts.py component/test_ncc_integration_shared.py component/test_memory_limits.py component/test_base_image_updation.py component/test_deploy_vm_userdata_reg.py component/test_project_limits.py component/test_baremetal.py component/test_acl_isolatednetwork.py component/test_brocade_vcs.py component/test_bigswitch_bcf.py component/test_concurrent_create_volume_from_snapshot.py component/test_escalations_networks.py component/test_custom_hostname.py component/test_network_offering.py component/test_browse_templates.py component/test_project_configs.py component/test_VirtualRouter_alerts.py component/test_vpc_vms_deployment.py component/test_vpc_offerings.py component/test_acl_sharednetwork_deployVM-impersonation.py component/test_vpn_service.py component/test_implicit_planner.py component/test_deploy_vm_userdata_multi_nic.py component/test_affinity_groups.py component/test_redundant_router_upgrades.py component/test_advancedsg_networks.py component/test_vpc_network_pfrules.py component/test_ldap.py component/test_netscaler_configs.py component/test_update_vm.py component/test_egress_fw_rules.py component/test_multiple_ip_ranges.py component/test_ps_limits.py component/test_ip_reservation.py component/test_ps_domain_limits.py component/test_region_vpc.py component/test_dynamic_compute_offering.py component/test_ss_limits.py component/test_ps_project_limits.py component/test_escalation_listTemplateDomainAdmin.py component/test_multiple_public_interfaces.py component/test_escalations_securitygroups.py component/test_vpc_distributed_routing_offering.py component/test_non_contiguous_vlan.py component/test_shared_networks.py component/test_portable_ip.py component/test_escalations_isos.py component/test_list_nics.py component/test_snapshots_improvement.py component/test_vpc_network_lbrules.py component/test_egress_rules.py component/test_vpc_routers.py component/test_vmware_drs.py component/test_acl_sharednetwork.py component/test_concurrent_snapshots_limit.py component/test_vpc_network_staticnatrule.py component/test_list_pod.py component/test_migrate_vol_to_maintained_pool.py component/test_snapshot_limits.py component/test_redundant_router_services.py |
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
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, nice work @borisstoyanov ! (I saw no test failures till now, check on the marvin host, hopefully we should see less failure in the next result)
Pinging for additional review - @nvazquez @DaanHoogland @wido @marcaurele @rafaelweingartner and others, thanks. |
Trillian test result (tid-1912)
Additional tests completed. 88 look ok, 41 have error(s)
|
Many of these are issues that are not fixed yet, others may be regressions, but the failure number is down to 41 from ~100 |
cls).getClsTestClient() | ||
cls.api_client = cls.testClient.getApiClient() | ||
cls.services = cls.testClient.getParsedTestDataConfig() | ||
cls.hostConfig = cls.config.__dict__["zones"][0].__dict__["pods"][0].__dict__["clusters"][0].__dict__["hosts"][0].__dict__ |
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.
Is cls.config
set here or should it be cls.services
? Can we add basic check for each dictionary key?
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.
This is a configuration, not a test data which we consider as 'services'.
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.
Ok, sorry, copy that. Please ignore similar comments, thanks
cls).getClsTestClient() | ||
cls.api_client = cls.testClient.getApiClient() | ||
cls.services = cls.testClient.getParsedTestDataConfig() | ||
cls.hostConfig = cls.config.__dict__["zones"][0].__dict__["pods"][0].__dict__["clusters"][0].__dict__["hosts"][0].__dict__ |
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.
Same question here
self.services = self.testClient.getParsedTestDataConfig() | ||
self.zone = get_zone(self.apiclient, self.testClient.getZoneForTests()) | ||
self.pod = get_pod(self.apiclient, self.zone.id) | ||
self.hostConfig = self.config.__dict__["zones"][0].__dict__["pods"][0].__dict__["clusters"][0].__dict__["hosts"][0].__dict__ |
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.
Same here
cls.testdata["configurableData"]["password"] = "xenroot" | ||
#cls.testdata["configurableData"]["password"] = "xenroot" | ||
cls.hypervisor = testClient.getHypervisorInfo() | ||
cls.hostConfig = cls.config.__dict__["zones"][0].__dict__["pods"][0].__dict__["clusters"][0].__dict__["hosts"][0].__dict__ |
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.
Same here
cls.zone = get_zone(cls.apiclient) | ||
cls.testdata["mode"] = cls.zone.networktype | ||
cls.testdata["configurableData"]["password"] = "xenroot" | ||
#cls.testdata["configurableData"]["password"] = "xenroot" |
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.
Can we remove this commented line?
cls.services["zoneid"] = cls.zone.id | ||
cls.services["virtual_machine"]["zoneid"] = cls.zone.id | ||
cls.services["virtual_machine"]["template"] = template.id | ||
#cls.testdata["zoneid"] = cls.zone.id |
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.
Multiple commented lines
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, thanks for the hard work @borisstoyanov. I just left some minor comments/questions
…/cloudstack into component-tests-sanity
Thanks @nvazquez I've addressed your comments, @rhtyd I think we're in a good position to merge now, lets address the new failures in separate PRs. |
Thanks @borisstoyanov I'll merge this, as we'll deal with fixing rest of the failures in other PRs. Will merge this based on improved component test results and 2lgtms code reviews, and as soon as Travis is green. |
I've fixed quite a few component tests, most of them should pass now.