)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10980,"name":"Doug Wiegley","email":"dougwig@parkside.io","username":"dougw"},"change_message_id":"0b41779fd8c3eaae849fcb293de52241bdcf8142","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add code for load balancer status tree"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"So far the feature of retrieve a specific Load Balancer\u0027s Status Tree"},{"line_number":10,"context_line":"is not implemented in the neutronclient code, we need to add feature"},{"line_number":11,"context_line":"code and related tests."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"da6ed579_bde3a032","line":9,"range":{"start_line":9,"start_character":22,"end_line":9,"end_character":30},"updated":"2016-01-13 19:18:33.000000000","message":"nit: retrieving (note, do not re-spin simply for nits; only correct if you have to do a new patchset anyway.)","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":14605,"name":"vikram.choudhary","email":"vikschw@gmail.com","username":"vikram.choudhary"},"change_message_id":"c1c24c0bd16fbf119c84754f644de6281733c512","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"So far the feature of retrieving a specific Load Balancer\u0027s Status Tree"},{"line_number":10,"context_line":"is not implemented in the neutronclient code, we need to add feature"},{"line_number":11,"context_line":"code and related tests."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ia7804ab6baac674830c6834f67cfd411ebf4d14f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"da6ed579_22408933","line":11,"updated":"2016-01-15 09:53:58.000000000","message":"Please add\n\n1. DocImpact Flag. IMO, this is required as we are adding a new CLI.\n2. Release notes","commit_id":"f1688be255125ccba436e05778da667be8e81c41"},{"author":{"_account_id":12999,"name":"James Arendt","email":"james.arendt@hpe.com","username":"jwarendt"},"change_message_id":"084a9b4805f8f22c8e6274398c8161b9483a033f","unresolved":false,"context_lines":[{"line_number":10,"context_line":"is not implemented in the neutronclient code, we need to add feature"},{"line_number":11,"context_line":"code and related tests."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"1. DocImpact Flag. IMO, this is required as we are adding a new CLI."},{"line_number":14,"context_line":"2. Release notes"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: Ia7804ab6baac674830c6834f67cfd411ebf4d14f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"da6ed579_8dc86a56","line":13,"updated":"2016-01-15 18:27:27.000000000","message":"Should be \"DocImpact\" on a line by itself, to trigger normal autogenerate of related openstack-manuals bug.","commit_id":"eef3720377a9f23b7f983c2c8cf6085b3aee38c1"},{"author":{"_account_id":12999,"name":"James Arendt","email":"james.arendt@hpe.com","username":"jwarendt"},"change_message_id":"084a9b4805f8f22c8e6274398c8161b9483a033f","unresolved":false,"context_lines":[{"line_number":11,"context_line":"code and related tests."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"1. DocImpact Flag. IMO, this is required as we are adding a new CLI."},{"line_number":14,"context_line":"2. Release notes"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: Ia7804ab6baac674830c6834f67cfd411ebf4d14f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"da6ed579_ed97a6e9","line":14,"updated":"2016-01-15 18:27:27.000000000","message":"Believe this was a request to add actual release notes, not just to add release note as line to commit message.","commit_id":"eef3720377a9f23b7f983c2c8cf6085b3aee38c1"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"c3de268e270198791c5c220f0c6caad172a1702e","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"So far the feature of retrieving a specific Load Balancer\u0027s Status Tree"},{"line_number":10,"context_line":"is not implemented in the neutronclient code, we need to add feature"},{"line_number":11,"context_line":"code and related tests."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"DocImpact"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":13,"id":"ba63e191_4f06cf21","line":11,"updated":"2016-01-19 02:20:58.000000000","message":"nit: This paragraph looks unnecessary...","commit_id":"ea3c6e57c9e1c75ef9dcee3990ac773e7a7aac0f"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"0cae5286bf2e6b56a56e919dc42f6509ddbb9655","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"So far the feature of retrieving a specific Load Balancer\u0027s Status Tree"},{"line_number":10,"context_line":"is not implemented in the neutronclient code, we need to add feature"},{"line_number":11,"context_line":"code and related tests."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"DocImpact"},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":13,"id":"ba63e191_65f2a4d4","line":11,"in_reply_to":"ba63e191_4f06cf21","updated":"2016-01-19 18:02:47.000000000","message":"i will keep it, for people dont know the background of this patch, it is a good hint.","commit_id":"ea3c6e57c9e1c75ef9dcee3990ac773e7a7aac0f"},{"author":{"_account_id":14605,"name":"vikram.choudhary","email":"vikschw@gmail.com","username":"vikram.choudhary"},"change_message_id":"87d26d728428f30899a5f83e2fc71bc8a18a2609","unresolved":false,"context_lines":[{"line_number":10,"context_line":"is not implemented in the neutronclient code, we need to add feature"},{"line_number":11,"context_line":"code and related tests."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"DocImpact"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Ia7804ab6baac674830c6834f67cfd411ebf4d14f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":13,"id":"da6ed579_0c78e1f7","line":13,"updated":"2016-01-16 14:09:17.000000000","message":"Please write a word about the change.","commit_id":"ea3c6e57c9e1c75ef9dcee3990ac773e7a7aac0f"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"40988144806a8a5d454e000cdfb008f0e4dc80e3","unresolved":false,"context_lines":[{"line_number":10,"context_line":"is not implemented in the neutronclient code, we need to add feature"},{"line_number":11,"context_line":"code and related tests."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"DocImpact"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: Ia7804ab6baac674830c6834f67cfd411ebf4d14f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":13,"id":"ba63e191_00ec2cc6","line":13,"in_reply_to":"da6ed579_0c78e1f7","updated":"2016-01-19 17:24:02.000000000","message":"will edit it in the next version.","commit_id":"ea3c6e57c9e1c75ef9dcee3990ac773e7a7aac0f"}],"neutronclient/neutron/v2_0/__init__.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"e2bd9fd1eb34ef24daa9bd568bcf3dc81895479b","unresolved":false,"context_lines":[{"line_number":766,"context_line":"            return None"},{"line_number":767,"context_line":""},{"line_number":768,"context_line":""},{"line_number":769,"context_line":"class ShowJSONCommand(NeutronCommand):"},{"line_number":770,"context_line":"    \"\"\"Show JSON information of  a given resource.\"\"\""},{"line_number":771,"context_line":""},{"line_number":772,"context_line":"    api \u003d \u0027network\u0027"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_59015304","line":769,"updated":"2016-01-13 21:14:56.000000000","message":"This command is not generalized enough. Detail comments are found below. If you don\u0027t have a plan to generalize more, it is better to define a command in lb/v2/.\n\nWe already have some commands which directly inherit NeutronCommand (for example, RouterInterfaceCommand, SetGatewayRouter).","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"27da818c6bdeb760ae1898d7639cb8e8865f7b70","unresolved":false,"context_lines":[{"line_number":766,"context_line":"            return None"},{"line_number":767,"context_line":""},{"line_number":768,"context_line":""},{"line_number":769,"context_line":"class ShowJSONCommand(NeutronCommand):"},{"line_number":770,"context_line":"    \"\"\"Show JSON information of  a given resource.\"\"\""},{"line_number":771,"context_line":""},{"line_number":772,"context_line":"    api \u003d \u0027network\u0027"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_bfa86f78","line":769,"in_reply_to":"da6ed579_59015304","updated":"2016-01-13 22:10:07.000000000","message":"will move back to lb/v2/loadbalancer.py instead of put it in the init file","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"e2bd9fd1eb34ef24daa9bd568bcf3dc81895479b","unresolved":false,"context_lines":[{"line_number":767,"context_line":""},{"line_number":768,"context_line":""},{"line_number":769,"context_line":"class ShowJSONCommand(NeutronCommand):"},{"line_number":770,"context_line":"    \"\"\"Show JSON information of  a given resource.\"\"\""},{"line_number":771,"context_line":""},{"line_number":772,"context_line":"    api \u003d \u0027network\u0027"},{"line_number":773,"context_line":"    log \u003d None"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_0945ca6e","line":770,"updated":"2016-01-13 21:14:56.000000000","message":"double spaces before \u0027a given ..\u0027","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"27da818c6bdeb760ae1898d7639cb8e8865f7b70","unresolved":false,"context_lines":[{"line_number":767,"context_line":""},{"line_number":768,"context_line":""},{"line_number":769,"context_line":"class ShowJSONCommand(NeutronCommand):"},{"line_number":770,"context_line":"    \"\"\"Show JSON information of  a given resource.\"\"\""},{"line_number":771,"context_line":""},{"line_number":772,"context_line":"    api \u003d \u0027network\u0027"},{"line_number":773,"context_line":"    log \u003d None"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_5fb853c8","line":770,"in_reply_to":"da6ed579_0945ca6e","updated":"2016-01-13 22:10:07.000000000","message":"will change in the next version.","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":12999,"name":"James Arendt","email":"james.arendt@hpe.com","username":"jwarendt"},"change_message_id":"3bab68e1b85d8e6698534716f3db2e607073488a","unresolved":false,"context_lines":[{"line_number":768,"context_line":""},{"line_number":769,"context_line":"class ShowJSONCommand(NeutronCommand):"},{"line_number":770,"context_line":"    \"\"\"Show JSON information of  a given resource.\"\"\""},{"line_number":771,"context_line":""},{"line_number":772,"context_line":"    api \u003d \u0027network\u0027"},{"line_number":773,"context_line":"    log \u003d None"},{"line_number":774,"context_line":"    allow_names \u003d True"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_38725eae","line":771,"updated":"2016-01-13 19:54:50.000000000","message":"Nit: extra space","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"27da818c6bdeb760ae1898d7639cb8e8865f7b70","unresolved":false,"context_lines":[{"line_number":768,"context_line":""},{"line_number":769,"context_line":"class ShowJSONCommand(NeutronCommand):"},{"line_number":770,"context_line":"    \"\"\"Show JSON information of  a given resource.\"\"\""},{"line_number":771,"context_line":""},{"line_number":772,"context_line":"    api \u003d \u0027network\u0027"},{"line_number":773,"context_line":"    log \u003d None"},{"line_number":774,"context_line":"    allow_names \u003d True"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_1fae5b81","line":771,"in_reply_to":"da6ed579_38725eae","updated":"2016-01-13 22:10:07.000000000","message":"will change in the next version.","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"e2bd9fd1eb34ef24daa9bd568bcf3dc81895479b","unresolved":false,"context_lines":[{"line_number":776,"context_line":"    def get_parser(self, prog_name):"},{"line_number":777,"context_line":"        parser \u003d super(ShowJSONCommand, self).get_parser(prog_name)"},{"line_number":778,"context_line":"        if self.allow_names:"},{"line_number":779,"context_line":"            help_str \u003d _(\u0027ID or name of %s to show JSON info.\u0027)"},{"line_number":780,"context_line":"        else:"},{"line_number":781,"context_line":"            help_str \u003d _(\u0027ID of %s to show JSON info.\u0027)"},{"line_number":782,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_d9430341","line":779,"range":{"start_line":779,"start_character":50,"end_line":779,"end_character":60},"updated":"2016-01-13 21:14:56.000000000","message":"\"JSON info\" looks unnecessary. If necessary, it is better to be describe in the command help instead.","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"27da818c6bdeb760ae1898d7639cb8e8865f7b70","unresolved":false,"context_lines":[{"line_number":776,"context_line":"    def get_parser(self, prog_name):"},{"line_number":777,"context_line":"        parser \u003d super(ShowJSONCommand, self).get_parser(prog_name)"},{"line_number":778,"context_line":"        if self.allow_names:"},{"line_number":779,"context_line":"            help_str \u003d _(\u0027ID or name of %s to show JSON info.\u0027)"},{"line_number":780,"context_line":"        else:"},{"line_number":781,"context_line":"            help_str \u003d _(\u0027ID of %s to show JSON info.\u0027)"},{"line_number":782,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_7fe437e0","line":779,"range":{"start_line":779,"start_character":50,"end_line":779,"end_character":60},"in_reply_to":"da6ed579_d9430341","updated":"2016-01-13 22:10:07.000000000","message":"will delete the json info, the reason that i use this style is to be consistent with the code above, e.g in class  showcommand .","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"e2bd9fd1eb34ef24daa9bd568bcf3dc81895479b","unresolved":false,"context_lines":[{"line_number":778,"context_line":"        if self.allow_names:"},{"line_number":779,"context_line":"            help_str \u003d _(\u0027ID or name of %s to show JSON info.\u0027)"},{"line_number":780,"context_line":"        else:"},{"line_number":781,"context_line":"            help_str \u003d _(\u0027ID of %s to show JSON info.\u0027)"},{"line_number":782,"context_line":"        parser.add_argument("},{"line_number":783,"context_line":"            \u0027loadbalancer\u0027, metavar\u003dself.resource.upper(),"},{"line_number":784,"context_line":"            help\u003dhelp_str % self.resource)"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_599dd3ba","line":781,"range":{"start_line":781,"start_character":43,"end_line":781,"end_character":52},"updated":"2016-01-13 21:14:56.000000000","message":"ditto","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"e2bd9fd1eb34ef24daa9bd568bcf3dc81895479b","unresolved":false,"context_lines":[{"line_number":780,"context_line":"        else:"},{"line_number":781,"context_line":"            help_str \u003d _(\u0027ID of %s to show JSON info.\u0027)"},{"line_number":782,"context_line":"        parser.add_argument("},{"line_number":783,"context_line":"            \u0027loadbalancer\u0027, metavar\u003dself.resource.upper(),"},{"line_number":784,"context_line":"            help\u003dhelp_str % self.resource)"},{"line_number":785,"context_line":""},{"line_number":786,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_191e7b5f","line":783,"range":{"start_line":783,"start_character":12,"end_line":783,"end_character":26},"updated":"2016-01-13 21:14:56.000000000","message":"As a generic command, it should be \u0027id\u0027.","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"27da818c6bdeb760ae1898d7639cb8e8865f7b70","unresolved":false,"context_lines":[{"line_number":780,"context_line":"        else:"},{"line_number":781,"context_line":"            help_str \u003d _(\u0027ID of %s to show JSON info.\u0027)"},{"line_number":782,"context_line":"        parser.add_argument("},{"line_number":783,"context_line":"            \u0027loadbalancer\u0027, metavar\u003dself.resource.upper(),"},{"line_number":784,"context_line":"            help\u003dhelp_str % self.resource)"},{"line_number":785,"context_line":""},{"line_number":786,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_bfd62fef","line":783,"range":{"start_line":783,"start_character":12,"end_line":783,"end_character":26},"in_reply_to":"da6ed579_191e7b5f","updated":"2016-01-13 22:10:07.000000000","message":"once move back to loadbalancer.py, i will use loadbalancer.","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"e2bd9fd1eb34ef24daa9bd568bcf3dc81895479b","unresolved":false,"context_lines":[{"line_number":788,"context_line":"    def run(self, parsed_args):"},{"line_number":789,"context_line":"        self.log.debug(\u0027run(%s)\u0027 % parsed_args)"},{"line_number":790,"context_line":"        neutron_client \u003d self.get_client()"},{"line_number":791,"context_line":"        neutron_client.format \u003d parsed_args.request_format"},{"line_number":792,"context_line":"        lb_id \u003d find_resourceid_by_name_or_id("},{"line_number":793,"context_line":"            neutron_client, \u0027loadbalancer\u0027, parsed_args.loadbalancer)"},{"line_number":794,"context_line":"        params \u003d {}"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_197afb77","line":791,"updated":"2016-01-13 21:14:56.000000000","message":"It is no longer required.","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"27da818c6bdeb760ae1898d7639cb8e8865f7b70","unresolved":false,"context_lines":[{"line_number":788,"context_line":"    def run(self, parsed_args):"},{"line_number":789,"context_line":"        self.log.debug(\u0027run(%s)\u0027 % parsed_args)"},{"line_number":790,"context_line":"        neutron_client \u003d self.get_client()"},{"line_number":791,"context_line":"        neutron_client.format \u003d parsed_args.request_format"},{"line_number":792,"context_line":"        lb_id \u003d find_resourceid_by_name_or_id("},{"line_number":793,"context_line":"            neutron_client, \u0027loadbalancer\u0027, parsed_args.loadbalancer)"},{"line_number":794,"context_line":"        params \u003d {}"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_5f061387","line":791,"in_reply_to":"da6ed579_197afb77","updated":"2016-01-13 22:10:07.000000000","message":"will delete .","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"e2bd9fd1eb34ef24daa9bd568bcf3dc81895479b","unresolved":false,"context_lines":[{"line_number":789,"context_line":"        self.log.debug(\u0027run(%s)\u0027 % parsed_args)"},{"line_number":790,"context_line":"        neutron_client \u003d self.get_client()"},{"line_number":791,"context_line":"        neutron_client.format \u003d parsed_args.request_format"},{"line_number":792,"context_line":"        lb_id \u003d find_resourceid_by_name_or_id("},{"line_number":793,"context_line":"            neutron_client, \u0027loadbalancer\u0027, parsed_args.loadbalancer)"},{"line_number":794,"context_line":"        params \u003d {}"},{"line_number":795,"context_line":"        data \u003d neutron_client.retrieve_loadbalancer_status(lb_id, **params)"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_b98baf67","line":792,"range":{"start_line":792,"start_character":8,"end_line":792,"end_character":13},"updated":"2016-01-13 21:14:56.000000000","message":"It should honor \u0027allow_names\u0027. More concrete example is found at L.747-754.\n\nAlso, lb_id is not generic:\n\n  lb_id -\u003e _id","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"27da818c6bdeb760ae1898d7639cb8e8865f7b70","unresolved":false,"context_lines":[{"line_number":789,"context_line":"        self.log.debug(\u0027run(%s)\u0027 % parsed_args)"},{"line_number":790,"context_line":"        neutron_client \u003d self.get_client()"},{"line_number":791,"context_line":"        neutron_client.format \u003d parsed_args.request_format"},{"line_number":792,"context_line":"        lb_id \u003d find_resourceid_by_name_or_id("},{"line_number":793,"context_line":"            neutron_client, \u0027loadbalancer\u0027, parsed_args.loadbalancer)"},{"line_number":794,"context_line":"        params \u003d {}"},{"line_number":795,"context_line":"        data \u003d neutron_client.retrieve_loadbalancer_status(lb_id, **params)"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_3f86bfdb","line":792,"range":{"start_line":792,"start_character":8,"end_line":792,"end_character":13},"in_reply_to":"da6ed579_b98baf67","updated":"2016-01-13 22:10:07.000000000","message":"if we move back to loadbalancer.py, this name will make sense.","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"e2bd9fd1eb34ef24daa9bd568bcf3dc81895479b","unresolved":false,"context_lines":[{"line_number":790,"context_line":"        neutron_client \u003d self.get_client()"},{"line_number":791,"context_line":"        neutron_client.format \u003d parsed_args.request_format"},{"line_number":792,"context_line":"        lb_id \u003d find_resourceid_by_name_or_id("},{"line_number":793,"context_line":"            neutron_client, \u0027loadbalancer\u0027, parsed_args.loadbalancer)"},{"line_number":794,"context_line":"        params \u003d {}"},{"line_number":795,"context_line":"        data \u003d neutron_client.retrieve_loadbalancer_status(lb_id, **params)"},{"line_number":796,"context_line":"        self.format_output_data(data)"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_d9154340","line":793,"range":{"start_line":793,"start_character":28,"end_line":793,"end_character":42},"updated":"2016-01-13 21:14:56.000000000","message":"Use self.resource instead.","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"27da818c6bdeb760ae1898d7639cb8e8865f7b70","unresolved":false,"context_lines":[{"line_number":790,"context_line":"        neutron_client \u003d self.get_client()"},{"line_number":791,"context_line":"        neutron_client.format \u003d parsed_args.request_format"},{"line_number":792,"context_line":"        lb_id \u003d find_resourceid_by_name_or_id("},{"line_number":793,"context_line":"            neutron_client, \u0027loadbalancer\u0027, parsed_args.loadbalancer)"},{"line_number":794,"context_line":"        params \u003d {}"},{"line_number":795,"context_line":"        data \u003d neutron_client.retrieve_loadbalancer_status(lb_id, **params)"},{"line_number":796,"context_line":"        self.format_output_data(data)"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_5f2db3fc","line":793,"range":{"start_line":793,"start_character":28,"end_line":793,"end_character":42},"in_reply_to":"da6ed579_d9154340","updated":"2016-01-13 22:10:07.000000000","message":"will do.","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":12999,"name":"James Arendt","email":"james.arendt@hpe.com","username":"jwarendt"},"change_message_id":"3bab68e1b85d8e6698534716f3db2e607073488a","unresolved":false,"context_lines":[{"line_number":792,"context_line":"        lb_id \u003d find_resourceid_by_name_or_id("},{"line_number":793,"context_line":"            neutron_client, \u0027loadbalancer\u0027, parsed_args.loadbalancer)"},{"line_number":794,"context_line":"        params \u003d {}"},{"line_number":795,"context_line":"        data \u003d neutron_client.retrieve_loadbalancer_status(lb_id, **params)"},{"line_number":796,"context_line":"        self.format_output_data(data)"},{"line_number":797,"context_line":"        res \u003d data[self.filter]"},{"line_number":798,"context_line":"        if self.filter in data:"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_189afa63","line":795,"updated":"2016-01-13 19:54:50.000000000","message":"Hardcoding LBaaS command here makes no sense to me, is a real hack.  Like ability to have json tree as a default output but that has general applicability.  LBaaS specific methods should be in the loadbalancer code.","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"e2bd9fd1eb34ef24daa9bd568bcf3dc81895479b","unresolved":false,"context_lines":[{"line_number":792,"context_line":"        lb_id \u003d find_resourceid_by_name_or_id("},{"line_number":793,"context_line":"            neutron_client, \u0027loadbalancer\u0027, parsed_args.loadbalancer)"},{"line_number":794,"context_line":"        params \u003d {}"},{"line_number":795,"context_line":"        data \u003d neutron_client.retrieve_loadbalancer_status(lb_id, **params)"},{"line_number":796,"context_line":"        self.format_output_data(data)"},{"line_number":797,"context_line":"        res \u003d data[self.filter]"},{"line_number":798,"context_line":"        if self.filter in data:"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_19567bce","line":795,"in_reply_to":"da6ed579_189afa63","updated":"2016-01-13 21:14:56.000000000","message":"Yes, this needs to be generalized.","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"27da818c6bdeb760ae1898d7639cb8e8865f7b70","unresolved":false,"context_lines":[{"line_number":792,"context_line":"        lb_id \u003d find_resourceid_by_name_or_id("},{"line_number":793,"context_line":"            neutron_client, \u0027loadbalancer\u0027, parsed_args.loadbalancer)"},{"line_number":794,"context_line":"        params \u003d {}"},{"line_number":795,"context_line":"        data \u003d neutron_client.retrieve_loadbalancer_status(lb_id, **params)"},{"line_number":796,"context_line":"        self.format_output_data(data)"},{"line_number":797,"context_line":"        res \u003d data[self.filter]"},{"line_number":798,"context_line":"        if self.filter in data:"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_df41a3aa","line":795,"in_reply_to":"da6ed579_189afa63","updated":"2016-01-13 22:10:07.000000000","message":"will move back to loadbalancer.py","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"e2bd9fd1eb34ef24daa9bd568bcf3dc81895479b","unresolved":false,"context_lines":[{"line_number":793,"context_line":"            neutron_client, \u0027loadbalancer\u0027, parsed_args.loadbalancer)"},{"line_number":794,"context_line":"        params \u003d {}"},{"line_number":795,"context_line":"        data \u003d neutron_client.retrieve_loadbalancer_status(lb_id, **params)"},{"line_number":796,"context_line":"        self.format_output_data(data)"},{"line_number":797,"context_line":"        res \u003d data[self.filter]"},{"line_number":798,"context_line":"        if self.filter in data:"},{"line_number":799,"context_line":"            print(jsonutils.dumps(res, indent\u003d4, sort_keys\u003dFalse,"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_b9ec0ff4","line":796,"updated":"2016-01-13 21:14:56.000000000","message":"If we show JSON data to stdout, there is no need to call format_output_data().This method converts a list to a string delimited by \\n.","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"27da818c6bdeb760ae1898d7639cb8e8865f7b70","unresolved":false,"context_lines":[{"line_number":793,"context_line":"            neutron_client, \u0027loadbalancer\u0027, parsed_args.loadbalancer)"},{"line_number":794,"context_line":"        params \u003d {}"},{"line_number":795,"context_line":"        data \u003d neutron_client.retrieve_loadbalancer_status(lb_id, **params)"},{"line_number":796,"context_line":"        self.format_output_data(data)"},{"line_number":797,"context_line":"        res \u003d data[self.filter]"},{"line_number":798,"context_line":"        if self.filter in data:"},{"line_number":799,"context_line":"            print(jsonutils.dumps(res, indent\u003d4, sort_keys\u003dFalse,"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_bf44efb6","line":796,"in_reply_to":"da6ed579_b9ec0ff4","updated":"2016-01-13 22:10:07.000000000","message":"will delete it in the next version.","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"e2bd9fd1eb34ef24daa9bd568bcf3dc81895479b","unresolved":false,"context_lines":[{"line_number":794,"context_line":"        params \u003d {}"},{"line_number":795,"context_line":"        data \u003d neutron_client.retrieve_loadbalancer_status(lb_id, **params)"},{"line_number":796,"context_line":"        self.format_output_data(data)"},{"line_number":797,"context_line":"        res \u003d data[self.filter]"},{"line_number":798,"context_line":"        if self.filter in data:"},{"line_number":799,"context_line":"            print(jsonutils.dumps(res, indent\u003d4, sort_keys\u003dFalse,"},{"line_number":800,"context_line":"                                  separators\u003d(\u0027,\u0027, \u0027: \u0027)))"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_19ae7b81","line":797,"range":{"start_line":797,"start_character":19,"end_line":797,"end_character":30},"updated":"2016-01-13 21:14:56.000000000","message":"It is too specific to some operation. It should be covered by a specific implementation like RetrieveLoadBalancerStatus.","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"27da818c6bdeb760ae1898d7639cb8e8865f7b70","unresolved":false,"context_lines":[{"line_number":794,"context_line":"        params \u003d {}"},{"line_number":795,"context_line":"        data \u003d neutron_client.retrieve_loadbalancer_status(lb_id, **params)"},{"line_number":796,"context_line":"        self.format_output_data(data)"},{"line_number":797,"context_line":"        res \u003d data[self.filter]"},{"line_number":798,"context_line":"        if self.filter in data:"},{"line_number":799,"context_line":"            print(jsonutils.dumps(res, indent\u003d4, sort_keys\u003dFalse,"},{"line_number":800,"context_line":"                                  separators\u003d(\u0027,\u0027, \u0027: \u0027)))"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_3f0f9f89","line":797,"range":{"start_line":797,"start_character":19,"end_line":797,"end_character":30},"in_reply_to":"da6ed579_19ae7b81","updated":"2016-01-13 22:10:07.000000000","message":"will do in the next version.","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"}],"neutronclient/neutron/v2_0/lb/v2/loadbalancer.py":[{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"8d617e7aff16e5f9071504355fbfb5cb4ae1db70","unresolved":false,"context_lines":[{"line_number":130,"context_line":"            return list(zip(*sorted(stats.items())))"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"class RetrieveLBStatuses(neutronV20.ShowCommand):"},{"line_number":134,"context_line":"    \"\"\"Retrieve status tree for a given loadbalancer.\"\"\""},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"    resource \u003d \u0027loadbalancer\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa69d971_4a37c4fe","line":133,"range":{"start_line":133,"start_character":6,"end_line":133,"end_character":24},"updated":"2016-01-07 01:19:46.000000000","message":"Can we reword Statuses to Status ( too many S´s)","commit_id":"63a0a1809e2e5c06c0b3d2cd615125ee0b8f2cf1"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"cbc8ddee2aaf4d0e1533ffd2e64f179afe38573a","unresolved":false,"context_lines":[{"line_number":130,"context_line":"            return list(zip(*sorted(stats.items())))"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"class RetrieveLBStatuses(neutronV20.ShowCommand):"},{"line_number":134,"context_line":"    \"\"\"Retrieve status tree for a given loadbalancer.\"\"\""},{"line_number":135,"context_line":""},{"line_number":136,"context_line":"    resource \u003d \u0027loadbalancer\u0027"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa69d971_2d992320","line":133,"range":{"start_line":133,"start_character":6,"end_line":133,"end_character":24},"in_reply_to":"fa69d971_4a37c4fe","updated":"2016-01-07 18:29:32.000000000","message":"yes, we can change the name and will update it in the next version","commit_id":"63a0a1809e2e5c06c0b3d2cd615125ee0b8f2cf1"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"0fa7f1bbfe57b7f5570c4e1066d26bd631372e83","unresolved":false,"context_lines":[{"line_number":113,"context_line":"        data \u003d neutron_client.retrieve_loadbalancer_stats(loadbalancer_id,"},{"line_number":114,"context_line":"                                                          **params)"},{"line_number":115,"context_line":"        self.format_output_data(data)"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"        stats \u003d data[\u0027stats\u0027]"},{"line_number":118,"context_line":"        if \u0027stats\u0027 in data:"},{"line_number":119,"context_line":"            # To render the output table like:"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_06598b8e","line":116,"updated":"2016-01-08 01:34:33.000000000","message":"Not required","commit_id":"a6f4f4ba15bade77e72c329db749196346afdcc6"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"61dfe1096534b16e32d1468f195cf90f6181c68b","unresolved":false,"context_lines":[{"line_number":113,"context_line":"        data \u003d neutron_client.retrieve_loadbalancer_stats(loadbalancer_id,"},{"line_number":114,"context_line":"                                                          **params)"},{"line_number":115,"context_line":"        self.format_output_data(data)"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"        stats \u003d data[\u0027stats\u0027]"},{"line_number":118,"context_line":"        if \u0027stats\u0027 in data:"},{"line_number":119,"context_line":"            # To render the output table like:"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_a1cb3b52","line":116,"in_reply_to":"fa69d971_06598b8e","updated":"2016-01-11 17:52:40.000000000","message":"will change in the next version","commit_id":"a6f4f4ba15bade77e72c329db749196346afdcc6"},{"author":{"_account_id":8410,"name":"Richard Theis","email":"rtheis@us.ibm.com","username":"rtheis"},"change_message_id":"521bdc6cc6ce9a7b8be28b186bd983602beea521","unresolved":false,"context_lines":[{"line_number":131,"context_line":"            return list(zip(*sorted(stats.items())))"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"class RetrieveLBStatus(neutronV20.ShowCommand):"},{"line_number":135,"context_line":"    \"\"\"Retrieve status tree for a given loadbalancer.\"\"\""},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"    resource \u003d \u0027loadbalancer\u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_a20cf23b","line":134,"updated":"2016-01-08 14:17:42.000000000","message":"I think that the class should be called \"RetrieveLoadBalancerStatus\" for consistency with the other classes.","commit_id":"a6f4f4ba15bade77e72c329db749196346afdcc6"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"61dfe1096534b16e32d1468f195cf90f6181c68b","unresolved":false,"context_lines":[{"line_number":131,"context_line":"            return list(zip(*sorted(stats.items())))"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"class RetrieveLBStatus(neutronV20.ShowCommand):"},{"line_number":135,"context_line":"    \"\"\"Retrieve status tree for a given loadbalancer.\"\"\""},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"    resource \u003d \u0027loadbalancer\u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_81d5d7e5","line":134,"in_reply_to":"fa69d971_a20cf23b","updated":"2016-01-11 17:52:40.000000000","message":"yes, we can do that, and will change it in the next version","commit_id":"a6f4f4ba15bade77e72c329db749196346afdcc6"},{"author":{"_account_id":12999,"name":"James Arendt","email":"james.arendt@hpe.com","username":"jwarendt"},"change_message_id":"b28606a944eea7ce1444b563e0cc0f9440367053","unresolved":false,"context_lines":[{"line_number":149,"context_line":"        data \u003d neutron_client.retrieve_loadbalancer_status(loadbalancer_id,"},{"line_number":150,"context_line":"                                                           **params)"},{"line_number":151,"context_line":"        self.format_output_data(data)"},{"line_number":152,"context_line":"        statusess \u003d data[\u0027statuses\u0027]"},{"line_number":153,"context_line":"        if \u0027statuses\u0027 in data:"},{"line_number":154,"context_line":"            return list(zip(*sorted(statusess.items())))"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_6b824754","line":152,"updated":"2016-01-08 19:09:26.000000000","message":"Is this \u0027statusess\u0027 intentionally?  And what does this value do?","commit_id":"a6f4f4ba15bade77e72c329db749196346afdcc6"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"61dfe1096534b16e32d1468f195cf90f6181c68b","unresolved":false,"context_lines":[{"line_number":149,"context_line":"        data \u003d neutron_client.retrieve_loadbalancer_status(loadbalancer_id,"},{"line_number":150,"context_line":"                                                           **params)"},{"line_number":151,"context_line":"        self.format_output_data(data)"},{"line_number":152,"context_line":"        statusess \u003d data[\u0027statuses\u0027]"},{"line_number":153,"context_line":"        if \u0027statuses\u0027 in data:"},{"line_number":154,"context_line":"            return list(zip(*sorted(statusess.items())))"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_41311f59","line":152,"in_reply_to":"fa69d971_6b824754","updated":"2016-01-11 17:52:40.000000000","message":"it should be statuses, it is coming from here: \nhttps://wiki.openstack.org/wiki/Neutron/LBaaS/API_2.0#Retrieve_a_specific_Load_Balancer.27s_Status_Tree","commit_id":"a6f4f4ba15bade77e72c329db749196346afdcc6"},{"author":{"_account_id":12999,"name":"James Arendt","email":"james.arendt@hpe.com","username":"jwarendt"},"change_message_id":"f28a87ae3618b96f57d70d33090f841ec3da10d2","unresolved":false,"context_lines":[{"line_number":134,"context_line":"class RetrieveLoadBalancerStatus(neutronV20.NeutronCommand):"},{"line_number":135,"context_line":"    \"\"\"Retrieve status for a given loadbalancer."},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"    The only valid output here is a formatted JSON tree,it does not support"},{"line_number":138,"context_line":"    such functions as sort, query,etc, and the table format does not support"},{"line_number":139,"context_line":"    this type of data."},{"line_number":140,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":10,"id":"da6ed579_4e0f3f88","line":137,"updated":"2016-01-14 00:29:16.000000000","message":"Nit: add space after \u0027tree,\u0027 here and \u0027query,\u0027 in next line.  Only worth fixing if doing a new revision anyway.","commit_id":"f1688be255125ccba436e05778da667be8e81c41"},{"author":{"_account_id":14605,"name":"vikram.choudhary","email":"vikschw@gmail.com","username":"vikram.choudhary"},"change_message_id":"c1c24c0bd16fbf119c84754f644de6281733c512","unresolved":false,"context_lines":[{"line_number":147,"context_line":"        if self.allow_names:"},{"line_number":148,"context_line":"            help_str \u003d _(\u0027ID or name of %s to show.\u0027)"},{"line_number":149,"context_line":"        else:"},{"line_number":150,"context_line":"            help_str \u003d _(\u0027ID of %s to show.\u0027)"},{"line_number":151,"context_line":"        parser.add_argument("},{"line_number":152,"context_line":"            self.resource, metavar\u003dself.resource.upper(),"},{"line_number":153,"context_line":"            help\u003dhelp_str % self.resource)"}],"source_content_type":"text/x-python","patch_set":10,"id":"da6ed579_275dfb46","line":150,"updated":"2016-01-15 09:53:58.000000000","message":"When this condition can hit? I can find allow_names \u003d True in L143.","commit_id":"f1688be255125ccba436e05778da667be8e81c41"},{"author":{"_account_id":14605,"name":"vikram.choudhary","email":"vikschw@gmail.com","username":"vikram.choudhary"},"change_message_id":"c1c24c0bd16fbf119c84754f644de6281733c512","unresolved":false,"context_lines":[{"line_number":154,"context_line":""},{"line_number":155,"context_line":"        return parser"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"    def run(self, parsed_args):"},{"line_number":158,"context_line":"        self.log.debug(\u0027run(%s)\u0027 % parsed_args)"},{"line_number":159,"context_line":"        neutron_client \u003d self.get_client()"},{"line_number":160,"context_line":"        lb_id \u003d neutronV20.find_resourceid_by_name_or_id("}],"source_content_type":"text/x-python","patch_set":10,"id":"da6ed579_6746937c","line":157,"updated":"2016-01-15 09:53:58.000000000","message":"run is replaced by take_action. For details plz refer to https://review.openstack.org/#/c/265286/","commit_id":"f1688be255125ccba436e05778da667be8e81c41"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"c3de268e270198791c5c220f0c6caad172a1702e","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"from neutronclient._i18n import _"},{"line_number":19,"context_line":"from neutronclient.neutron import v2_0 as neutronV20"},{"line_number":20,"context_line":"from oslo_serialization import jsonutils"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"class ListLoadBalancer(neutronV20.ListCommand):"}],"source_content_type":"text/x-python","patch_set":13,"id":"ba63e191_2f17dba4","line":20,"updated":"2016-01-19 02:20:58.000000000","message":"This is a third-party library. Please move it to L.17 and add a blank line before neutronclient imports.","commit_id":"ea3c6e57c9e1c75ef9dcee3990ac773e7a7aac0f"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"c3de268e270198791c5c220f0c6caad172a1702e","unresolved":false,"context_lines":[{"line_number":134,"context_line":"class RetrieveLoadBalancerStatus(neutronV20.NeutronCommand):"},{"line_number":135,"context_line":"    \"\"\"Retrieve status for a given loadbalancer."},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"    The only valid output here is a formatted JSON tree, it does not support"},{"line_number":138,"context_line":"    such functions as sort, query, etc, and the table format does not support"},{"line_number":139,"context_line":"    this type of data."},{"line_number":140,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":13,"id":"ba63e191_81349bde","line":137,"range":{"start_line":137,"start_character":4,"end_line":137,"end_character":30},"updated":"2016-01-19 02:20:58.000000000","message":"-\u003e The only output","commit_id":"ea3c6e57c9e1c75ef9dcee3990ac773e7a7aac0f"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"c3de268e270198791c5c220f0c6caad172a1702e","unresolved":false,"context_lines":[{"line_number":134,"context_line":"class RetrieveLoadBalancerStatus(neutronV20.NeutronCommand):"},{"line_number":135,"context_line":"    \"\"\"Retrieve status for a given loadbalancer."},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"    The only valid output here is a formatted JSON tree, it does not support"},{"line_number":138,"context_line":"    such functions as sort, query, etc, and the table format does not support"},{"line_number":139,"context_line":"    this type of data."},{"line_number":140,"context_line":"    \"\"\""},{"line_number":141,"context_line":"    api \u003d \u0027network\u0027"}],"source_content_type":"text/x-python","patch_set":13,"id":"ba63e191_8f02873d","line":138,"range":{"start_line":137,"start_character":57,"end_line":138,"end_character":39},"updated":"2016-01-19 02:20:58.000000000","message":"This is unnecessary in the help message.","commit_id":"ea3c6e57c9e1c75ef9dcee3990ac773e7a7aac0f"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"40988144806a8a5d454e000cdfb008f0e4dc80e3","unresolved":false,"context_lines":[{"line_number":134,"context_line":"class RetrieveLoadBalancerStatus(neutronV20.NeutronCommand):"},{"line_number":135,"context_line":"    \"\"\"Retrieve status for a given loadbalancer."},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"    The only valid output here is a formatted JSON tree, it does not support"},{"line_number":138,"context_line":"    such functions as sort, query, etc, and the table format does not support"},{"line_number":139,"context_line":"    this type of data."},{"line_number":140,"context_line":"    \"\"\""},{"line_number":141,"context_line":"    api \u003d \u0027network\u0027"}],"source_content_type":"text/x-python","patch_set":13,"id":"ba63e191_e4445a75","line":138,"range":{"start_line":137,"start_character":57,"end_line":138,"end_character":39},"in_reply_to":"ba63e191_8f02873d","updated":"2016-01-19 17:24:02.000000000","message":"the above two changes will be updated in the next patch.","commit_id":"ea3c6e57c9e1c75ef9dcee3990ac773e7a7aac0f"},{"author":{"_account_id":14605,"name":"vikram.choudhary","email":"vikschw@gmail.com","username":"vikram.choudhary"},"change_message_id":"87d26d728428f30899a5f83e2fc71bc8a18a2609","unresolved":false,"context_lines":[{"line_number":138,"context_line":"    such functions as sort, query, etc, and the table format does not support"},{"line_number":139,"context_line":"    this type of data."},{"line_number":140,"context_line":"    \"\"\""},{"line_number":141,"context_line":"    api \u003d \u0027network\u0027"},{"line_number":142,"context_line":"    resource \u003d \u0027loadbalancer\u0027"},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"    def get_parser(self, prog_name):"}],"source_content_type":"text/x-python","patch_set":13,"id":"da6ed579_6c4abdae","line":141,"updated":"2016-01-16 14:09:17.000000000","message":"api attribute is no more required. You can refer to https://review.openstack.org/#/c/265284/","commit_id":"ea3c6e57c9e1c75ef9dcee3990ac773e7a7aac0f"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"c3de268e270198791c5c220f0c6caad172a1702e","unresolved":false,"context_lines":[{"line_number":138,"context_line":"    such functions as sort, query, etc, and the table format does not support"},{"line_number":139,"context_line":"    this type of data."},{"line_number":140,"context_line":"    \"\"\""},{"line_number":141,"context_line":"    api \u003d \u0027network\u0027"},{"line_number":142,"context_line":"    resource \u003d \u0027loadbalancer\u0027"},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"    def get_parser(self, prog_name):"}],"source_content_type":"text/x-python","patch_set":13,"id":"ba63e191_af866bb7","line":141,"in_reply_to":"da6ed579_6c4abdae","updated":"2016-01-19 02:20:58.000000000","message":"api \u003d \u0027network\u0027 itself is unnecessary. \u0027api\u0027 attribute is defined in NeutronCommand already.","commit_id":"ea3c6e57c9e1c75ef9dcee3990ac773e7a7aac0f"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"40988144806a8a5d454e000cdfb008f0e4dc80e3","unresolved":false,"context_lines":[{"line_number":138,"context_line":"    such functions as sort, query, etc, and the table format does not support"},{"line_number":139,"context_line":"    this type of data."},{"line_number":140,"context_line":"    \"\"\""},{"line_number":141,"context_line":"    api \u003d \u0027network\u0027"},{"line_number":142,"context_line":"    resource \u003d \u0027loadbalancer\u0027"},{"line_number":143,"context_line":""},{"line_number":144,"context_line":"    def get_parser(self, prog_name):"}],"source_content_type":"text/x-python","patch_set":13,"id":"ba63e191_40e224d7","line":141,"in_reply_to":"da6ed579_6c4abdae","updated":"2016-01-19 17:24:02.000000000","message":"will remove it in the next version.","commit_id":"ea3c6e57c9e1c75ef9dcee3990ac773e7a7aac0f"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"c3de268e270198791c5c220f0c6caad172a1702e","unresolved":false,"context_lines":[{"line_number":158,"context_line":"        data \u003d neutron_client.retrieve_loadbalancer_status(lb_id, **params)"},{"line_number":159,"context_line":"        res \u003d data[\u0027statuses\u0027]"},{"line_number":160,"context_line":"        if \u0027statuses\u0027 in data:"},{"line_number":161,"context_line":"            print(jsonutils.dumps(res, indent\u003d4, sort_keys\u003dFalse,"},{"line_number":162,"context_line":"                                  separators\u003d(\u0027,\u0027, \u0027: \u0027)))"}],"source_content_type":"text/x-python","patch_set":13,"id":"ba63e191_cf3abfe7","line":161,"range":{"start_line":161,"start_character":49,"end_line":161,"end_character":64},"updated":"2016-01-19 02:20:58.000000000","message":"False is the default value. There is no need to specify it here.","commit_id":"ea3c6e57c9e1c75ef9dcee3990ac773e7a7aac0f"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"40988144806a8a5d454e000cdfb008f0e4dc80e3","unresolved":false,"context_lines":[{"line_number":158,"context_line":"        data \u003d neutron_client.retrieve_loadbalancer_status(lb_id, **params)"},{"line_number":159,"context_line":"        res \u003d data[\u0027statuses\u0027]"},{"line_number":160,"context_line":"        if \u0027statuses\u0027 in data:"},{"line_number":161,"context_line":"            print(jsonutils.dumps(res, indent\u003d4, sort_keys\u003dFalse,"},{"line_number":162,"context_line":"                                  separators\u003d(\u0027,\u0027, \u0027: \u0027)))"}],"source_content_type":"text/x-python","patch_set":13,"id":"ba63e191_6482aa96","line":161,"range":{"start_line":161,"start_character":49,"end_line":161,"end_character":64},"in_reply_to":"ba63e191_cf3abfe7","updated":"2016-01-19 17:24:02.000000000","message":"will edit it in the next patch","commit_id":"ea3c6e57c9e1c75ef9dcee3990ac773e7a7aac0f"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"c3de268e270198791c5c220f0c6caad172a1702e","unresolved":false,"context_lines":[{"line_number":159,"context_line":"        res \u003d data[\u0027statuses\u0027]"},{"line_number":160,"context_line":"        if \u0027statuses\u0027 in data:"},{"line_number":161,"context_line":"            print(jsonutils.dumps(res, indent\u003d4, sort_keys\u003dFalse,"},{"line_number":162,"context_line":"                                  separators\u003d(\u0027,\u0027, \u0027: \u0027)))"}],"source_content_type":"text/x-python","patch_set":13,"id":"ba63e191_8f5e0744","line":162,"range":{"start_line":162,"start_character":34,"end_line":162,"end_character":56},"updated":"2016-01-19 02:20:58.000000000","message":"Why do you need to use a different set of separators?\nAren\u0027t the default separators enough?","commit_id":"ea3c6e57c9e1c75ef9dcee3990ac773e7a7aac0f"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"40988144806a8a5d454e000cdfb008f0e4dc80e3","unresolved":false,"context_lines":[{"line_number":159,"context_line":"        res \u003d data[\u0027statuses\u0027]"},{"line_number":160,"context_line":"        if \u0027statuses\u0027 in data:"},{"line_number":161,"context_line":"            print(jsonutils.dumps(res, indent\u003d4, sort_keys\u003dFalse,"},{"line_number":162,"context_line":"                                  separators\u003d(\u0027,\u0027, \u0027: \u0027)))"}],"source_content_type":"text/x-python","patch_set":13,"id":"ba63e191_847d5eb3","line":162,"range":{"start_line":162,"start_character":34,"end_line":162,"end_character":56},"in_reply_to":"ba63e191_8f5e0744","updated":"2016-01-19 17:24:02.000000000","message":"will edit it in the next version.","commit_id":"ea3c6e57c9e1c75ef9dcee3990ac773e7a7aac0f"}],"neutronclient/shell.py":[{"author":{"_account_id":12999,"name":"James Arendt","email":"james.arendt@hpe.com","username":"jwarendt"},"change_message_id":"b28606a944eea7ce1444b563e0cc0f9440367053","unresolved":false,"context_lines":[{"line_number":204,"context_line":"    \u0027lbaas-loadbalancer-update\u0027: lbaas_loadbalancer.UpdateLoadBalancer,"},{"line_number":205,"context_line":"    \u0027lbaas-loadbalancer-delete\u0027: lbaas_loadbalancer.DeleteLoadBalancer,"},{"line_number":206,"context_line":"    \u0027lbaas-loadbalancer-stats\u0027: lbaas_loadbalancer.RetrieveLoadBalancerStats,"},{"line_number":207,"context_line":"    \u0027lbaas-loadbalancer-status\u0027: lbaas_loadbalancer.RetrieveLBStatus,"},{"line_number":208,"context_line":"    \u0027lbaas-listener-list\u0027: lbaas_listener.ListListener,"},{"line_number":209,"context_line":"    \u0027lbaas-listener-show\u0027: lbaas_listener.ShowListener,"},{"line_number":210,"context_line":"    \u0027lbaas-listener-create\u0027: lbaas_listener.CreateListener,"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_90b9eacd","line":207,"updated":"2016-01-08 19:09:26.000000000","message":"Why change to \"LB\" instead of \"LoadBalancer\"?  Inconsistent with lines above such as Stats.","commit_id":"a6f4f4ba15bade77e72c329db749196346afdcc6"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"61dfe1096534b16e32d1468f195cf90f6181c68b","unresolved":false,"context_lines":[{"line_number":204,"context_line":"    \u0027lbaas-loadbalancer-update\u0027: lbaas_loadbalancer.UpdateLoadBalancer,"},{"line_number":205,"context_line":"    \u0027lbaas-loadbalancer-delete\u0027: lbaas_loadbalancer.DeleteLoadBalancer,"},{"line_number":206,"context_line":"    \u0027lbaas-loadbalancer-stats\u0027: lbaas_loadbalancer.RetrieveLoadBalancerStats,"},{"line_number":207,"context_line":"    \u0027lbaas-loadbalancer-status\u0027: lbaas_loadbalancer.RetrieveLBStatus,"},{"line_number":208,"context_line":"    \u0027lbaas-listener-list\u0027: lbaas_listener.ListListener,"},{"line_number":209,"context_line":"    \u0027lbaas-listener-show\u0027: lbaas_listener.ShowListener,"},{"line_number":210,"context_line":"    \u0027lbaas-listener-create\u0027: lbaas_listener.CreateListener,"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_c1a18f95","line":207,"in_reply_to":"fa69d971_90b9eacd","updated":"2016-01-11 17:52:40.000000000","message":"I can change it to be LoadBlanacer, LB is short for that so this line\u0027s character will not exceed 80, but in order for the consistency of the name, i will figure out a way for it.","commit_id":"a6f4f4ba15bade77e72c329db749196346afdcc6"},{"author":{"_account_id":12999,"name":"James Arendt","email":"james.arendt@hpe.com","username":"jwarendt"},"change_message_id":"3bab68e1b85d8e6698534716f3db2e607073488a","unresolved":false,"context_lines":[{"line_number":204,"context_line":"    \u0027lbaas-loadbalancer-update\u0027: lbaas_loadbalancer.UpdateLoadBalancer,"},{"line_number":205,"context_line":"    \u0027lbaas-loadbalancer-delete\u0027: lbaas_loadbalancer.DeleteLoadBalancer,"},{"line_number":206,"context_line":"    \u0027lbaas-loadbalancer-stats\u0027: lbaas_loadbalancer.RetrieveLoadBalancerStats,"},{"line_number":207,"context_line":"    \u0027lbaas-loadbalancer-status\u0027: lbaas_loadbalancer.RetrieveLoadBalancerStatus,"},{"line_number":208,"context_line":"    \u0027lbaas-listener-list\u0027: lbaas_listener.ListListener,"},{"line_number":209,"context_line":"    \u0027lbaas-listener-show\u0027: lbaas_listener.ShowListener,"},{"line_number":210,"context_line":"    \u0027lbaas-listener-create\u0027: lbaas_listener.CreateListener,"}],"source_content_type":"text/x-python","patch_set":9,"id":"da6ed579_d8c5428e","line":207,"updated":"2016-01-13 19:54:50.000000000","message":"Dislike that this lacks a verb, so not of form \u003cresource\u003e\u003caction\u003e like other command line entities.","commit_id":"381720fa6aae484ea8fb9c67fa824f1303b63481"}],"neutronclient/tests/unit/lb/v2/test_cli20_loadbalancer.py":[{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"8d617e7aff16e5f9071504355fbfb5cb4ae1db70","unresolved":false,"context_lines":[{"line_number":165,"context_line":"        self.assertIn(\u0027bytes_in\u0027, _str)"},{"line_number":166,"context_line":"        self.assertIn(\u00271234\u0027, _str)"},{"line_number":167,"context_line":"        self.assertIn(\u0027bytes_out\u0027, _str)"},{"line_number":168,"context_line":"        self.assertIn(\u00274321\u0027, _str)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa69d971_ea10f85a","line":168,"range":{"start_line":168,"start_character":23,"end_line":168,"end_character":27},"updated":"2016-01-07 01:19:46.000000000","message":"What changed here?","commit_id":"63a0a1809e2e5c06c0b3d2cd615125ee0b8f2cf1"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"0fa7f1bbfe57b7f5570c4e1066d26bd631372e83","unresolved":false,"context_lines":[{"line_number":173,"context_line":"        cmd \u003d lb.RetrieveLBStatus(test_cli20.MyApp(sys.stdout), None)"},{"line_number":174,"context_line":"        my_id \u003d self.test_id"},{"line_number":175,"context_line":"        fields \u003d [\u0027operating_status\u0027, \u0027provisioning_status\u0027]"},{"line_number":176,"context_line":"        args \u003d [\u0027--fields\u0027, \u0027operating_status\u0027, \u0027--fields\u0027,"},{"line_number":177,"context_line":"                \u0027provisioning_status\u0027, my_id]"},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"        self.mox.StubOutWithMock(cmd, \"get_client\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_069e6b07","line":176,"range":{"start_line":176,"start_character":51,"end_line":176,"end_character":57},"updated":"2016-01-08 01:34:33.000000000","message":"are two --fields required? ( this is just a query, not a review comment)","commit_id":"a6f4f4ba15bade77e72c329db749196346afdcc6"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"4c7fb782f6aab502366a22870804735ae1979572","unresolved":false,"context_lines":[{"line_number":173,"context_line":"        cmd \u003d lb.RetrieveLBStatus(test_cli20.MyApp(sys.stdout), None)"},{"line_number":174,"context_line":"        my_id \u003d self.test_id"},{"line_number":175,"context_line":"        fields \u003d [\u0027operating_status\u0027, \u0027provisioning_status\u0027]"},{"line_number":176,"context_line":"        args \u003d [\u0027--fields\u0027, \u0027operating_status\u0027, \u0027--fields\u0027,"},{"line_number":177,"context_line":"                \u0027provisioning_status\u0027, my_id]"},{"line_number":178,"context_line":""},{"line_number":179,"context_line":"        self.mox.StubOutWithMock(cmd, \"get_client\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_a229448b","line":176,"range":{"start_line":176,"start_character":51,"end_line":176,"end_character":57},"in_reply_to":"fa69d971_069e6b07","updated":"2016-01-08 17:58:37.000000000","message":"yes, since  we are testing the statuses of loadbalancer, it is necessary to check these 2 fields.","commit_id":"a6f4f4ba15bade77e72c329db749196346afdcc6"}],"neutronclient/v2_0/client.py":[{"author":{"_account_id":12999,"name":"James Arendt","email":"james.arendt@hpe.com","username":"jwarendt"},"change_message_id":"b28606a944eea7ce1444b563e0cc0f9440367053","unresolved":false,"context_lines":[{"line_number":343,"context_line":"    lbaas_loadbalancer_path \u003d \"/lbaas/loadbalancers/%s\""},{"line_number":344,"context_line":"    lbaas_loadbalancer_path_stats \u003d \"/lbaas/loadbalancers/%s/stats\""},{"line_number":345,"context_line":"    lbaas_loadbalancer_path_status \u003d \"/lbaas/loadbalancers/%s/statuses\""},{"line_number":346,"context_line":"    lbaas_listeners_path \u003d \"/lbaas/listeners\""},{"line_number":347,"context_line":"    lbaas_listener_path \u003d \"/lbaas/listeners/%s\""},{"line_number":348,"context_line":"    lbaas_pools_path \u003d \"/lbaas/pools\""},{"line_number":349,"context_line":"    lbaas_pool_path \u003d \"/lbaas/pools/%s\""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_2b576f03","line":346,"updated":"2016-01-08 19:09:26.000000000","message":"Not a fan of this URI.  Gets into the EXTED_PLURALS logic that involves \u0027es\u0027 instead of standard \u0027s\u0027.","commit_id":"a6f4f4ba15bade77e72c329db749196346afdcc6"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"61dfe1096534b16e32d1468f195cf90f6181c68b","unresolved":false,"context_lines":[{"line_number":343,"context_line":"    lbaas_loadbalancer_path \u003d \"/lbaas/loadbalancers/%s\""},{"line_number":344,"context_line":"    lbaas_loadbalancer_path_stats \u003d \"/lbaas/loadbalancers/%s/stats\""},{"line_number":345,"context_line":"    lbaas_loadbalancer_path_status \u003d \"/lbaas/loadbalancers/%s/statuses\""},{"line_number":346,"context_line":"    lbaas_listeners_path \u003d \"/lbaas/listeners\""},{"line_number":347,"context_line":"    lbaas_listener_path \u003d \"/lbaas/listeners/%s\""},{"line_number":348,"context_line":"    lbaas_pools_path \u003d \"/lbaas/pools\""},{"line_number":349,"context_line":"    lbaas_pool_path \u003d \"/lbaas/pools/%s\""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_01b80774","line":346,"in_reply_to":"fa69d971_2b576f03","updated":"2016-01-11 17:52:40.000000000","message":"I prefer to use statuses instead of status to keep consistency.","commit_id":"a6f4f4ba15bade77e72c329db749196346afdcc6"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"0fa7f1bbfe57b7f5570c4e1066d26bd631372e83","unresolved":false,"context_lines":[{"line_number":947,"context_line":""},{"line_number":948,"context_line":"    @APIParamsCall"},{"line_number":949,"context_line":"    def retrieve_loadbalancer_status(self, loadbalancer, **_params):"},{"line_number":950,"context_line":"        \"\"\"Retrieves statuses for a certain load balancer.\"\"\""},{"line_number":951,"context_line":"        return self.get(self.lbaas_loadbalancer_path_status % (loadbalancer),"},{"line_number":952,"context_line":"                        params\u003d_params)"},{"line_number":953,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_a602ff09","line":950,"range":{"start_line":950,"start_character":21,"end_line":950,"end_character":29},"updated":"2016-01-08 01:34:33.000000000","message":"status","commit_id":"a6f4f4ba15bade77e72c329db749196346afdcc6"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"834b6fb4cf8b0a7a55b5e756432ff1bfb9f0bce7","unresolved":false,"context_lines":[{"line_number":947,"context_line":""},{"line_number":948,"context_line":"    @APIParamsCall"},{"line_number":949,"context_line":"    def retrieve_loadbalancer_status(self, loadbalancer, **_params):"},{"line_number":950,"context_line":"        \"\"\"Retrieves statuses for a certain load balancer.\"\"\""},{"line_number":951,"context_line":"        return self.get(self.lbaas_loadbalancer_path_status % (loadbalancer),"},{"line_number":952,"context_line":"                        params\u003d_params)"},{"line_number":953,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"da6ed579_5d269b17","line":950,"range":{"start_line":950,"start_character":21,"end_line":950,"end_character":29},"in_reply_to":"fa69d971_22ab14d8","updated":"2016-01-12 00:08:31.000000000","message":"I am not a fan of statuses personally but yes, if you are fetching two values, then maybe you can use statuses","commit_id":"a6f4f4ba15bade77e72c329db749196346afdcc6"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"4c7fb782f6aab502366a22870804735ae1979572","unresolved":false,"context_lines":[{"line_number":947,"context_line":""},{"line_number":948,"context_line":"    @APIParamsCall"},{"line_number":949,"context_line":"    def retrieve_loadbalancer_status(self, loadbalancer, **_params):"},{"line_number":950,"context_line":"        \"\"\"Retrieves statuses for a certain load balancer.\"\"\""},{"line_number":951,"context_line":"        return self.get(self.lbaas_loadbalancer_path_status % (loadbalancer),"},{"line_number":952,"context_line":"                        params\u003d_params)"},{"line_number":953,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_22ab14d8","line":950,"range":{"start_line":950,"start_character":21,"end_line":950,"end_character":29},"in_reply_to":"fa69d971_a602ff09","updated":"2016-01-08 17:58:37.000000000","message":"I actually have a question here, since we have two statuses to return back from the status tree:  operating_status and provisioning_status, in this case, should we still use status or statuses.","commit_id":"a6f4f4ba15bade77e72c329db749196346afdcc6"}],"releasenotes/source/old_relnotes.rst":[{"author":{"_account_id":14605,"name":"vikram.choudhary","email":"vikschw@gmail.com","username":"vikram.choudhary"},"change_message_id":"87d26d728428f30899a5f83e2fc71bc8a18a2609","unresolved":false,"context_lines":[{"line_number":10,"context_line":"* made the publicURL the default endpoint instead of adminURL"},{"line_number":11,"context_line":"* add ability to update security group name (requires 2013.2-Havana or later)"},{"line_number":12,"context_line":"* add flake8 and pbr support for testing and building"},{"line_number":13,"context_line":"* add ability to retrieve a specific load balancer\u0027s status tree"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"2.2.0"},{"line_number":16,"context_line":"-----"}],"source_content_type":"text/x-rst","patch_set":13,"id":"da6ed579_cc58898d","line":13,"updated":"2016-01-16 14:09:17.000000000","message":"You got to add a separate yaml file. Refer to https://review.openstack.org/#/c/242547/8/releasenotes/notes/default-subnetpool-support-c0d34870e9d3e814.yaml","commit_id":"ea3c6e57c9e1c75ef9dcee3990ac773e7a7aac0f"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"dd92c2bc7199f0e9ec9af0d2c6be025807261c9d","unresolved":false,"context_lines":[{"line_number":10,"context_line":"* made the publicURL the default endpoint instead of adminURL"},{"line_number":11,"context_line":"* add ability to update security group name (requires 2013.2-Havana or later)"},{"line_number":12,"context_line":"* add flake8 and pbr support for testing and building"},{"line_number":13,"context_line":"* add ability to retrieve a specific load balancer\u0027s status tree"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"2.2.0"},{"line_number":16,"context_line":"-----"}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba63e191_e5f3f408","line":13,"in_reply_to":"ba63e191_6faab372","updated":"2016-01-19 18:00:49.000000000","message":"where should i put the release version 4.1.0","commit_id":"ea3c6e57c9e1c75ef9dcee3990ac773e7a7aac0f"},{"author":{"_account_id":10477,"name":"min wang","email":"swiftwangster@gmail.com","username":"minwang"},"change_message_id":"40988144806a8a5d454e000cdfb008f0e4dc80e3","unresolved":false,"context_lines":[{"line_number":10,"context_line":"* made the publicURL the default endpoint instead of adminURL"},{"line_number":11,"context_line":"* add ability to update security group name (requires 2013.2-Havana or later)"},{"line_number":12,"context_line":"* add flake8 and pbr support for testing and building"},{"line_number":13,"context_line":"* add ability to retrieve a specific load balancer\u0027s status tree"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"2.2.0"},{"line_number":16,"context_line":"-----"}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba63e191_d9581ffd","line":13,"in_reply_to":"ba63e191_6faab372","updated":"2016-01-19 17:24:02.000000000","message":"will add it in the next patch.","commit_id":"ea3c6e57c9e1c75ef9dcee3990ac773e7a7aac0f"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"c3de268e270198791c5c220f0c6caad172a1702e","unresolved":false,"context_lines":[{"line_number":10,"context_line":"* made the publicURL the default endpoint instead of adminURL"},{"line_number":11,"context_line":"* add ability to update security group name (requires 2013.2-Havana or later)"},{"line_number":12,"context_line":"* add flake8 and pbr support for testing and building"},{"line_number":13,"context_line":"* add ability to retrieve a specific load balancer\u0027s status tree"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"2.2.0"},{"line_number":16,"context_line":"-----"}],"source_content_type":"text/x-rst","patch_set":13,"id":"ba63e191_6faab372","line":13,"in_reply_to":"da6ed579_cc58898d","updated":"2016-01-19 02:20:58.000000000","message":"Yes, you need to create a new file. The next release will 4.1.0.\n\nFor more detail instruction, see http://docs.openstack.org/developer/reno/usage.html#creating-new-release-notes","commit_id":"ea3c6e57c9e1c75ef9dcee3990ac773e7a7aac0f"}]}