)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"90c524496691afdc02b06a25701557e02d54dc95","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Implement openstack share show metadata command"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This commit adds openstack share show metadata command, which implements manila"},{"line_number":10,"context_line":"metadata-show functionality."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Partially implements bp openstack-client-support."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1f621f24_2ad7344d","line":9,"updated":"2020-11-17 13:20:22.000000000","message":"NIT Wrap title to 50 chars and subsequent text to 72 chars (https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure)","commit_id":"af452479d6934fa618d9facaf4411b4fb7ff7fc7"},{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"9cbe30b7cb83c2459782e2e3f7c065314d275c33","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Implement openstack share properties show command"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This commit adds openstack share show metadata command, which implements manila"},{"line_number":10,"context_line":"metadata-show functionality."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Partially implements bp openstack-client-support."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fffc6b78_637ddae6","line":9,"range":{"start_line":9,"start_character":17,"end_line":9,"end_character":46},"updated":"2020-11-19 19:47:56.000000000","message":"Here too: \u0027openstack share properties show\u0027\n\nAlso, +1 to what Victoria said about wrapping the line to 72 chars.","commit_id":"a59c8e5fb295c87aee8d43819272fa2a139d6d64"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"d0249e68a3837ba7f193d44bfa5429747330fff2","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Implement openstack share properties show command"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This commit adds openstack share show metadata command, which implements manila"},{"line_number":10,"context_line":"metadata-show functionality."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Partially implements bp openstack-client-support."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fffc6b78_6699681e","line":9,"range":{"start_line":9,"start_character":17,"end_line":9,"end_character":46},"in_reply_to":"fffc6b78_637ddae6","updated":"2020-11-19 21:00:04.000000000","message":"+1","commit_id":"a59c8e5fb295c87aee8d43819272fa2a139d6d64"},{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"9cbe30b7cb83c2459782e2e3f7c065314d275c33","unresolved":false,"context_lines":[{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Partially implements bp openstack-client-support."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Updated with working unit test."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: Ibde82cf9c36be41403a0e68ecac17cb211725d84"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fffc6b78_43da5617","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":31},"updated":"2020-11-19 19:47:56.000000000","message":"No need for this comment, it is still a single commit and we can see the history in the reviews/comments section :)","commit_id":"a59c8e5fb295c87aee8d43819272fa2a139d6d64"}],"doc/source/cli/osc/v2/index.rst":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"db03f3dfa2a99baa02e4fe09ebffd6b4e43ec26d","unresolved":true,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":".. autoprogram-cliff:: openstack.share.v2"},{"line_number":42,"context_line":"    :command: share export location list"},{"line_number":43,"context_line":"    :command: share properties show"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":6,"id":"6f15733b_ec79d84a","line":43,"range":{"start_line":43,"start_character":0,"end_line":43,"end_character":35},"updated":"2021-01-29 19:24:31.000000000","message":"Move this after the share unset command on lines 26-27, and you need to use a different section for this command:\n\n  .. autoprogram-cliff:: openstack.share.v2\n     :command: share properties show","commit_id":"1df4c32be3e71f9dfbc9da3906a1db34d942c393"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"e2e99a1175a4fc87055c77cd2fd9dec4cf38bcc7","unresolved":true,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":".. autoprogram-cliff:: openstack.share.v2"},{"line_number":27,"context_line":"    :command: share unset"},{"line_number":28,"context_line":"    :command: share properties show"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":".. autoprogram-cliff:: openstack.share.v2"},{"line_number":31,"context_line":"    :command: share resize"}],"source_content_type":"text/x-rst","patch_set":7,"id":"ae8126eb_126c7a12","line":28,"range":{"start_line":28,"start_character":0,"end_line":28,"end_character":35},"updated":"2021-01-29 20:45:06.000000000","message":"You\u0027ll need to put this into its own cliff section:\n\n .. autoprogram-cliff:: openstack.share.v2\n    :command: share properties show","commit_id":"4be5d3975869cb6b0a8fd0052d9d023162f5939d"}],"manilaclient/osc/v2/share.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"db03f3dfa2a99baa02e4fe09ebffd6b4e43ec26d","unresolved":true,"context_lines":[{"line_number":1014,"context_line":""},{"line_number":1015,"context_line":"    def take_action(self, parsed_args):"},{"line_number":1016,"context_line":"        share_client \u003d self.app.client_manager.share"},{"line_number":1017,"context_line":""},{"line_number":1018,"context_line":"        share \u003d apiutils.find_resource(share_client.shares,"},{"line_number":1019,"context_line":"                                       parsed_args.share)"},{"line_number":1020,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"d7fb00e5_00f30e60","line":1017,"range":{"start_line":1017,"start_character":0,"end_line":1017,"end_character":0},"updated":"2021-01-29 19:24:31.000000000","message":"unrelated new line change, please remove if you make another patch","commit_id":"1df4c32be3e71f9dfbc9da3906a1db34d942c393"}],"manilaclient/tests/unit/osc/v2/test_share.py":[{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"90c524496691afdc02b06a25701557e02d54dc95","unresolved":false,"context_lines":[{"line_number":1207,"context_line":"                          self.check_parser, self.cmd, arglist, verifylist)"},{"line_number":1208,"context_line":""},{"line_number":1209,"context_line":""},{"line_number":1210,"context_line":"# class TestShowShareProperties(TestShare):"},{"line_number":1211,"context_line":""},{"line_number":1212,"context_line":"#     def setUp(self):"},{"line_number":1213,"context_line":"#         super(TestShowShareProperties, self).setUp()"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f621f24_eab89c11","line":1210,"updated":"2020-11-17 13:20:22.000000000","message":"Comment out? :D","commit_id":"af452479d6934fa618d9facaf4411b4fb7ff7fc7"},{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"9cbe30b7cb83c2459782e2e3f7c065314d275c33","unresolved":false,"context_lines":[{"line_number":23,"context_line":"from manilaclient.common.apiclient import exceptions"},{"line_number":24,"context_line":"from manilaclient.common import cliutils"},{"line_number":25,"context_line":"from manilaclient.osc.v2 import share as osc_shares"},{"line_number":26,"context_line":"from manilaclient.tests.unit.osc import osc_fakes as osc_fakes"},{"line_number":27,"context_line":"from manilaclient.tests.unit.osc import osc_utils"},{"line_number":28,"context_line":"from manilaclient.tests.unit.osc.v2 import fakes as manila_fakes"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fffc6b78_03cb2e53","line":27,"range":{"start_line":26,"start_character":0,"end_line":27,"end_character":49},"updated":"2020-11-19 19:47:56.000000000","message":"Combine the two lines:\nfrom manilaclient.tests.unit.osc import osc_utils, osc_fakes","commit_id":"a59c8e5fb295c87aee8d43819272fa2a139d6d64"},{"author":{"_account_id":31213,"name":"Maari Tamm","email":"maari.tamm@citynetwork.eu","username":"maaritamm"},"change_message_id":"5c68287306040af394bc8b905f83f912590269c0","unresolved":true,"context_lines":[{"line_number":23,"context_line":"from manilaclient.common.apiclient import exceptions"},{"line_number":24,"context_line":"from manilaclient.common import cliutils"},{"line_number":25,"context_line":"from manilaclient.osc.v2 import share as osc_shares"},{"line_number":26,"context_line":"from manilaclient.tests.unit.osc import osc_fakes as osc_fakes"},{"line_number":27,"context_line":"from manilaclient.tests.unit.osc import osc_utils"},{"line_number":28,"context_line":"from manilaclient.tests.unit.osc.v2 import fakes as manila_fakes"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"41c27445_2541fb2a","line":27,"range":{"start_line":26,"start_character":0,"end_line":27,"end_character":49},"in_reply_to":"6d302cd2_f915cc88","updated":"2020-11-29 09:45:06.000000000","message":"Actually, this will not make pep8 happy (H301: one import per line). Please ignore my first comment, keep the imports in separate lines but no need to add \u0027as osc_fakes\u0027 if you are not giving the import a different name. :) Sorry for the confusion.","commit_id":"a59c8e5fb295c87aee8d43819272fa2a139d6d64"},{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"88b351f9000a7a39f506d10891523ae75b7bd6b9","unresolved":false,"context_lines":[{"line_number":23,"context_line":"from manilaclient.common.apiclient import exceptions"},{"line_number":24,"context_line":"from manilaclient.common import cliutils"},{"line_number":25,"context_line":"from manilaclient.osc.v2 import share as osc_shares"},{"line_number":26,"context_line":"from manilaclient.tests.unit.osc import osc_fakes as osc_fakes"},{"line_number":27,"context_line":"from manilaclient.tests.unit.osc import osc_utils"},{"line_number":28,"context_line":"from manilaclient.tests.unit.osc.v2 import fakes as manila_fakes"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"6d302cd2_f915cc88","line":27,"range":{"start_line":26,"start_character":0,"end_line":27,"end_character":49},"in_reply_to":"fffc6b78_03cb2e53","updated":"2020-11-26 20:46:06.000000000","message":"+1","commit_id":"a59c8e5fb295c87aee8d43819272fa2a139d6d64"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"db03f3dfa2a99baa02e4fe09ebffd6b4e43ec26d","unresolved":true,"context_lines":[{"line_number":1732,"context_line":"            self._share.id"},{"line_number":1733,"context_line":"        ]"},{"line_number":1734,"context_line":"        verifylist \u003d ["},{"line_number":1735,"context_line":""},{"line_number":1736,"context_line":"            (\u0027share\u0027, self._share.id)"},{"line_number":1737,"context_line":"        ]"},{"line_number":1738,"context_line":"        parsed_args \u003d self.check_parser(self.cmd, arglist, verifylist)"}],"source_content_type":"text/x-python","patch_set":6,"id":"d423ea84_f8acc7cb","line":1735,"range":{"start_line":1735,"start_character":0,"end_line":1735,"end_character":0},"updated":"2021-01-29 19:24:31.000000000","message":"unrelated new line change, please remove if you make another patch","commit_id":"1df4c32be3e71f9dfbc9da3906a1db34d942c393"}]}