)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"bf274cd173d3f18dc64df2f31919041756e314d9","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"At the same time, this patch also relaxes the check so that it just verifies"},{"line_number":20,"context_line":"that member pools are load balanced at least once. Verifying that the the"},{"line_number":21,"context_line":"number of requests are spread evenly accross is always going to be prone to"},{"line_number":22,"context_line":"transient condition and it is beyond the scope of a scenario test that validates"},{"line_number":23,"context_line":"whether Neutron LBaaS is wiring things correctly."},{"line_number":24,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"baa201ad_9cabdb88","line":21,"updated":"2014-09-29 18:24:04.000000000","message":"femtonit: s/accross/across/","commit_id":"dd6e6610a564a0dbea707c02490064a0574542c2"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"5c3c54b626bca2ba3cc616408b8ca58166e4dd2e","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"At the same time, this patch also relaxes the check so that it just verifies"},{"line_number":20,"context_line":"that member pools are load balanced at least once. Verifying that the the"},{"line_number":21,"context_line":"number of requests are spread evenly accross is always going to be prone to"},{"line_number":22,"context_line":"transient condition and it is beyond the scope of a scenario test that validates"},{"line_number":23,"context_line":"whether Neutron LBaaS is wiring things correctly."},{"line_number":24,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"baa201ad_7c363705","line":21,"in_reply_to":"baa201ad_9cabdb88","updated":"2014-09-29 18:34:01.000000000","message":"doh!","commit_id":"dd6e6610a564a0dbea707c02490064a0574542c2"}],"tempest/scenario/test_load_balancer_basic.py":[{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"bf274cd173d3f18dc64df2f31919041756e314d9","unresolved":false,"context_lines":[{"line_number":301,"context_line":"            counters[server] +\u003d 1"},{"line_number":302,"context_line":"        # Assert that each member of the pool gets balanced at least once"},{"line_number":303,"context_line":"        for member, counter in counters.iteritems():"},{"line_number":304,"context_line":"            self.assertTrue(counter \u003e 0, \u0027Member %s never balanced\u0027 % member)"},{"line_number":305,"context_line":""},{"line_number":306,"context_line":"    @test.services(\u0027compute\u0027, \u0027network\u0027)"},{"line_number":307,"context_line":"    def test_load_balancer_basic(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"baa201ad_7ce2b7d6","line":304,"updated":"2014-09-29 18:24:04.000000000","message":"You should probably use assertGreater() here instead of an assertTrue.","commit_id":"dd6e6610a564a0dbea707c02490064a0574542c2"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"5c3c54b626bca2ba3cc616408b8ca58166e4dd2e","unresolved":false,"context_lines":[{"line_number":301,"context_line":"            counters[server] +\u003d 1"},{"line_number":302,"context_line":"        # Assert that each member of the pool gets balanced at least once"},{"line_number":303,"context_line":"        for member, counter in counters.iteritems():"},{"line_number":304,"context_line":"            self.assertTrue(counter \u003e 0, \u0027Member %s never balanced\u0027 % member)"},{"line_number":305,"context_line":""},{"line_number":306,"context_line":"    @test.services(\u0027compute\u0027, \u0027network\u0027)"},{"line_number":307,"context_line":"    def test_load_balancer_basic(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"baa201ad_1c2feba2","line":304,"in_reply_to":"baa201ad_7ce2b7d6","updated":"2014-09-29 18:34:01.000000000","message":"The only reason why I chose this is because I wasn\u0027t sure it worked for py26. If we don\u0027t care about py26 in Tempest I\u0027ll switch.\n\nhttps://docs.python.org/2/library/unittest.html#unittest.TestCase.assertGreater","commit_id":"dd6e6610a564a0dbea707c02490064a0574542c2"}]}