)]}'
{"openstack/block_storage/v3/_proxy.py":[{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"0eebf2f4e70f48b4a60f92c9ab41090a1b79af2e","unresolved":true,"context_lines":[{"line_number":389,"context_line":""},{"line_number":390,"context_line":"        :param name_or_id: Name, unique ID of the volume or a volume dict."},{"line_number":391,"context_line":"        :param bool readonly: Whether the volume should be a read-only volume"},{"line_number":392,"context_line":"                               or not"},{"line_number":393,"context_line":""},{"line_number":394,"context_line":"        :raises: OpenStackCloudTimeout if wait time exceeded."},{"line_number":395,"context_line":"        :raises: OpenStackCloudException on operation error."}],"source_content_type":"text/x-python","patch_set":4,"id":"a3290f66_3a80f35b","line":392,"updated":"2021-03-03 10:30:10.000000000","message":"would you please align this properly (4 chars after :param please)","commit_id":"9ffbe191c53f2dfe3ddc9dcf0d02db9316a2ecf3"},{"author":{"_account_id":32349,"name":"Thiago Paiva Brito","email":"thiago.brito@windriver.com","username":"outbrito"},"change_message_id":"41c92f46e429ef5450b7b2393389629a20a1b1fd","unresolved":false,"context_lines":[{"line_number":389,"context_line":""},{"line_number":390,"context_line":"        :param name_or_id: Name, unique ID of the volume or a volume dict."},{"line_number":391,"context_line":"        :param bool readonly: Whether the volume should be a read-only volume"},{"line_number":392,"context_line":"                               or not"},{"line_number":393,"context_line":""},{"line_number":394,"context_line":"        :raises: OpenStackCloudTimeout if wait time exceeded."},{"line_number":395,"context_line":"        :raises: OpenStackCloudException on operation error."}],"source_content_type":"text/x-python","patch_set":4,"id":"66f53772_978e76a2","line":392,"in_reply_to":"a3290f66_3a80f35b","updated":"2021-03-03 15:11:59.000000000","message":"Done","commit_id":"9ffbe191c53f2dfe3ddc9dcf0d02db9316a2ecf3"}],"openstack/tests/unit/block_storage/v3/test_volume.py":[{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"0eebf2f4e70f48b4a60f92c9ab41090a1b79af2e","unresolved":true,"context_lines":[{"line_number":131,"context_line":"    def test_set_volume_readonly(self):"},{"line_number":132,"context_line":"        sot \u003d volume.Volume(**VOLUME)"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"        self.assertIsNone(sot.set_readonly(self.sess, True))"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"        url \u003d \u0027volumes/%s/action\u0027 % FAKE_ID"},{"line_number":137,"context_line":"        body \u003d {\u0027os-update_readonly_flag\u0027: {\u0027readonly\u0027: True}}"}],"source_content_type":"text/x-python","patch_set":4,"id":"a43f378f_c66ac058","line":134,"updated":"2021-03-03 10:30:10.000000000","message":"don\u0027t you want to verify the default value in this test (not passing True at all)?","commit_id":"9ffbe191c53f2dfe3ddc9dcf0d02db9316a2ecf3"},{"author":{"_account_id":32349,"name":"Thiago Paiva Brito","email":"thiago.brito@windriver.com","username":"outbrito"},"change_message_id":"41c92f46e429ef5450b7b2393389629a20a1b1fd","unresolved":true,"context_lines":[{"line_number":131,"context_line":"    def test_set_volume_readonly(self):"},{"line_number":132,"context_line":"        sot \u003d volume.Volume(**VOLUME)"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"        self.assertIsNone(sot.set_readonly(self.sess, True))"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"        url \u003d \u0027volumes/%s/action\u0027 % FAKE_ID"},{"line_number":137,"context_line":"        body \u003d {\u0027os-update_readonly_flag\u0027: {\u0027readonly\u0027: True}}"}],"source_content_type":"text/x-python","patch_set":4,"id":"c04fbc4a_1534c82b","line":134,"in_reply_to":"a43f378f_c66ac058","updated":"2021-03-03 15:11:59.000000000","message":"In fact, the default behavior is implemented only at the proxy layer. The call on Volume doesn\u0027t have a default.","commit_id":"9ffbe191c53f2dfe3ddc9dcf0d02db9316a2ecf3"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"164512653855a1c052d102962976fdb2ea16803b","unresolved":true,"context_lines":[{"line_number":131,"context_line":"    def test_set_volume_readonly(self):"},{"line_number":132,"context_line":"        sot \u003d volume.Volume(**VOLUME)"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"        self.assertIsNone(sot.set_readonly(self.sess, True))"},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"        url \u003d \u0027volumes/%s/action\u0027 % FAKE_ID"},{"line_number":137,"context_line":"        body \u003d {\u0027os-update_readonly_flag\u0027: {\u0027readonly\u0027: True}}"}],"source_content_type":"text/x-python","patch_set":4,"id":"43a0ac95_be7d57aa","line":134,"in_reply_to":"c04fbc4a_1534c82b","updated":"2021-03-03 15:39:24.000000000","message":"right, I confused myself.","commit_id":"9ffbe191c53f2dfe3ddc9dcf0d02db9316a2ecf3"}]}