Repository navigation
cleanup test resources #3693
Description
This is to clean-up tests that create resources and do not clean-up after themselves. The idea is that the entire test-suite should be able to be run in a production environment without harming or messing up. In smoke tests running in Trillian we found the following tests to leave stuff around (the checks fixed in scope of this pr are just for simulator):
- test_accounts cleanup for resources left by test_accounts and test_project #3657 test cleanup #4473
- test_affinity_groups_projects test cleanup #4473
- test_async_job test cleanup #4473
- test_deploy_vm_iso
- test_deploy_vms_with_varied_deploymentplanners default teardown methods with reversed() handling #4289
- test_deploy_vm_with_userdata
- test_diagnostics default teardown methods with reversed() handling #4289
- test_domain_service_offerings
- test_internal_lb Marvin3 #4727
- test_list_ids_parameter
- test_loadbalance
- test_metrics_api
- test_multipleips_per_nic
- test_nested_virtualization
- test_network_acl
- test_network
- test_nic_adapter_type
- test_password_server
- test_portforwardingrules
- test_privategw_acl
- test_projects cleanup for resources left by test_accounts and test_project #3657
- test_reset_vm_on_reboot
- test_resource_accounting
- test_router_dhcphosts cleanup of unused code and cleanup of cleanup procedure #5562
- test_router_dns
- test_router_dnsservice
- test_routers_iptables_default_policy
- test_routers_network_ops
- test_routers default teardown methods with reversed() handling #4289
- test_service_offerings
- test_snapshots
- test_templates
- test_vm_deployment_planner smoke test: fix test_vm_deployment_planner #7183 - some work might be remaining due to the way VMs are created in this test-suite
- test_vpc_redundant
- test_vpc_router_nics
- test_vpc_vpn default teardown methods with reversed() handling #4289
- test_hostha_kvm cleanup some smoke tests #8196 - disabling and enabling host HA, not sure if this should be a smoke test!?
ISSUE TYPE
- Bug Report
- Other: test improvements
Activity
yadvr commentedon Aug 12, 2020
@DaanHoogland is this still valid or can be closed?
DaanHoogland commentedon Sep 18, 2020
no @rhtyd , it is still very valid. It is also limited to simulator tests and should be expanded to all tests. I will expand on this the coming weeks (given 4.15 is not asking more attention)
nvazquez commentedon Jul 27, 2021
@DaanHoogland is this still valid?
DaanHoogland commentedon Jul 27, 2021
yes it is, there is a lot of antiquated code in the smoke tests still, as they are all fixed/passing it is low priority.
DaanHoogland commentedon Jul 27, 2021
also it has been touched since a year ago, so removing the 1year old label
yadvr commentedon Aug 9, 2021
Maybe review/close or update this issue @DaanHoogland ?
DaanHoogland commentedon Aug 17, 2021
@rhtyd @nvazquez this will be open for a long time unless it's getting priority, which it won't as it concerned data and entities being left around in test environments when running smoke tests. I've updated the list and am addressing some for component tests and will address any further I encounter during other work.
Maybe there should be a milestone "ongoing" or something?
29 remaining items
github-actions commentedon May 8, 2026
This issue is stale because it has been open for 120 days with no activity. It may be removed by administrators of this project at any time. Remove the stale label or comment to request for removal of it to prevent this.
test: cleanup resources in test_deploy_vm_iso, use base class tearDown