)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"0aa5ee33e2f42e9c2dc54a97ba4dc44f049c040f","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Rodrigo Barbieri \u003crodrigo.barbieri@fit-tecnologia.org.br\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-02-17 14:16:42 -0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove revert-to-snapshot unused test option"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This option are used to control how revert-to-snapshot"},{"line_number":10,"context_line":"extra-spec is used when creating a share type for several tests,"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da36d5c6_4e368304","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":44},"updated":"2017-02-20 13:30:13.000000000","message":"It is false statement. It is used, but can be considered as redundant.","commit_id":"52861993bc5a7d3104b54e73ad8741bb54fa05c8"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"3d7f725b3e13e9b5d06aa3c9e02d931007d7c7df","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Rodrigo Barbieri \u003crodrigo.barbieri@fit-tecnologia.org.br\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-02-17 14:16:42 -0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove revert-to-snapshot unused test option"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This option are used to control how revert-to-snapshot"},{"line_number":10,"context_line":"extra-spec is used when creating a share type for several tests,"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da36d5c6_31636fb3","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":44},"in_reply_to":"da36d5c6_4e368304","updated":"2017-02-20 15:52:30.000000000","message":"Done","commit_id":"52861993bc5a7d3104b54e73ad8741bb54fa05c8"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"0aa5ee33e2f42e9c2dc54a97ba4dc44f049c040f","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove revert-to-snapshot unused test option"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This option are used to control how revert-to-snapshot"},{"line_number":10,"context_line":"extra-spec is used when creating a share type for several tests,"},{"line_number":11,"context_line":"but for the same tests, it is more appropriate to omit this"},{"line_number":12,"context_line":"extra-spec so it assumes the \"don\u0027t care\" behavior."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da36d5c6_0e472bbc","line":9,"range":{"start_line":9,"start_character":12,"end_line":9,"end_character":15},"updated":"2017-02-20 13:30:13.000000000","message":"s/are/is/","commit_id":"52861993bc5a7d3104b54e73ad8741bb54fa05c8"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"3d7f725b3e13e9b5d06aa3c9e02d931007d7c7df","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove revert-to-snapshot unused test option"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This option are used to control how revert-to-snapshot"},{"line_number":10,"context_line":"extra-spec is used when creating a share type for several tests,"},{"line_number":11,"context_line":"but for the same tests, it is more appropriate to omit this"},{"line_number":12,"context_line":"extra-spec so it assumes the \"don\u0027t care\" behavior."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da36d5c6_5158e3fa","line":9,"range":{"start_line":9,"start_character":12,"end_line":9,"end_character":15},"in_reply_to":"da36d5c6_0e472bbc","updated":"2017-02-20 15:52:30.000000000","message":"Done","commit_id":"52861993bc5a7d3104b54e73ad8741bb54fa05c8"},{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"0aa5ee33e2f42e9c2dc54a97ba4dc44f049c040f","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This option are used to control how revert-to-snapshot"},{"line_number":10,"context_line":"extra-spec is used when creating a share type for several tests,"},{"line_number":11,"context_line":"but for the same tests, it is more appropriate to omit this"},{"line_number":12,"context_line":"extra-spec so it assumes the \"don\u0027t care\" behavior."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"TrivialFix"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da36d5c6_4e4f6392","line":11,"range":{"start_line":11,"start_character":8,"end_line":11,"end_character":16},"updated":"2017-02-20 13:30:13.000000000","message":"s/the same/all other/ ?","commit_id":"52861993bc5a7d3104b54e73ad8741bb54fa05c8"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"3d7f725b3e13e9b5d06aa3c9e02d931007d7c7df","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This option are used to control how revert-to-snapshot"},{"line_number":10,"context_line":"extra-spec is used when creating a share type for several tests,"},{"line_number":11,"context_line":"but for the same tests, it is more appropriate to omit this"},{"line_number":12,"context_line":"extra-spec so it assumes the \"don\u0027t care\" behavior."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"TrivialFix"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da36d5c6_b1cbdf71","line":11,"range":{"start_line":11,"start_character":8,"end_line":11,"end_character":16},"in_reply_to":"da36d5c6_4e4f6392","updated":"2017-02-20 15:52:30.000000000","message":"see if it is more clear now.","commit_id":"52861993bc5a7d3104b54e73ad8741bb54fa05c8"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"0f06c4ca458184b60de88a6731f36d9bfac5bece","unresolved":false,"context_lines":[{"line_number":13,"context_line":"behavior) or do so based on the value of"},{"line_number":14,"context_line":"run_revert_to_snapshot_tests."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"TrivialFix"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: Ie8a0030a896634eb8deaff17ea72371419eecdea"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"da36d5c6_c73411e0","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":10},"updated":"2017-03-17 19:59:02.000000000","message":"Not a trivial fix. This option has been advertised to downstream CIs.","commit_id":"30403d05bfcf6fb11ebfc88a4f1a4b48bde6eb70"},{"author":{"_account_id":2417,"name":"Ben Swartzlander","email":"ben@swartzlander.org","username":"bswartz"},"change_message_id":"876cd2fe78ba62f24f2a4c30439a8bf650fadcfe","unresolved":false,"context_lines":[{"line_number":13,"context_line":"behavior) or do so based on the value of"},{"line_number":14,"context_line":"run_revert_to_snapshot_tests."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"TrivialFix"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: Ie8a0030a896634eb8deaff17ea72371419eecdea"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1a1ced50_4d491f5f","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":10},"in_reply_to":"da36d5c6_c73411e0","updated":"2017-03-17 20:33:43.000000000","message":"I agree not a trivial fix, but also not a bug or a blueprint. I\u0027m not sure what would be better here, other than to leave \"trivialfix\" out, because it doesn\u0027t affect how we\u0027ll treat this patch either way.","commit_id":"30403d05bfcf6fb11ebfc88a4f1a4b48bde6eb70"}],"manila_tempest_tests/config.py":[{"author":{"_account_id":8851,"name":"Valeriy Ponomaryov","email":"kiparis.kh@gmail.com","username":"vponomaryov"},"change_message_id":"5275d3fdb5eccb711f0da1cd1155d1fd2a82f3e6","unresolved":false,"context_lines":[{"line_number":103,"context_line":"                     \"Defaults to the value of run_snapshot_tests. Set it to \""},{"line_number":104,"context_line":"                     \"False if the driver being tested does not support \""},{"line_number":105,"context_line":"                     \"creating shares from snapshots.\"),"},{"line_number":106,"context_line":"    cfg.BoolOpt(\"capability_revert_to_snapshot_support\","},{"line_number":107,"context_line":"                help\u003d\"Defines extra spec that satisfies specific back end \""},{"line_number":108,"context_line":"                     \"capability called \u0027revert_to_snapshot_support\u0027 \""},{"line_number":109,"context_line":"                     \"and will be used for setting up custom share type. \""},{"line_number":110,"context_line":"                     \"Defaults to the value of run_revert_to_snapshot_tests.\"),"},{"line_number":111,"context_line":"    cfg.StrOpt(\"share_network_id\","},{"line_number":112,"context_line":"               default\u003d\"\","},{"line_number":113,"context_line":"               help\u003d\"Some backend drivers requires share network \""}],"source_content_type":"text/x-python","patch_set":2,"id":"da36d5c6_39ffdb02","side":"PARENT","line":110,"range":{"start_line":106,"start_character":4,"end_line":110,"end_character":79},"updated":"2017-02-20 13:31:09.000000000","message":"We cannot just remove options if they were released. Need to follow \"deprecation\" approach saying it is not used anymore.","commit_id":"595a2bd73caf09548f14921f12d3ee5550b061df"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"3d7f725b3e13e9b5d06aa3c9e02d931007d7c7df","unresolved":false,"context_lines":[{"line_number":103,"context_line":"                     \"Defaults to the value of run_snapshot_tests. Set it to \""},{"line_number":104,"context_line":"                     \"False if the driver being tested does not support \""},{"line_number":105,"context_line":"                     \"creating shares from snapshots.\"),"},{"line_number":106,"context_line":"    cfg.BoolOpt(\"capability_revert_to_snapshot_support\","},{"line_number":107,"context_line":"                help\u003d\"Defines extra spec that satisfies specific back end \""},{"line_number":108,"context_line":"                     \"capability called \u0027revert_to_snapshot_support\u0027 \""},{"line_number":109,"context_line":"                     \"and will be used for setting up custom share type. \""},{"line_number":110,"context_line":"                     \"Defaults to the value of run_revert_to_snapshot_tests.\"),"},{"line_number":111,"context_line":"    cfg.StrOpt(\"share_network_id\","},{"line_number":112,"context_line":"               default\u003d\"\","},{"line_number":113,"context_line":"               help\u003d\"Some backend drivers requires share network \""}],"source_content_type":"text/x-python","patch_set":2,"id":"da36d5c6_91c41b93","side":"PARENT","line":110,"range":{"start_line":106,"start_character":4,"end_line":110,"end_character":79},"in_reply_to":"da36d5c6_39ffdb02","updated":"2017-02-20 15:52:30.000000000","message":"Done","commit_id":"595a2bd73caf09548f14921f12d3ee5550b061df"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"01feb07bd80de4fb0b906d86d1837d90503c8ef7","unresolved":false,"context_lines":[{"line_number":103,"context_line":"                     \"Defaults to the value of run_snapshot_tests. Set it to \""},{"line_number":104,"context_line":"                     \"False if the driver being tested does not support \""},{"line_number":105,"context_line":"                     \"creating shares from snapshots.\"),"},{"line_number":106,"context_line":"    cfg.BoolOpt(\"capability_revert_to_snapshot_support\","},{"line_number":107,"context_line":"                deprecated_for_removal\u003dTrue,"},{"line_number":108,"context_line":"                deprecated_reason\u003d\"Redundant configuration option. Please use \""},{"line_number":109,"context_line":"                                  \"\u0027run_revert_to_snapshot_tests\u0027 config \""},{"line_number":110,"context_line":"                                  \"option instead.\","},{"line_number":111,"context_line":"                help\u003d\"Defines extra spec that satisfies specific back end \""},{"line_number":112,"context_line":"                     \"capability called \u0027revert_to_snapshot_support\u0027 \""},{"line_number":113,"context_line":"                     \"and will be used for setting up custom share type. \""}],"source_content_type":"text/x-python","patch_set":3,"id":"1a1ced50_a520e79c","line":110,"range":{"start_line":106,"start_character":0,"end_line":110,"end_character":52},"updated":"2017-03-17 23:57:45.000000000","message":"With this patch, we will stop honoring this in the tests tests. Why keep this option around?","commit_id":"30403d05bfcf6fb11ebfc88a4f1a4b48bde6eb70"},{"author":{"_account_id":14567,"name":"Rodrigo Barbieri","email":"rodrigo.barbieri2010@gmail.com","username":"ganso"},"change_message_id":"0bf8b646b7cc91d15b486be102c7d57dc2ed0137","unresolved":false,"context_lines":[{"line_number":103,"context_line":"                     \"Defaults to the value of run_snapshot_tests. Set it to \""},{"line_number":104,"context_line":"                     \"False if the driver being tested does not support \""},{"line_number":105,"context_line":"                     \"creating shares from snapshots.\"),"},{"line_number":106,"context_line":"    cfg.BoolOpt(\"capability_revert_to_snapshot_support\","},{"line_number":107,"context_line":"                deprecated_for_removal\u003dTrue,"},{"line_number":108,"context_line":"                deprecated_reason\u003d\"Redundant configuration option. Please use \""},{"line_number":109,"context_line":"                                  \"\u0027run_revert_to_snapshot_tests\u0027 config \""},{"line_number":110,"context_line":"                                  \"option instead.\","},{"line_number":111,"context_line":"                help\u003d\"Defines extra spec that satisfies specific back end \""},{"line_number":112,"context_line":"                     \"capability called \u0027revert_to_snapshot_support\u0027 \""},{"line_number":113,"context_line":"                     \"and will be used for setting up custom share type. \""}],"source_content_type":"text/x-python","patch_set":3,"id":"1a1ced50_65a18fa9","line":110,"range":{"start_line":106,"start_character":0,"end_line":110,"end_character":52},"in_reply_to":"1a1ced50_a520e79c","updated":"2017-03-18 00:32:45.000000000","message":"I was in for removing directly... but some would argue for following the deprecation process for testing config options too.","commit_id":"30403d05bfcf6fb11ebfc88a4f1a4b48bde6eb70"}]}