)]}'
{"octaviaclient/api/load_balancer_v2.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"6bb430268cd6d5ecc9d73403e92bddf643e3e371","unresolved":false,"context_lines":[{"line_number":188,"context_line":"            A dict of the created pool\u0027s settings"},{"line_number":189,"context_line":"        \"\"\""},{"line_number":190,"context_line":"        url \u003d const.BASE_POOL_URL"},{"line_number":191,"context_line":"        pool \u003d self.create(url, method\u003d\u0027POST\u0027, **kwargs)"},{"line_number":192,"context_line":""},{"line_number":193,"context_line":"        return pool"},{"line_number":194,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_93b28b48","line":191,"range":{"start_line":191,"start_character":32,"end_line":191,"end_character":45},"updated":"2017-06-02 23:27:05.000000000","message":"Post is default and this param is not on the load balancer create.  We should be consistent and either remove this one or add it to load balancer","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"75e74492f76d1509cddf1cfb6991c2f3b13f83bb","unresolved":false,"context_lines":[{"line_number":188,"context_line":"            A dict of the created pool\u0027s settings"},{"line_number":189,"context_line":"        \"\"\""},{"line_number":190,"context_line":"        url \u003d const.BASE_POOL_URL"},{"line_number":191,"context_line":"        pool \u003d self.create(url, method\u003d\u0027POST\u0027, **kwargs)"},{"line_number":192,"context_line":""},{"line_number":193,"context_line":"        return pool"},{"line_number":194,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_36d81aed","line":191,"range":{"start_line":191,"start_character":32,"end_line":191,"end_character":45},"in_reply_to":"df140735_93b28b48","updated":"2017-06-06 16:48:50.000000000","message":"Done","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"}],"octaviaclient/osc/v2/constants.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"6bb430268cd6d5ecc9d73403e92bddf643e3e371","unresolved":false,"context_lines":[{"line_number":65,"context_line":"    \u0027Connection Limit\u0027,"},{"line_number":66,"context_line":"    \u0027Loadbalancers\u0027)"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"POOL_ROWS \u003d ("},{"line_number":69,"context_line":"    \u0027Admin State Up\u0027,"},{"line_number":70,"context_line":"    \u0027Created At\u0027,"},{"line_number":71,"context_line":"    \u0027Description\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_53d87379","line":68,"range":{"start_line":68,"start_character":0,"end_line":68,"end_character":13},"updated":"2017-06-02 23:27:05.000000000","message":"Same question here about these being pretty names vs. the API names.","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"75e74492f76d1509cddf1cfb6991c2f3b13f83bb","unresolved":false,"context_lines":[{"line_number":65,"context_line":"    \u0027Connection Limit\u0027,"},{"line_number":66,"context_line":"    \u0027Loadbalancers\u0027)"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"POOL_ROWS \u003d ("},{"line_number":69,"context_line":"    \u0027Admin State Up\u0027,"},{"line_number":70,"context_line":"    \u0027Created At\u0027,"},{"line_number":71,"context_line":"    \u0027Description\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_f6a4e24c","line":68,"range":{"start_line":68,"start_character":0,"end_line":68,"end_character":13},"in_reply_to":"df140735_53d87379","updated":"2017-06-06 16:48:50.000000000","message":"Done","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"6bb430268cd6d5ecc9d73403e92bddf643e3e371","unresolved":false,"context_lines":[{"line_number":69,"context_line":"    \u0027Admin State Up\u0027,"},{"line_number":70,"context_line":"    \u0027Created At\u0027,"},{"line_number":71,"context_line":"    \u0027Description\u0027,"},{"line_number":72,"context_line":"    \u0027ID\u0027,"},{"line_number":73,"context_line":"    \u0027LB Algorithm\u0027,"},{"line_number":74,"context_line":"    \u0027Listeners\u0027,"},{"line_number":75,"context_line":"    \u0027Loadbalancers\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_33b3bf0c","line":72,"updated":"2017-06-02 23:27:05.000000000","message":"healthmonitor_id is missing","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"75e74492f76d1509cddf1cfb6991c2f3b13f83bb","unresolved":false,"context_lines":[{"line_number":69,"context_line":"    \u0027Admin State Up\u0027,"},{"line_number":70,"context_line":"    \u0027Created At\u0027,"},{"line_number":71,"context_line":"    \u0027Description\u0027,"},{"line_number":72,"context_line":"    \u0027ID\u0027,"},{"line_number":73,"context_line":"    \u0027LB Algorithm\u0027,"},{"line_number":74,"context_line":"    \u0027Listeners\u0027,"},{"line_number":75,"context_line":"    \u0027Loadbalancers\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_966b66ef","line":72,"in_reply_to":"df140735_33b3bf0c","updated":"2017-06-06 16:48:50.000000000","message":"Done","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"6bb430268cd6d5ecc9d73403e92bddf643e3e371","unresolved":false,"context_lines":[{"line_number":82,"context_line":"    \u0027Session Persistence\u0027,"},{"line_number":83,"context_line":"    \u0027Updated At\u0027)"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"POOL_COLUMNS \u003d ("},{"line_number":86,"context_line":"    \u0027ID\u0027,"},{"line_number":87,"context_line":"    \u0027Name\u0027,"},{"line_number":88,"context_line":"    \u0027Project ID\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_f3fae772","line":85,"range":{"start_line":85,"start_character":0,"end_line":85,"end_character":12},"updated":"2017-06-02 23:27:05.000000000","message":"the legacy client had \"admin_state_up\" here, we should probably add it.","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"75e74492f76d1509cddf1cfb6991c2f3b13f83bb","unresolved":false,"context_lines":[{"line_number":82,"context_line":"    \u0027Session Persistence\u0027,"},{"line_number":83,"context_line":"    \u0027Updated At\u0027)"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"POOL_COLUMNS \u003d ("},{"line_number":86,"context_line":"    \u0027ID\u0027,"},{"line_number":87,"context_line":"    \u0027Name\u0027,"},{"line_number":88,"context_line":"    \u0027Project ID\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_167f56ab","line":85,"range":{"start_line":85,"start_character":0,"end_line":85,"end_character":12},"in_reply_to":"df140735_f3fae772","updated":"2017-06-06 16:48:50.000000000","message":"Done","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"6bb430268cd6d5ecc9d73403e92bddf643e3e371","unresolved":false,"context_lines":[{"line_number":87,"context_line":"    \u0027Name\u0027,"},{"line_number":88,"context_line":"    \u0027Project ID\u0027,"},{"line_number":89,"context_line":"    \u0027Provisioning Status\u0027,"},{"line_number":90,"context_line":"    \u0027Members\u0027,"},{"line_number":91,"context_line":"    \u0027Protocol\u0027,"},{"line_number":92,"context_line":"    \u0027LB Algorithm\u0027,"},{"line_number":93,"context_line":"    \u0027Loadbalancers\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_73ded711","line":90,"range":{"start_line":90,"start_character":4,"end_line":90,"end_character":14},"updated":"2017-06-02 23:27:05.000000000","message":"Members should not be here.","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"75e74492f76d1509cddf1cfb6991c2f3b13f83bb","unresolved":false,"context_lines":[{"line_number":87,"context_line":"    \u0027Name\u0027,"},{"line_number":88,"context_line":"    \u0027Project ID\u0027,"},{"line_number":89,"context_line":"    \u0027Provisioning Status\u0027,"},{"line_number":90,"context_line":"    \u0027Members\u0027,"},{"line_number":91,"context_line":"    \u0027Protocol\u0027,"},{"line_number":92,"context_line":"    \u0027LB Algorithm\u0027,"},{"line_number":93,"context_line":"    \u0027Loadbalancers\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_f67982c4","line":90,"range":{"start_line":90,"start_character":4,"end_line":90,"end_character":14},"in_reply_to":"df140735_73ded711","updated":"2017-06-06 16:48:50.000000000","message":"Done","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"6bb430268cd6d5ecc9d73403e92bddf643e3e371","unresolved":false,"context_lines":[{"line_number":90,"context_line":"    \u0027Members\u0027,"},{"line_number":91,"context_line":"    \u0027Protocol\u0027,"},{"line_number":92,"context_line":"    \u0027LB Algorithm\u0027,"},{"line_number":93,"context_line":"    \u0027Loadbalancers\u0027,"},{"line_number":94,"context_line":"    \u0027Listeners\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_f3d1c7fc","line":93,"range":{"start_line":93,"start_character":4,"end_line":93,"end_character":20},"updated":"2017-06-02 23:27:05.000000000","message":"loadbalancers should not be here","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"75e74492f76d1509cddf1cfb6991c2f3b13f83bb","unresolved":false,"context_lines":[{"line_number":90,"context_line":"    \u0027Members\u0027,"},{"line_number":91,"context_line":"    \u0027Protocol\u0027,"},{"line_number":92,"context_line":"    \u0027LB Algorithm\u0027,"},{"line_number":93,"context_line":"    \u0027Loadbalancers\u0027,"},{"line_number":94,"context_line":"    \u0027Listeners\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_d64e7e95","line":93,"range":{"start_line":93,"start_character":4,"end_line":93,"end_character":20},"in_reply_to":"df140735_f3d1c7fc","updated":"2017-06-06 16:48:50.000000000","message":"Done","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"6bb430268cd6d5ecc9d73403e92bddf643e3e371","unresolved":false,"context_lines":[{"line_number":91,"context_line":"    \u0027Protocol\u0027,"},{"line_number":92,"context_line":"    \u0027LB Algorithm\u0027,"},{"line_number":93,"context_line":"    \u0027Loadbalancers\u0027,"},{"line_number":94,"context_line":"    \u0027Listeners\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_b3d7cfee","line":94,"range":{"start_line":94,"start_character":4,"end_line":94,"end_character":15},"updated":"2017-06-02 23:27:05.000000000","message":"listeners should not be here","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"75e74492f76d1509cddf1cfb6991c2f3b13f83bb","unresolved":false,"context_lines":[{"line_number":91,"context_line":"    \u0027Protocol\u0027,"},{"line_number":92,"context_line":"    \u0027LB Algorithm\u0027,"},{"line_number":93,"context_line":"    \u0027Loadbalancers\u0027,"},{"line_number":94,"context_line":"    \u0027Listeners\u0027)"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_b651ca38","line":94,"range":{"start_line":94,"start_character":4,"end_line":94,"end_character":15},"in_reply_to":"df140735_b3d7cfee","updated":"2017-06-06 16:48:50.000000000","message":"Done","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"}],"octaviaclient/osc/v2/pool.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"0ff7c24bb32c7cd8bedb6e464431d289e70575cb","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    def get_parser(self, prog_name):"},{"line_number":28,"context_line":"        parser \u003d super(CreatePool, self).get_parser(prog_name)"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"        parser.add_argument("},{"line_number":31,"context_line":"            \u0027loadbalancer\u0027,"},{"line_number":32,"context_line":"            metavar\u003d\u0027\u003cload balancer\u003e\u0027,"},{"line_number":33,"context_line":"            help\u003d\"Load balncer to add the pool to (name or ID)\""},{"line_number":34,"context_line":"        )"},{"line_number":35,"context_line":"        parser.add_argument("},{"line_number":36,"context_line":"            \u0027--name\u0027,"},{"line_number":37,"context_line":"            metavar\u003d\u0027\u003cname\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_fdd196fc","line":34,"range":{"start_line":30,"start_character":8,"end_line":34,"end_character":9},"updated":"2017-06-02 21:18:12.000000000","message":"The user can specify either a loadbalancer or a listener (which implies the loadbalancer).","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"6bb430268cd6d5ecc9d73403e92bddf643e3e371","unresolved":false,"context_lines":[{"line_number":50,"context_line":"            help\u003d\"Set the pool protocol\""},{"line_number":51,"context_line":"        )"},{"line_number":52,"context_line":"        parser.add_argument("},{"line_number":53,"context_line":"            \u0027--listener\u0027,"},{"line_number":54,"context_line":"            metavar\u003d\u0027\u003clistener\u003e\u0027,"},{"line_number":55,"context_line":"            help\u003d\"Listener to add to pool (name or ID)\""},{"line_number":56,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_13855bcf","line":53,"range":{"start_line":53,"start_character":13,"end_line":53,"end_character":23},"updated":"2017-06-02 23:27:05.000000000","message":"This should probably be in a add_mutually_exclusive_group with the loadbalancer option.\nWe should also add a check that at least one is specified.","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"75e74492f76d1509cddf1cfb6991c2f3b13f83bb","unresolved":false,"context_lines":[{"line_number":50,"context_line":"            help\u003d\"Set the pool protocol\""},{"line_number":51,"context_line":"        )"},{"line_number":52,"context_line":"        parser.add_argument("},{"line_number":53,"context_line":"            \u0027--listener\u0027,"},{"line_number":54,"context_line":"            metavar\u003d\u0027\u003clistener\u003e\u0027,"},{"line_number":55,"context_line":"            help\u003d\"Listener to add to pool (name or ID)\""},{"line_number":56,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_416746ea","line":53,"range":{"start_line":53,"start_character":13,"end_line":53,"end_character":23},"in_reply_to":"df140735_13855bcf","updated":"2017-06-06 16:48:50.000000000","message":"Done","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"6bb430268cd6d5ecc9d73403e92bddf643e3e371","unresolved":false,"context_lines":[{"line_number":56,"context_line":"        )"},{"line_number":57,"context_line":"        parser.add_argument("},{"line_number":58,"context_line":"            \u0027--session-persistence\u0027,"},{"line_number":59,"context_line":"            metavar\u003d\u0027\u003csession persistence\u003e\u0027,"},{"line_number":60,"context_line":"            help\u003d\"Set the session persistence for the listener (key\u003dvalue)\""},{"line_number":61,"context_line":"        )"},{"line_number":62,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_3343bf6e","line":59,"range":{"start_line":59,"start_character":21,"end_line":59,"end_character":42},"updated":"2017-06-02 23:27:05.000000000","message":"\u003cheader\u003dvalue,...\u003e\n\nThis could be: {\"cookie_name\": \"my_app_cookie\", \"type\": \"APP_COOKIE\"}\nor just:\u003e\n{\"type\": \"SOURCE_IP\"}","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"6bb430268cd6d5ecc9d73403e92bddf643e3e371","unresolved":false,"context_lines":[{"line_number":57,"context_line":"        parser.add_argument("},{"line_number":58,"context_line":"            \u0027--session-persistence\u0027,"},{"line_number":59,"context_line":"            metavar\u003d\u0027\u003csession persistence\u003e\u0027,"},{"line_number":60,"context_line":"            help\u003d\"Set the session persistence for the listener (key\u003dvalue)\""},{"line_number":61,"context_line":"        )"},{"line_number":62,"context_line":"        parser.add_argument("},{"line_number":63,"context_line":"            \u0027--lb-algorithm\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_1348bb84","line":60,"range":{"start_line":60,"start_character":64,"end_line":60,"end_character":73},"updated":"2017-06-02 23:27:05.000000000","message":"Same, should handle the case where there is a cookie_name","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"0ff7c24bb32c7cd8bedb6e464431d289e70575cb","unresolved":false,"context_lines":[{"line_number":66,"context_line":"            choices\u003d[\u0027SOURCE_IP\u0027, \u0027ROUND_ROBIN\u0027, \u0027LEAST_CONNECTIONS\u0027],"},{"line_number":67,"context_line":"            help\u003d\"Load balancing algorithm to use\""},{"line_number":68,"context_line":"        )"},{"line_number":69,"context_line":"        parser.add_argument("},{"line_number":70,"context_line":"            \u0027--project\u0027,"},{"line_number":71,"context_line":"            metavar\u003d\u0027\u003cproject\u003e\u0027,"},{"line_number":72,"context_line":"            help\u003d\"Set the ID of the project owning this pool\""}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_ddc93259","line":69,"updated":"2017-06-02 21:18:12.000000000","message":"We do not allow specifying project_id on objects other than the load balancer.","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"75e74492f76d1509cddf1cfb6991c2f3b13f83bb","unresolved":false,"context_lines":[{"line_number":66,"context_line":"            choices\u003d[\u0027SOURCE_IP\u0027, \u0027ROUND_ROBIN\u0027, \u0027LEAST_CONNECTIONS\u0027],"},{"line_number":67,"context_line":"            help\u003d\"Load balancing algorithm to use\""},{"line_number":68,"context_line":"        )"},{"line_number":69,"context_line":"        parser.add_argument("},{"line_number":70,"context_line":"            \u0027--project\u0027,"},{"line_number":71,"context_line":"            metavar\u003d\u0027\u003cproject\u003e\u0027,"},{"line_number":72,"context_line":"            help\u003d\"Set the ID of the project owning this pool\""}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_87a68e0e","line":69,"in_reply_to":"df140735_ddc93259","updated":"2017-06-06 16:48:50.000000000","message":"Done","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"6bb430268cd6d5ecc9d73403e92bddf643e3e371","unresolved":false,"context_lines":[{"line_number":123,"context_line":"        data \u003d self.app.client_manager.load_balancer.pool_delete("},{"line_number":124,"context_line":"            pool_id\u003dpool_id)"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"        return data"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"class ListPool(lister.Lister):"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_33adfee8","line":126,"range":{"start_line":126,"start_character":8,"end_line":126,"end_character":19},"updated":"2017-06-02 23:27:05.000000000","message":"This command is returning \"\u003cResponse [204]\u003e\" when it should not return any output.\n\n(octaviaclient) michjohn@devstackpy27:~/.../testclient/python-octaviaclient$ openstack loadbalancer pool delete pool1\n\u003cResponse [204]\u003e","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"75e74492f76d1509cddf1cfb6991c2f3b13f83bb","unresolved":false,"context_lines":[{"line_number":123,"context_line":"        data \u003d self.app.client_manager.load_balancer.pool_delete("},{"line_number":124,"context_line":"            pool_id\u003dpool_id)"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"        return data"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"class ListPool(lister.Lister):"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_56102e57","line":126,"range":{"start_line":126,"start_character":8,"end_line":126,"end_character":19},"in_reply_to":"df140735_33adfee8","updated":"2017-06-06 16:48:50.000000000","message":"Done","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"6bb430268cd6d5ecc9d73403e92bddf643e3e371","unresolved":false,"context_lines":[{"line_number":200,"context_line":"            metavar\u003d\u0027\u003cdescription\u003e\u0027,"},{"line_number":201,"context_line":"            help\u003d\"Set pool description\""},{"line_number":202,"context_line":"        )"},{"line_number":203,"context_line":"        parser.add_argument("},{"line_number":204,"context_line":"            \u0027--protocol\u0027,"},{"line_number":205,"context_line":"            metavar\u003d\u0027\u003cprotocol\u003e\u0027,"},{"line_number":206,"context_line":"            choices\u003d[\u0027TERMINATED_HTTPS\u0027, \u0027HTTP\u0027, \u0027HTTPS\u0027, \u0027TCP\u0027, \u0027PROXY\u0027],"},{"line_number":207,"context_line":"            help\u003d\"Set protocol\""},{"line_number":208,"context_line":"        )"},{"line_number":209,"context_line":"        parser.add_argument("},{"line_number":210,"context_line":"            \u0027--loadbalancer\u0027,"},{"line_number":211,"context_line":"            metavar\u003d\u0027\u003cload_balancer\u003e\u0027,"},{"line_number":212,"context_line":"            help\u003d\"Load balncer to add to pool  (name or ID)\""},{"line_number":213,"context_line":"        )"},{"line_number":214,"context_line":"        parser.add_argument("},{"line_number":215,"context_line":"            \u0027--listener\u0027,"},{"line_number":216,"context_line":"            metavar\u003d\u0027\u003clistener\u003e\u0027,"},{"line_number":217,"context_line":"            help\u003d\"Listener to add to pool (name or ID)\""},{"line_number":218,"context_line":"        )"},{"line_number":219,"context_line":"        parser.add_argument("},{"line_number":220,"context_line":"            \u0027--session-persistence\u0027,"},{"line_number":221,"context_line":"            metavar\u003d\u0027\u003csession_persistence\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_d377e29f","line":218,"range":{"start_line":203,"start_character":0,"end_line":218,"end_character":9},"updated":"2017-06-02 23:27:05.000000000","message":"We don\u0027t allow the user to change these settings.","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"6bb430268cd6d5ecc9d73403e92bddf643e3e371","unresolved":false,"context_lines":[{"line_number":252,"context_line":"        response \u003d self.app.client_manager.load_balancer.pool_set("},{"line_number":253,"context_line":"            pool_id, json\u003dbody)"},{"line_number":254,"context_line":""},{"line_number":255,"context_line":"        return response"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_f35d4615","line":255,"range":{"start_line":255,"start_character":8,"end_line":255,"end_character":23},"updated":"2017-06-02 23:27:05.000000000","message":"This returns raw json output, but it shouldn\u0027t return anything.","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"}],"octaviaclient/osc/v2/utils.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"f3b5c5bcc3dc2e3ffb49de0e5865fbda19a085e9","unresolved":false,"context_lines":[{"line_number":163,"context_line":"            \u0027loadbalancers\u0027,"},{"line_number":164,"context_line":"            client_manager.load_balancer.load_balancer_list"},{"line_number":165,"context_line":"        ),"},{"line_number":166,"context_line":"        \u0027lb_algorithm\u0027: (\u0027lb_algorithm\u0027, str.upper),"},{"line_number":167,"context_line":"        \u0027listener\u0027: ("},{"line_number":168,"context_line":"            \u0027listener_id\u0027,"},{"line_number":169,"context_line":"            \u0027listeners\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_fd2876e2","line":166,"updated":"2017-06-02 21:20:50.000000000","message":"This errors:\nTypeError: descriptor \u0027upper\u0027 requires a \u0027str\u0027 object but received a \u0027unicode\u0027","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"6bb430268cd6d5ecc9d73403e92bddf643e3e371","unresolved":false,"context_lines":[{"line_number":163,"context_line":"            \u0027loadbalancers\u0027,"},{"line_number":164,"context_line":"            client_manager.load_balancer.load_balancer_list"},{"line_number":165,"context_line":"        ),"},{"line_number":166,"context_line":"        \u0027lb_algorithm\u0027: (\u0027lb_algorithm\u0027, str.upper),"},{"line_number":167,"context_line":"        \u0027listener\u0027: ("},{"line_number":168,"context_line":"            \u0027listener_id\u0027,"},{"line_number":169,"context_line":"            \u0027listeners\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_73ced6b4","line":166,"in_reply_to":"df140735_fd2876e2","updated":"2017-06-02 23:27:05.000000000","message":"BTW, I am testing with python2.7","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"},{"author":{"_account_id":25674,"name":"Jude Cross","email":"jucross@blizzard.com","username":"problem_v"},"change_message_id":"75e74492f76d1509cddf1cfb6991c2f3b13f83bb","unresolved":false,"context_lines":[{"line_number":163,"context_line":"            \u0027loadbalancers\u0027,"},{"line_number":164,"context_line":"            client_manager.load_balancer.load_balancer_list"},{"line_number":165,"context_line":"        ),"},{"line_number":166,"context_line":"        \u0027lb_algorithm\u0027: (\u0027lb_algorithm\u0027, str.upper),"},{"line_number":167,"context_line":"        \u0027listener\u0027: ("},{"line_number":168,"context_line":"            \u0027listener_id\u0027,"},{"line_number":169,"context_line":"            \u0027listeners\u0027,"}],"source_content_type":"text/x-python","patch_set":8,"id":"df140735_6776da72","line":166,"in_reply_to":"df140735_fd2876e2","updated":"2017-06-06 16:48:50.000000000","message":"Done","commit_id":"d12df786c3ef671737670cb96629b6a1e1753a06"}]}