)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"f35b341b4fc4c196b848fa248d0ebdd9d7a11df3","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2014-11-11 00:56:51 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"quota testcases use tenanat_name should use tenanat_id"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Test case related to quota usage (test_show_quota_usage,"},{"line_number":10,"context_line":"test_cinder_quota_defaults and test_cinder_quota_show)"},{"line_number":11,"context_line":"uses tenant name, ideally it should use tenant id as quota"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"5a890539_7aaaa471","line":8,"updated":"2014-11-11 09:12:20.000000000","message":"There is a small typo. \"tenanat\" should be tenant.","commit_id":"59c1a4b6af6d2e05bae551a7942a95e3d0cf02dc"},{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"02c03f87bb5732d49c5e944a8f37ad4ac194bd4b","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Test case related to quota usage (test_show_quota_usage,"},{"line_number":10,"context_line":"test_cinder_quota_defaults and test_cinder_quota_show)"},{"line_number":11,"context_line":"uses tenant name, ideally it should use tenant id as quota"},{"line_number":12,"context_line":"requires UUID of tenant and not tenant name."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Icab25afd482b34290bc74be2ba420f6b016dde18"},{"line_number":15,"context_line":"Closes-Bug: #1321617"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"5a890539_bff155cf","line":12,"updated":"2014-11-11 14:27:58.000000000","message":"I feel this description is not sufficient to understand this bug. As Zhi\u0027s question, I think we should put the reason why we should use tenant id instead of tenant name in this commit message.","commit_id":"a19a2f1959e03690a4ca96c92c499e5c0cc077e2"}],"tempest/cli/simple_read_only/volume/test_cinder.py":[{"author":{"_account_id":8576,"name":"Yair Fried","email":"yfried@redhat.com","username":"yfried"},"change_message_id":"024bb2230fbfbb3148e038eb95b64026f8df7b8f","unresolved":false,"context_lines":[{"line_number":86,"context_line":"        self.assertTableStruct(roles, [\u0027Property\u0027, \u0027Value\u0027])"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"    def test_cinder_quota_defaults(self):"},{"line_number":89,"context_line":"        \"\"\"This CLI can accept and string as param.\"\"\""},{"line_number":90,"context_line":"        id_cl \u003d clients.AdminManager().identity_client"},{"line_number":91,"context_line":"        tenant \u003d id_cl.get_tenant_by_name(CONF.identity.admin_tenant_name)"},{"line_number":92,"context_line":"        admin_tenant_id \u003d tenant[\u0027id\u0027]"}],"source_content_type":"text/x-python","patch_set":6,"id":"5a890539_391c8fc1","line":89,"updated":"2014-11-12 07:42:27.000000000","message":"nit: this can be done once in resource_setup instead of in the tests","commit_id":"ef7ec1e8a2ae1b0714ce8e197e3258b38c2c3502"}],"tempest/config.py":[{"author":{"_account_id":8576,"name":"Yair Fried","email":"yfried@redhat.com","username":"yfried"},"change_message_id":"936eaf760c2b0c7ad65cc444f2fe68d6df848e07","unresolved":false,"context_lines":[{"line_number":121,"context_line":"    cfg.StrOpt(\u0027admin_tenant_name\u0027,"},{"line_number":122,"context_line":"               help\u003d\"Administrative Tenant name to use for Keystone API \""},{"line_number":123,"context_line":"                    \"requests.\"),"},{"line_number":124,"context_line":"    cfg.StrOpt(\u0027admin_tenant_id\u0027,"},{"line_number":125,"context_line":"               help\u003d\"Administrative Tenant id to use for Keystone API \""},{"line_number":126,"context_line":"                    \"requests.\"),"},{"line_number":127,"context_line":"    cfg.StrOpt(\u0027admin_password\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"5a890539_9ad3d037","line":124,"updated":"2014-11-11 09:07:39.000000000","message":"why add a conf value that can be easily retrieved during runtime?\nYou can get tenant_id from tenant_name\n\nAlso, in summit it was agreed that IDs in conf are bad","commit_id":"59c1a4b6af6d2e05bae551a7942a95e3d0cf02dc"}]}