)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"3c966cdc6d311c240e36bc72e4ddaeca11bee3aa","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Most modules are switched to using decorators.idempotent_id and"},{"line_number":10,"context_line":"this patch covers the remaining and removes the old decorator check."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"NOTE: The old decorator test.idempotent_id is still necessary to be"},{"line_number":13,"context_line":"      used on the other projects."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Ice7f593fbc109863eba11402c2f3af75fc6ab9a1"},{"line_number":16,"context_line":"Closes-Bug: #1616913"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a461143_4b2e0ca6","line":13,"range":{"start_line":12,"start_character":0,"end_line":13,"end_character":33},"updated":"2017-01-30 04:30:29.000000000","message":"yes. please keep check-uuid support of it for now.","commit_id":"54822e806f983aaa03ecc2c3eebf7b2237129dd5"}],"tempest/lib/cmd/check_uuid.py":[{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"ccca9580eaf9202f6fe9690b7b16c959cb1f8d4b","unresolved":false,"context_lines":[{"line_number":27,"context_line":"import six.moves.urllib.parse as urlparse"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"# TODO(oomichi): Need to remove this after switching all modules to decorators"},{"line_number":30,"context_line":"# on all OpenStack projects because they runs check_uuid on their own gates."},{"line_number":31,"context_line":"OLD_DECORATOR_MODULE \u003d \u0027test\u0027"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"DECORATOR_MODULE \u003d \u0027decorators\u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a430d35_9d38efd4","line":30,"range":{"start_line":30,"start_character":46,"end_line":30,"end_character":56},"updated":"2017-02-08 22:35:02.000000000","message":"check-uuid","commit_id":"9c83006d4415f36040692a7e86d782ea1a3740b4"},{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"56cbffab72821947a43bed8209ff701e28148b0d","unresolved":false,"context_lines":[{"line_number":304,"context_line":"        return bool(self._filter_tests(report, tests))"},{"line_number":305,"context_line":""},{"line_number":306,"context_line":"    def report_untagged(self, tests):"},{"line_number":307,"context_line":"        \"\"\"Reports untagged tests if there are any"},{"line_number":308,"context_line":""},{"line_number":309,"context_line":"        Returns true if untagged tests exist."},{"line_number":310,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":6,"id":"fa31d9ce_f43248f1","line":307,"updated":"2017-02-13 22:42:00.000000000","message":"i guess it makes sense to detect and report the use of the old decorator. (don\u0027t make the command fail, just warnings)\nit isn\u0027t necessarily to be a part of this patch though.","commit_id":"44f01273cff6edaf2f2fb079895105e79afbf1c1"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"aa1716f4c395250aaa068f67a1993ef4e9501ec1","unresolved":false,"context_lines":[{"line_number":304,"context_line":"        return bool(self._filter_tests(report, tests))"},{"line_number":305,"context_line":""},{"line_number":306,"context_line":"    def report_untagged(self, tests):"},{"line_number":307,"context_line":"        \"\"\"Reports untagged tests if there are any"},{"line_number":308,"context_line":""},{"line_number":309,"context_line":"        Returns true if untagged tests exist."},{"line_number":310,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":6,"id":"fa31d9ce_8f38153f","line":307,"in_reply_to":"fa31d9ce_f43248f1","updated":"2017-02-13 23:39:30.000000000","message":"I may not got your comment completely but this function for untagged tests and error with new decorator is ok here.","commit_id":"44f01273cff6edaf2f2fb079895105e79afbf1c1"}],"tempest/test.py":[{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"ccca9580eaf9202f6fe9690b7b16c959cb1f8d4b","unresolved":false,"context_lines":[{"line_number":41,"context_line":""},{"line_number":42,"context_line":"# TODO(oomichi): This test.idempotent_id should be removed after all projects"},{"line_number":43,"context_line":"# switch to use decorators.idempotent_id."},{"line_number":44,"context_line":"idempotent_id \u003d decorators.idempotent_id"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"def attr(**kwargs):"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a430d35_fd9d73bc","line":44,"updated":"2017-02-08 22:35:02.000000000","message":"consider using debtcollector.moves.moved_function","commit_id":"9c83006d4415f36040692a7e86d782ea1a3740b4"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"f0ec67bf65259aa21f0d33dd4c596da832696c2b","unresolved":false,"context_lines":[{"line_number":43,"context_line":"# switch to use decorators.idempotent_id."},{"line_number":44,"context_line":"idempotent_id \u003d debtcollector.moves.moved_function("},{"line_number":45,"context_line":"    decorators.idempotent_id, \u0027idempotent_id\u0027, __name__,"},{"line_number":46,"context_line":"    version\u003d\u0027Ocata\u0027, removal_version\u003d\u0027Q\u0027)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"def attr(**kwargs):"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa31d9ce_5f772f02","line":46,"range":{"start_line":46,"start_character":21,"end_line":46,"end_character":40},"updated":"2017-02-10 00:54:08.000000000","message":"Q might be too late to keep this ?\n\nAs you know I am fixing all plugins for this [1] and around 70% done. Can we make it P.\n\n..1 https://review.openstack.org/#/q/branch:master+topic:bug/1616913+owner:+ghanshyam.mann%2540nectechnologies.in","commit_id":"40123fa75e3fabc5cb0ec747fe82990b527ef91b"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"0a1e3d360f3c251cb96659015efd0e49d50c9347","unresolved":false,"context_lines":[{"line_number":43,"context_line":"# switch to use decorators.idempotent_id."},{"line_number":44,"context_line":"idempotent_id \u003d debtcollector.moves.moved_function("},{"line_number":45,"context_line":"    decorators.idempotent_id, \u0027idempotent_id\u0027, __name__,"},{"line_number":46,"context_line":"    version\u003d\u0027Ocata\u0027, removal_version\u003d\u0027Q\u0027)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"def attr(**kwargs):"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa31d9ce_9cf98ecb","line":46,"range":{"start_line":46,"start_character":21,"end_line":46,"end_character":40},"in_reply_to":"fa31d9ce_020549c1","updated":"2017-02-10 17:01:44.000000000","message":"Haha, OK. Everyone is confident even if users attach us due to this removal.\nLet\u0027s put ? here as \"a string of ‘?’ will denote this will be removed in some future unknown version\"","commit_id":"40123fa75e3fabc5cb0ec747fe82990b527ef91b"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"6bbef62fc91ec0104a95760329e01f02c6b6dc9f","unresolved":false,"context_lines":[{"line_number":43,"context_line":"# switch to use decorators.idempotent_id."},{"line_number":44,"context_line":"idempotent_id \u003d debtcollector.moves.moved_function("},{"line_number":45,"context_line":"    decorators.idempotent_id, \u0027idempotent_id\u0027, __name__,"},{"line_number":46,"context_line":"    version\u003d\u0027Ocata\u0027, removal_version\u003d\u0027Q\u0027)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"def attr(**kwargs):"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa31d9ce_020549c1","line":46,"range":{"start_line":46,"start_character":21,"end_line":46,"end_character":40},"in_reply_to":"fa31d9ce_3f227bba","updated":"2017-02-10 09:33:14.000000000","message":"This is not a stable interface so we are not bound by any deprecation policy. I think it would be fine to simply have the comment oomichi already added in there, and as soon it\u0027s removed by all known plugins just drop it.\nWe can never be sure who\u0027s using this in their downstream code, but that can\u0027t be helped really.\n\nPersonally I would put \u0027?\u0027 as removal_version as per http://docs.openstack.org/developer/debtcollector/api.html#module-debtcollector.removals as I don\u0027t want to commit to keeping this around until Q.","commit_id":"40123fa75e3fabc5cb0ec747fe82990b527ef91b"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"297a72170e6b0111d8148de0fceae4e41576349f","unresolved":false,"context_lines":[{"line_number":43,"context_line":"# switch to use decorators.idempotent_id."},{"line_number":44,"context_line":"idempotent_id \u003d debtcollector.moves.moved_function("},{"line_number":45,"context_line":"    decorators.idempotent_id, \u0027idempotent_id\u0027, __name__,"},{"line_number":46,"context_line":"    version\u003d\u0027Ocata\u0027, removal_version\u003d\u0027Q\u0027)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"def attr(**kwargs):"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa31d9ce_3f227bba","line":46,"range":{"start_line":46,"start_character":21,"end_line":46,"end_character":40},"in_reply_to":"fa31d9ce_5f772f02","updated":"2017-02-10 01:03:52.000000000","message":"I think Q is normal deprecation period in OpenStack.","commit_id":"40123fa75e3fabc5cb0ec747fe82990b527ef91b"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"1a12154b3975f8027480e81b7942dd96cdbc14e6","unresolved":false,"context_lines":[{"line_number":43,"context_line":"# switch to use decorators.idempotent_id."},{"line_number":44,"context_line":"idempotent_id \u003d debtcollector.moves.moved_function("},{"line_number":45,"context_line":"    decorators.idempotent_id, \u0027idempotent_id\u0027, __name__,"},{"line_number":46,"context_line":"    version\u003d\u0027Ocata\u0027, removal_version\u003d\u0027?\u0027)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"def attr(**kwargs):"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa31d9ce_4a039a42","line":46,"updated":"2017-02-13 06:31:30.000000000","message":"I actually feel \u0027?\u0027 is just confusing for users, though...","commit_id":"2d0f561f8bdc0f2de904c24fd4ecebc2f123e70e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"9d94c6aad339366116b69a5fa7ef267069e082ed","unresolved":false,"context_lines":[{"line_number":43,"context_line":"# switch to use decorators.idempotent_id."},{"line_number":44,"context_line":"idempotent_id \u003d debtcollector.moves.moved_function("},{"line_number":45,"context_line":"    decorators.idempotent_id, \u0027idempotent_id\u0027, __name__,"},{"line_number":46,"context_line":"    version\u003d\u0027Ocata\u0027, removal_version\u003d\u0027?\u0027)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"def attr(**kwargs):"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa31d9ce_316ed92f","line":46,"range":{"start_line":46,"start_character":12,"end_line":46,"end_character":18},"updated":"2017-02-13 09:04:47.000000000","message":"actually this is available since Mitaka. Because warning says it is moved to new location in \u0027ocata\u0027-\n\nDeprecationWarning: Function \u0027tempest.test.idempotent_id()\u0027 has moved to \u0027tempest.lib.decorators.idempotent_id()\u0027 in version \u0027Ocata\u0027 and will be removed in a future version","commit_id":"2d0f561f8bdc0f2de904c24fd4ecebc2f123e70e"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"7d72f63b5f2985980f6d7e07b22b187c555ad418","unresolved":false,"context_lines":[{"line_number":43,"context_line":"# switch to use decorators.idempotent_id."},{"line_number":44,"context_line":"idempotent_id \u003d debtcollector.moves.moved_function("},{"line_number":45,"context_line":"    decorators.idempotent_id, \u0027idempotent_id\u0027, __name__,"},{"line_number":46,"context_line":"    version\u003d\u0027Ocata\u0027, removal_version\u003d\u0027?\u0027)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"def attr(**kwargs):"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa31d9ce_59defa3f","line":46,"range":{"start_line":46,"start_character":12,"end_line":46,"end_character":18},"in_reply_to":"fa31d9ce_316ed92f","updated":"2017-02-13 17:29:46.000000000","message":"Maybe you are just saying \"Here should be Mitaka instead of Ocata\" because new one has been implemented since Mitaka.","commit_id":"2d0f561f8bdc0f2de904c24fd4ecebc2f123e70e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"9d94c6aad339366116b69a5fa7ef267069e082ed","unresolved":false,"context_lines":[{"line_number":43,"context_line":"# switch to use decorators.idempotent_id."},{"line_number":44,"context_line":"idempotent_id \u003d debtcollector.moves.moved_function("},{"line_number":45,"context_line":"    decorators.idempotent_id, \u0027idempotent_id\u0027, __name__,"},{"line_number":46,"context_line":"    version\u003d\u0027Ocata\u0027, removal_version\u003d\u0027?\u0027)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"def attr(**kwargs):"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa31d9ce_b62b9fea","line":46,"in_reply_to":"fa31d9ce_4a039a42","updated":"2017-02-13 09:04:47.000000000","message":"yea but seems like doc and code says so and should be fine-http://docs.openstack.org/developer/debtcollector/api.html\nhttps://github.com/openstack/debtcollector/blob/master/debtcollector/_utils.py#L80","commit_id":"2d0f561f8bdc0f2de904c24fd4ecebc2f123e70e"},{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"56cbffab72821947a43bed8209ff701e28148b0d","unresolved":false,"context_lines":[{"line_number":43,"context_line":"# switch to use decorators.idempotent_id."},{"line_number":44,"context_line":"idempotent_id \u003d debtcollector.moves.moved_function("},{"line_number":45,"context_line":"    decorators.idempotent_id, \u0027idempotent_id\u0027, __name__,"},{"line_number":46,"context_line":"    version\u003d\u0027Mitaka\u0027, removal_version\u003d\u0027?\u0027)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"def attr(**kwargs):"}],"source_content_type":"text/x-python","patch_set":6,"id":"fa31d9ce_f4b7a837","line":46,"range":{"start_line":46,"start_character":39,"end_line":46,"end_character":40},"updated":"2017-02-13 22:42:00.000000000","message":"i prefer to follow the standard procedure, especially when we can\u0027t save much by the early removal.","commit_id":"44f01273cff6edaf2f2fb079895105e79afbf1c1"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"aa1716f4c395250aaa068f67a1993ef4e9501ec1","unresolved":false,"context_lines":[{"line_number":43,"context_line":"# switch to use decorators.idempotent_id."},{"line_number":44,"context_line":"idempotent_id \u003d debtcollector.moves.moved_function("},{"line_number":45,"context_line":"    decorators.idempotent_id, \u0027idempotent_id\u0027, __name__,"},{"line_number":46,"context_line":"    version\u003d\u0027Mitaka\u0027, removal_version\u003d\u0027?\u0027)"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"def attr(**kwargs):"}],"source_content_type":"text/x-python","patch_set":6,"id":"fa31d9ce_0f1c25c0","line":46,"range":{"start_line":46,"start_character":39,"end_line":46,"end_character":40},"in_reply_to":"fa31d9ce_f4b7a837","updated":"2017-02-13 23:39:30.000000000","message":"Sure Yamamoto san but these interface never been stable one and tempest did not guarantee not to change/remove those any time. But yea for smooth transition we will keep till all plugins being fixed for this.","commit_id":"44f01273cff6edaf2f2fb079895105e79afbf1c1"}]}