From 49d2558ddbefcfab8f8dc8710a9c5e0c62a15fd6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ren=C3=A9=20Radoi?= <103035318+reneradoi@users.noreply.github.com> Date: Mon, 3 Jun 2024 10:58:30 +0200 Subject: [PATCH] [DPE-4130] Remove workarounds for removing the OpenSearch application (#317) ## Issue https://github.com/canonical/opensearch-operator/issues/243 ## Solution The issue itself could not be reproduced with the current main branch (revision 95 in channel 2/edge). It was presumably fixed with https://github.com/canonical/opensearch-operator/pull/312. This PR adds integration test coverage. --- tests/integration/ha/test_backups.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/tests/integration/ha/test_backups.py b/tests/integration/ha/test_backups.py index 1b4d0b2e9..03197fcbc 100644 --- a/tests/integration/ha/test_backups.py +++ b/tests/integration/ha/test_backups.py @@ -574,9 +574,6 @@ async def test_restore_to_new_cluster( """ app = (await app_name(ops_test) or APP_NAME) if deploy_type == "small" else "main" logging.info("Destroying the application") - for machine in ops_test.model.state.machines.values(): - # Needed due to canonical/opensearch-operator#243 - await machine.destroy(force=True) # Now, remove the applications await asyncio.gather( ops_test.model.remove_application(S3_INTEGRATOR, block_until_done=True),