)]}'
{"openstack/load_balancer/v2/_proxy.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"cdaf595884d2194c58a1e5498022ccfd925177b0","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"from openstack.load_balancer.v2 import amphora as _amphora"},{"line_number":14,"context_line":"from openstack.load_balancer.v2 import availability_zone as _availability_zone"},{"line_number":15,"context_line":"from openstack.load_balancer.v2 import availability_zone_profile as \\"},{"line_number":16,"context_line":"    _availability_zone_profile"},{"line_number":17,"context_line":"from openstack.load_balancer.v2 import flavor as _flavor"},{"line_number":18,"context_line":"from openstack.load_balancer.v2 import flavor_profile as _flavor_profile"}],"source_content_type":"text/x-python","patch_set":3,"id":"df33271e_e64b9884","line":15,"range":{"start_line":15,"start_character":68,"end_line":15,"end_character":69},"updated":"2020-03-24 00:06:36.000000000","message":"The octavia team doesn\u0027t allow \\, but prefers ()\nObviously this is SDK so there is not hacking rule for that.\nLet\u0027s see what rm_work says.","commit_id":"f9edfb9d105d8ed7a750f8f75f4a21b3a4f0c097"},{"author":{"_account_id":3031,"name":"Sam Morrison","email":"sorrison@gmail.com","username":"sorrison"},"change_message_id":"6595c0daf8458c5d7a639f0b01b090792655b0f8","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"from openstack.load_balancer.v2 import amphora as _amphora"},{"line_number":14,"context_line":"from openstack.load_balancer.v2 import availability_zone as _availability_zone"},{"line_number":15,"context_line":"from openstack.load_balancer.v2 import availability_zone_profile as \\"},{"line_number":16,"context_line":"    _availability_zone_profile"},{"line_number":17,"context_line":"from openstack.load_balancer.v2 import flavor as _flavor"},{"line_number":18,"context_line":"from openstack.load_balancer.v2 import flavor_profile as _flavor_profile"}],"source_content_type":"text/x-python","patch_set":3,"id":"df33271e_ecdc171a","line":15,"range":{"start_line":15,"start_character":68,"end_line":15,"end_character":69},"in_reply_to":"df33271e_e64b9884","updated":"2020-03-24 03:42:14.000000000","message":"I see this used in eg.\n\nopenstack/identity/v3/_proxy.py","commit_id":"f9edfb9d105d8ed7a750f8f75f4a21b3a4f0c097"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"cd124fc6358370e58ce810ab6b9a53c2e0fe2902","unresolved":false,"context_lines":[{"line_number":1090,"context_line":"            :class:`~openstack.load_balancer.v2.availability_zone_profile.AvailabilityZoneProfile`"},{"line_number":1091,"context_line":"            instance"},{"line_number":1092,"context_line":"        :param dict attrs: The attributes to update on the availability_zone"},{"line_number":1093,"context_line":"                           profile represented by"},{"line_number":1094,"context_line":"                           ``availability_zone_profile``."},{"line_number":1095,"context_line":""},{"line_number":1096,"context_line":"        :returns: The updated availability zone profile"}],"source_content_type":"text/x-python","patch_set":5,"id":"df33271e_23eb65b0","line":1093,"updated":"2020-03-27 14:48:50.000000000","message":"nit: mixed indentation looks really weird, also that each function has different indent in docstrings","commit_id":"ba82d79a304ef1e64472e662e178c6a310f67e8a"}],"openstack/load_balancer/v2/availability_zone.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"cdaf595884d2194c58a1e5498022ccfd925177b0","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    allow_list \u003d True"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    _query_mapping \u003d resource.QueryParameters("},{"line_number":30,"context_line":"        \u0027id\u0027, \u0027name\u0027, \u0027description\u0027, \u0027availability_zone_profile_id\u0027,"},{"line_number":31,"context_line":"        is_enabled\u003d\u0027enabled\u0027"},{"line_number":32,"context_line":"    )"},{"line_number":33,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"df33271e_868f2ccb","line":30,"range":{"start_line":30,"start_character":8,"end_line":30,"end_character":12},"updated":"2020-03-24 00:06:36.000000000","message":"id is not needed here.","commit_id":"f9edfb9d105d8ed7a750f8f75f4a21b3a4f0c097"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"cdaf595884d2194c58a1e5498022ccfd925177b0","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    # Properties"},{"line_number":35,"context_line":"    #: The ID of the availability zone."},{"line_number":36,"context_line":"    id \u003d resource.Body(\u0027id\u0027)"},{"line_number":37,"context_line":"    #: The name of the availability zone."},{"line_number":38,"context_line":"    name \u003d resource.Body(\u0027name\u0027)"},{"line_number":39,"context_line":"    #: The availability zone description."}],"source_content_type":"text/x-python","patch_set":3,"id":"df33271e_66b14890","line":36,"range":{"start_line":36,"start_character":4,"end_line":36,"end_character":28},"updated":"2020-03-24 00:06:36.000000000","message":"We don\u0027t allow the setting of ID, this should be removed.","commit_id":"f9edfb9d105d8ed7a750f8f75f4a21b3a4f0c097"},{"author":{"_account_id":3031,"name":"Sam Morrison","email":"sorrison@gmail.com","username":"sorrison"},"change_message_id":"6595c0daf8458c5d7a639f0b01b090792655b0f8","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    # Properties"},{"line_number":35,"context_line":"    #: The ID of the availability zone."},{"line_number":36,"context_line":"    id \u003d resource.Body(\u0027id\u0027)"},{"line_number":37,"context_line":"    #: The name of the availability zone."},{"line_number":38,"context_line":"    name \u003d resource.Body(\u0027name\u0027)"},{"line_number":39,"context_line":"    #: The availability zone description."}],"source_content_type":"text/x-python","patch_set":3,"id":"df33271e_6cd007f9","line":36,"range":{"start_line":36,"start_character":4,"end_line":36,"end_character":28},"in_reply_to":"df33271e_66b14890","updated":"2020-03-24 03:42:14.000000000","message":"OK makes sense. I just copied from flavors so I assume they are wrong too.\n\nStill slightly confused as although you don\u0027t set it you do need the use the id attribute. Not sure how this all fits together","commit_id":"f9edfb9d105d8ed7a750f8f75f4a21b3a4f0c097"}],"openstack/load_balancer/v2/availability_zone_profile.py":[{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"cdaf595884d2194c58a1e5498022ccfd925177b0","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    allow_list \u003d True"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"    _query_mapping \u003d resource.QueryParameters("},{"line_number":30,"context_line":"        \u0027id\u0027, \u0027name\u0027, \u0027provider_name\u0027, \u0027availability_zone_data\u0027"},{"line_number":31,"context_line":"    )"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    # Properties"}],"source_content_type":"text/x-python","patch_set":3,"id":"df33271e_0672fce5","line":30,"range":{"start_line":30,"start_character":8,"end_line":30,"end_character":12},"updated":"2020-03-24 00:06:36.000000000","message":"Same, ID is not needed here.","commit_id":"f9edfb9d105d8ed7a750f8f75f4a21b3a4f0c097"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"cdaf595884d2194c58a1e5498022ccfd925177b0","unresolved":false,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":"    # Properties"},{"line_number":34,"context_line":"    #: The ID of the availability zone profile."},{"line_number":35,"context_line":"    id \u003d resource.Body(\u0027id\u0027)"},{"line_number":36,"context_line":"    #: The name of the availability zone profile."},{"line_number":37,"context_line":"    name \u003d resource.Body(\u0027name\u0027)"},{"line_number":38,"context_line":"    #: The provider this availability zone profile is for."}],"source_content_type":"text/x-python","patch_set":3,"id":"df33271e_667668db","line":35,"range":{"start_line":35,"start_character":4,"end_line":35,"end_character":28},"updated":"2020-03-24 00:06:36.000000000","message":"Same, ID is not needed here.","commit_id":"f9edfb9d105d8ed7a750f8f75f4a21b3a4f0c097"}]}