)]}'
{"octavia/api/v2/controllers/load_balancer.py":[{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"f3ca6c90d19c3b59b5d7b7dfaf57e53bceea62d9","unresolved":false,"context_lines":[{"line_number":197,"context_line":"            # TODO(johnsom) Remove provider and flavor from the lb_dict"},{"line_number":198,"context_line":"            # as they have not been implemented beyond the API yet."},{"line_number":199,"context_line":"            # Remove these lines as they are implemented."},{"line_number":200,"context_line":"            if \u0027provider\u0027 in lb_dict:"},{"line_number":201,"context_line":"                del lb_dict[\u0027provider\u0027]"},{"line_number":202,"context_line":"            if \u0027flavor\u0027 in lb_dict:"},{"line_number":203,"context_line":"                del lb_dict[\u0027flavor\u0027]"}],"source_content_type":"text/x-python","patch_set":4,"id":"5f201791_f7bfc5f8","line":200,"updated":"2017-06-22 21:18:25.000000000","message":"You *could* also just do:\nlb_dict.pop(\u0027provider\u0027, None)\nlb_dict.pop(\u0027flavor\u0027, None)","commit_id":"af71a91603f8dd861d90361d390e2d4af9f7b7f9"}],"octavia/common/constants.py":[{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"b8cc7c1762c32b901266f2e3cdfadb408d321201","unresolved":false,"context_lines":[{"line_number":434,"context_line":"# PROVIDERS"},{"line_number":435,"context_line":"# TODO(johnsom) When providers are implemented, this should be removed."},{"line_number":436,"context_line":"OCTAVIA \u003d \u0027octavia\u0027"},{"line_number":437,"context_line":"SUPPORTED_PROVIDERS \u003d (OCTAVIA)"},{"line_number":438,"context_line":""},{"line_number":439,"context_line":"# FLAVORS"},{"line_number":440,"context_line":"# TODO(johnsom) When flavors are implemented, this should be removed."}],"source_content_type":"text/x-python","patch_set":3,"id":"5f201791_31eaf2ab","line":437,"updated":"2017-06-22 17:26:30.000000000","message":"This isn\u0027t a list without a training comma.\nThough apparently it doesn\u0027t actually matter if it\u0027s a list, because tests pass? But, then these are redundant parens.\nOh, or more likely it\u0027s treating the string as a list, so valid providers include \u0027o\u0027, \u0027c\u0027, \u0027t\u0027, etc :P","commit_id":"0f52c427502a6a684d2c021bc5d32dd0264bba4a"},{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"f3ca6c90d19c3b59b5d7b7dfaf57e53bceea62d9","unresolved":false,"context_lines":[{"line_number":450,"context_line":"# PROVIDERS"},{"line_number":451,"context_line":"# TODO(johnsom) When providers are implemented, this should be removed."},{"line_number":452,"context_line":"OCTAVIA \u003d \u0027octavia\u0027"},{"line_number":453,"context_line":"SUPPORTED_PROVIDERS \u003d OCTAVIA,"},{"line_number":454,"context_line":""},{"line_number":455,"context_line":"# FLAVORS"},{"line_number":456,"context_line":"# TODO(johnsom) When flavors are implemented, this should be removed."}],"source_content_type":"text/x-python","patch_set":4,"id":"5f201791_1746c122","line":453,"updated":"2017-06-22 21:18:25.000000000","message":"Wow, you went ALL the way to minimum characters! I had meant to do (OCTAVIA,) which is what I normally see, but this is probably better. :P","commit_id":"af71a91603f8dd861d90361d390e2d4af9f7b7f9"}]}