)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4257,"name":"Zane Bitter","email":"zbitter@redhat.com","username":"zaneb"},"change_message_id":"ea6590f11214622f6005b5f91f28e39fb971db86","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Vijendar Komalla \u003cvijendar.komalla@RACKSPACE.COM\u003e"},{"line_number":5,"context_line":"CommitDate: 2013-10-15 14:55:31 -0500"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Openstack DbaaS(Database as a Service)/Trove resource"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This is patch-1 in the two patch series to implement trove"},{"line_number":10,"context_line":"(DBaaS) resource. This patch adds troveclient to requirements"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAATH%2F%2Fats%3D","line":7,"updated":"2013-10-16 13:53:32.000000000","message":"This is not an accurate headline and will be really unhelpful to anyone looking through the history. Please change it to something like \"Provide access to Trove client\".","commit_id":"3edc8c68f427226815707700e2e2ae539460879d"},{"author":{"_account_id":4257,"name":"Zane Bitter","email":"zbitter@redhat.com","username":"zaneb"},"change_message_id":"ea6590f11214622f6005b5f91f28e39fb971db86","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Openstack DbaaS(Database as a Service)/Trove resource"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This is patch-1 in the two patch series to implement trove"},{"line_number":10,"context_line":"(DBaaS) resource. This patch adds troveclient to requirements"},{"line_number":11,"context_line":"and updates clients.py to get trove client."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAATH%2F%2FapU%3D","line":9,"updated":"2013-10-16 13:53:32.000000000","message":"This kind of comment is really not necessary when the headline is accurate. It\u0027s trivial to see which patches are part of a series in Gerrit, and irrelevant once they are committed.","commit_id":"3edc8c68f427226815707700e2e2ae539460879d"}],"heat/engine/clients.py":[{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"92bae014e4a932939b3bd3a0c855b02ab28fa2e7","unresolved":false,"context_lines":[{"line_number":41,"context_line":"    logger.info(\u0027cinderclient not available\u0027)"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"try:"},{"line_number":44,"context_line":"    from troveclient import client as troveclient"},{"line_number":45,"context_line":"except ImportError:"},{"line_number":46,"context_line":"    troveclient \u003d None"},{"line_number":47,"context_line":"    logger.info(\u0027troveclient not available\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2Fd7c%3D","line":44,"updated":"2013-11-05 11:05:26.000000000","message":"Please update this lines. Because troveclient had alot of changes. And now packaging placement abit changed.","commit_id":"3edc8c68f427226815707700e2e2ae539460879d"}],"heat/engine/resources/os_dbinstance.py":[{"author":{"_account_id":4257,"name":"Zane Bitter","email":"zbitter@redhat.com","username":"zaneb"},"change_message_id":"d8d5dfe265ea7cf11f28858d421df0e631fcfa99","unresolved":false,"context_lines":[{"line_number":30,"context_line":"    Openstack Dbaas cloud database resource."},{"line_number":31,"context_line":"    \u0027\u0027\u0027"},{"line_number":32,"context_line":"    database_schema \u003d {"},{"line_number":33,"context_line":"        \"character_set\": {"},{"line_number":34,"context_line":"            \"Type\": \"String\","},{"line_number":35,"context_line":"            \"Default\": \"utf8\","},{"line_number":36,"context_line":"            \"Required\": False"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATH%2F%2FfL4%3D","line":33,"updated":"2013-10-15 15:02:04.000000000","message":"Please add descriptions for all of these properties, so that they will show up in the autogenerated docs.\n\nAlso, not a show stopper, but for new resources it would be preferable to use the Schema class, e.g.:\n\n  \"character_set\": properties.Schema(\n      properties.STRING,\n      _(\"Please add a description here\"),\n       default\u003d\"utf8\"),\n  \"collate\": properties.Schema(\n      properties.STRING,\n      _(\"Please add a description here\"),\n      default\u003d\"utf8_general_ci\"),\n  \"name\": properties.Schema(\n      properties.STRING,\n      _(\"Please add a description here\"),\n      required\u003dTrue,\n      constraints\u003d[properties.Length(max\u003d64),\n                   properties.AllowedPattern(\"[a-zA-Z0-9_]+[a-zA-Z0-9_@?#\\s]*[a-zA-Z0-9_]+\")])\n\n(Don\u0027t necessarily copy this formatting ;)","commit_id":"9954d44bb0ce873c74b6355c1ec84e988e417443"},{"author":{"_account_id":4257,"name":"Zane Bitter","email":"zbitter@redhat.com","username":"zaneb"},"change_message_id":"d8d5dfe265ea7cf11f28858d421df0e631fcfa99","unresolved":false,"context_lines":[{"line_number":208,"context_line":"        except NotFound:"},{"line_number":209,"context_line":"            pass"},{"line_number":210,"context_line":"        else:"},{"line_number":211,"context_line":"            scheduler.TaskRunner(self._delete_dbinstance, instance)()"},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"        self.resource_id \u003d None"},{"line_number":214,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATH%2F%2FfM4%3D","line":211,"updated":"2013-10-15 15:02:04.000000000","message":"We should just start the deletion here and use check_delete_complete() to poll it.","commit_id":"9954d44bb0ce873c74b6355c1ec84e988e417443"}],"requirements.txt":[{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"92bae014e4a932939b3bd3a0c855b02ab28fa2e7","unresolved":false,"context_lines":[{"line_number":20,"context_line":"python-neutronclient\u003e\u003d2.3.0,\u003c3"},{"line_number":21,"context_line":"python-ceilometerclient\u003e\u003d1.0.5"},{"line_number":22,"context_line":"python-cinderclient\u003e\u003d1.0.5"},{"line_number":23,"context_line":"python-troveclient"},{"line_number":24,"context_line":"PyYAML\u003e\u003d3.1.0"},{"line_number":25,"context_line":"paramiko\u003e\u003d1.8.0"},{"line_number":26,"context_line":"Babel\u003e\u003d1.3"}],"source_content_type":"text/plain","patch_set":3,"id":"AAAATn%2F%2Fd7M%3D","line":23,"updated":"2013-11-05 11:05:26.000000000","message":"It would be nice to fix version, because now troveclient has functionality that doesn\u0027t fits to Trove server side.","commit_id":"3edc8c68f427226815707700e2e2ae539460879d"}]}