)]}'
{"octaviaclient/osc/v2/pool.py":[{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"8b1173fc0c73a89e200526a84179c66960d8b2d9","unresolved":false,"context_lines":[{"line_number":361,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":362,"context_line":"            help\u003d\"Clear the certificate reference for this pool.\""},{"line_number":363,"context_line":"        )"},{"line_number":364,"context_line":"        return parser"},{"line_number":365,"context_line":""},{"line_number":366,"context_line":"    def take_action(self, parsed_args):"},{"line_number":367,"context_line":"        unset_args \u003d v2_utils.get_unsets(parsed_args)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7faddb67_b255a452","line":364,"updated":"2019-07-26 22:14:42.000000000","message":"tls_enabled is missing","commit_id":"89bfad3f87d99eb2c0983bd9df888614fa549378"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"c01b9fe605be1264be7047583f69eafcdab7ea68","unresolved":false,"context_lines":[{"line_number":361,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":362,"context_line":"            help\u003d\"Clear the certificate reference for this pool.\""},{"line_number":363,"context_line":"        )"},{"line_number":364,"context_line":"        return parser"},{"line_number":365,"context_line":""},{"line_number":366,"context_line":"    def take_action(self, parsed_args):"},{"line_number":367,"context_line":"        unset_args \u003d v2_utils.get_unsets(parsed_args)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7faddb67_c6e65e17","line":364,"in_reply_to":"7faddb67_46d26e5e","updated":"2019-07-27 09:33:05.000000000","message":"Meh, ok. Let\u0027s merge and come back to this if we want to.","commit_id":"89bfad3f87d99eb2c0983bd9df888614fa549378"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"e79d38570528b0faef9898e3ece0f355411707fe","unresolved":false,"context_lines":[{"line_number":361,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":362,"context_line":"            help\u003d\"Clear the certificate reference for this pool.\""},{"line_number":363,"context_line":"        )"},{"line_number":364,"context_line":"        return parser"},{"line_number":365,"context_line":""},{"line_number":366,"context_line":"    def take_action(self, parsed_args):"},{"line_number":367,"context_line":"        unset_args \u003d v2_utils.get_unsets(parsed_args)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7faddb67_46d26e5e","line":364,"in_reply_to":"7faddb67_7535d6d9","updated":"2019-07-27 08:52:53.000000000","message":"I was on the fence with this one because on the member unset patch, we support unsetting \"backup\" which is also a boolean so I was for uniformization. Perhaps we should remove the backup unset?","commit_id":"89bfad3f87d99eb2c0983bd9df888614fa549378"},{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"139ceeb6b9c3e4208305d0da19cde926d369e288","unresolved":false,"context_lines":[{"line_number":361,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":362,"context_line":"            help\u003d\"Clear the certificate reference for this pool.\""},{"line_number":363,"context_line":"        )"},{"line_number":364,"context_line":"        return parser"},{"line_number":365,"context_line":""},{"line_number":366,"context_line":"    def take_action(self, parsed_args):"},{"line_number":367,"context_line":"        unset_args \u003d v2_utils.get_unsets(parsed_args)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7faddb67_7535d6d9","line":364,"in_reply_to":"7faddb67_b255a452","updated":"2019-07-26 22:43:07.000000000","message":"enable-tls / disable-tls is a boolean-switch pair. I don\u0027t see why it\u0027d need an unset... There is no third state there :D","commit_id":"89bfad3f87d99eb2c0983bd9df888614fa549378"}]}