)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"4421c8eb1536f856ba1d125b2a18700e1a33b00b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"4300ab97_3c3ec5ca","updated":"2022-08-18 06:58:52.000000000","message":"Thanks Franca! The code looks good, I tested in my environment and it worked as expected, great job! I\u0027ve added a few nits inline, what do you think about adding a negative test case?","commit_id":"76e60288f60f392b28a89aae40fedf3f12b6d2e0"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"ee616c4514c7d05ddc1ce89a5df6c3d5520cfaeb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"308f9789_ecec48f8","updated":"2022-08-18 22:10:22.000000000","message":"LGTM, passed all tests in my local env. Thanks!","commit_id":"0aadd2f88f08ad926ac8c16ef6a8dee0a52a5444"},{"author":{"_account_id":34123,"name":"Franca Mgbogu","email":"franciedigital@gmail.com","username":"franciedigital"},"change_message_id":"418e4982c74531c6b7f0b2596a5a3fe2b2132ad4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"4c141787_8b1f68f8","updated":"2022-08-19 08:31:53.000000000","message":"Sure  I would rebase","commit_id":"0aadd2f88f08ad926ac8c16ef6a8dee0a52a5444"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"9641a2929c12784bf5f237e9c0758c0eadf0b303","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"a96a6f49_8c1c0c6d","updated":"2022-08-18 22:13:45.000000000","message":"When the other change merges, this will possibly need a rebase","commit_id":"0aadd2f88f08ad926ac8c16ef6a8dee0a52a5444"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"988da84457100621f60a7780ce01e0a4ba1af55c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"737f9de0_a3c52ea5","in_reply_to":"a96a6f49_8c1c0c6d","updated":"2022-08-18 22:14:06.000000000","message":"Could you please take a look at it? :)","commit_id":"0aadd2f88f08ad926ac8c16ef6a8dee0a52a5444"},{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"a8433ec68344d7212973b7df348e51c4d51b098c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"1e6c0c0d_3262234a","updated":"2022-08-22 07:15:04.000000000","message":"Thanks Franca! :) ","commit_id":"20638a500008edfbe4cf62d4bc4bb4e57fb38d0b"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"73a419976d491f87784f73d56ee0ab998c497a7c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"7b0b384e_b2bf1dad","updated":"2022-08-22 20:55:11.000000000","message":"Thanks Franca; there are a couple of minor issues; could you please address these in a following patch? ","commit_id":"20638a500008edfbe4cf62d4bc4bb4e57fb38d0b"}],"manilaclient/osc/v2/share_servers.py":[{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"4421c8eb1536f856ba1d125b2a18700e1a33b00b","unresolved":true,"context_lines":[{"line_number":421,"context_line":"        parser.add_argument("},{"line_number":422,"context_line":"            \u0027share_server\u0027,"},{"line_number":423,"context_line":"            metavar\u003d\u0027\u003cshare_server\u003e\u0027,"},{"line_number":424,"context_line":"            help\u003d\u0027ID of share server to complete migration.\u0027"},{"line_number":425,"context_line":"        )"},{"line_number":426,"context_line":"        parser.add_argument("},{"line_number":427,"context_line":"            \u0027host\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"588e9e06_177d78bb","line":424,"range":{"start_line":424,"start_character":17,"end_line":424,"end_character":60},"updated":"2022-08-18 06:58:52.000000000","message":"nit: add translation support.","commit_id":"76e60288f60f392b28a89aae40fedf3f12b6d2e0"},{"author":{"_account_id":34123,"name":"Franca Mgbogu","email":"franciedigital@gmail.com","username":"franciedigital"},"change_message_id":"5519e0766265ab0689898201d9fcf0af2fec13ca","unresolved":false,"context_lines":[{"line_number":421,"context_line":"        parser.add_argument("},{"line_number":422,"context_line":"            \u0027share_server\u0027,"},{"line_number":423,"context_line":"            metavar\u003d\u0027\u003cshare_server\u003e\u0027,"},{"line_number":424,"context_line":"            help\u003d\u0027ID of share server to complete migration.\u0027"},{"line_number":425,"context_line":"        )"},{"line_number":426,"context_line":"        parser.add_argument("},{"line_number":427,"context_line":"            \u0027host\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"9b9040af_ce5ab749","line":424,"range":{"start_line":424,"start_character":17,"end_line":424,"end_character":60},"in_reply_to":"588e9e06_177d78bb","updated":"2022-08-18 11:18:18.000000000","message":"Done","commit_id":"76e60288f60f392b28a89aae40fedf3f12b6d2e0"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"73a419976d491f87784f73d56ee0ab998c497a7c","unresolved":true,"context_lines":[{"line_number":517,"context_line":"        parser.add_argument("},{"line_number":518,"context_line":"            \u0027share_server\u0027,"},{"line_number":519,"context_line":"            metavar\u003d\u0027\u003cshare_server\u003e\u0027,"},{"line_number":520,"context_line":"            help\u003d_(\u0027ID of share server to complete migration.\u0027)"},{"line_number":521,"context_line":"        )"},{"line_number":522,"context_line":"        parser.add_argument("},{"line_number":523,"context_line":"            \u0027host\u0027,"}],"source_content_type":"text/x-python","patch_set":9,"id":"3aaf7106_72ff52f8","line":520,"range":{"start_line":520,"start_character":42,"end_line":520,"end_character":50},"updated":"2022-08-22 20:55:11.000000000","message":"start","commit_id":"20638a500008edfbe4cf62d4bc4bb4e57fb38d0b"},{"author":{"_account_id":34123,"name":"Franca Mgbogu","email":"franciedigital@gmail.com","username":"franciedigital"},"change_message_id":"d77f1b72ce36a46d6db810d4063b894d8eb70093","unresolved":false,"context_lines":[{"line_number":517,"context_line":"        parser.add_argument("},{"line_number":518,"context_line":"            \u0027share_server\u0027,"},{"line_number":519,"context_line":"            metavar\u003d\u0027\u003cshare_server\u003e\u0027,"},{"line_number":520,"context_line":"            help\u003d_(\u0027ID of share server to complete migration.\u0027)"},{"line_number":521,"context_line":"        )"},{"line_number":522,"context_line":"        parser.add_argument("},{"line_number":523,"context_line":"            \u0027host\u0027,"}],"source_content_type":"text/x-python","patch_set":9,"id":"53de3c75_9c3c3b91","line":520,"range":{"start_line":520,"start_character":42,"end_line":520,"end_character":50},"in_reply_to":"3aaf7106_72ff52f8","updated":"2022-08-23 10:08:32.000000000","message":"Done","commit_id":"20638a500008edfbe4cf62d4bc4bb4e57fb38d0b"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"73a419976d491f87784f73d56ee0ab998c497a7c","unresolved":true,"context_lines":[{"line_number":538,"context_line":"            metavar\u003d\u0027\u003cTrue|False\u003e\u0027,"},{"line_number":539,"context_line":"            choices\u003d[\u0027True\u0027, \u0027False\u0027],"},{"line_number":540,"context_line":"            required\u003dTrue,"},{"line_number":541,"context_line":"            help\u003d_(\"Enforces migration to keep all its shares writable\""},{"line_number":542,"context_line":"                   \"while contents are being moved.\")"},{"line_number":543,"context_line":"        )"},{"line_number":544,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":9,"id":"45f1b3d3_05915f9b","line":541,"range":{"start_line":541,"start_character":70,"end_line":541,"end_character":71},"updated":"2022-08-22 20:55:11.000000000","message":"missing space before quote","commit_id":"20638a500008edfbe4cf62d4bc4bb4e57fb38d0b"},{"author":{"_account_id":34123,"name":"Franca Mgbogu","email":"franciedigital@gmail.com","username":"franciedigital"},"change_message_id":"d77f1b72ce36a46d6db810d4063b894d8eb70093","unresolved":false,"context_lines":[{"line_number":538,"context_line":"            metavar\u003d\u0027\u003cTrue|False\u003e\u0027,"},{"line_number":539,"context_line":"            choices\u003d[\u0027True\u0027, \u0027False\u0027],"},{"line_number":540,"context_line":"            required\u003dTrue,"},{"line_number":541,"context_line":"            help\u003d_(\"Enforces migration to keep all its shares writable\""},{"line_number":542,"context_line":"                   \"while contents are being moved.\")"},{"line_number":543,"context_line":"        )"},{"line_number":544,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":9,"id":"414556c0_680082a6","line":541,"range":{"start_line":541,"start_character":70,"end_line":541,"end_character":71},"in_reply_to":"45f1b3d3_05915f9b","updated":"2022-08-23 10:08:32.000000000","message":"Done","commit_id":"20638a500008edfbe4cf62d4bc4bb4e57fb38d0b"}],"manilaclient/tests/unit/osc/v2/test_share_servers.py":[{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"4421c8eb1536f856ba1d125b2a18700e1a33b00b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"63c52072_d6b74a07","line":565,"updated":"2022-08-18 06:58:52.000000000","message":"we could add an exception case test for the microversion check :)","commit_id":"76e60288f60f392b28a89aae40fedf3f12b6d2e0"},{"author":{"_account_id":34123,"name":"Franca Mgbogu","email":"franciedigital@gmail.com","username":"franciedigital"},"change_message_id":"5519e0766265ab0689898201d9fcf0af2fec13ca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"d5e91d91_a3934a9d","line":565,"in_reply_to":"63c52072_d6b74a07","updated":"2022-08-18 11:18:18.000000000","message":"Done","commit_id":"76e60288f60f392b28a89aae40fedf3f12b6d2e0"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"73a419976d491f87784f73d56ee0ab998c497a7c","unresolved":true,"context_lines":[{"line_number":683,"context_line":"        ]"},{"line_number":684,"context_line":""},{"line_number":685,"context_line":"        parsed_args \u003d self.check_parser(self.cmd, arglist, verifylist)"},{"line_number":686,"context_line":""},{"line_number":687,"context_line":"        self.servers_mock.migration_start.side_effect \u003d Exception()"},{"line_number":688,"context_line":""},{"line_number":689,"context_line":"        self.assertRaises("},{"line_number":690,"context_line":"            exceptions.CommandError,"}],"source_content_type":"text/x-python","patch_set":9,"id":"ddc19927_86a2d022","line":687,"range":{"start_line":686,"start_character":0,"end_line":687,"end_character":67},"updated":"2022-08-22 20:55:11.000000000","message":"this clobbers the test, and isn\u0027t needed..","commit_id":"20638a500008edfbe4cf62d4bc4bb4e57fb38d0b"},{"author":{"_account_id":34123,"name":"Franca Mgbogu","email":"franciedigital@gmail.com","username":"franciedigital"},"change_message_id":"d77f1b72ce36a46d6db810d4063b894d8eb70093","unresolved":false,"context_lines":[{"line_number":683,"context_line":"        ]"},{"line_number":684,"context_line":""},{"line_number":685,"context_line":"        parsed_args \u003d self.check_parser(self.cmd, arglist, verifylist)"},{"line_number":686,"context_line":""},{"line_number":687,"context_line":"        self.servers_mock.migration_start.side_effect \u003d Exception()"},{"line_number":688,"context_line":""},{"line_number":689,"context_line":"        self.assertRaises("},{"line_number":690,"context_line":"            exceptions.CommandError,"}],"source_content_type":"text/x-python","patch_set":9,"id":"ed41de94_f78d2e67","line":687,"range":{"start_line":686,"start_character":0,"end_line":687,"end_character":67},"in_reply_to":"ddc19927_86a2d022","updated":"2022-08-23 10:08:32.000000000","message":"Done","commit_id":"20638a500008edfbe4cf62d4bc4bb4e57fb38d0b"}]}