)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"e26999e46af33317289e12bbd2f75aec6ca9c7fe","unresolved":false,"context_lines":[{"line_number":7,"context_line":"add pool member operating status"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"in some clouds neutron might support also operating_status for the "},{"line_number":10,"context_line":"neutron.load_balancer.pool_member, so do handle it correctly"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Idad2435303f235045c1eb41b6aa456b937ca0695"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_35660ac3","line":10,"updated":"2018-08-14 08:33:21.000000000","message":"I\u0027m sorry if I\u0027m being picky, but I don\u0027t see operating_status in the API REF for Neutron: https://developer.openstack.org/api-ref/network/v2/index.html#pool-members. Was it added recently? I can only find references to operating_status on the balancers themselves.","commit_id":"89436850d266d30831fc414f3d27e0c5072f9769"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"61b54d622470625d8448f0e0df172c1442b6fb91","unresolved":false,"context_lines":[{"line_number":7,"context_line":"add pool member operating status"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"in some clouds neutron might support also operating_status for the "},{"line_number":10,"context_line":"neutron.load_balancer.pool_member, so do handle it correctly"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Idad2435303f235045c1eb41b6aa456b937ca0695"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_751b22c9","line":10,"in_reply_to":"3f79a3b5_35660ac3","updated":"2018-08-14 08:56:27.000000000","message":"Unfortunately Neutron LB API description is \"very\" buggy. I was fixing already couple of attributes, which are not listed in the official doc, but are present in at least our cloud. Upon query to Neutron Team they said there were multiple similar issues already and that this part of API is not documented/maintained anymore. Main point is that Octavia API is made compatible with Neutron, therefore it is safe to add attributes, which are described at least in Octavia (https://developer.openstack.org/api-ref/load-balancer/v2/#list-members)","commit_id":"89436850d266d30831fc414f3d27e0c5072f9769"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"dbe38fe08992f60a45f9612a1a219b7bbdf18cd5","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"add pool member operating status"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"in some clouds neutron might support also operating_status for the "},{"line_number":10,"context_line":"neutron.load_balancer.pool_member, so do handle it correctly"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Idad2435303f235045c1eb41b6aa456b937ca0695"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3f79a3b5_1598ae19","line":9,"updated":"2018-08-14 09:03:25.000000000","message":"s/neutron/octavia/?","commit_id":"fd474700d304d11ece92732ffd7c90faeea2f1dc"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"f0d091cb43f07b6765e486c67a9afe14e7142a7c","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"add pool member operating status"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"in some clouds neutron might support also operating_status for the "},{"line_number":10,"context_line":"neutron.load_balancer.pool_member, so do handle it correctly"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Idad2435303f235045c1eb41b6aa456b937ca0695"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3f79a3b5_15c18ee5","line":9,"in_reply_to":"3f79a3b5_1598ae19","updated":"2018-08-14 09:07:46.000000000","message":"no, neutron","commit_id":"fd474700d304d11ece92732ffd7c90faeea2f1dc"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"221734441bb00edb23bd4f99af5763987bcf903f","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"add pool member operating status"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"in some clouds neutron might support also operating_status for the "},{"line_number":10,"context_line":"neutron.load_balancer.pool_member, so do handle it correctly"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Idad2435303f235045c1eb41b6aa456b937ca0695"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3f79a3b5_e0077217","line":9,"in_reply_to":"3f79a3b5_15c18ee5","updated":"2018-08-14 09:15:24.000000000","message":"well, according to neutron\u0027s api-ref it does not support operating_status for pool members. but octavia does.","commit_id":"fd474700d304d11ece92732ffd7c90faeea2f1dc"},{"author":{"_account_id":22623,"name":"Erik Olof Gunnar Andersson","email":"eandersson@blizzard.com","username":"eoandersson"},"change_message_id":"df3b19e8a20f8c6468028dd16b82135c9382b1f3","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"add pool member operating status"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"in some clouds neutron might support also operating_status for the "},{"line_number":10,"context_line":"neutron.load_balancer.pool_member, so do handle it correctly"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Idad2435303f235045c1eb41b6aa456b937ca0695"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3f79a3b5_209ecab8","line":9,"in_reply_to":"3f79a3b5_6049225e","updated":"2018-08-14 09:25:46.000000000","message":"Look at the link I provided, the api-ref is correct, although the field is available in the database it’s not exposed in the api call, it’s only available in the status call for neutron-lbaas.","commit_id":"fd474700d304d11ece92732ffd7c90faeea2f1dc"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"2ad455b5cd790134570a81e8a55fa749805c9d20","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"add pool member operating status"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"in some clouds neutron might support also operating_status for the "},{"line_number":10,"context_line":"neutron.load_balancer.pool_member, so do handle it correctly"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Idad2435303f235045c1eb41b6aa456b937ca0695"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3f79a3b5_6049225e","line":9,"in_reply_to":"3f79a3b5_e0077217","updated":"2018-08-14 09:21:44.000000000","message":"api-ref is unfortunately not the single place of truth. I have experienced issues with it in Neutron, Cinder (not existing attributes, not documented existing attributes and methods). I.e. api-ref for cinder does not list backup restore functionality, but it is implemented in cinderclient.","commit_id":"fd474700d304d11ece92732ffd7c90faeea2f1dc"}],"openstack/network/v2/pool_member.py":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"e26999e46af33317289e12bbd2f75aec6ca9c7fe","unresolved":false,"context_lines":[{"line_number":55,"context_line":"    #: with weight of 2."},{"line_number":56,"context_line":"    weight \u003d resource.Body(\u0027weight\u0027, type\u003dint)"},{"line_number":57,"context_line":"    #: Status of pool member operating, e.g. ONLINE, OFFLINE."},{"line_number":58,"context_line":"    operating_status \u003d resource.Body(\u0027operating_status\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_1563ceb3","line":58,"updated":"2018-08-14 08:33:21.000000000","message":"nit: ideally the field should be sorted","commit_id":"89436850d266d30831fc414f3d27e0c5072f9769"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"61b54d622470625d8448f0e0df172c1442b6fb91","unresolved":false,"context_lines":[{"line_number":55,"context_line":"    #: with weight of 2."},{"line_number":56,"context_line":"    weight \u003d resource.Body(\u0027weight\u0027, type\u003dint)"},{"line_number":57,"context_line":"    #: Status of pool member operating, e.g. ONLINE, OFFLINE."},{"line_number":58,"context_line":"    operating_status \u003d resource.Body(\u0027operating_status\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_35312a5b","line":58,"in_reply_to":"3f79a3b5_1563ceb3","updated":"2018-08-14 08:56:27.000000000","message":"Done","commit_id":"89436850d266d30831fc414f3d27e0c5072f9769"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"dbe38fe08992f60a45f9612a1a219b7bbdf18cd5","unresolved":false,"context_lines":[{"line_number":43,"context_line":"    is_admin_state_up \u003d resource.Body(\u0027admin_state_up\u0027, type\u003dbool)"},{"line_number":44,"context_line":"    #: Name of the pool member."},{"line_number":45,"context_line":"    name \u003d resource.Body(\u0027name\u0027)"},{"line_number":46,"context_line":"    #: Status of pool member operating, e.g. ONLINE, OFFLINE."},{"line_number":47,"context_line":"    operating_status \u003d resource.Body(\u0027operating_status\u0027)"},{"line_number":48,"context_line":"    #: The ID of the project this pool member is associated with."},{"line_number":49,"context_line":"    project_id \u003d resource.Body(\u0027tenant_id\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_75f7a25a","line":46,"updated":"2018-08-14 09:03:25.000000000","message":"Let us explicitly state it\u0027s Octavia only","commit_id":"fd474700d304d11ece92732ffd7c90faeea2f1dc"}]}
