canonical-ubuntu-qa team mailing list archive

[Tim Andersson <mp+479824@xxxxxxxxxxxxxxxxxx>]

Tim Andersson has proposed merging ~andersson123/autopkgtest-cloud:fix-noport into autopkgtest-cloud:master.

Requested reviews:
  Canonical's Ubuntu QA (canonical-ubuntu-qa)

For more details, see:
https://code.launchpad.net/~andersson123/autopkgtest-cloud/+git/autopkgtest-cloud/+merge/479824

Just a small bugfix :)
-- 
Your team Canonical's Ubuntu QA is requested to review the proposed merge of ~andersson123/autopkgtest-cloud:fix-noport into autopkgtest-cloud:master.

diff --git a/charms/focal/autopkgtest-cloud-worker/autopkgtest-cloud/tools/copy-security-group b/charms/focal/autopkgtest-cloud-worker/autopkgtest-cloud/tools/copy-security-group
index 3675da3..1fb88f0 100755
--- a/charms/focal/autopkgtest-cloud-worker/autopkgtest-cloud/tools/copy-security-group
+++ b/charms/focal/autopkgtest-cloud-worker/autopkgtest-cloud/tools/copy-security-group
@@ -114,7 +114,7 @@ def main():
         for port in existing_ports:
             if target["id"] in port["security_groups"]:
                 print("Deleting port in group:", target["id"])
-                neutron_callback(neutron.delete_port)
+                neutron_callback(neutron.delete_port, port["id"])
 
         neutron_callback(neutron.delete_security_group, target["id"])