Skip to content

Commit

Permalink
Add TODO
Browse files Browse the repository at this point in the history
  • Loading branch information
bschimke95 committed Oct 23, 2024
1 parent e56f66c commit 7edc7f5
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion tests/integration/tests/test_util/harness/lxd.py
Original file line number Diff line number Diff line change
Expand Up @@ -230,7 +230,10 @@ def delete_instance(self, instance_id: str):
try:
# There are cases where the instance is not deleted properly and this command is stuck.
# A timeout prevents this.
run(["lxc", "rm", instance_id, "--force", "--debug"], timeout=60 * 5)
# TODO(ben): This is a workaround for an issue that arises because of our use of
# privileged containers. We eventually move away from this (not supported >24.10)
# which should also fix this issue and make this timeout unnecessary.
run(["lxc", "rm", instance_id, "--force"], timeout=60 * 5)
except subprocess.CalledProcessError as e:
raise HarnessError(f"failed to delete instance {instance_id}") from e
except subprocess.TimeoutExpired:
Expand Down

0 comments on commit 7edc7f5

Please sign in to comment.