From 149979407eb2c3c1720a5b20ff66e0543ebd04e7 Mon Sep 17 00:00:00 2001 From: Kristine Bujold Date: Wed, 28 Nov 2018 16:15:21 -0500 Subject: [PATCH] Adding TODO comment Adding a clarification to earlier commit 7c1c082b. Related-Bug: 1802397 Change-Id: I9c008db783fa32c5747e416844f76315bd4dc3ba Signed-off-by: Kristine Bujold --- dcorch/engine/sync_services/network.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/dcorch/engine/sync_services/network.py b/dcorch/engine/sync_services/network.py index d04fa4222..76e3e263d 100644 --- a/dcorch/engine/sync_services/network.py +++ b/dcorch/engine/sync_services/network.py @@ -420,6 +420,8 @@ class NetworkSyncThread(SyncThread): def same_security_group(self, qc1, qc2): # Fetch the tenant name from the project. Tenant ids are different # between regions. + # TODO(kbujold): This solution only works if we have one domain within + # keystone. qc1_tenant_name = sdk.OpenStackDriver().get_project_by_id( qc1['tenant_id']).name qc2_tenant_name = sdk.OpenStackDriver(