)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"3855adb97b1102df14db376e63ec0d827ab4b108","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Eugene Nikanorov \u003cenikanorov@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-03-24 15:39:21 +0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"unskip load balancer basic scenario test"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Idff6d27611adda502e6e3f7e0d37f3acbda41d03"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAWH%2F%2BEBY%3D","line":7,"updated":"2014-03-31 07:25:07.000000000","message":"Please refer the corresponding bug in the commit message","commit_id":"98e3f1de68fd6428234ec54964e3c97e8d099982"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"e56c386d78f448cfb77a9c34315920d66e241e9e","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Eugene Nikanorov \u003cenikanorov@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-03-24 15:39:21 +0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"unskip load balancer basic scenario test"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Idff6d27611adda502e6e3f7e0d37f3acbda41d03"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAWH%2F%2BDrI%3D","line":7,"in_reply_to":"AAAAWH%2F%2BEBY%3D","updated":"2014-03-31 08:49:25.000000000","message":"Done","commit_id":"98e3f1de68fd6428234ec54964e3c97e8d099982"},{"author":{"_account_id":7227,"name":"Timur Nurlygayanov","email":"tnurlygayanov@mirantis.com","username":"tnurlygayanov"},"change_message_id":"b75a18946f7b77101201a0d75220e6fd773f3ee7","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Eugene Nikanorov \u003cenikanorov@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-04-02 11:17:43 +0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"unskip load balancer basic scenario test"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Corresponding bug 1294603 was fixed by commit"},{"line_number":10,"context_line":"I0dc4d01b0c1c04887ec6ad5766ec7c6c96903faa"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"AAAAWH%2F9w%2Bk%3D","line":7,"updated":"2014-04-02 09:26:22.000000000","message":"Let\u0027s use capital \u0027U\u0027 there, like:\n\nUnskip load balancer basic scenario test","commit_id":"6624aab1e54953d5fed716844d6629cbc85ec342"},{"author":{"_account_id":7139,"name":"Zhi Kun Liu","username":"zhikunliu","inactive":true},"change_message_id":"a875ac23377b6fba284aab3c6b097a2a488b103d","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Corresponding bug 1294603 was fixed by commit"},{"line_number":10,"context_line":"I0dc4d01b0c1c04887ec6ad5766ec7c6c96903faa"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Also add rules to allow ingress and egress traffic"},{"line_number":13,"context_line":"for ports 80 and 88 for a created security group."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Idff6d27611adda502e6e3f7e0d37f3acbda41d03"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"AAAAWH%2F9xYk%3D","line":12,"updated":"2014-04-02 08:40:05.000000000","message":"only add \u0027ingress\u0027 now.","commit_id":"6624aab1e54953d5fed716844d6629cbc85ec342"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"5cbdd95eba0c63dbfdd0c51a0bebe2c0da9161d3","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Corresponding bug 1294603 was fixed by commit"},{"line_number":10,"context_line":"I0dc4d01b0c1c04887ec6ad5766ec7c6c96903faa"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Also add rules to allow ingress and egress traffic"},{"line_number":13,"context_line":"for ports 80 and 88 for a created security group."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Idff6d27611adda502e6e3f7e0d37f3acbda41d03"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"AAAAWH%2F9w0E%3D","line":12,"in_reply_to":"AAAAWH%2F9w9w%3D","updated":"2014-04-02 09:46:53.000000000","message":"Timur, egress is not needed.","commit_id":"6624aab1e54953d5fed716844d6629cbc85ec342"},{"author":{"_account_id":7227,"name":"Timur Nurlygayanov","email":"tnurlygayanov@mirantis.com","username":"tnurlygayanov"},"change_message_id":"b75a18946f7b77101201a0d75220e6fd773f3ee7","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Corresponding bug 1294603 was fixed by commit"},{"line_number":10,"context_line":"I0dc4d01b0c1c04887ec6ad5766ec7c6c96903faa"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Also add rules to allow ingress and egress traffic"},{"line_number":13,"context_line":"for ports 80 and 88 for a created security group."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Idff6d27611adda502e6e3f7e0d37f3acbda41d03"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"AAAAWH%2F9w9w%3D","line":12,"in_reply_to":"AAAAWH%2F9xYk%3D","updated":"2014-04-02 09:26:22.000000000","message":"+1 for this, need to add rule for \u0027egress traffic\u0027 too.","commit_id":"6624aab1e54953d5fed716844d6629cbc85ec342"}],"tempest/scenario/test_load_balancer_basic.py":[{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"f6a1c8268435f3d75cdea96e353618caedde70e7","unresolved":false,"context_lines":[{"line_number":85,"context_line":"            secgroup\u003dself.security_groups[self.tenant_id],"},{"line_number":86,"context_line":"            tenant_id\u003dself.tenant_id,"},{"line_number":87,"context_line":"            **rules)"},{"line_number":88,"context_line":"        rules.update(dict(direction\u003d\u0027egress\u0027))"},{"line_number":89,"context_line":"        self._create_security_group_rule("},{"line_number":90,"context_line":"            client\u003dself.network_client,"},{"line_number":91,"context_line":"            secgroup\u003dself.security_groups[self.tenant_id],"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAWH%2F9yCM%3D","line":88,"updated":"2014-04-02 07:15:28.000000000","message":"I am not sure is the egress rules are really required.","commit_id":"c07365b697bc0792309bebe0a0379705992011ba"}]}
