)]}'
{"tempest/api/identity/v2/test_ec2_credentials.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"2c90262235c264c735c8a882101b499f73ce03ac","unresolved":false,"context_lines":[{"line_number":33,"context_line":"        cls.creds \u003d cls.os.credentials"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    @test.idempotent_id(\u0027b580fab9-7ae9-46e8-8138-417260cb6f9f\u0027)"},{"line_number":36,"context_line":"    def test_create_ec2_credentials(self):"},{"line_number":37,"context_line":"        \"\"\"Create user ec2 credentials.\"\"\""},{"line_number":38,"context_line":"        resp \u003d self.non_admin_users_client.create_user_ec2_credential("},{"line_number":39,"context_line":"            self.creds.user_id,"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a9d85d2_6e36a6e8","line":36,"range":{"start_line":36,"start_character":34,"end_line":36,"end_character":35},"updated":"2016-06-21 06:34:42.000000000","message":"I think this should be removed. The others are also same.","commit_id":"f6f6b7be8ec3f309e66d4c25b6cf577002399705"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"8aa343c9faf5bf899473c08dfbcf88e26a1f024a","unresolved":false,"context_lines":[{"line_number":33,"context_line":"        cls.creds \u003d cls.os.credentials"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    @test.idempotent_id(\u0027b580fab9-7ae9-46e8-8138-417260cb6f9f\u0027)"},{"line_number":36,"context_line":"    def test_create_ec2_credentials(self):"},{"line_number":37,"context_line":"        \"\"\"Create user ec2 credentials.\"\"\""},{"line_number":38,"context_line":"        resp \u003d self.non_admin_users_client.create_user_ec2_credential("},{"line_number":39,"context_line":"            self.creds.user_id,"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a9d85d2_649247a1","line":36,"range":{"start_line":36,"start_character":34,"end_line":36,"end_character":35},"in_reply_to":"5a9d85d2_6e36a6e8","updated":"2016-06-21 07:33:57.000000000","message":"Done. Thanks","commit_id":"f6f6b7be8ec3f309e66d4c25b6cf577002399705"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"5b4ad7a8f34f7b8db87bb819d5e528ed3b1d53bf","unresolved":false,"context_lines":[{"line_number":33,"context_line":"        cls.creds \u003d cls.os.credentials"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    @test.idempotent_id(\u0027b580fab9-7ae9-46e8-8138-417260cb6f9f\u0027)"},{"line_number":36,"context_line":"    def test_create_ec2_credentials(self):"},{"line_number":37,"context_line":"        \"\"\"Create user ec2 credentials.\"\"\""},{"line_number":38,"context_line":"        resp \u003d self.non_admin_users_client.create_user_ec2_credentials("},{"line_number":39,"context_line":"            self.creds.user_id,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3aaa91ec_365b9556","side":"PARENT","line":36,"range":{"start_line":36,"start_character":0,"end_line":36,"end_character":35},"updated":"2016-06-21 18:48:57.000000000","message":"Masayuki,\n\nDid you check these tests are not used on Refstack side?\nBasically, we are avoiding to rename test classes/methods names because refstack is specifying tests by names.","commit_id":"e612bf2b25265f50ee289902c57f682c31dfc691"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"9eeab09519f6cbc7caed66fbe1277e18abd7689d","unresolved":false,"context_lines":[{"line_number":33,"context_line":"        cls.creds \u003d cls.os.credentials"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    @test.idempotent_id(\u0027b580fab9-7ae9-46e8-8138-417260cb6f9f\u0027)"},{"line_number":36,"context_line":"    def test_create_ec2_credentials(self):"},{"line_number":37,"context_line":"        \"\"\"Create user ec2 credentials.\"\"\""},{"line_number":38,"context_line":"        resp \u003d self.non_admin_users_client.create_user_ec2_credentials("},{"line_number":39,"context_line":"            self.creds.user_id,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3aaa91ec_5ee748e4","side":"PARENT","line":36,"range":{"start_line":36,"start_character":0,"end_line":36,"end_character":35},"in_reply_to":"3aaa91ec_365b9556","updated":"2016-06-22 01:55:39.000000000","message":"Do you mean defcore? I checked defcore repo[1] and there\u0027s no \u0027ec2_credentials\u0027 word in it.\n\nBut hmm, what\u0027s the purpose of the \u0027idempotent_id\u0027s if we can\u0027t change the method name?\n\n[1] http://codesearch.openstack.org/?q\u003dec2_credentials\u0026i\u003dnope\u0026files\u003d\u0026repos\u003ddefcore","commit_id":"e612bf2b25265f50ee289902c57f682c31dfc691"}]}
