)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"803b71c0f5114c7b029abcb3858a22fe63cda1e6","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Carlos Goncalves \u003ccarlos.goncalves@neclab.eu\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-01-25 11:04:14 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Adds API definition for data plane status extension"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Related-Bug: #1598081"},{"line_number":10,"context_line":"Related-Bug: #1575146"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"3a461143_54d627c6","line":7,"updated":"2017-01-26 00:07:50.000000000","message":"Nit: API definition and reference (docs)","commit_id":"91a3d26fec5a84a8507b5782b2265b47ecc1b471"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"865ca0e945615c1c12f038d0a5fa8bc6d75aa552","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Carlos Goncalves \u003ccarlos.goncalves@neclab.eu\u003e"},{"line_number":5,"context_line":"CommitDate: 2017-01-25 11:04:14 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Adds API definition for data plane status extension"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Related-Bug: #1598081"},{"line_number":10,"context_line":"Related-Bug: #1575146"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"3a461143_70416c0b","line":7,"in_reply_to":"3a461143_54d627c6","updated":"2017-01-26 00:37:45.000000000","message":"Done","commit_id":"91a3d26fec5a84a8507b5782b2265b47ecc1b471"}],"api-ref/source/v2/data-plane-status.inc":[{"author":{"_account_id":22220,"name":"Anindita Das","email":"anindita.das@utexas.edu","username":"anudas"},"change_message_id":"5d016e1917548403bc8f33d3726b673360c53958","unresolved":false,"context_lines":[{"line_number":1,"context_line":".. -*- rst -*-"},{"line_number":2,"context_line":".. needs:method_verification"},{"line_number":3,"context_line":".. needs:parameter_verification"},{"line_number":4,"context_line":".. needs:example_verification"},{"line_number":5,"context_line":".. needs:body_verification"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":8,"context_line":"Data plane status extenssion (ports)"}],"source_content_type":"text/x-c++src","patch_set":3,"id":"5a3905b3_da02460b","line":5,"range":{"start_line":2,"start_character":0,"end_line":5,"end_character":26},"updated":"2017-01-25 05:10:12.000000000","message":"Can you please delete these lines after verification?","commit_id":"6630159cd7fd2e7542838b55b8842e9e81c69ddf"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"e93d6b949e75199f957adc4f94660b58f6ed261d","unresolved":false,"context_lines":[{"line_number":1,"context_line":".. -*- rst -*-"},{"line_number":2,"context_line":".. needs:method_verification"},{"line_number":3,"context_line":".. needs:parameter_verification"},{"line_number":4,"context_line":".. needs:example_verification"},{"line_number":5,"context_line":".. needs:body_verification"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":8,"context_line":"Data plane status extenssion (ports)"}],"source_content_type":"text/x-c++src","patch_set":3,"id":"5a3905b3_5a300d9f","line":5,"range":{"start_line":2,"start_character":0,"end_line":5,"end_character":26},"in_reply_to":"5a3905b3_572f08f6","updated":"2017-01-25 10:23:12.000000000","message":"Done","commit_id":"6630159cd7fd2e7542838b55b8842e9e81c69ddf"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"1b510236f9ac31cd755bf8a77ea1b7b0f6511c5d","unresolved":false,"context_lines":[{"line_number":1,"context_line":".. -*- rst -*-"},{"line_number":2,"context_line":".. needs:method_verification"},{"line_number":3,"context_line":".. needs:parameter_verification"},{"line_number":4,"context_line":".. needs:example_verification"},{"line_number":5,"context_line":".. needs:body_verification"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":8,"context_line":"Data plane status extenssion (ports)"}],"source_content_type":"text/x-c++src","patch_set":3,"id":"5a3905b3_572f08f6","line":5,"range":{"start_line":2,"start_character":0,"end_line":5,"end_character":26},"in_reply_to":"5a3905b3_da02460b","updated":"2017-01-25 09:17:09.000000000","message":"+1","commit_id":"6630159cd7fd2e7542838b55b8842e9e81c69ddf"},{"author":{"_account_id":22220,"name":"Anindita Das","email":"anindita.das@utexas.edu","username":"anudas"},"change_message_id":"5d016e1917548403bc8f33d3726b673360c53958","unresolved":false,"context_lines":[{"line_number":5,"context_line":".. needs:body_verification"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":8,"context_line":"Data plane status extenssion (ports)"},{"line_number":9,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"The data_plane_status extension attribute is available"}],"source_content_type":"text/x-c++src","patch_set":3,"id":"5a3905b3_1d03e80e","line":8,"range":{"start_line":8,"start_character":18,"end_line":8,"end_character":28},"updated":"2017-01-25 05:10:12.000000000","message":"/s/extenssion/extension","commit_id":"6630159cd7fd2e7542838b55b8842e9e81c69ddf"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"1b510236f9ac31cd755bf8a77ea1b7b0f6511c5d","unresolved":false,"context_lines":[{"line_number":5,"context_line":".. needs:body_verification"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":8,"context_line":"Data plane status extenssion (ports)"},{"line_number":9,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"The data_plane_status extension attribute is available"}],"source_content_type":"text/x-c++src","patch_set":3,"id":"5a3905b3_b73e6c27","line":8,"range":{"start_line":8,"start_character":18,"end_line":8,"end_character":28},"in_reply_to":"5a3905b3_1d03e80e","updated":"2017-01-25 09:17:09.000000000","message":"+1","commit_id":"6630159cd7fd2e7542838b55b8842e9e81c69ddf"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"1b510236f9ac31cd755bf8a77ea1b7b0f6511c5d","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The data_plane_port extension adds a new attribute ``data_plane_status`` to"},{"line_number":14,"context_line":"represent the status of the underlay data plane. This attribute is to be"},{"line_number":15,"context_line":"managed by entities outsideof Neutron, while the ``status`` attribute is"},{"line_number":16,"context_line":"managed by Neutron. Both status attributes are independent from one another."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Supported data plane status values:"}],"source_content_type":"text/x-c++src","patch_set":3,"id":"5a3905b3_171d80bd","line":15,"range":{"start_line":15,"start_character":20,"end_line":15,"end_character":29},"updated":"2017-01-25 09:17:09.000000000","message":"outside of","commit_id":"6630159cd7fd2e7542838b55b8842e9e81c69ddf"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"e93d6b949e75199f957adc4f94660b58f6ed261d","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The data_plane_port extension adds a new attribute ``data_plane_status`` to"},{"line_number":14,"context_line":"represent the status of the underlay data plane. This attribute is to be"},{"line_number":15,"context_line":"managed by entities outsideof Neutron, while the ``status`` attribute is"},{"line_number":16,"context_line":"managed by Neutron. Both status attributes are independent from one another."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Supported data plane status values:"}],"source_content_type":"text/x-c++src","patch_set":3,"id":"5a3905b3_bae65114","line":15,"range":{"start_line":15,"start_character":20,"end_line":15,"end_character":29},"in_reply_to":"5a3905b3_171d80bd","updated":"2017-01-25 10:23:12.000000000","message":"Done","commit_id":"6630159cd7fd2e7542838b55b8842e9e81c69ddf"},{"author":{"_account_id":22220,"name":"Anindita Das","email":"anindita.das@utexas.edu","username":"anudas"},"change_message_id":"5d016e1917548403bc8f33d3726b673360c53958","unresolved":false,"context_lines":[{"line_number":61,"context_line":""},{"line_number":62,"context_line":"Normal response codees: 200"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"Error response codes: 400,401,404"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"Request"},{"line_number":67,"context_line":"-------"}],"source_content_type":"text/x-c++src","patch_set":3,"id":"5a3905b3_7d122c5a","line":64,"range":{"start_line":64,"start_character":25,"end_line":64,"end_character":26},"updated":"2017-01-25 05:10:12.000000000","message":"nit: space after each comma","commit_id":"6630159cd7fd2e7542838b55b8842e9e81c69ddf"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"e93d6b949e75199f957adc4f94660b58f6ed261d","unresolved":false,"context_lines":[{"line_number":61,"context_line":""},{"line_number":62,"context_line":"Normal response codees: 200"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"Error response codes: 400,401,404"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"Request"},{"line_number":67,"context_line":"-------"}],"source_content_type":"text/x-c++src","patch_set":3,"id":"5a3905b3_faf0d9cb","line":64,"range":{"start_line":64,"start_character":25,"end_line":64,"end_character":26},"in_reply_to":"5a3905b3_7d122c5a","updated":"2017-01-25 10:23:12.000000000","message":"Done. Also need to add 403 (policy violation)","commit_id":"6630159cd7fd2e7542838b55b8842e9e81c69ddf"},{"author":{"_account_id":22220,"name":"Anindita Das","email":"anindita.das@utexas.edu","username":"anudas"},"change_message_id":"5d016e1917548403bc8f33d3726b673360c53958","unresolved":false,"context_lines":[{"line_number":75,"context_line":""},{"line_number":76,"context_line":".. literalinclude:: samples/ports/port-data-plane-status-update-request.json"},{"line_number":77,"context_line":"   :language: javascript"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"Response Example"},{"line_number":80,"context_line":"----------------"},{"line_number":81,"context_line":""}],"source_content_type":"text/x-c++src","patch_set":3,"id":"5a3905b3_dd19607b","line":78,"updated":"2017-01-25 05:10:12.000000000","message":"Missing Response section","commit_id":"6630159cd7fd2e7542838b55b8842e9e81c69ddf"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"e93d6b949e75199f957adc4f94660b58f6ed261d","unresolved":false,"context_lines":[{"line_number":75,"context_line":""},{"line_number":76,"context_line":".. literalinclude:: samples/ports/port-data-plane-status-update-request.json"},{"line_number":77,"context_line":"   :language: javascript"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"Response Example"},{"line_number":80,"context_line":"----------------"},{"line_number":81,"context_line":""}],"source_content_type":"text/x-c++src","patch_set":3,"id":"5a3905b3_9aafd593","line":78,"in_reply_to":"5a3905b3_dd19607b","updated":"2017-01-25 10:23:12.000000000","message":"Done","commit_id":"6630159cd7fd2e7542838b55b8842e9e81c69ddf"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"803b71c0f5114c7b029abcb3858a22fe63cda1e6","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Data plane status extension (ports)"},{"line_number":5,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"The data_plane_status extension attribute is available"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The data_plane_port extension adds a new attribute ``data_plane_status`` to"},{"line_number":10,"context_line":"represent the status of the underlay data plane. This attribute is to be"}],"source_content_type":"text/x-c++src","patch_set":6,"id":"3a461143_b493cbf9","line":7,"updated":"2017-01-26 00:07:50.000000000","message":"This sentence seems to be unneeded, or perhaps incomplete.","commit_id":"91a3d26fec5a84a8507b5782b2265b47ecc1b471"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"865ca0e945615c1c12f038d0a5fa8bc6d75aa552","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Data plane status extension (ports)"},{"line_number":5,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"The data_plane_status extension attribute is available"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The data_plane_port extension adds a new attribute ``data_plane_status`` to"},{"line_number":10,"context_line":"represent the status of the underlay data plane. This attribute is to be"}],"source_content_type":"text/x-c++src","patch_set":6,"id":"3a461143_d06f8090","line":7,"in_reply_to":"3a461143_b493cbf9","updated":"2017-01-26 00:37:45.000000000","message":"Done. Forgot to delete it.","commit_id":"91a3d26fec5a84a8507b5782b2265b47ecc1b471"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"803b71c0f5114c7b029abcb3858a22fe63cda1e6","unresolved":false,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Supported data plane status values:"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- None: no status being reported; default value"},{"line_number":17,"context_line":"- ACTIVE: the underlying data plane is up and running"},{"line_number":18,"context_line":"- DOWN: no traffic can flow from/to the Neutron port"},{"line_number":19,"context_line":""}],"source_content_type":"text/x-c++src","patch_set":6,"id":"3a461143_f462b315","line":16,"range":{"start_line":16,"start_character":2,"end_line":16,"end_character":6},"updated":"2017-01-26 00:07:50.000000000","message":"Nit: use ``the words`` to format as code on these.","commit_id":"91a3d26fec5a84a8507b5782b2265b47ecc1b471"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"865ca0e945615c1c12f038d0a5fa8bc6d75aa552","unresolved":false,"context_lines":[{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Supported data plane status values:"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- None: no status being reported; default value"},{"line_number":17,"context_line":"- ACTIVE: the underlying data plane is up and running"},{"line_number":18,"context_line":"- DOWN: no traffic can flow from/to the Neutron port"},{"line_number":19,"context_line":""}],"source_content_type":"text/x-c++src","patch_set":6,"id":"3a461143_106ae880","line":16,"range":{"start_line":16,"start_character":2,"end_line":16,"end_character":6},"in_reply_to":"3a461143_f462b315","updated":"2017-01-26 00:37:45.000000000","message":"Done","commit_id":"91a3d26fec5a84a8507b5782b2265b47ecc1b471"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"8c84a6e32305e6107009f81b1246027d44f41ce0","unresolved":false,"context_lines":[{"line_number":23,"context_line":"Shows status of the underlying data plane for a port."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Normal response code: 200"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Request"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":".. rest_parameters:: parameters.yaml"}],"source_content_type":"text/x-c++src","patch_set":7,"id":"3a461143_70baef6f","line":26,"updated":"2017-01-27 19:47:51.000000000","message":"No error response codes??","commit_id":"a8779543310a4a1ce30c78529950a821f38f74dc"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"6811680dcb898a7229d17539290dc5062f995e0d","unresolved":false,"context_lines":[{"line_number":23,"context_line":"Shows status of the underlying data plane for a port."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"Normal response code: 200"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Request"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":".. rest_parameters:: parameters.yaml"}],"source_content_type":"text/x-c++src","patch_set":7,"id":"1a430d35_2754a51f","line":26,"in_reply_to":"3a461143_70baef6f","updated":"2017-02-07 09:53:58.000000000","message":"Fixed by merging into the port API. Similar to Change-Id: Ia6b5966e597f3f619222a27f9a1eca03d6ed59b7","commit_id":"a8779543310a4a1ce30c78529950a821f38f74dc"}],"api-ref/source/v2/parameters.yaml":[{"author":{"_account_id":22220,"name":"Anindita Das","email":"anindita.das@utexas.edu","username":"anudas"},"change_message_id":"5d016e1917548403bc8f33d3726b673360c53958","unresolved":false,"context_lines":[{"line_number":708,"context_line":"  description: |"},{"line_number":709,"context_line":"    Status of the underlying data plane of a port."},{"line_number":710,"context_line":"  in: body"},{"line_number":711,"context_line":"  required: false"},{"line_number":712,"context_line":"  type: string"},{"line_number":713,"context_line":"default:"},{"line_number":714,"context_line":"  description: |"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"5a3905b3_fd4b7c5d","line":711,"range":{"start_line":711,"start_character":12,"end_line":711,"end_character":17},"updated":"2017-01-25 05:10:12.000000000","message":"Shouldn\u0027t this be true?","commit_id":"6630159cd7fd2e7542838b55b8842e9e81c69ddf"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"e93d6b949e75199f957adc4f94660b58f6ed261d","unresolved":false,"context_lines":[{"line_number":708,"context_line":"  description: |"},{"line_number":709,"context_line":"    Status of the underlying data plane of a port."},{"line_number":710,"context_line":"  in: body"},{"line_number":711,"context_line":"  required: false"},{"line_number":712,"context_line":"  type: string"},{"line_number":713,"context_line":"default:"},{"line_number":714,"context_line":"  description: |"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"5a3905b3_1aa76550","line":711,"range":{"start_line":711,"start_character":12,"end_line":711,"end_character":17},"in_reply_to":"5a3905b3_fd4b7c5d","updated":"2017-01-25 10:23:12.000000000","message":"Done","commit_id":"6630159cd7fd2e7542838b55b8842e9e81c69ddf"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"7b230e0f6cedd769de8f2401eaf3b74bd9f7d713","unresolved":false,"context_lines":[{"line_number":731,"context_line":"  in: body"},{"line_number":732,"context_line":"  required: true"},{"line_number":733,"context_line":"  type: string"},{"line_number":734,"context_line":"data_plane_status:"},{"line_number":735,"context_line":"  description: |"},{"line_number":736,"context_line":"    Status of the underlying data plane of a port."},{"line_number":737,"context_line":"  in: body"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"1a430d35_b13f94b2","line":734,"updated":"2017-02-09 00:58:13.000000000","message":"We should prepare \"data_plane_status-request\" which has required\u003dfalse.","commit_id":"37ce52196e0c82349c5d28e5fb2de99c0863503a"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"b00afc2d1a52ee3f9870be22bb5aa68036abbb96","unresolved":false,"context_lines":[{"line_number":731,"context_line":"  in: body"},{"line_number":732,"context_line":"  required: true"},{"line_number":733,"context_line":"  type: string"},{"line_number":734,"context_line":"data_plane_status:"},{"line_number":735,"context_line":"  description: |"},{"line_number":736,"context_line":"    Status of the underlying data plane of a port."},{"line_number":737,"context_line":"  in: body"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"1a430d35_a3b81c12","line":734,"in_reply_to":"1a430d35_b13f94b2","updated":"2017-02-09 10:42:29.000000000","message":"Done","commit_id":"37ce52196e0c82349c5d28e5fb2de99c0863503a"}],"api-ref/source/v2/ports.inc":[{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"0516492389622ac1f654acc94abdbcf72006999c","unresolved":false,"context_lines":[{"line_number":20,"context_line":"Data plane status extension"},{"line_number":21,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"The data plane port extension adds a new attribute ``data_plane_status`` to"},{"line_number":24,"context_line":"represent the status of the underlay data plane. This attribute is to be"},{"line_number":25,"context_line":"managed by entities outside of Neutron, while the ``status`` attribute is"},{"line_number":26,"context_line":"managed by Neutron. Both status attributes are independent from one another."}],"source_content_type":"text/x-c++src","patch_set":8,"id":"1a430d35_b347f375","line":23,"range":{"start_line":23,"start_character":29,"end_line":23,"end_character":30},"updated":"2017-02-07 11:29:15.000000000","message":"It would be nice if we could note the extension name (like above) here. e.g.::\n\n   \"port extension (``data-plane-status``) adds a new...\"","commit_id":"f67251f33766b013a1e7b8e85ef8077784b22626"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"5a99e41da3f31462ee48ad4a9476c4076bbf1c35","unresolved":false,"context_lines":[{"line_number":20,"context_line":"Data plane status extension"},{"line_number":21,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"The data plane port extension adds a new attribute ``data_plane_status`` to"},{"line_number":24,"context_line":"represent the status of the underlay data plane. This attribute is to be"},{"line_number":25,"context_line":"managed by entities outside of Neutron, while the ``status`` attribute is"},{"line_number":26,"context_line":"managed by Neutron. Both status attributes are independent from one another."}],"source_content_type":"text/x-c++src","patch_set":8,"id":"1a430d35_93f59258","line":23,"range":{"start_line":23,"start_character":29,"end_line":23,"end_character":30},"in_reply_to":"1a430d35_b347f375","updated":"2017-02-07 13:37:32.000000000","message":"Done","commit_id":"f67251f33766b013a1e7b8e85ef8077784b22626"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"0516492389622ac1f654acc94abdbcf72006999c","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Supported data plane status values:"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"- ``None``: no status being reported; default value"},{"line_number":31,"context_line":"- ``ACTIVE``: the underlying data plane is up and running"},{"line_number":32,"context_line":"- ``DOWN``: no traffic can flow from/to the Neutron port"},{"line_number":33,"context_line":""}],"source_content_type":"text/x-c++src","patch_set":8,"id":"1a430d35_4e0706ca","line":30,"range":{"start_line":30,"start_character":2,"end_line":30,"end_character":10},"updated":"2017-02-07 11:29:15.000000000","message":"I\u0027m thinking this should be ``null`` and not the string \"None\"??","commit_id":"f67251f33766b013a1e7b8e85ef8077784b22626"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"5a99e41da3f31462ee48ad4a9476c4076bbf1c35","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Supported data plane status values:"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"- ``None``: no status being reported; default value"},{"line_number":31,"context_line":"- ``ACTIVE``: the underlying data plane is up and running"},{"line_number":32,"context_line":"- ``DOWN``: no traffic can flow from/to the Neutron port"},{"line_number":33,"context_line":""}],"source_content_type":"text/x-c++src","patch_set":8,"id":"1a430d35_939cb210","line":30,"range":{"start_line":30,"start_character":2,"end_line":30,"end_character":10},"in_reply_to":"1a430d35_4e0706ca","updated":"2017-02-07 13:37:32.000000000","message":"Oops, you\u0027re right.","commit_id":"f67251f33766b013a1e7b8e85ef8077784b22626"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"0516492389622ac1f654acc94abdbcf72006999c","unresolved":false,"context_lines":[{"line_number":97,"context_line":"Response Example (admin user)"},{"line_number":98,"context_line":"-----------------------------"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":".. literalinclude:: samples/ports/port-bind-show-response.json"},{"line_number":101,"context_line":"   :language: javascript"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"Update port"}],"source_content_type":"text/x-c++src","patch_set":8,"id":"1a430d35_ceb13649","line":100,"updated":"2017-02-07 11:29:15.000000000","message":"Shouldn\u0027t we also add data_plane_status to this sample response?","commit_id":"f67251f33766b013a1e7b8e85ef8077784b22626"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"5a99e41da3f31462ee48ad4a9476c4076bbf1c35","unresolved":false,"context_lines":[{"line_number":97,"context_line":"Response Example (admin user)"},{"line_number":98,"context_line":"-----------------------------"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":".. literalinclude:: samples/ports/port-bind-show-response.json"},{"line_number":101,"context_line":"   :language: javascript"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"Update port"}],"source_content_type":"text/x-c++src","patch_set":8,"id":"1a430d35_b3b44e82","line":100,"in_reply_to":"1a430d35_ceb13649","updated":"2017-02-07 13:37:32.000000000","message":"Done","commit_id":"f67251f33766b013a1e7b8e85ef8077784b22626"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"0516492389622ac1f654acc94abdbcf72006999c","unresolved":false,"context_lines":[{"line_number":125,"context_line":"removed from the port."},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"Only admins and users with a specific role can update the data plane status"},{"line_number":128,"context_line":"(default role: data_plane_integrator)."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"Normal response codes: 200"}],"source_content_type":"text/x-c++src","patch_set":8,"id":"1a430d35_933917f5","line":128,"range":{"start_line":128,"start_character":15,"end_line":128,"end_character":36},"updated":"2017-02-07 11:29:15.000000000","message":"Nit use ``data_plane_integrator`` formatting.","commit_id":"f67251f33766b013a1e7b8e85ef8077784b22626"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"5a99e41da3f31462ee48ad4a9476c4076bbf1c35","unresolved":false,"context_lines":[{"line_number":125,"context_line":"removed from the port."},{"line_number":126,"context_line":""},{"line_number":127,"context_line":"Only admins and users with a specific role can update the data plane status"},{"line_number":128,"context_line":"(default role: data_plane_integrator)."},{"line_number":129,"context_line":""},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"Normal response codes: 200"}],"source_content_type":"text/x-c++src","patch_set":8,"id":"1a430d35_d3b9da7a","line":128,"range":{"start_line":128,"start_character":15,"end_line":128,"end_character":36},"in_reply_to":"1a430d35_933917f5","updated":"2017-02-07 13:37:32.000000000","message":"Done","commit_id":"f67251f33766b013a1e7b8e85ef8077784b22626"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"0516492389622ac1f654acc94abdbcf72006999c","unresolved":false,"context_lines":[{"line_number":304,"context_line":"Response Example (admin user)"},{"line_number":305,"context_line":"-----------------------------"},{"line_number":306,"context_line":""},{"line_number":307,"context_line":".. literalinclude:: samples/ports/ports-bind-list-response.json"},{"line_number":308,"context_line":"   :language: javascript"},{"line_number":309,"context_line":""},{"line_number":310,"context_line":"Create port"}],"source_content_type":"text/x-c++src","patch_set":8,"id":"1a430d35_0ee6de43","line":307,"updated":"2017-02-07 11:29:15.000000000","message":"Ditto: missing the data_plane_status in sample response.","commit_id":"f67251f33766b013a1e7b8e85ef8077784b22626"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"5a99e41da3f31462ee48ad4a9476c4076bbf1c35","unresolved":false,"context_lines":[{"line_number":304,"context_line":"Response Example (admin user)"},{"line_number":305,"context_line":"-----------------------------"},{"line_number":306,"context_line":""},{"line_number":307,"context_line":".. literalinclude:: samples/ports/ports-bind-list-response.json"},{"line_number":308,"context_line":"   :language: javascript"},{"line_number":309,"context_line":""},{"line_number":310,"context_line":"Create port"}],"source_content_type":"text/x-c++src","patch_set":8,"id":"1a430d35_1377e2d3","line":307,"in_reply_to":"1a430d35_0ee6de43","updated":"2017-02-07 13:37:32.000000000","message":"It\u0027s there already.","commit_id":"f67251f33766b013a1e7b8e85ef8077784b22626"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"0516492389622ac1f654acc94abdbcf72006999c","unresolved":false,"context_lines":[{"line_number":412,"context_line":""},{"line_number":413,"context_line":"Error response codes: 400, 401, 403, 404, 409"},{"line_number":414,"context_line":""},{"line_number":415,"context_line":"Request"},{"line_number":416,"context_line":"-------"},{"line_number":417,"context_line":""},{"line_number":418,"context_line":".. rest_parameters:: parameters.yaml"}],"source_content_type":"text/x-c++src","patch_set":8,"id":"1a430d35_0e197e22","line":415,"updated":"2017-02-07 11:29:15.000000000","message":"Can\u0027t set the status on bulk create?","commit_id":"f67251f33766b013a1e7b8e85ef8077784b22626"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"5a99e41da3f31462ee48ad4a9476c4076bbf1c35","unresolved":false,"context_lines":[{"line_number":412,"context_line":""},{"line_number":413,"context_line":"Error response codes: 400, 401, 403, 404, 409"},{"line_number":414,"context_line":""},{"line_number":415,"context_line":"Request"},{"line_number":416,"context_line":"-------"},{"line_number":417,"context_line":""},{"line_number":418,"context_line":".. rest_parameters:: parameters.yaml"}],"source_content_type":"text/x-c++src","patch_set":8,"id":"1a430d35_132522c0","line":415,"in_reply_to":"1a430d35_0e197e22","updated":"2017-02-07 13:37:32.000000000","message":"No POST. Not even on single port create.","commit_id":"f67251f33766b013a1e7b8e85ef8077784b22626"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"a1cef193b1036b5760644c87540c76a272076e05","unresolved":false,"context_lines":[{"line_number":299,"context_line":"Response Example"},{"line_number":300,"context_line":"----------------"},{"line_number":301,"context_line":""},{"line_number":302,"context_line":".. literalinclude:: samples/ports/ports-list-response.json"},{"line_number":303,"context_line":"   :language: javascript"},{"line_number":304,"context_line":""},{"line_number":305,"context_line":"Response Example (admin user)"}],"source_content_type":"text/x-c++src","patch_set":9,"id":"1a430d35_464b2f59","line":302,"range":{"start_line":302,"start_character":20,"end_line":302,"end_character":58},"updated":"2017-02-08 14:59:13.000000000","message":"Looks like we\u0027re still missing the data plane statue attr in this response: https://github.com/openstack/neutron-lib/blob/master/api-ref/source/v2/samples/ports/ports-list-response.json","commit_id":"9b8f180fdfb18374226cad8e4806972a3b5c4399"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"d13161e87332ad105ed6df30a2d7dbe2be687050","unresolved":false,"context_lines":[{"line_number":299,"context_line":"Response Example"},{"line_number":300,"context_line":"----------------"},{"line_number":301,"context_line":""},{"line_number":302,"context_line":".. literalinclude:: samples/ports/ports-list-response.json"},{"line_number":303,"context_line":"   :language: javascript"},{"line_number":304,"context_line":""},{"line_number":305,"context_line":"Response Example (admin user)"}],"source_content_type":"text/x-c++src","patch_set":9,"id":"1a430d35_cb8723b3","line":302,"range":{"start_line":302,"start_character":20,"end_line":302,"end_character":58},"in_reply_to":"1a430d35_464b2f59","updated":"2017-02-08 16:23:54.000000000","message":"Done","commit_id":"9b8f180fdfb18374226cad8e4806972a3b5c4399"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"a1cef193b1036b5760644c87540c76a272076e05","unresolved":false,"context_lines":[{"line_number":375,"context_line":"   - description: description"},{"line_number":376,"context_line":"   - device_id: device_id"},{"line_number":377,"context_line":"   - device_owner: device_owner"},{"line_number":378,"context_line":"   - extra_dhcp_opts: extra_dhcp_opts"},{"line_number":379,"context_line":"   - fixed_ips: fixed_ips"},{"line_number":380,"context_line":"   - id: id"},{"line_number":381,"context_line":"   - mac_address: mac_address"}],"source_content_type":"text/x-c++src","patch_set":9,"id":"1a430d35_46908f95","line":378,"updated":"2017-02-08 14:59:13.000000000","message":"Don\u0027t we need the data plane status here too?","commit_id":"9b8f180fdfb18374226cad8e4806972a3b5c4399"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"d13161e87332ad105ed6df30a2d7dbe2be687050","unresolved":false,"context_lines":[{"line_number":375,"context_line":"   - description: description"},{"line_number":376,"context_line":"   - device_id: device_id"},{"line_number":377,"context_line":"   - device_owner: device_owner"},{"line_number":378,"context_line":"   - extra_dhcp_opts: extra_dhcp_opts"},{"line_number":379,"context_line":"   - fixed_ips: fixed_ips"},{"line_number":380,"context_line":"   - id: id"},{"line_number":381,"context_line":"   - mac_address: mac_address"}],"source_content_type":"text/x-c++src","patch_set":9,"id":"1a430d35_0b8e8bcf","line":378,"in_reply_to":"1a430d35_46908f95","updated":"2017-02-08 16:23:54.000000000","message":"Done","commit_id":"9b8f180fdfb18374226cad8e4806972a3b5c4399"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"7b230e0f6cedd769de8f2401eaf3b74bd9f7d713","unresolved":false,"context_lines":[{"line_number":145,"context_line":"   - binding:host_id: binding:host_id-request"},{"line_number":146,"context_line":"   - binding:profile: binding:profile-request"},{"line_number":147,"context_line":"   - binding:vnic_type: binding:vnic_type-request"},{"line_number":148,"context_line":"   - data_plane_status: data_plane_status"},{"line_number":149,"context_line":"   - description: description-request"},{"line_number":150,"context_line":"   - device_id: device_id-request"},{"line_number":151,"context_line":"   - device_owner: device_owner-request"}],"source_content_type":"text/x-c++src","patch_set":10,"id":"1a430d35_119ea8b8","line":148,"range":{"start_line":148,"start_character":5,"end_line":148,"end_character":41},"updated":"2017-02-09 00:58:13.000000000","message":"It should be Optional. Could you use \"data_plane_status-request\" as I commented in parameters.yaml?","commit_id":"37ce52196e0c82349c5d28e5fb2de99c0863503a"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"b00afc2d1a52ee3f9870be22bb5aa68036abbb96","unresolved":false,"context_lines":[{"line_number":145,"context_line":"   - binding:host_id: binding:host_id-request"},{"line_number":146,"context_line":"   - binding:profile: binding:profile-request"},{"line_number":147,"context_line":"   - binding:vnic_type: binding:vnic_type-request"},{"line_number":148,"context_line":"   - data_plane_status: data_plane_status"},{"line_number":149,"context_line":"   - description: description-request"},{"line_number":150,"context_line":"   - device_id: device_id-request"},{"line_number":151,"context_line":"   - device_owner: device_owner-request"}],"source_content_type":"text/x-c++src","patch_set":10,"id":"1a430d35_e33144a3","line":148,"range":{"start_line":148,"start_character":5,"end_line":148,"end_character":41},"in_reply_to":"1a430d35_119ea8b8","updated":"2017-02-09 10:42:29.000000000","message":"Eagle eye :-) Done","commit_id":"37ce52196e0c82349c5d28e5fb2de99c0863503a"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"90e33637459a9c8ba161b40f57df3cc75aa3c2fd","unresolved":false,"context_lines":[{"line_number":21,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"The data plane port extension (``data-plane-status``) adds a new attribute"},{"line_number":24,"context_line":"``data_plane_status`` to represent the status of the underlay data plane. This"},{"line_number":25,"context_line":"attribute is to be managed by entities outside of Neutron, while the ``status``"},{"line_number":26,"context_line":"attribute is managed by Neutron. Both status attributes are independent from"},{"line_number":27,"context_line":"one another."}],"source_content_type":"text/x-c++src","patch_set":11,"id":"fa31d9ce_fb08f02a","line":24,"range":{"start_line":24,"start_character":53,"end_line":24,"end_character":61},"updated":"2017-02-10 04:52:11.000000000","message":"Looking at the definition of \u0027underlay\u0027 on-line, it is either a verb or a noun: https://www.merriam-webster.com/dictionary/underlay. In this case, you want to use this word as an adjective applied to data plane. So I recommend using \u0027underlying\u0027, like you do in the rest of this patchset","commit_id":"141f7bb87b04c0ce68ad9f2dfdd6961fa79da062"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"c814a240be77256f98fdd4aa39092a12b7b9f395","unresolved":false,"context_lines":[{"line_number":21,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"The data plane port extension (``data-plane-status``) adds a new attribute"},{"line_number":24,"context_line":"``data_plane_status`` to represent the status of the underlay data plane. This"},{"line_number":25,"context_line":"attribute is to be managed by entities outside of Neutron, while the ``status``"},{"line_number":26,"context_line":"attribute is managed by Neutron. Both status attributes are independent from"},{"line_number":27,"context_line":"one another."}],"source_content_type":"text/x-c++src","patch_set":11,"id":"fa31d9ce_b8fcb03f","line":24,"range":{"start_line":24,"start_character":53,"end_line":24,"end_character":61},"in_reply_to":"fa31d9ce_fb08f02a","updated":"2017-02-10 10:24:43.000000000","message":"Done","commit_id":"141f7bb87b04c0ce68ad9f2dfdd6961fa79da062"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"02f5c3bd321cce9517084d3e7a0e8c5d916d618b","unresolved":false,"context_lines":[{"line_number":23,"context_line":"The data plane port extension (``data-plane-status``) adds a new attribute"},{"line_number":24,"context_line":"``data_plane_status`` to represent the status of the underlying data plane."},{"line_number":25,"context_line":"This attribute is to be managed by entities outside of Neutron, while the"},{"line_number":26,"context_line":"``status`` attribute is managed by Neutron. Both status attributes are"},{"line_number":27,"context_line":"independent from one another."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Supported data plane status values:"}],"source_content_type":"text/x-c++src","patch_set":13,"id":"ba2be162_2318cf74","line":26,"range":{"start_line":26,"start_character":35,"end_line":26,"end_character":42},"updated":"2017-03-05 16:12:21.000000000","message":"We don\u0027t use \"Neutron\u0027 in the API reference. Use \u0027the Networking service\u0027 instead.","commit_id":"16d86ed5e1e67594d5ce5cc60309acbfda3cdef9"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"2d53c0aa2a375e9ff01235cfae30d899119a5073","unresolved":false,"context_lines":[{"line_number":23,"context_line":"The data plane port extension (``data-plane-status``) adds a new attribute"},{"line_number":24,"context_line":"``data_plane_status`` to represent the status of the underlying data plane."},{"line_number":25,"context_line":"This attribute is to be managed by entities outside of Neutron, while the"},{"line_number":26,"context_line":"``status`` attribute is managed by Neutron. Both status attributes are"},{"line_number":27,"context_line":"independent from one another."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Supported data plane status values:"}],"source_content_type":"text/x-c++src","patch_set":13,"id":"ba2be162_bc33fdf4","line":26,"range":{"start_line":26,"start_character":35,"end_line":26,"end_character":42},"in_reply_to":"ba2be162_2318cf74","updated":"2017-03-05 21:12:42.000000000","message":"Done","commit_id":"16d86ed5e1e67594d5ce5cc60309acbfda3cdef9"}],"api-ref/source/v2/samples/extensions/extensions-list-response.json":[{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"cc63c6f36e09d5dfa7b0ab1da86cbad2d3aff3ac","unresolved":false,"context_lines":[{"line_number":125,"context_line":"            \"links\": [],"},{"line_number":126,"context_line":"            \"alias\": \"data-plane-status\","},{"line_number":127,"context_line":"            \"description\": \"Status of the underlying data plane.\""},{"line_number":128,"context_line":"        },"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"    ]"},{"line_number":131,"context_line":"}"}],"source_content_type":"application/json","patch_set":11,"id":"fa31d9ce_54c1d17f","line":128,"range":{"start_line":128,"start_character":9,"end_line":128,"end_character":10},"updated":"2017-02-10 01:48:44.000000000","message":"extra comma?","commit_id":"141f7bb87b04c0ce68ad9f2dfdd6961fa79da062"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"c814a240be77256f98fdd4aa39092a12b7b9f395","unresolved":false,"context_lines":[{"line_number":125,"context_line":"            \"links\": [],"},{"line_number":126,"context_line":"            \"alias\": \"data-plane-status\","},{"line_number":127,"context_line":"            \"description\": \"Status of the underlying data plane.\""},{"line_number":128,"context_line":"        },"},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"    ]"},{"line_number":131,"context_line":"}"}],"source_content_type":"application/json","patch_set":11,"id":"fa31d9ce_38b6008b","line":128,"range":{"start_line":128,"start_character":9,"end_line":128,"end_character":10},"in_reply_to":"fa31d9ce_54c1d17f","updated":"2017-02-10 10:24:43.000000000","message":"Done","commit_id":"141f7bb87b04c0ce68ad9f2dfdd6961fa79da062"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"fcc683a778dd8822638b656a2ef2b25056dcf267","unresolved":false,"context_lines":[{"line_number":121,"context_line":"        },"},{"line_number":122,"context_line":"        {"},{"line_number":123,"context_line":"            \"updated\": \"2016-01-24T10:00:00-00:00\","},{"line_number":124,"context_line":"            \"name\": \"Neutron Port Data Plane Status\","},{"line_number":125,"context_line":"            \"links\": [],"},{"line_number":126,"context_line":"            \"alias\": \"data-plane-status\","},{"line_number":127,"context_line":"            \"description\": \"Status of the underlying data plane.\""}],"source_content_type":"application/json","patch_set":12,"id":"fa31d9ce_775dd3e8","line":124,"updated":"2017-02-10 17:31:53.000000000","message":"This doesn\u0027t match https://review.openstack.org/#/c/424868/12/neutron_lib/api/definitions/data_plane_status.py@37","commit_id":"0f9aaefb17c1999234cf05f11a3539d97ab0e966"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"fcc683a778dd8822638b656a2ef2b25056dcf267","unresolved":false,"context_lines":[{"line_number":124,"context_line":"            \"name\": \"Neutron Port Data Plane Status\","},{"line_number":125,"context_line":"            \"links\": [],"},{"line_number":126,"context_line":"            \"alias\": \"data-plane-status\","},{"line_number":127,"context_line":"            \"description\": \"Status of the underlying data plane.\""},{"line_number":128,"context_line":"        }"},{"line_number":129,"context_line":"    ]"},{"line_number":130,"context_line":"}"}],"source_content_type":"application/json","patch_set":12,"id":"fa31d9ce_f71703eb","line":127,"updated":"2017-02-10 17:31:53.000000000","message":"This doesn\u0027t match https://review.openstack.org/#/c/424868/12/neutron_lib/api/definitions/data_plane_status.py@40","commit_id":"0f9aaefb17c1999234cf05f11a3539d97ab0e966"}],"api-ref/source/v2/samples/ports/port-bind-create-response.json":[{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"eb69ae3ddd5345110c44d7dedf7bf1b59bea1c60","unresolved":false,"context_lines":[{"line_number":15,"context_line":"        \"binding:vif_details\": {},"},{"line_number":16,"context_line":"        \"binding:vif_type\": \"unbound\","},{"line_number":17,"context_line":"        \"binding:vnic_type\": \"other\","},{"line_number":18,"context_line":"        \"data_plane_status\": \"ACTIVE\","},{"line_number":19,"context_line":"        \"description\": \"\","},{"line_number":20,"context_line":"        \"device_id\": \"d90a13da-be41-461f-9f99-1dbcf438fdf2\","},{"line_number":21,"context_line":"        \"device_owner\": \"baremetal:none\","}],"source_content_type":"application/json","patch_set":11,"id":"fa31d9ce_ff5b832c","line":18,"updated":"2017-02-10 00:32:06.000000000","message":"Per https://review.openstack.org/#/c/424868/11/api-ref/source/v2/ports.inc@31, isn\u0027t the default value of this attribute null? And the only way to set it is a put, if I understand correctly. If both of the premises is true, I think it is inconsistent that we show an ACTIVE value right after the port has been created. It should be null. Am I missing something?","commit_id":"141f7bb87b04c0ce68ad9f2dfdd6961fa79da062"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"c814a240be77256f98fdd4aa39092a12b7b9f395","unresolved":false,"context_lines":[{"line_number":15,"context_line":"        \"binding:vif_details\": {},"},{"line_number":16,"context_line":"        \"binding:vif_type\": \"unbound\","},{"line_number":17,"context_line":"        \"binding:vnic_type\": \"other\","},{"line_number":18,"context_line":"        \"data_plane_status\": \"ACTIVE\","},{"line_number":19,"context_line":"        \"description\": \"\","},{"line_number":20,"context_line":"        \"device_id\": \"d90a13da-be41-461f-9f99-1dbcf438fdf2\","},{"line_number":21,"context_line":"        \"device_owner\": \"baremetal:none\","}],"source_content_type":"application/json","patch_set":11,"id":"fa31d9ce_3309c12d","line":18,"in_reply_to":"fa31d9ce_b4e84d18","updated":"2017-02-10 10:24:43.000000000","message":"Done","commit_id":"141f7bb87b04c0ce68ad9f2dfdd6961fa79da062"},{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"cc63c6f36e09d5dfa7b0ab1da86cbad2d3aff3ac","unresolved":false,"context_lines":[{"line_number":15,"context_line":"        \"binding:vif_details\": {},"},{"line_number":16,"context_line":"        \"binding:vif_type\": \"unbound\","},{"line_number":17,"context_line":"        \"binding:vnic_type\": \"other\","},{"line_number":18,"context_line":"        \"data_plane_status\": \"ACTIVE\","},{"line_number":19,"context_line":"        \"description\": \"\","},{"line_number":20,"context_line":"        \"device_id\": \"d90a13da-be41-461f-9f99-1dbcf438fdf2\","},{"line_number":21,"context_line":"        \"device_owner\": \"baremetal:none\","}],"source_content_type":"application/json","patch_set":11,"id":"fa31d9ce_b4e84d18","line":18,"in_reply_to":"fa31d9ce_ff5b832c","updated":"2017-02-10 01:48:44.000000000","message":"+1","commit_id":"141f7bb87b04c0ce68ad9f2dfdd6961fa79da062"}],"api-ref/source/v2/samples/ports/port-bind-update-request.json":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"02f5c3bd321cce9517084d3e7a0e8c5d916d618b","unresolved":false,"context_lines":[{"line_number":3,"context_line":"        \"binding:host_id\": \"test_for_port_update_host\","},{"line_number":4,"context_line":"        \"device_id\": \"d90a13da-be41-461f-9f99-1dbcf438fdf2\","},{"line_number":5,"context_line":"        \"device_owner\": \"compute:nova\","},{"line_number":6,"context_line":"        \"data_plane_status\": \"DOWN\""},{"line_number":7,"context_line":"    }"},{"line_number":8,"context_line":"}"}],"source_content_type":"application/json","patch_set":13,"id":"ba2be162_632c8799","line":6,"updated":"2017-03-05 16:12:21.000000000","message":"Alphabetical order is preferred.","commit_id":"16d86ed5e1e67594d5ce5cc60309acbfda3cdef9"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"2d53c0aa2a375e9ff01235cfae30d899119a5073","unresolved":false,"context_lines":[{"line_number":3,"context_line":"        \"binding:host_id\": \"test_for_port_update_host\","},{"line_number":4,"context_line":"        \"device_id\": \"d90a13da-be41-461f-9f99-1dbcf438fdf2\","},{"line_number":5,"context_line":"        \"device_owner\": \"compute:nova\","},{"line_number":6,"context_line":"        \"data_plane_status\": \"DOWN\""},{"line_number":7,"context_line":"    }"},{"line_number":8,"context_line":"}"}],"source_content_type":"application/json","patch_set":13,"id":"ba2be162_5c592134","line":6,"in_reply_to":"ba2be162_632c8799","updated":"2017-03-05 21:12:42.000000000","message":"Done","commit_id":"16d86ed5e1e67594d5ce5cc60309acbfda3cdef9"}],"api-ref/source/v2/samples/ports/port-create-response.json":[{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"0516492389622ac1f654acc94abdbcf72006999c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"        \"allowed_address_pairs\": [],"},{"line_number":5,"context_line":"        \"created_at\": \"2016-03-08T20:19:41\","},{"line_number":6,"context_line":"        \"description\": \"\","},{"line_number":7,"context_line":"        \"data_plane_status\": \"None\","},{"line_number":8,"context_line":"        \"device_id\": \"\","},{"line_number":9,"context_line":"        \"device_owner\": \"\","},{"line_number":10,"context_line":"        \"extra_dhcp_opts\": [],"}],"source_content_type":"application/json","patch_set":8,"id":"1a430d35_0e541ec8","line":7,"range":{"start_line":7,"start_character":29,"end_line":7,"end_character":35},"updated":"2017-02-07 11:29:15.000000000","message":"Shouldn\u0027t this just be null and not the string \"None\"?","commit_id":"f67251f33766b013a1e7b8e85ef8077784b22626"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"5a99e41da3f31462ee48ad4a9476c4076bbf1c35","unresolved":false,"context_lines":[{"line_number":4,"context_line":"        \"allowed_address_pairs\": [],"},{"line_number":5,"context_line":"        \"created_at\": \"2016-03-08T20:19:41\","},{"line_number":6,"context_line":"        \"description\": \"\","},{"line_number":7,"context_line":"        \"data_plane_status\": \"None\","},{"line_number":8,"context_line":"        \"device_id\": \"\","},{"line_number":9,"context_line":"        \"device_owner\": \"\","},{"line_number":10,"context_line":"        \"extra_dhcp_opts\": [],"}],"source_content_type":"application/json","patch_set":8,"id":"1a430d35_332e9ea5","line":7,"range":{"start_line":7,"start_character":29,"end_line":7,"end_character":35},"in_reply_to":"1a430d35_0e541ec8","updated":"2017-02-07 13:37:32.000000000","message":"Done","commit_id":"f67251f33766b013a1e7b8e85ef8077784b22626"},{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"cc63c6f36e09d5dfa7b0ab1da86cbad2d3aff3ac","unresolved":false,"context_lines":[{"line_number":4,"context_line":"        \"allowed_address_pairs\": [],"},{"line_number":5,"context_line":"        \"created_at\": \"2016-03-08T20:19:41\","},{"line_number":6,"context_line":"        \"description\": \"\","},{"line_number":7,"context_line":"        \"data_plane_status\": null,"},{"line_number":8,"context_line":"        \"device_id\": \"\","},{"line_number":9,"context_line":"        \"device_owner\": \"\","},{"line_number":10,"context_line":"        \"extra_dhcp_opts\": [],"}],"source_content_type":"application/json","patch_set":11,"id":"fa31d9ce_d4786192","line":7,"updated":"2017-02-10 01:48:44.000000000","message":"probably it\u0027s better to keep alphabetical order?","commit_id":"141f7bb87b04c0ce68ad9f2dfdd6961fa79da062"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"c814a240be77256f98fdd4aa39092a12b7b9f395","unresolved":false,"context_lines":[{"line_number":4,"context_line":"        \"allowed_address_pairs\": [],"},{"line_number":5,"context_line":"        \"created_at\": \"2016-03-08T20:19:41\","},{"line_number":6,"context_line":"        \"description\": \"\","},{"line_number":7,"context_line":"        \"data_plane_status\": null,"},{"line_number":8,"context_line":"        \"device_id\": \"\","},{"line_number":9,"context_line":"        \"device_owner\": \"\","},{"line_number":10,"context_line":"        \"extra_dhcp_opts\": [],"}],"source_content_type":"application/json","patch_set":11,"id":"fa31d9ce_15366503","line":7,"in_reply_to":"fa31d9ce_d4786192","updated":"2017-02-10 10:24:43.000000000","message":"Done","commit_id":"141f7bb87b04c0ce68ad9f2dfdd6961fa79da062"}],"api-ref/source/v2/samples/ports/ports-bind-list-response.json":[{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"0516492389622ac1f654acc94abdbcf72006999c","unresolved":false,"context_lines":[{"line_number":11,"context_line":"            },"},{"line_number":12,"context_line":"            \"binding:vif_type\": \"ovs\","},{"line_number":13,"context_line":"            \"binding:vnic_type\": \"normal\","},{"line_number":14,"context_line":"            \"data_plane_status\": \"None\","},{"line_number":15,"context_line":"            \"description\": \"\","},{"line_number":16,"context_line":"            \"device_id\": \"9ae135f4-b6e0-4dad-9e91-3c223e385824\","},{"line_number":17,"context_line":"            \"device_owner\": \"network:router_gateway\","}],"source_content_type":"application/json","patch_set":8,"id":"1a430d35_4e60e6a5","line":14,"range":{"start_line":14,"start_character":34,"end_line":14,"end_character":38},"updated":"2017-02-07 11:29:15.000000000","message":"Ditto","commit_id":"f67251f33766b013a1e7b8e85ef8077784b22626"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"5a99e41da3f31462ee48ad4a9476c4076bbf1c35","unresolved":false,"context_lines":[{"line_number":11,"context_line":"            },"},{"line_number":12,"context_line":"            \"binding:vif_type\": \"ovs\","},{"line_number":13,"context_line":"            \"binding:vnic_type\": \"normal\","},{"line_number":14,"context_line":"            \"data_plane_status\": \"None\","},{"line_number":15,"context_line":"            \"description\": \"\","},{"line_number":16,"context_line":"            \"device_id\": \"9ae135f4-b6e0-4dad-9e91-3c223e385824\","},{"line_number":17,"context_line":"            \"device_owner\": \"network:router_gateway\","}],"source_content_type":"application/json","patch_set":8,"id":"1a430d35_731ae67e","line":14,"range":{"start_line":14,"start_character":34,"end_line":14,"end_character":38},"in_reply_to":"1a430d35_4e60e6a5","updated":"2017-02-07 13:37:32.000000000","message":"Done","commit_id":"f67251f33766b013a1e7b8e85ef8077784b22626"}],"api-ref/source/v2/samples/ports/ports-bulk-create-response.json":[{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"0516492389622ac1f654acc94abdbcf72006999c","unresolved":false,"context_lines":[{"line_number":3,"context_line":"        {"},{"line_number":4,"context_line":"            \"admin_state_up\": false,"},{"line_number":5,"context_line":"            \"allowed_address_pairs\": [],"},{"line_number":6,"context_line":"            \"data_plane_status\": \"None\","},{"line_number":7,"context_line":"            \"description\": \"\","},{"line_number":8,"context_line":"            \"device_id\": \"\","},{"line_number":9,"context_line":"            \"device_owner\": \"\","}],"source_content_type":"application/json","patch_set":8,"id":"1a430d35_8e0d6eea","line":6,"range":{"start_line":6,"start_character":34,"end_line":6,"end_character":38},"updated":"2017-02-07 11:29:15.000000000","message":"ditto","commit_id":"f67251f33766b013a1e7b8e85ef8077784b22626"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"5a99e41da3f31462ee48ad4a9476c4076bbf1c35","unresolved":false,"context_lines":[{"line_number":3,"context_line":"        {"},{"line_number":4,"context_line":"            \"admin_state_up\": false,"},{"line_number":5,"context_line":"            \"allowed_address_pairs\": [],"},{"line_number":6,"context_line":"            \"data_plane_status\": \"None\","},{"line_number":7,"context_line":"            \"description\": \"\","},{"line_number":8,"context_line":"            \"device_id\": \"\","},{"line_number":9,"context_line":"            \"device_owner\": \"\","}],"source_content_type":"application/json","patch_set":8,"id":"1a430d35_d3155a49","line":6,"range":{"start_line":6,"start_character":34,"end_line":6,"end_character":38},"in_reply_to":"1a430d35_8e0d6eea","updated":"2017-02-07 13:37:32.000000000","message":"Done","commit_id":"f67251f33766b013a1e7b8e85ef8077784b22626"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"0516492389622ac1f654acc94abdbcf72006999c","unresolved":false,"context_lines":[{"line_number":27,"context_line":"        {"},{"line_number":28,"context_line":"            \"admin_state_up\": false,"},{"line_number":29,"context_line":"            \"allowed_address_pairs\": [],"},{"line_number":30,"context_line":"            \"data_plane_status\": \"None\","},{"line_number":31,"context_line":"            \"description\": \"\","},{"line_number":32,"context_line":"            \"device_id\": \"\","},{"line_number":33,"context_line":"            \"device_owner\": \"\","}],"source_content_type":"application/json","patch_set":8,"id":"1a430d35_ae122a07","line":30,"range":{"start_line":30,"start_character":34,"end_line":30,"end_character":38},"updated":"2017-02-07 11:29:15.000000000","message":"ditto","commit_id":"f67251f33766b013a1e7b8e85ef8077784b22626"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"5a99e41da3f31462ee48ad4a9476c4076bbf1c35","unresolved":false,"context_lines":[{"line_number":27,"context_line":"        {"},{"line_number":28,"context_line":"            \"admin_state_up\": false,"},{"line_number":29,"context_line":"            \"allowed_address_pairs\": [],"},{"line_number":30,"context_line":"            \"data_plane_status\": \"None\","},{"line_number":31,"context_line":"            \"description\": \"\","},{"line_number":32,"context_line":"            \"device_id\": \"\","},{"line_number":33,"context_line":"            \"device_owner\": \"\","}],"source_content_type":"application/json","patch_set":8,"id":"1a430d35_130c4240","line":30,"range":{"start_line":30,"start_character":34,"end_line":30,"end_character":38},"in_reply_to":"1a430d35_ae122a07","updated":"2017-02-07 13:37:32.000000000","message":"Done","commit_id":"f67251f33766b013a1e7b8e85ef8077784b22626"}],"neutron_lib/api/definitions/data_plane_status.py":[{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"1b510236f9ac31cd755bf8a77ea1b7b0f6511c5d","unresolved":false,"context_lines":[{"line_number":55,"context_line":"# The resource attribute map for the extension."},{"line_number":56,"context_line":"RESOURCE_ATTRIBUTE_MAP \u003d {"},{"line_number":57,"context_line":"    COLLECTION_NAME: {"},{"line_number":58,"context_line":"        DATA_PLANE_STATUS: {\u0027allow_post\u0027: False, \u0027allow_put\u0027: True,"},{"line_number":59,"context_line":"                            \u0027default\u0027: constants.ATTR_NOT_SPECIFIED,"},{"line_number":60,"context_line":"                            \u0027validate\u0027: {\u0027type:values\u0027: DPS_SUPPORTED},"},{"line_number":61,"context_line":"                            \u0027is_visible\u0027: True,"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a3905b3_37ed1c8f","line":58,"range":{"start_line":58,"start_character":28,"end_line":58,"end_character":67},"updated":"2017-01-25 09:17:09.000000000","message":"So you cannot create it but update it....??","commit_id":"6630159cd7fd2e7542838b55b8842e9e81c69ddf"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"e93d6b949e75199f957adc4f94660b58f6ed261d","unresolved":false,"context_lines":[{"line_number":55,"context_line":"# The resource attribute map for the extension."},{"line_number":56,"context_line":"RESOURCE_ATTRIBUTE_MAP \u003d {"},{"line_number":57,"context_line":"    COLLECTION_NAME: {"},{"line_number":58,"context_line":"        DATA_PLANE_STATUS: {\u0027allow_post\u0027: False, \u0027allow_put\u0027: True,"},{"line_number":59,"context_line":"                            \u0027default\u0027: constants.ATTR_NOT_SPECIFIED,"},{"line_number":60,"context_line":"                            \u0027validate\u0027: {\u0027type:values\u0027: DPS_SUPPORTED},"},{"line_number":61,"context_line":"                            \u0027is_visible\u0027: True,"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a3905b3_65860a67","line":58,"range":{"start_line":58,"start_character":28,"end_line":58,"end_character":67},"in_reply_to":"5a3905b3_37ed1c8f","updated":"2017-01-25 10:23:12.000000000","message":"Port create will default it to None. At port creation we actually cannot really tell what\u0027s the status of the underlying data plane for the port.","commit_id":"6630159cd7fd2e7542838b55b8842e9e81c69ddf"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"7757ece0816346e829949f13763686419983d66b","unresolved":false,"context_lines":[{"line_number":15,"context_line":"from neutron_lib import constants"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"DATA_PLANE_STATUS_ACTIVE \u003d \"ACTIVE\""},{"line_number":19,"context_line":"DATA_PLANE_STATUS_DOWN \u003d \"DOWN\""},{"line_number":20,"context_line":"DPS_SUPPORTED \u003d [None, DATA_PLANE_STATUS_ACTIVE, DATA_PLANE_STATUS_DOWN]"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"# The alias of the extension."}],"source_content_type":"text/x-python","patch_set":7,"id":"3a461143_5679c892","line":19,"range":{"start_line":18,"start_character":0,"end_line":19,"end_character":31},"updated":"2017-01-26 01:51:02.000000000","message":"It\u0027s better to move them into constants.py","commit_id":"a8779543310a4a1ce30c78529950a821f38f74dc"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"2eae2f15ccc5b152a8227d0a8d15a1f082beb592","unresolved":false,"context_lines":[{"line_number":15,"context_line":"from neutron_lib import constants"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"DATA_PLANE_STATUS_ACTIVE \u003d \"ACTIVE\""},{"line_number":19,"context_line":"DATA_PLANE_STATUS_DOWN \u003d \"DOWN\""},{"line_number":20,"context_line":"DPS_SUPPORTED \u003d [None, DATA_PLANE_STATUS_ACTIVE, DATA_PLANE_STATUS_DOWN]"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"# The alias of the extension."}],"source_content_type":"text/x-python","patch_set":7,"id":"3a461143_9cc42693","line":19,"range":{"start_line":18,"start_character":0,"end_line":19,"end_character":31},"in_reply_to":"3a461143_5679c892","updated":"2017-01-26 11:00:35.000000000","message":"Done","commit_id":"a8779543310a4a1ce30c78529950a821f38f74dc"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"02f5c3bd321cce9517084d3e7a0e8c5d916d618b","unresolved":false,"context_lines":[{"line_number":15,"context_line":"from neutron_lib import constants"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"DPS_SUPPORTED \u003d [None, constants.DATA_PLANE_STATUS_ACTIVE,"},{"line_number":19,"context_line":"                 constants.DATA_PLANE_STATUS_DOWN]"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"# The alias of the extension."}],"source_content_type":"text/x-python","patch_set":13,"id":"ba2be162_c33d33c5","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":13},"updated":"2017-03-05 16:12:21.000000000","message":"I think \"DPS\" is \"data plane status\" but it is not clear.\nThis is the definition of data_plane_status API, so I think \"SUPPORTED_VALUES\" or \"VALID_VALUES\" looks better.","commit_id":"16d86ed5e1e67594d5ce5cc60309acbfda3cdef9"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"2d53c0aa2a375e9ff01235cfae30d899119a5073","unresolved":false,"context_lines":[{"line_number":15,"context_line":"from neutron_lib import constants"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"DPS_SUPPORTED \u003d [None, constants.DATA_PLANE_STATUS_ACTIVE,"},{"line_number":19,"context_line":"                 constants.DATA_PLANE_STATUS_DOWN]"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"# The alias of the extension."}],"source_content_type":"text/x-python","patch_set":13,"id":"ba2be162_9c3f19b3","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":13},"in_reply_to":"ba2be162_c33d33c5","updated":"2017-03-05 21:12:42.000000000","message":"Done","commit_id":"16d86ed5e1e67594d5ce5cc60309acbfda3cdef9"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"215dd81f4c1133d43902199e9dd39300a2586365","unresolved":false,"context_lines":[{"line_number":15,"context_line":"from neutron_lib import constants"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"VALID_VALUES \u003d [None, constants.DATA_PLANE_STATUS_ACTIVE,"},{"line_number":19,"context_line":"                constants.DATA_PLANE_STATUS_DOWN]"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"# The alias of the extension."}],"source_content_type":"text/x-python","patch_set":14,"id":"ba2be162_681603f3","line":18,"updated":"2017-03-06 16:18:42.000000000","message":"See my comments on constants.py","commit_id":"313ebac4abd60afdc43a88d387666f3b7ba275d8"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"94b33d8543e688b7bd0bdda3177b0c0eb9447d57","unresolved":false,"context_lines":[{"line_number":15,"context_line":"from neutron_lib import constants"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"VALID_VALUES \u003d [None, constants.DATA_PLANE_STATUS_ACTIVE,"},{"line_number":19,"context_line":"                constants.DATA_PLANE_STATUS_DOWN]"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"# The alias of the extension."}],"source_content_type":"text/x-python","patch_set":14,"id":"9a30ddce_48bca06c","line":18,"in_reply_to":"ba2be162_681603f3","updated":"2017-03-07 23:09:23.000000000","message":"Done","commit_id":"313ebac4abd60afdc43a88d387666f3b7ba275d8"}],"neutron_lib/constants.py":[{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"215dd81f4c1133d43902199e9dd39300a2586365","unresolved":false,"context_lines":[{"line_number":30,"context_line":"FLOATINGIP_STATUS_DOWN \u003d \u0027DOWN\u0027"},{"line_number":31,"context_line":"FLOATINGIP_STATUS_ERROR \u003d \u0027ERROR\u0027"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"DATA_PLANE_STATUS_ACTIVE \u003d \"ACTIVE\""},{"line_number":34,"context_line":"DATA_PLANE_STATUS_DOWN \u003d \"DOWN\""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"# Service operation status constants"}],"source_content_type":"text/x-python","patch_set":14,"id":"ba2be162_a82f1b3d","line":33,"updated":"2017-03-06 16:18:42.000000000","message":"I\u0027m not sure why we keep redefining the same constants over and over. I\u0027d rather see us just use ACTIVE and DOWN from this module.","commit_id":"313ebac4abd60afdc43a88d387666f3b7ba275d8"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"116a2e8dc9b5df3899fb4741a0b7813122b2c4f0","unresolved":false,"context_lines":[{"line_number":30,"context_line":"FLOATINGIP_STATUS_DOWN \u003d \u0027DOWN\u0027"},{"line_number":31,"context_line":"FLOATINGIP_STATUS_ERROR \u003d \u0027ERROR\u0027"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"DATA_PLANE_STATUS_ACTIVE \u003d \"ACTIVE\""},{"line_number":34,"context_line":"DATA_PLANE_STATUS_DOWN \u003d \"DOWN\""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"# Service operation status constants"}],"source_content_type":"text/x-python","patch_set":14,"id":"9a30ddce_0471a2e0","line":33,"in_reply_to":"9a30ddce_7e3d95ba","updated":"2017-03-08 10:28:49.000000000","message":"I\u0027ve updated the code in patch set 15 to use existing ACTIVE and DOWN constants.","commit_id":"313ebac4abd60afdc43a88d387666f3b7ba275d8"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"2e0c00163db0fcca262a17da658a3a4f250f7822","unresolved":false,"context_lines":[{"line_number":30,"context_line":"FLOATINGIP_STATUS_DOWN \u003d \u0027DOWN\u0027"},{"line_number":31,"context_line":"FLOATINGIP_STATUS_ERROR \u003d \u0027ERROR\u0027"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"DATA_PLANE_STATUS_ACTIVE \u003d \"ACTIVE\""},{"line_number":34,"context_line":"DATA_PLANE_STATUS_DOWN \u003d \"DOWN\""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"# Service operation status constants"}],"source_content_type":"text/x-python","patch_set":14,"id":"9a30ddce_7e3d95ba","line":33,"in_reply_to":"9a30ddce_88d138d0","updated":"2017-03-08 02:35:58.000000000","message":"Valid question. I wonder why we have dedicated status definitions for network, port and floating IP. I don\u0027t think we need dedicated definitions and agree to use ACTIVE and DOWN below.","commit_id":"313ebac4abd60afdc43a88d387666f3b7ba275d8"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"94b33d8543e688b7bd0bdda3177b0c0eb9447d57","unresolved":false,"context_lines":[{"line_number":30,"context_line":"FLOATINGIP_STATUS_DOWN \u003d \u0027DOWN\u0027"},{"line_number":31,"context_line":"FLOATINGIP_STATUS_ERROR \u003d \u0027ERROR\u0027"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"DATA_PLANE_STATUS_ACTIVE \u003d \"ACTIVE\""},{"line_number":34,"context_line":"DATA_PLANE_STATUS_DOWN \u003d \"DOWN\""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"# Service operation status constants"}],"source_content_type":"text/x-python","patch_set":14,"id":"9a30ddce_88d138d0","line":33,"in_reply_to":"ba2be162_a82f1b3d","updated":"2017-03-07 23:09:23.000000000","message":"That\u0027s a valid question, indeed. Reason why I redefined was because there\u0027s also specific constants for port status in addition to constants ACTIVE (L37) and DOWN (L38) so it could be for historical reasons. I can reuse ACTIVE and DOWN, unless someone argues otherwise preferring keeping as is.","commit_id":"313ebac4abd60afdc43a88d387666f3b7ba275d8"}],"releasenotes/notes/data-plane-status-ext-c3452a01ef5007ce.yaml":[{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"eb69ae3ddd5345110c44d7dedf7bf1b59bea1c60","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - API definition and reference for the port data plane status extension."}],"source_content_type":"text/x-yaml","patch_set":11,"id":"fa31d9ce_5fe72f40","line":3,"range":{"start_line":3,"start_character":23,"end_line":3,"end_character":32},"updated":"2017-02-10 00:32:06.000000000","message":"reference documentation","commit_id":"141f7bb87b04c0ce68ad9f2dfdd6961fa79da062"},{"author":{"_account_id":6469,"name":"Carlos Gonçalves","display_name":"Carlos Goncalves","email":"cgoncalves@redhat.com","username":"cgoncalves"},"change_message_id":"c814a240be77256f98fdd4aa39092a12b7b9f395","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - API definition and reference for the port data plane status extension."}],"source_content_type":"text/x-yaml","patch_set":11,"id":"fa31d9ce_38444073","line":3,"range":{"start_line":3,"start_character":23,"end_line":3,"end_character":32},"in_reply_to":"fa31d9ce_5fe72f40","updated":"2017-02-10 10:24:43.000000000","message":"Done","commit_id":"141f7bb87b04c0ce68ad9f2dfdd6961fa79da062"}]}