)]}'
{"zun/api/controllers/v1/containers.py":[{"author":{"_account_id":16277,"name":"Pradeep Kumar Singh","email":"ps4openstack@gmail.com","username":"pradeep_singh"},"change_message_id":"4dd1648e36af3278d8f897769e2f88b609d7ae19","unresolved":false,"context_lines":[{"line_number":285,"context_line":""},{"line_number":286,"context_line":"        self.compute_api \u003d compute_api.API(context)"},{"line_number":287,"context_line":"        if run:"},{"line_number":288,"context_line":"            self.compute_api.container_run(context, new_container)"},{"line_number":289,"context_line":"        else:"},{"line_number":290,"context_line":"            self.compute_api.container_create(context, new_container)"},{"line_number":291,"context_line":"        # Set the HTTP Location Header"}],"source_content_type":"text/x-python","patch_set":21,"id":"ba5201f7_3e69608c","line":288,"updated":"2017-01-10 16:08:24.000000000","message":"Any special reason for not using compute_api for other APIs like stop/start/kill etc? I think if compute API expose all such methods then it would be good.","commit_id":"9101e4fad35259f5ca02b0d7e5586bfcb05b874a"}],"zun/compute/rpcapi.py":[{"author":{"_account_id":10206,"name":"Madhuri Kumari","email":"madhuri.kumari@intel.com","username":"Madhuri"},"change_message_id":"7164030aad03b9b1483bb22379c8766ca3e1bcfc","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"    def container_run(self, context, container):"},{"line_number":42,"context_line":"        host \u003d container.host"},{"line_number":43,"context_line":"        return self._cast(host, \u0027container_run\u0027, container\u003dcontainer)"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    def container_delete(self, context, container, force):"},{"line_number":46,"context_line":"        host \u003d container.host"}],"source_content_type":"text/x-python","patch_set":21,"id":"ba5201f7_21106e4e","line":43,"range":{"start_line":43,"start_character":26,"end_line":43,"end_character":30},"updated":"2017-01-10 08:14:54.000000000","message":"Can\u0027t we directly access \u0027container.host\u0027 here?","commit_id":"9101e4fad35259f5ca02b0d7e5586bfcb05b874a"}],"zun/container/docker/driver.py":[{"author":{"_account_id":10206,"name":"Madhuri Kumari","email":"madhuri.kumari@intel.com","username":"Madhuri"},"change_message_id":"7164030aad03b9b1483bb22379c8766ca3e1bcfc","unresolved":false,"context_lines":[{"line_number":300,"context_line":"        name \u003d self.get_sandbox_name(container)"},{"line_number":301,"context_line":"        # NOTE(hongbin): The format of availability zone is ZONE:HOST:NODE"},{"line_number":302,"context_line":"        # However, we just want to specify host, so it is \u0027:HOST:\u0027"},{"line_number":303,"context_line":"        az \u003d \u0027:%s:\u0027 % CONF.host"},{"line_number":304,"context_line":"        sandbox \u003d novaclient.create_server(name\u003dname, image\u003dimage,"},{"line_number":305,"context_line":"                                           flavor\u003dflavor, key_name\u003dkey_name,"},{"line_number":306,"context_line":"                                           nics\u003dnics, availability_zone\u003daz)"}],"source_content_type":"text/x-python","patch_set":21,"id":"ba5201f7_8104bac6","line":303,"range":{"start_line":303,"start_character":22,"end_line":303,"end_character":31},"updated":"2017-01-10 08:14:54.000000000","message":"the sandbox exists on the same node as container. Right?\nIf so, this host should be the one passed in container.host?","commit_id":"9101e4fad35259f5ca02b0d7e5586bfcb05b874a"},{"author":{"_account_id":16277,"name":"Pradeep Kumar Singh","email":"ps4openstack@gmail.com","username":"pradeep_singh"},"change_message_id":"4dd1648e36af3278d8f897769e2f88b609d7ae19","unresolved":false,"context_lines":[{"line_number":300,"context_line":"        name \u003d self.get_sandbox_name(container)"},{"line_number":301,"context_line":"        # NOTE(hongbin): The format of availability zone is ZONE:HOST:NODE"},{"line_number":302,"context_line":"        # However, we just want to specify host, so it is \u0027:HOST:\u0027"},{"line_number":303,"context_line":"        az \u003d \u0027:%s:\u0027 % CONF.host"},{"line_number":304,"context_line":"        sandbox \u003d novaclient.create_server(name\u003dname, image\u003dimage,"},{"line_number":305,"context_line":"                                           flavor\u003dflavor, key_name\u003dkey_name,"},{"line_number":306,"context_line":"                                           nics\u003dnics, availability_zone\u003daz)"}],"source_content_type":"text/x-python","patch_set":21,"id":"ba5201f7_2841180c","line":303,"range":{"start_line":303,"start_character":22,"end_line":303,"end_character":31},"in_reply_to":"ba5201f7_8104bac6","updated":"2017-01-10 16:08:24.000000000","message":"here i think, container.host and CONF.host is same? So its better to use container.host for consistency?","commit_id":"9101e4fad35259f5ca02b0d7e5586bfcb05b874a"}]}