From 64745c1fc525d821fc40490232f8d8692b4a1079 Mon Sep 17 00:00:00 2001 From: Hongbin Lu Date: Tue, 5 Jun 2018 00:58:05 +0000 Subject: [PATCH] Remove the unused url for container exec This url is not needed anymore since we are using websocket proxy for doing interactive execute now. Depends-On: I68d2b3daaaf8fb551d00774f01ad0ee88538f7ae Change-Id: I5792d01fcb3f360864c49ea7368dc470181a5a75 Closes-Bug: #1735076 --- zun/compute/manager.py | 4 ---- 1 file changed, 4 deletions(-) diff --git a/zun/compute/manager.py b/zun/compute/manager.py index af800bc5f..1fc48d59d 100644 --- a/zun/compute/manager.py +++ b/zun/compute/manager.py @@ -731,11 +731,9 @@ class Manager(periodic_task.PeriodicTasks): interactive) if run: output, exit_code = self.driver.execute_run(exec_id, command) - # TODO(hongbin): remove url once bug #1735076 is fixed return {"output": output, "exit_code": exit_code, "exec_id": None, - "url": None, "token": None} else: token = uuidutils.generate_uuid() @@ -744,11 +742,9 @@ class Manager(periodic_task.PeriodicTasks): context, container_id=container.id, exec_id=exec_id, url=url, token=token) exec_instace.create(context) - # TODO(hongbin): remove url once bug #1735076 is fixed return {'output': None, 'exit_code': None, 'exec_id': exec_id, - 'url': url, 'token': token} except exception.DockerError as e: LOG.error("Error occurred while calling Docker exec API: %s",