)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2861,"name":"Vincent Hou","email":"shou@us.ibm.com","username":"houshengbo"},"change_message_id":"8517674abbac8623e2c0c664fdfe85a1f6d48bc9","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Matthew Edmonds \u003cedmondsw@us.ibm.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-07-29 14:54:39 -0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"move oslo.vmware to test-requirements"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"oslo.vmware is currently listed in requirements.txt, but as a driver"},{"line_number":10,"context_line":"dependency this should be listed in test-requirements.txt instead."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"3a50d1a3_b98008c1","line":7,"updated":"2015-07-30 05:45:30.000000000","message":"Move at first.","commit_id":"06dc8c841c16101da4b661f9149051325d276a2c"},{"author":{"_account_id":2861,"name":"Vincent Hou","email":"shou@us.ibm.com","username":"houshengbo"},"change_message_id":"8517674abbac8623e2c0c664fdfe85a1f6d48bc9","unresolved":false,"context_lines":[{"line_number":7,"context_line":"move oslo.vmware to test-requirements"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"oslo.vmware is currently listed in requirements.txt, but as a driver"},{"line_number":10,"context_line":"dependency this should be listed in test-requirements.txt instead."},{"line_number":11,"context_line":"Moving to test-requirements.txt accordingly."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I9b00edc38f0700304a1a164f0679a734f8701ebe"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"3a50d1a3_19633c43","line":10,"updated":"2015-07-30 05:45:30.000000000","message":"This line needs better explanations: \n\u0027should be listed in test-requirements.txt instead\u0027. How and where? No test-requirements.txt changed. Otherwise, you implement somewhere else. OK, explain more here.","commit_id":"06dc8c841c16101da4b661f9149051325d276a2c"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"98e84fb0c9fd52da8cb38d381b4b3774711edd85","unresolved":false,"context_lines":[{"line_number":7,"context_line":"move oslo.vmware to test-requirements"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"oslo.vmware is currently listed in requirements.txt, but as a driver"},{"line_number":10,"context_line":"dependency this should be listed in test-requirements.txt instead."},{"line_number":11,"context_line":"Moving to test-requirements.txt accordingly."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I9b00edc38f0700304a1a164f0679a734f8701ebe"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"3a50d1a3_e950e9d6","line":10,"in_reply_to":"3a50d1a3_19633c43","updated":"2015-07-30 14:37:49.000000000","message":"You\u0027re absolutely correct... I forgot to update the commit message with the latest changes.","commit_id":"06dc8c841c16101da4b661f9149051325d276a2c"}],"cinder/volume/drivers/vmware/vmdk.py":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"438ad3e436c90ff4b38db07b2b9530546b1d7a7b","unresolved":false,"context_lines":[{"line_number":276,"context_line":"        if oslo_vmware is None:"},{"line_number":277,"context_line":"            msg \u003d _(\"Missing \u0027oslo_vmware\u0027 python module, ensure the library\""},{"line_number":278,"context_line":"                    \" is installed and available.\")"},{"line_number":279,"context_line":"            raise exception.VolumeDriverException(msg)"},{"line_number":280,"context_line":""},{"line_number":281,"context_line":"        # Throw error if required parameters are not set."},{"line_number":282,"context_line":"        required_params \u003d [\u0027vmware_host_ip\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"3a50d1a3_7b943b36","line":279,"updated":"2015-08-02 18:53:11.000000000","message":"exception.VolumeDriverException(message\u003dmsg)","commit_id":"2006390cac12ddc43f4b513b27a291860cd579c1"}],"test-requirements.txt":[{"author":{"_account_id":9171,"name":"Vipin Balachandran","email":"vipin.bl@gmail.com","username":"vbala"},"change_message_id":"424a2d85264290e485b1ab140269c75eb9ec2103","unresolved":false,"context_lines":[{"line_number":21,"context_line":"oslosphinx\u003e\u003d2.5.0 # Apache-2.0"},{"line_number":22,"context_line":"os-testr\u003e\u003d0.1.0"},{"line_number":23,"context_line":"tempest-lib\u003e\u003d0.6.1"},{"line_number":24,"context_line":"oslo.vmware\u003e\u003d1.16.0 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":4,"id":"3a50d1a3_d3e7883e","line":24,"updated":"2015-07-27 14:34:46.000000000","message":"How about adding a comment \"vmdk driver specific dependency\"?","commit_id":"07611994dd705908c6bab8afa191a02cebb16d32"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"9c47280c7a61a9d73788f04bfebea4f205be8737","unresolved":false,"context_lines":[{"line_number":22,"context_line":"os-testr\u003e\u003d0.1.0"},{"line_number":23,"context_line":"tempest-lib\u003e\u003d0.6.1"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"# For vmdk driver"},{"line_number":26,"context_line":"oslo.vmware\u003e\u003d1.16.0 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":5,"id":"3a50d1a3_3eab138b","line":25,"updated":"2015-07-27 14:57:17.000000000","message":"Is it ok to have this in a separate section?  Will this comment stay here when we update reqs?","commit_id":"e5d083e9bb2fbb0730c9f17147b84ccdef264226"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"950b829beddb735ce05425d4b2fcf284e6d783ab","unresolved":false,"context_lines":[{"line_number":22,"context_line":"os-testr\u003e\u003d0.1.0"},{"line_number":23,"context_line":"tempest-lib\u003e\u003d0.6.1"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"# For vmdk driver"},{"line_number":26,"context_line":"oslo.vmware\u003e\u003d1.16.0 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":5,"id":"3a50d1a3_5e385f3d","line":25,"in_reply_to":"3a50d1a3_3eab138b","updated":"2015-07-27 15:00:59.000000000","message":"I see comments like this in test-requirements for other projects, so this should be fine.","commit_id":"e5d083e9bb2fbb0730c9f17147b84ccdef264226"}]}