)]}'
{"octavia_tempest_plugin/tests/scenario/v2/test_pool.py":[{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"0608b61567b8d12e95988e8f5585522d2c3cfa9c","unresolved":false,"context_lines":[{"line_number":51,"context_line":"                                CONF.load_balancer.lb_build_interval,"},{"line_number":52,"context_line":"                                CONF.load_balancer.lb_build_timeout)"},{"line_number":53,"context_line":"        cls.protocol \u003d const.HTTP"},{"line_number":54,"context_line":"        cls.lb_feature_enabled \u003d CONF.loadbalancer_feature_enabled"},{"line_number":55,"context_line":"        if not cls.lb_feature_enabled.l7_protocol_enabled:"},{"line_number":56,"context_line":"            cls.protocol \u003d cls.lb_feature_enabled.l4_protocol"},{"line_number":57,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5fc1f717_9cbafebd","line":54,"range":{"start_line":54,"start_character":8,"end_line":54,"end_character":11},"updated":"2019-03-22 07:57:36.000000000","message":"we can avoid cls here, cant we? We arent using the lb_feature_enabled anywhere except the current function, so it can be local to this scope only","commit_id":"3b41a36c0fec1488dcf0f6f19230a782ce4ccd55"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"c4394078eeefb014cc03f8d349c747d08fd603a2","unresolved":false,"context_lines":[{"line_number":51,"context_line":"                                CONF.load_balancer.lb_build_interval,"},{"line_number":52,"context_line":"                                CONF.load_balancer.lb_build_timeout)"},{"line_number":53,"context_line":"        cls.protocol \u003d const.HTTP"},{"line_number":54,"context_line":"        cls.lb_feature_enabled \u003d CONF.loadbalancer_feature_enabled"},{"line_number":55,"context_line":"        if not cls.lb_feature_enabled.l7_protocol_enabled:"},{"line_number":56,"context_line":"            cls.protocol \u003d cls.lb_feature_enabled.l4_protocol"},{"line_number":57,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5fc1f717_3c0d0a90","line":54,"range":{"start_line":54,"start_character":8,"end_line":54,"end_character":11},"in_reply_to":"5fc1f717_5c37b64e","updated":"2019-03-22 08:02:51.000000000","message":"oh sorry ... I thought those were some other options. But thanks, this is useful for my next patch :D","commit_id":"3b41a36c0fec1488dcf0f6f19230a782ce4ccd55"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"4af1130c74b88e89bf1a6f349df190ad7f93cc2e","unresolved":false,"context_lines":[{"line_number":51,"context_line":"                                CONF.load_balancer.lb_build_interval,"},{"line_number":52,"context_line":"                                CONF.load_balancer.lb_build_timeout)"},{"line_number":53,"context_line":"        cls.protocol \u003d const.HTTP"},{"line_number":54,"context_line":"        cls.lb_feature_enabled \u003d CONF.loadbalancer_feature_enabled"},{"line_number":55,"context_line":"        if not cls.lb_feature_enabled.l7_protocol_enabled:"},{"line_number":56,"context_line":"            cls.protocol \u003d cls.lb_feature_enabled.l4_protocol"},{"line_number":57,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5fc1f717_5c37b64e","line":54,"range":{"start_line":54,"start_character":8,"end_line":54,"end_character":11},"in_reply_to":"5fc1f717_9cbafebd","updated":"2019-03-22 08:01:52.000000000","message":"L166, L191","commit_id":"3b41a36c0fec1488dcf0f6f19230a782ce4ccd55"}]}