)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"986df98cb9500d472b6b4c912c800d435f0d40cc","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     ranjit-ray \u003cranjitray@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-05-15 10:32:41 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"New test added for trove db-configuration"},{"line_number":8,"context_line":"    1) List all configuration"},{"line_number":9,"context_line":"    2) Get configuration details"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7adec928_8be327a3","line":7,"updated":"2014-05-16 09:04:40.000000000","message":"need a blank line between title and content.\n\nrefer to https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_GIT_commit_message_structure","commit_id":"e1708cf462c88fe02c64a83ec950fbc33febde30"}],"tempest/api/database/base.py":[{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"986df98cb9500d472b6b4c912c800d435f0d40cc","unresolved":false,"context_lines":[{"line_number":12,"context_line":"#    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the"},{"line_number":13,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":14,"context_line":"#    under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from tempest import config"},{"line_number":17,"context_line":"from tempest.openstack.common import log as logging"},{"line_number":18,"context_line":"import tempest.test"}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_4b7e7f1a","side":"PARENT","line":15,"updated":"2014-05-16 09:04:40.000000000","message":"why delete copyright and license? Do we have the intention to do that?","commit_id":"b4171c7bbfaf32ad647d1d31f65b2ee7059b3125"},{"author":{"_account_id":8824,"name":"Sergey Murashov","email":"smurashov@mirantis.com","username":"SergeyMurashov"},"change_message_id":"e064f29e148b7f506aee7b8658c2e8c9a38bf21f","unresolved":false,"context_lines":[{"line_number":20,"context_line":"            raise cls.skipException(skip_msg)"},{"line_number":21,"context_line":"        cls.catalog_type \u003d CONF.database.catalog_type"},{"line_number":22,"context_line":"        cls.db_flavor_ref \u003d CONF.database.db_flavor_ref"},{"line_number":23,"context_line":"\u003c\u003c\u003c\u003c\u003c\u003c\u003c Updated upstream"},{"line_number":24,"context_line":"        cls.db_current_version \u003d CONF.database.db_current_version"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"        os \u003d cls.get_client_manager()"}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_e81e7f5e","line":23,"updated":"2014-05-15 11:53:15.000000000","message":"please remove it","commit_id":"e1708cf462c88fe02c64a83ec950fbc33febde30"},{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"986df98cb9500d472b6b4c912c800d435f0d40cc","unresolved":false,"context_lines":[{"line_number":20,"context_line":"            raise cls.skipException(skip_msg)"},{"line_number":21,"context_line":"        cls.catalog_type \u003d CONF.database.catalog_type"},{"line_number":22,"context_line":"        cls.db_flavor_ref \u003d CONF.database.db_flavor_ref"},{"line_number":23,"context_line":"\u003c\u003c\u003c\u003c\u003c\u003c\u003c Updated upstream"},{"line_number":24,"context_line":"        cls.db_current_version \u003d CONF.database.db_current_version"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"        os \u003d cls.get_client_manager()"}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_0b81d7fa","line":23,"in_reply_to":"7adec928_e81e7f5e","updated":"2014-05-16 09:04:40.000000000","message":"The code has conflict, you should resolve conflict before committing, not only remove these lines which have \u0027\u003c\u003c\u003c\u003c\u003c\u003c\u003c\u0027 and \u0027\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u0027.","commit_id":"e1708cf462c88fe02c64a83ec950fbc33febde30"},{"author":{"_account_id":8824,"name":"Sergey Murashov","email":"smurashov@mirantis.com","username":"SergeyMurashov"},"change_message_id":"e064f29e148b7f506aee7b8658c2e8c9a38bf21f","unresolved":false,"context_lines":[{"line_number":27,"context_line":"        cls.os \u003d os"},{"line_number":28,"context_line":"        cls.database_flavors_client \u003d cls.os.database_flavors_client"},{"line_number":29,"context_line":"        cls.database_versions_client \u003d cls.os.database_versions_client"},{"line_number":30,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":31,"context_line":"        os \u003d cls.get_client_manager()"},{"line_number":32,"context_line":"        cls.os \u003d os"},{"line_number":33,"context_line":"        cls.database_flavors_client \u003d cls.os.database_flavors_client"}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_283ce7c7","line":30,"updated":"2014-05-15 11:53:15.000000000","message":"please remove it","commit_id":"e1708cf462c88fe02c64a83ec950fbc33febde30"},{"author":{"_account_id":8824,"name":"Sergey Murashov","email":"smurashov@mirantis.com","username":"SergeyMurashov"},"change_message_id":"e064f29e148b7f506aee7b8658c2e8c9a38bf21f","unresolved":false,"context_lines":[{"line_number":33,"context_line":"        cls.database_flavors_client \u003d cls.os.database_flavors_client"},{"line_number":34,"context_line":"        cls.database_configuration_client \u003d \\"},{"line_number":35,"context_line":"           cls.os.database_configuration_client"},{"line_number":36,"context_line":"\u003e\u003e\u003e\u003e\u003e\u003e\u003e Stashed changes"}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_483933d6","line":36,"updated":"2014-05-15 11:53:15.000000000","message":"please remove it","commit_id":"e1708cf462c88fe02c64a83ec950fbc33febde30"}],"tempest/api/database/configurations/test_configuration.py":[{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"986df98cb9500d472b6b4c912c800d435f0d40cc","unresolved":false,"context_lines":[{"line_number":11,"context_line":"#    distributed under the License is distributed on an \"AS IS\" BASIS, WITHOUT"},{"line_number":12,"context_line":"#    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the"},{"line_number":13,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":14,"context_line":"#    under the License."},{"line_number":15,"context_line":"from tempest.api.database import base"},{"line_number":16,"context_line":"from tempest import test"},{"line_number":17,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_4ba59fa0","line":14,"updated":"2014-05-16 09:04:40.000000000","message":"need a blank line between license and module import.","commit_id":"e1708cf462c88fe02c64a83ec950fbc33febde30"},{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"986df98cb9500d472b6b4c912c800d435f0d40cc","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":27,"context_line":"    def test_list_configuration(self):"},{"line_number":28,"context_line":"        resp, configuration \u003d self.client.list_db_configuration()"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":31,"context_line":"    def test_get_db_configuration(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_4bccbf6a","line":28,"updated":"2014-05-16 09:04:40.000000000","message":"it need check the response code and configuration like other list tests.\n\nfor example:\n    @test.attr(type\u003d\u0027smoke\u0027)\n    def test_list_db_versions(self):\n        resp, versions \u003d self.client.list_db_versions()\n        self.assertEqual(200, resp.status)\n        self.assertTrue(len(versions) \u003e 0, \"No database versions found\")","commit_id":"e1708cf462c88fe02c64a83ec950fbc33febde30"},{"author":{"_account_id":8824,"name":"Sergey Murashov","email":"smurashov@mirantis.com","username":"SergeyMurashov"},"change_message_id":"e064f29e148b7f506aee7b8658c2e8c9a38bf21f","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":27,"context_line":"    def test_list_configuration(self):"},{"line_number":28,"context_line":"        resp, configuration \u003d self.client.list_db_configuration()"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":31,"context_line":"    def test_get_db_configuration(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_085bab2e","line":28,"updated":"2014-05-15 11:53:15.000000000","message":"unused variables resp and configuration","commit_id":"e1708cf462c88fe02c64a83ec950fbc33febde30"},{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"986df98cb9500d472b6b4c912c800d435f0d40cc","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":31,"context_line":"    def test_get_db_configuration(self):"},{"line_number":32,"context_line":"        # The expected configuration details should be returned"},{"line_number":33,"context_line":"        resp, db_configuration_id \u003d self.client.list_db_configuration()"},{"line_number":34,"context_line":"        if len(db_configuration_id):"},{"line_number":35,"context_line":"            db_configuration_id \u003d configurations[\u0027id\u0027]"},{"line_number":36,"context_line":"            resp, configuration_get \u003d \\"}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_eb0bcb8b","line":33,"updated":"2014-05-16 09:04:40.000000000","message":"db_configuration_id -\u003e configurations ?","commit_id":"e1708cf462c88fe02c64a83ec950fbc33febde30"},{"author":{"_account_id":8824,"name":"Sergey Murashov","email":"smurashov@mirantis.com","username":"SergeyMurashov"},"change_message_id":"e064f29e148b7f506aee7b8658c2e8c9a38bf21f","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    @test.attr(type\u003d\u0027smoke\u0027)"},{"line_number":31,"context_line":"    def test_get_db_configuration(self):"},{"line_number":32,"context_line":"        # The expected configuration details should be returned"},{"line_number":33,"context_line":"        resp, db_configuration_id \u003d self.client.list_db_configuration()"},{"line_number":34,"context_line":"        if len(db_configuration_id):"},{"line_number":35,"context_line":"            db_configuration_id \u003d configurations[\u0027id\u0027]"},{"line_number":36,"context_line":"            resp, configuration_get \u003d \\"}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_a84f176a","line":33,"updated":"2014-05-15 11:53:15.000000000","message":"unused variable resp","commit_id":"e1708cf462c88fe02c64a83ec950fbc33febde30"},{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"986df98cb9500d472b6b4c912c800d435f0d40cc","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    def test_get_db_configuration(self):"},{"line_number":32,"context_line":"        # The expected configuration details should be returned"},{"line_number":33,"context_line":"        resp, db_configuration_id \u003d self.client.list_db_configuration()"},{"line_number":34,"context_line":"        if len(db_configuration_id):"},{"line_number":35,"context_line":"            db_configuration_id \u003d configurations[\u0027id\u0027]"},{"line_number":36,"context_line":"            resp, configuration_get \u003d \\"},{"line_number":37,"context_line":"                self.client.get_db_configuration_details(db_configuration_id)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_0bfd576e","line":34,"updated":"2014-05-16 09:04:40.000000000","message":"db_configuration_id -\u003e configurations ?","commit_id":"e1708cf462c88fe02c64a83ec950fbc33febde30"},{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"986df98cb9500d472b6b4c912c800d435f0d40cc","unresolved":false,"context_lines":[{"line_number":34,"context_line":"        if len(db_configuration_id):"},{"line_number":35,"context_line":"            db_configuration_id \u003d configurations[\u0027id\u0027]"},{"line_number":36,"context_line":"            resp, configuration_get \u003d \\"},{"line_number":37,"context_line":"                self.client.get_db_configuration_details(db_configuration_id)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_abd623ef","line":37,"updated":"2014-05-16 09:04:40.000000000","message":"need check the response code and configuration_get like other list tests.","commit_id":"e1708cf462c88fe02c64a83ec950fbc33febde30"}],"tempest/clients.py":[{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"986df98cb9500d472b6b4c912c800d435f0d40cc","unresolved":false,"context_lines":[{"line_number":29,"context_line":"from tempest.services.compute.json.aggregates_client import \\"},{"line_number":30,"context_line":"    AggregatesClientJSON"},{"line_number":31,"context_line":"from tempest.services.database.json.user_create_client import \\"},{"line_number":32,"context_line":"    DatabaseUserCreateClientJSON"},{"line_number":33,"context_line":"from tempest.services.database.json.configuration_client import \\"},{"line_number":34,"context_line":"    DatabaseConfigurationClientJSON"},{"line_number":35,"context_line":"from tempest.services.compute.json.availability_zone_client import \\"}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_cbc9af27","line":32,"updated":"2014-05-16 09:04:40.000000000","message":"this line is wrong. there\u0027s no user_create_client.","commit_id":"e1708cf462c88fe02c64a83ec950fbc33febde30"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"1cacd34943e990977f90f742daaa106af241ab85","unresolved":false,"context_lines":[{"line_number":351,"context_line":"            self.hosts_v3_client \u003d HostsV3ClientJSON(self.auth_provider)"},{"line_number":352,"context_line":"            self.database_flavors_client \u003d DatabaseFlavorsClientJSON("},{"line_number":353,"context_line":"                self.auth_provider)"},{"line_number":354,"context_line":"\u003c\u003c\u003c\u003c\u003c\u003c\u003c Updated upstream"},{"line_number":355,"context_line":"            self.database_versions_client \u003d DatabaseVersionsClientJSON("},{"line_number":356,"context_line":"                self.auth_provider)"},{"line_number":357,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_08100ba7","line":354,"updated":"2014-05-15 12:35:09.000000000","message":"Please, remove this.","commit_id":"e1708cf462c88fe02c64a83ec950fbc33febde30"}],"tempest/services/database/json/configuration_client.py":[{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"986df98cb9500d472b6b4c912c800d435f0d40cc","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"class DatabaseConfigurationClientJSON(rest_client.RestClient):"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    def __init__(self, auth_provider):"},{"line_number":27,"context_line":"        super(DatabaseConfigurationClientJSON, self).__init__(auth_provider)"},{"line_number":28,"context_line":"        self.service \u003d CONF.database.catalog_type"}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_ab1b83c8","line":25,"updated":"2014-05-16 09:04:40.000000000","message":"It\u0027s better to add some comments about the function of this class.","commit_id":"e1708cf462c88fe02c64a83ec950fbc33febde30"},{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"986df98cb9500d472b6b4c912c800d435f0d40cc","unresolved":false,"context_lines":[{"line_number":27,"context_line":"        super(DatabaseConfigurationClientJSON, self).__init__(auth_provider)"},{"line_number":28,"context_line":"        self.service \u003d CONF.database.catalog_type"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"    def list_db_configuration(self, params\u003dNone):"},{"line_number":31,"context_line":"        url \u003d \u0027configurations\u0027"},{"line_number":32,"context_line":"        if params:"},{"line_number":33,"context_line":"            url +\u003d \u0027?%s\u0027 % urllib.urlencode(params)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7adec928_6b3edb29","line":30,"updated":"2014-05-16 09:04:40.000000000","message":"list_db_configuration -\u003e list_db_configurations","commit_id":"e1708cf462c88fe02c64a83ec950fbc33febde30"}]}