)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"357dcac0d300c2c05a1cded8615c12fdf0348c71","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- Fix so that Content-Type of the v1 modify vnflcm request is"},{"line_number":10,"context_line":"  \"application/merge-patch+json\"."},{"line_number":11,"context_line":"- Fix so that as the same, Content-Type of all patch request is"},{"line_number":12,"context_line":"  \"application/merge-patch+json\"."},{"line_number":13,"context_line":"- Fix so that Accept of all patch request and put vnfpkgm request"},{"line_number":14,"context_line":"  is \"application/json\"."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"c3ffae02_096ad9e2","line":11,"range":{"start_line":11,"start_character":43,"end_line":11,"end_character":60},"updated":"2024-08-20 08:31:07.000000000","message":"As commented in the bug report [1], I feel that the scope should be made a little more clear in this CommitMessage or BugReport [1]. \n(Is this patch\u0027s fixes for \"all patch request\" of V1 API and V2 API shown in API reference [2]?\n\n[1] https://bugs.launchpad.net/tacker/+bug/2069701\n[2] https://docs.openstack.org/api-ref/nfv-orchestration/","commit_id":"48797e406463dc3af88459a59a4b5518f7c97ae3"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"576a11733106fea20996550d4355178bcc94de22","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- Fix so that Content-Type of the v1 modify vnflcm request is"},{"line_number":10,"context_line":"  \"application/merge-patch+json\"."},{"line_number":11,"context_line":"- Fix so that as the same, Content-Type of all patch request is"},{"line_number":12,"context_line":"  \"application/merge-patch+json\"."},{"line_number":13,"context_line":"- Fix so that Accept of all patch request and put vnfpkgm request"},{"line_number":14,"context_line":"  is \"application/json\"."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"8bb7d1fc_3a14941f","line":11,"range":{"start_line":11,"start_character":43,"end_line":11,"end_character":60},"in_reply_to":"34cd3ac9_2abdda19","updated":"2024-08-28 08:40:03.000000000","message":"Thanks for adding description, I\u0027ve confirmed them.","commit_id":"48797e406463dc3af88459a59a4b5518f7c97ae3"},{"author":{"_account_id":36997,"name":"Takahiro Miyajima","display_name":"f-miyajima","email":"fj6257jz@fujitsu.com","username":"f-miyajima"},"change_message_id":"bec15ce3ad12fa47b982b554048b5767b931a912","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- Fix so that Content-Type of the v1 modify vnflcm request is"},{"line_number":10,"context_line":"  \"application/merge-patch+json\"."},{"line_number":11,"context_line":"- Fix so that as the same, Content-Type of all patch request is"},{"line_number":12,"context_line":"  \"application/merge-patch+json\"."},{"line_number":13,"context_line":"- Fix so that Accept of all patch request and put vnfpkgm request"},{"line_number":14,"context_line":"  is \"application/json\"."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"34cd3ac9_2abdda19","line":11,"range":{"start_line":11,"start_character":43,"end_line":11,"end_character":60},"in_reply_to":"c3ffae02_096ad9e2","updated":"2024-08-28 07:06:47.000000000","message":"I added a list of all patch requests to be fixed (with SOL information) to the Bug Description. Please confirm.","commit_id":"48797e406463dc3af88459a59a4b5518f7c97ae3"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"357dcac0d300c2c05a1cded8615c12fdf0348c71","unresolved":true,"context_lines":[{"line_number":10,"context_line":"  \"application/merge-patch+json\"."},{"line_number":11,"context_line":"- Fix so that as the same, Content-Type of all patch request is"},{"line_number":12,"context_line":"  \"application/merge-patch+json\"."},{"line_number":13,"context_line":"- Fix so that Accept of all patch request and put vnfpkgm request"},{"line_number":14,"context_line":"  is \"application/json\"."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Closes-Bug: #2069701"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"d76ed30c_688b3e88","line":13,"range":{"start_line":13,"start_character":24,"end_line":13,"end_character":41},"updated":"2024-08-20 08:31:07.000000000","message":"ditto.","commit_id":"48797e406463dc3af88459a59a4b5518f7c97ae3"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"576a11733106fea20996550d4355178bcc94de22","unresolved":false,"context_lines":[{"line_number":10,"context_line":"  \"application/merge-patch+json\"."},{"line_number":11,"context_line":"- Fix so that as the same, Content-Type of all patch request is"},{"line_number":12,"context_line":"  \"application/merge-patch+json\"."},{"line_number":13,"context_line":"- Fix so that Accept of all patch request and put vnfpkgm request"},{"line_number":14,"context_line":"  is \"application/json\"."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Closes-Bug: #2069701"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"0e1e231c_2eccaaa1","line":13,"range":{"start_line":13,"start_character":24,"end_line":13,"end_character":41},"in_reply_to":"aa128b22_418c17d3","updated":"2024-08-28 08:40:03.000000000","message":"Acknowledged","commit_id":"48797e406463dc3af88459a59a4b5518f7c97ae3"},{"author":{"_account_id":36997,"name":"Takahiro Miyajima","display_name":"f-miyajima","email":"fj6257jz@fujitsu.com","username":"f-miyajima"},"change_message_id":"bec15ce3ad12fa47b982b554048b5767b931a912","unresolved":true,"context_lines":[{"line_number":10,"context_line":"  \"application/merge-patch+json\"."},{"line_number":11,"context_line":"- Fix so that as the same, Content-Type of all patch request is"},{"line_number":12,"context_line":"  \"application/merge-patch+json\"."},{"line_number":13,"context_line":"- Fix so that Accept of all patch request and put vnfpkgm request"},{"line_number":14,"context_line":"  is \"application/json\"."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Closes-Bug: #2069701"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"aa128b22_418c17d3","line":13,"range":{"start_line":13,"start_character":24,"end_line":13,"end_character":41},"in_reply_to":"d76ed30c_688b3e88","updated":"2024-08-28 07:06:47.000000000","message":"I added a list of all patch requests to be fixed to the commit message. Please confirm.","commit_id":"48797e406463dc3af88459a59a4b5518f7c97ae3"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"357dcac0d300c2c05a1cded8615c12fdf0348c71","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6cb98a36_db940b69","updated":"2024-08-20 08:31:07.000000000","message":"Thanks for your patch, almost LGTM.\nPlease kindly find my tiny comment, and there seems to be a comment on the bug report [1] of this patch, so it would be helpful if you could respond.\n\n[1] https://bugs.launchpad.net/tacker/+bug/2069701","commit_id":"48797e406463dc3af88459a59a4b5518f7c97ae3"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"576a11733106fea20996550d4355178bcc94de22","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6ff3f5d4_cc143628","updated":"2024-08-28 08:40:03.000000000","message":"Thanks for your update, LGTM.","commit_id":"046a136021ad7e5191a28cda7c583bb2184a6727"}],"tackerclient/tests/unit/osc/v1/test_vnflcm.py":[{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"357dcac0d300c2c05a1cded8615c12fdf0348c71","unresolved":true,"context_lines":[{"line_number":696,"context_line":""},{"line_number":697,"context_line":"        actual_message \u003d buffer.getvalue().strip()"},{"line_number":698,"context_line":""},{"line_number":699,"context_line":"        expected_message \u003d (\u0027Update vnf:\u0027 + vnf_instance[\u0027id\u0027])"},{"line_number":700,"context_line":""},{"line_number":701,"context_line":"        self.assertEqual(expected_message, actual_message)"},{"line_number":702,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"275c35b9_5d5f1bd8","line":699,"range":{"start_line":699,"start_character":28,"end_line":699,"end_character":62},"updated":"2024-08-20 08:31:07.000000000","message":"nit: Could you use f-string to concatenate string instead of \"+\"?","commit_id":"48797e406463dc3af88459a59a4b5518f7c97ae3"},{"author":{"_account_id":36997,"name":"Takahiro Miyajima","display_name":"f-miyajima","email":"fj6257jz@fujitsu.com","username":"f-miyajima"},"change_message_id":"bec15ce3ad12fa47b982b554048b5767b931a912","unresolved":true,"context_lines":[{"line_number":696,"context_line":""},{"line_number":697,"context_line":"        actual_message \u003d buffer.getvalue().strip()"},{"line_number":698,"context_line":""},{"line_number":699,"context_line":"        expected_message \u003d (\u0027Update vnf:\u0027 + vnf_instance[\u0027id\u0027])"},{"line_number":700,"context_line":""},{"line_number":701,"context_line":"        self.assertEqual(expected_message, actual_message)"},{"line_number":702,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"d00f892f_125f089d","line":699,"range":{"start_line":699,"start_character":28,"end_line":699,"end_character":62},"in_reply_to":"275c35b9_5d5f1bd8","updated":"2024-08-28 07:06:47.000000000","message":"I fixed to use f-string.","commit_id":"48797e406463dc3af88459a59a4b5518f7c97ae3"},{"author":{"_account_id":31857,"name":"Ayumu Ueha","email":"ueha.ayumu@fujitsu.com","username":"ueha"},"change_message_id":"576a11733106fea20996550d4355178bcc94de22","unresolved":false,"context_lines":[{"line_number":696,"context_line":""},{"line_number":697,"context_line":"        actual_message \u003d buffer.getvalue().strip()"},{"line_number":698,"context_line":""},{"line_number":699,"context_line":"        expected_message \u003d (\u0027Update vnf:\u0027 + vnf_instance[\u0027id\u0027])"},{"line_number":700,"context_line":""},{"line_number":701,"context_line":"        self.assertEqual(expected_message, actual_message)"},{"line_number":702,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"2778c67e_27360b28","line":699,"range":{"start_line":699,"start_character":28,"end_line":699,"end_character":62},"in_reply_to":"d00f892f_125f089d","updated":"2024-08-28 08:40:03.000000000","message":"Acknowledged","commit_id":"48797e406463dc3af88459a59a4b5518f7c97ae3"}]}