)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":19011,"name":"Anton Studenov","email":"tohin1332@gmail.com","username":"tohin"},"change_message_id":"384d0f82c1bbcf9c3f78b0ac5601f66c330783aa","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We should update the engine\u0027s deployment after deployment is"},{"line_number":10,"context_line":"updated."},{"line_number":11,"context_line":"Close-bug: #1675271"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I9c3b07c1de3bd61121757c8c6075777f001310cb"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dfeb2761_d65da147","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":10},"updated":"2017-04-05 10:27:50.000000000","message":"Closes-Bug:","commit_id":"314e4f775dac0d66d178c6c00f12cf11cd0d7fdd"},{"author":{"_account_id":21528,"name":"chenhb","email":"chen.haibing1@zte.com.cn","username":"chenhb"},"change_message_id":"33944d9c0539643cdff7cfac43dd688b73f68c6b","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We should update the engine\u0027s deployment after deployment is"},{"line_number":10,"context_line":"updated."},{"line_number":11,"context_line":"Close-bug: #1675271"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I9c3b07c1de3bd61121757c8c6075777f001310cb"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"bff0334d_8deb14af","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":10},"in_reply_to":"dfeb2761_d65da147","updated":"2017-04-06 00:58:51.000000000","message":"Done","commit_id":"314e4f775dac0d66d178c6c00f12cf11cd0d7fdd"}],"rally/deployment/engine.py":[{"author":{"_account_id":19011,"name":"Anton Studenov","email":"tohin1332@gmail.com","username":"tohin"},"change_message_id":"2e37a30c98832819fd87547487b7a749be7653c7","unresolved":false,"context_lines":[{"line_number":71,"context_line":"    \"\"\""},{"line_number":72,"context_line":"    def __init__(self, deployment):"},{"line_number":73,"context_line":"        self.deployment \u003d deployment"},{"line_number":74,"context_line":"        self.config \u003d deployment[\"config\"]"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"    def update_deployment(self, deployment):"},{"line_number":77,"context_line":"        self.deployment \u003d deployment"}],"source_content_type":"text/x-python","patch_set":1,"id":"dfeb2761_dbea610d","line":74,"range":{"start_line":74,"start_character":8,"end_line":74,"end_character":42},"updated":"2017-04-05 12:25:19.000000000","message":"Let\u0027s replace it with a property:\n    \n    @property\n    def config(self):\n        return self.deployment[\"config\"]\n\nThen we do not need to add \"update_deployment\" method.","commit_id":"314e4f775dac0d66d178c6c00f12cf11cd0d7fdd"},{"author":{"_account_id":21528,"name":"chenhb","email":"chen.haibing1@zte.com.cn","username":"chenhb"},"change_message_id":"33944d9c0539643cdff7cfac43dd688b73f68c6b","unresolved":false,"context_lines":[{"line_number":71,"context_line":"    \"\"\""},{"line_number":72,"context_line":"    def __init__(self, deployment):"},{"line_number":73,"context_line":"        self.deployment \u003d deployment"},{"line_number":74,"context_line":"        self.config \u003d deployment[\"config\"]"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"    def update_deployment(self, deployment):"},{"line_number":77,"context_line":"        self.deployment \u003d deployment"}],"source_content_type":"text/x-python","patch_set":1,"id":"bff0334d_2ddac816","line":74,"range":{"start_line":74,"start_character":8,"end_line":74,"end_character":42},"in_reply_to":"dfeb2761_5bfad1bc","updated":"2017-04-06 00:58:51.000000000","message":"Done","commit_id":"314e4f775dac0d66d178c6c00f12cf11cd0d7fdd"},{"author":{"_account_id":19011,"name":"Anton Studenov","email":"tohin1332@gmail.com","username":"tohin"},"change_message_id":"7dafd3a7990d0ad38c30373315a256c21ddd234c","unresolved":false,"context_lines":[{"line_number":71,"context_line":"    \"\"\""},{"line_number":72,"context_line":"    def __init__(self, deployment):"},{"line_number":73,"context_line":"        self.deployment \u003d deployment"},{"line_number":74,"context_line":"        self.config \u003d deployment[\"config\"]"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"    def update_deployment(self, deployment):"},{"line_number":77,"context_line":"        self.deployment \u003d deployment"}],"source_content_type":"text/x-python","patch_set":1,"id":"dfeb2761_5bfad1bc","line":74,"range":{"start_line":74,"start_character":8,"end_line":74,"end_character":42},"in_reply_to":"dfeb2761_dbea610d","updated":"2017-04-05 12:27:35.000000000","message":"And, let\u0027s remove this line https://github.com/openstack/rally/blob/master/rally/deployment/engines/multihost.py#L67","commit_id":"314e4f775dac0d66d178c6c00f12cf11cd0d7fdd"},{"author":{"_account_id":21528,"name":"chenhb","email":"chen.haibing1@zte.com.cn","username":"chenhb"},"change_message_id":"33944d9c0539643cdff7cfac43dd688b73f68c6b","unresolved":false,"context_lines":[{"line_number":71,"context_line":"    \"\"\""},{"line_number":72,"context_line":"    def __init__(self, deployment):"},{"line_number":73,"context_line":"        self.deployment \u003d deployment"},{"line_number":74,"context_line":"        self.config \u003d deployment[\"config\"]"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"    def update_deployment(self, deployment):"},{"line_number":77,"context_line":"        self.deployment \u003d deployment"}],"source_content_type":"text/x-python","patch_set":1,"id":"bff0334d_6dd0d0f8","line":74,"range":{"start_line":74,"start_character":8,"end_line":74,"end_character":42},"in_reply_to":"dfeb2761_dbea610d","updated":"2017-04-06 00:58:51.000000000","message":"Done","commit_id":"314e4f775dac0d66d178c6c00f12cf11cd0d7fdd"}],"tests/unit/deployment/test_lxc.py":[{"author":{"_account_id":19011,"name":"Anton Studenov","email":"tohin1332@gmail.com","username":"tohin"},"change_message_id":"c7a941bbd9f212c2d5d843f1fd98713d3fb6e94b","unresolved":false,"context_lines":[{"line_number":142,"context_line":"            self.assertSequenceEqual("},{"line_number":143,"context_line":"                expected_kwargs[\"info\"][\"forwarded_ports\"],"},{"line_number":144,"context_line":"                actual_kwargs[\"info\"][\"forwarded_ports\"])"},{"line_number":145,"context_line":"        from rally.common import objects"},{"line_number":146,"context_line":"        fake_deployment \u003d objects.Deployment(self.deployment)"},{"line_number":147,"context_line":"        fake_deployment.add_resource \u003d add_resource"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"        fake_deployment.config \u003d self.config"},{"line_number":150,"context_line":""},{"line_number":151,"context_line":"        with mock.patch.object(self.engine, \"deployment\", fake_deployment):"},{"line_number":152,"context_line":"            credentials \u003d self.engine.deploy()"}],"source_content_type":"text/x-python","patch_set":3,"id":"bff0334d_22dd2e14","line":149,"range":{"start_line":145,"start_character":8,"end_line":149,"end_character":44},"updated":"2017-04-06 06:32:18.000000000","message":"Let\u0027s replace it with:\n\n    fake_deployment \u003d mock.MagicMock()\n    fake_deployment.__getitem__.side_effect \u003d self.deployment.__getitem__\n    fake_deployment.add_resource \u003d add_resource\n\n\nAlso, the functional test should be updated. Please add this line:\n\n   self.assertIn(\"http://foo/\", self.rally(\"deployment show\"))\n\nto this line:\nhttps://github.com/openstack/rally/blob/master/tests/functional/test_cli_deployment.py#L100","commit_id":"e74d1cfd92f59a4bd7ce4962153772674204c879"},{"author":{"_account_id":21528,"name":"chenhb","email":"chen.haibing1@zte.com.cn","username":"chenhb"},"change_message_id":"f5cc8675a045df7c0af36f90b7e6f61b66288f1a","unresolved":false,"context_lines":[{"line_number":142,"context_line":"            self.assertSequenceEqual("},{"line_number":143,"context_line":"                expected_kwargs[\"info\"][\"forwarded_ports\"],"},{"line_number":144,"context_line":"                actual_kwargs[\"info\"][\"forwarded_ports\"])"},{"line_number":145,"context_line":"        from rally.common import objects"},{"line_number":146,"context_line":"        fake_deployment \u003d objects.Deployment(self.deployment)"},{"line_number":147,"context_line":"        fake_deployment.add_resource \u003d add_resource"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"        fake_deployment.config \u003d self.config"},{"line_number":150,"context_line":""},{"line_number":151,"context_line":"        with mock.patch.object(self.engine, \"deployment\", fake_deployment):"},{"line_number":152,"context_line":"            credentials \u003d self.engine.deploy()"}],"source_content_type":"text/x-python","patch_set":3,"id":"bff0334d_b0974491","line":149,"range":{"start_line":145,"start_character":8,"end_line":149,"end_character":44},"in_reply_to":"bff0334d_22dd2e14","updated":"2017-04-06 07:11:58.000000000","message":"Done","commit_id":"e74d1cfd92f59a4bd7ce4962153772674204c879"}]}