)]}'
{"zun/api/controllers/v1/containers.py":[{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"28f8471c0a5ebfdf0520c675e9b4c55e6039a0a0","unresolved":false,"context_lines":[{"line_number":377,"context_line":"    @exception.wrap_pecan_controller_exception"},{"line_number":378,"context_line":"    @validation.validate_query_param(pecan.request, schema.query_param_logs)"},{"line_number":379,"context_line":"    def logs(self, container_id, stdout\u003dTrue, stderr\u003dTrue,"},{"line_number":380,"context_line":"             timestamps\u003dFalse, tail\u003d\u0027all\u0027, since\u003dNone, follow\u003dFalse):"},{"line_number":381,"context_line":"        container \u003d _get_container(container_id)"},{"line_number":382,"context_line":"        check_policy_on_container(container.as_dict(), \"container:logs\")"},{"line_number":383,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":1,"id":"da36d5c6_bea4adb9","line":380,"range":{"start_line":380,"start_character":55,"end_line":380,"end_character":67},"updated":"2017-02-27 16:57:11.000000000","message":"It looks \u0027follow\u0027 will return a stream:\n\n  follow – 1/True/true or 0/False/false, return stream. Default false.\n\nI don\u0027t think we are currently support streaming via API so this parameter should always be False for now.","commit_id":"338478755bee98b4fb6847da680d516619e244c3"},{"author":{"_account_id":23365,"name":"feng.shengqin","email":"feng.shengqin@zte.com.cn","username":"FengShengqin"},"change_message_id":"7d0e9b9e5e74ad1b87a814836e78813c40b3b703","unresolved":false,"context_lines":[{"line_number":377,"context_line":"    @exception.wrap_pecan_controller_exception"},{"line_number":378,"context_line":"    @validation.validate_query_param(pecan.request, schema.query_param_logs)"},{"line_number":379,"context_line":"    def logs(self, container_id, stdout\u003dTrue, stderr\u003dTrue,"},{"line_number":380,"context_line":"             timestamps\u003dFalse, tail\u003d\u0027all\u0027, since\u003dNone, follow\u003dFalse):"},{"line_number":381,"context_line":"        container \u003d _get_container(container_id)"},{"line_number":382,"context_line":"        check_policy_on_container(container.as_dict(), \"container:logs\")"},{"line_number":383,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":1,"id":"da36d5c6_0b1d5dbd","line":380,"range":{"start_line":380,"start_character":55,"end_line":380,"end_character":67},"in_reply_to":"da36d5c6_bea4adb9","updated":"2017-02-28 08:04:52.000000000","message":"Done","commit_id":"338478755bee98b4fb6847da680d516619e244c3"},{"author":{"_account_id":10206,"name":"Madhuri Kumari","email":"madhuri.kumari@intel.com","username":"Madhuri"},"change_message_id":"107d5adba1ccb60fb06a8ecf3c4c1656997276bf","unresolved":false,"context_lines":[{"line_number":376,"context_line":"    @pecan.expose(\u0027json\u0027)"},{"line_number":377,"context_line":"    @exception.wrap_pecan_controller_exception"},{"line_number":378,"context_line":"    @validation.validate_query_param(pecan.request, schema.query_param_logs)"},{"line_number":379,"context_line":"    def logs(self, container_id, stdout\u003dTrue, stderr\u003dTrue,"},{"line_number":380,"context_line":"             timestamps\u003dFalse, tail\u003d\u0027all\u0027, since\u003dNone):"},{"line_number":381,"context_line":"        container \u003d _get_container(container_id)"},{"line_number":382,"context_line":"        check_policy_on_container(container.as_dict(), \"container:logs\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba2be162_4523dd05","line":379,"updated":"2017-02-28 16:40:53.000000000","message":"FEng, I see two more stream(which seems out of scope) and follow. Do you plan to add them later?","commit_id":"69d17afb0fffd8bd034a8852bed975cea14e7217"},{"author":{"_account_id":16277,"name":"Pradeep Kumar Singh","email":"ps4openstack@gmail.com","username":"pradeep_singh"},"change_message_id":"c1abe343e8f9a4fa2616669f4670f4d4eb0e70eb","unresolved":false,"context_lines":[{"line_number":376,"context_line":"    @pecan.expose(\u0027json\u0027)"},{"line_number":377,"context_line":"    @exception.wrap_pecan_controller_exception"},{"line_number":378,"context_line":"    @validation.validate_query_param(pecan.request, schema.query_param_logs)"},{"line_number":379,"context_line":"    def logs(self, container_id, stdout\u003dTrue, stderr\u003dTrue,"},{"line_number":380,"context_line":"             timestamps\u003dFalse, tail\u003d\u0027all\u0027, since\u003dNone):"},{"line_number":381,"context_line":"        container \u003d _get_container(container_id)"},{"line_number":382,"context_line":"        check_policy_on_container(container.as_dict(), \"container:logs\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"ba2be162_90cedf58","line":379,"in_reply_to":"ba2be162_4523dd05","updated":"2017-03-01 02:42:08.000000000","message":"I think its for streaming which is not supported in zun api yet.","commit_id":"69d17afb0fffd8bd034a8852bed975cea14e7217"}],"zun/tests/unit/api/controllers/v1/test_containers.py":[{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"28f8471c0a5ebfdf0520c675e9b4c55e6039a0a0","unresolved":false,"context_lines":[{"line_number":904,"context_line":"        container_uuid \u003d test_container.get(\u0027uuid\u0027)"},{"line_number":905,"context_line":"        response \u003d self.app.get("},{"line_number":906,"context_line":"            \u0027/v1/containers/%s/logs?stderr\u003dTrue\u0026stdout\u003dTrue\u0027"},{"line_number":907,"context_line":"            \u0027\u0026timestamps\u003dFalse\u0026follow\u003dFalse\u0027 % container_uuid)"},{"line_number":908,"context_line":"        self.assertEqual(200, response.status_int)"},{"line_number":909,"context_line":"        mock_container_logs.assert_called_once_with("},{"line_number":910,"context_line":"            mock.ANY, test_container_obj, True, True,"}],"source_content_type":"text/x-python","patch_set":1,"id":"da36d5c6_b5eccb93","line":907,"updated":"2017-02-27 16:57:11.000000000","message":"Consider adding test cases for the \u0027since\u0027 parameter as well.","commit_id":"338478755bee98b4fb6847da680d516619e244c3"},{"author":{"_account_id":23365,"name":"feng.shengqin","email":"feng.shengqin@zte.com.cn","username":"FengShengqin"},"change_message_id":"7d0e9b9e5e74ad1b87a814836e78813c40b3b703","unresolved":false,"context_lines":[{"line_number":904,"context_line":"        container_uuid \u003d test_container.get(\u0027uuid\u0027)"},{"line_number":905,"context_line":"        response \u003d self.app.get("},{"line_number":906,"context_line":"            \u0027/v1/containers/%s/logs?stderr\u003dTrue\u0026stdout\u003dTrue\u0027"},{"line_number":907,"context_line":"            \u0027\u0026timestamps\u003dFalse\u0026follow\u003dFalse\u0027 % container_uuid)"},{"line_number":908,"context_line":"        self.assertEqual(200, response.status_int)"},{"line_number":909,"context_line":"        mock_container_logs.assert_called_once_with("},{"line_number":910,"context_line":"            mock.ANY, test_container_obj, True, True,"}],"source_content_type":"text/x-python","patch_set":1,"id":"da36d5c6_ab6c5137","line":907,"in_reply_to":"da36d5c6_b5eccb93","updated":"2017-02-28 08:04:52.000000000","message":"Done","commit_id":"338478755bee98b4fb6847da680d516619e244c3"}]}
