)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"82074b91d38fdb60a5dabea04c84fe552eda6cea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"ff346bd7_da10fd02","updated":"2017-07-27 21:23:20.000000000","message":"Please look at the \u0027Summary of Git commit message structure\u0027 section here: https://wiki.openstack.org/wiki/GitCommitMessages. It explains how you have to format your commit messages. Please note that the first line should be 50 characters long or less. The rest of the lines should have 72 characters at most.","commit_id":"8e2b3023503bc574df9aa23840bfb05c2fff7638"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"0245509b38cbaa5ebc505462eeb21c5dd29fd6e1","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix API Doc LBaaS v2"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When a Listener is created, the parameter protocol_port is optional according to the official documentation, but in really it is required."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"If a pool is created, the parameter protocol is optional according to the official documentation, but in really it is a required parameter."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"ff346bd7_79c494f5","line":9,"updated":"2017-07-27 13:27:23.000000000","message":"Can you please shorten these lines","commit_id":"8e2b3023503bc574df9aa23840bfb05c2fff7638"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"82074b91d38fdb60a5dabea04c84fe552eda6cea","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix API Doc LBaaS v2"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When a Listener is created, the parameter protocol_port is optional according to the official documentation, but in really it is required."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"If a pool is created, the parameter protocol is optional according to the official documentation, but in really it is a required parameter."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"ff346bd7_ba9ad9b5","line":9,"range":{"start_line":9,"start_character":116,"end_line":9,"end_character":122},"updated":"2017-07-27 21:23:20.000000000","message":"reality","commit_id":"8e2b3023503bc574df9aa23840bfb05c2fff7638"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"0245509b38cbaa5ebc505462eeb21c5dd29fd6e1","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When a Listener is created, the parameter protocol_port is optional according to the official documentation, but in really it is required."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"If a pool is created, the parameter protocol is optional according to the official documentation, but in really it is a required parameter."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"The API calls which use ‘health_monitors’ string do not properly work and the right one is ‘healthmonitors’. This string should also be modified within the body."},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"ff346bd7_59b5507d","line":11,"updated":"2017-07-27 13:27:23.000000000","message":"How is this diff from the first claim above?","commit_id":"8e2b3023503bc574df9aa23840bfb05c2fff7638"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"782117afdca0652a271fc75a3307e6bd6035ba4e","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix API Doc LBaaS v2"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    When a Listener and pool are created,"},{"line_number":10,"context_line":"    the parameter protocol_port is optional,"},{"line_number":11,"context_line":"    according to the official documentation,"},{"line_number":12,"context_line":"    but in reality it is required."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"df3967d1_f27d42b2","line":9,"updated":"2017-08-03 16:59:42.000000000","message":"These should be wrapped at 72 chars, please see [1] in the \"Summary of Git commit message structure\" section and they don\u0027t need to be indented.\n\n\n[1] https://wiki.openstack.org/wiki/GitCommitMessages","commit_id":"9993dbc51f6a5afb6d6fd88403dfa74745d52661"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"1d7c3159b122ba25c48a37b9f4bf24f24cca6ffe","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix API Doc LBaaS v2"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    When a Listener and pool are created"},{"line_number":10,"context_line":"    the parameter protocol_port is optional"},{"line_number":11,"context_line":"    according to the official documentation,"},{"line_number":12,"context_line":"    but in reality it is required."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"bf3e73b7_afd599bb","line":9,"updated":"2017-08-07 13:16:30.000000000","message":"These lines can start at the 1st character in the line and wrap at 72 chars. For example, please see the commit message here: https://review.openstack.org/#/c/490613/","commit_id":"57dfda6b3c696a614d923eac9ab6d7ce52ec20fa"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"67219ca5bbaa88966a259008567b5bba4c9697ac","unresolved":false,"context_lines":[{"line_number":13,"context_line":"The API calls which use ‘health_monitors’"},{"line_number":14,"context_line":"string do not properly work and the right one is ‘healthmonitors’."},{"line_number":15,"context_line":"This string should also be modified within the body."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I6b97678c872fbb5798f0295dd262a8f6fc6c5a84"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":11,"id":"9f436f4f_e3a063a8","line":16,"updated":"2017-08-08 19:59:07.000000000","message":"Can you please use the closes bug tag here as explained in the \"Including external references\" section of [1]?\n\nThis allows the patchset to be linked to the bug; otherwise its not easy to correlate the two.\n\n\n[1] https://wiki.openstack.org/wiki/GitCommitMessages","commit_id":"2b61eca13cbc8e88205daaa4222f4c322bbc9f2c"}],"api-ref/source/v2/lbaas-v2.inc":[{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"0245509b38cbaa5ebc505462eeb21c5dd29fd6e1","unresolved":false,"context_lines":[{"line_number":1472,"context_line":""},{"line_number":1473,"context_line":".. rest_parameters:: parameters.yaml"},{"line_number":1474,"context_line":""},{"line_number":1475,"context_line":"   - health_monitor: health_monitor"},{"line_number":1476,"context_line":"   - admin_state_up: admin_state_up"},{"line_number":1477,"context_line":"   - delay: delay"},{"line_number":1478,"context_line":"   - expected_codes: expected_codes"}],"source_content_type":"text/x-c++src","patch_set":5,"id":"ff346bd7_39160c57","line":1475,"range":{"start_line":1475,"start_character":5,"end_line":1475,"end_character":19},"updated":"2017-07-27 13:27:23.000000000","message":"shouldn\u0027t this also be \u0027healthmonitor\u0027 to match the other changes?","commit_id":"8e2b3023503bc574df9aa23840bfb05c2fff7638"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"0245509b38cbaa5ebc505462eeb21c5dd29fd6e1","unresolved":false,"context_lines":[{"line_number":1496,"context_line":""},{"line_number":1497,"context_line":".. rest_parameters:: parameters.yaml"},{"line_number":1498,"context_line":""},{"line_number":1499,"context_line":"   - health_monitor: health_monitor"},{"line_number":1500,"context_line":"   - admin_state_up: admin_state_up"},{"line_number":1501,"context_line":"   - delay: delay"},{"line_number":1502,"context_line":"   - expected_codes: expected_codes-response"}],"source_content_type":"text/x-c++src","patch_set":5,"id":"ff346bd7_f90be426","line":1499,"range":{"start_line":1499,"start_character":5,"end_line":1499,"end_character":19},"updated":"2017-07-27 13:27:23.000000000","message":"ditto","commit_id":"8e2b3023503bc574df9aa23840bfb05c2fff7638"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"0245509b38cbaa5ebc505462eeb21c5dd29fd6e1","unresolved":false,"context_lines":[{"line_number":1546,"context_line":""},{"line_number":1547,"context_line":".. rest_parameters:: parameters.yaml"},{"line_number":1548,"context_line":""},{"line_number":1549,"context_line":"   - health_monitor: health_monitor"},{"line_number":1550,"context_line":"   - admin_state_up: admin_state_up"},{"line_number":1551,"context_line":"   - delay: delay"},{"line_number":1552,"context_line":"   - expected_codes: expected_codes-response"}],"source_content_type":"text/x-c++src","patch_set":5,"id":"ff346bd7_5406351e","line":1549,"range":{"start_line":1549,"start_character":21,"end_line":1549,"end_character":35},"updated":"2017-07-27 13:27:23.000000000","message":"ditto.","commit_id":"8e2b3023503bc574df9aa23840bfb05c2fff7638"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"0245509b38cbaa5ebc505462eeb21c5dd29fd6e1","unresolved":false,"context_lines":[{"line_number":1591,"context_line":".. rest_parameters:: parameters.yaml"},{"line_number":1592,"context_line":""},{"line_number":1593,"context_line":"   - health_monitor_id: health_monitor_id-path"},{"line_number":1594,"context_line":"   - health_monitor: health_monitor"},{"line_number":1595,"context_line":"   - admin_state_up: admin_state_up"},{"line_number":1596,"context_line":"   - delay: delay"},{"line_number":1597,"context_line":"   - expected_codes: expected_codes"}],"source_content_type":"text/x-c++src","patch_set":5,"id":"ff346bd7_14142d43","line":1594,"range":{"start_line":1594,"start_character":5,"end_line":1594,"end_character":19},"updated":"2017-07-27 13:27:23.000000000","message":"ditto","commit_id":"8e2b3023503bc574df9aa23840bfb05c2fff7638"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"ed528f8742e6f7b4485066d34034e6acd2ba4057","unresolved":false,"context_lines":[{"line_number":1472,"context_line":""},{"line_number":1473,"context_line":".. rest_parameters:: parameters.yaml"},{"line_number":1474,"context_line":""},{"line_number":1475,"context_line":"   - healthmonitor: healthmonitor"},{"line_number":1476,"context_line":"   - admin_state_up: admin_state_up"},{"line_number":1477,"context_line":"   - delay: delay"},{"line_number":1478,"context_line":"   - expected_codes: expected_codes"}],"source_content_type":"text/x-c++src","patch_set":12,"id":"9f436f4f_5c512ed8","line":1475,"range":{"start_line":1475,"start_character":20,"end_line":1475,"end_character":33},"updated":"2017-08-09 17:19:18.000000000","message":"This currently reads::\n\n  The associated healthmonitor, if any.\n\nBut appears to be a required top-level ``healthmonitor`` object based on the sample JSON request. So this should probably read something like::\n\n  A  ``healthmonitor`` object.","commit_id":"4c533976384eeda31a4bfe7be1d0c197ff5200e7"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"ed528f8742e6f7b4485066d34034e6acd2ba4057","unresolved":false,"context_lines":[{"line_number":1496,"context_line":""},{"line_number":1497,"context_line":".. rest_parameters:: parameters.yaml"},{"line_number":1498,"context_line":""},{"line_number":1499,"context_line":"   - healthmonitor: healthmonitor"},{"line_number":1500,"context_line":"   - admin_state_up: admin_state_up"},{"line_number":1501,"context_line":"   - delay: delay"},{"line_number":1502,"context_line":"   - expected_codes: expected_codes-response"}],"source_content_type":"text/x-c++src","patch_set":12,"id":"9f436f4f_fc5902fe","line":1499,"range":{"start_line":1499,"start_character":20,"end_line":1499,"end_character":33},"updated":"2017-08-09 17:19:18.000000000","message":"Ditto.","commit_id":"4c533976384eeda31a4bfe7be1d0c197ff5200e7"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"ed528f8742e6f7b4485066d34034e6acd2ba4057","unresolved":false,"context_lines":[{"line_number":1546,"context_line":""},{"line_number":1547,"context_line":".. rest_parameters:: parameters.yaml"},{"line_number":1548,"context_line":""},{"line_number":1549,"context_line":"   - healthmonitor: healthmonitor"},{"line_number":1550,"context_line":"   - admin_state_up: admin_state_up"},{"line_number":1551,"context_line":"   - delay: delay"},{"line_number":1552,"context_line":"   - expected_codes: expected_codes-response"}],"source_content_type":"text/x-c++src","patch_set":12,"id":"9f436f4f_7c6d1297","line":1549,"range":{"start_line":1549,"start_character":20,"end_line":1549,"end_character":33},"updated":"2017-08-09 17:19:18.000000000","message":"ditto","commit_id":"4c533976384eeda31a4bfe7be1d0c197ff5200e7"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"ed528f8742e6f7b4485066d34034e6acd2ba4057","unresolved":false,"context_lines":[{"line_number":1591,"context_line":".. rest_parameters:: parameters.yaml"},{"line_number":1592,"context_line":""},{"line_number":1593,"context_line":"   - healthmonitor_id: healthmonitor_id-path"},{"line_number":1594,"context_line":"   - healthmonitor: healthmonitor"},{"line_number":1595,"context_line":"   - admin_state_up: admin_state_up"},{"line_number":1596,"context_line":"   - delay: delay"},{"line_number":1597,"context_line":"   - expected_codes: expected_codes"}],"source_content_type":"text/x-c++src","patch_set":12,"id":"9f436f4f_fc0ea2e3","line":1594,"range":{"start_line":1594,"start_character":20,"end_line":1594,"end_character":33},"updated":"2017-08-09 17:19:18.000000000","message":"ditto.","commit_id":"4c533976384eeda31a4bfe7be1d0c197ff5200e7"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"ed528f8742e6f7b4485066d34034e6acd2ba4057","unresolved":false,"context_lines":[{"line_number":1611,"context_line":""},{"line_number":1612,"context_line":".. rest_parameters:: parameters.yaml"},{"line_number":1613,"context_line":""},{"line_number":1614,"context_line":"   - healthmonitor: healthmonitor"},{"line_number":1615,"context_line":"   - admin_state_up: admin_state_up"},{"line_number":1616,"context_line":"   - delay: delay"},{"line_number":1617,"context_line":"   - expected_codes: expected_codes-response"}],"source_content_type":"text/x-c++src","patch_set":12,"id":"9f436f4f_dc13de0c","line":1614,"range":{"start_line":1614,"start_character":20,"end_line":1614,"end_character":33},"updated":"2017-08-09 17:19:18.000000000","message":"ditto.","commit_id":"4c533976384eeda31a4bfe7be1d0c197ff5200e7"}],"api-ref/source/v2/parameters.yaml":[{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"0245509b38cbaa5ebc505462eeb21c5dd29fd6e1","unresolved":false,"context_lines":[{"line_number":2562,"context_line":"  type: string"},{"line_number":2563,"context_line":"healthmonitors-obj:"},{"line_number":2564,"context_line":"  description: |"},{"line_number":2565,"context_line":"    A list of ``health_monitor`` objects."},{"line_number":2566,"context_line":"  in: body"},{"line_number":2567,"context_line":"  required: true"},{"line_number":2568,"context_line":"  type: array"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"ff346bd7_7952543d","line":2565,"range":{"start_line":2565,"start_character":16,"end_line":2565,"end_character":30},"updated":"2017-07-27 13:27:23.000000000","message":"shouldn\u0027t this be ``heathmonitor`` now?","commit_id":"8e2b3023503bc574df9aa23840bfb05c2fff7638"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"3e161c8f6fb92d4086f5b2ef71c9a2fd2ab869db","unresolved":false,"context_lines":[{"line_number":97,"context_line":"  in: path"},{"line_number":98,"context_line":"  required: true"},{"line_number":99,"context_line":"  type: string"},{"line_number":100,"context_line":"health_monitor_id-path:"},{"line_number":101,"context_line":"  description: |"},{"line_number":102,"context_line":"    The ID for the health monitor."},{"line_number":103,"context_line":"healthmonitor_id-path:"},{"line_number":104,"context_line":"  description: |"},{"line_number":105,"context_line":"    The ID for the health monitor."}],"source_content_type":"text/x-yaml","patch_set":6,"id":"df3967d1_bbab8dd4","line":102,"range":{"start_line":100,"start_character":0,"end_line":102,"end_character":34},"updated":"2017-07-31 16:24:47.000000000","message":"It seems to me that you need to delete these lines. This is why the check is complaining about not finding the \u0027in\u0027 key in a section","commit_id":"5d114e03cd972708a36127b47d81bd17993d3238"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"782117afdca0652a271fc75a3307e6bd6035ba4e","unresolved":false,"context_lines":[{"line_number":2574,"context_line":"  type: array"},{"line_number":2575,"context_line":"healthmonitors-request:"},{"line_number":2576,"context_line":"  description: |"},{"line_number":2577,"context_line":"    List of health monitors to be associated with the pool."},{"line_number":2578,"context_line":"    The default is an empty list."},{"line_number":2579,"context_line":"  in: body"},{"line_number":2580,"context_line":"  required: false"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"df3967d1_0642c73e","line":2577,"range":{"start_line":2577,"start_character":12,"end_line":2577,"end_character":27},"updated":"2017-08-03 16:59:42.000000000","message":"IMO might be more consistent to say something like \"List of ``healthmonitor`` objects...\"","commit_id":"9993dbc51f6a5afb6d6fd88403dfa74745d52661"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"782117afdca0652a271fc75a3307e6bd6035ba4e","unresolved":false,"context_lines":[{"line_number":2581,"context_line":"  type: array"},{"line_number":2582,"context_line":"healthmonitors-response:"},{"line_number":2583,"context_line":"  description: |"},{"line_number":2584,"context_line":"    List of health monitors that are associated with the pool."},{"line_number":2585,"context_line":"  in: body"},{"line_number":2586,"context_line":"  required: true"},{"line_number":2587,"context_line":"  type: array"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"df3967d1_e63ea3b6","line":2584,"range":{"start_line":2584,"start_character":12,"end_line":2584,"end_character":27},"updated":"2017-08-03 16:59:42.000000000","message":"Ditto","commit_id":"9993dbc51f6a5afb6d6fd88403dfa74745d52661"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"782117afdca0652a271fc75a3307e6bd6035ba4e","unresolved":false,"context_lines":[{"line_number":2587,"context_line":"  type: array"},{"line_number":2588,"context_line":"healthmonitors_status:"},{"line_number":2589,"context_line":"  description: |"},{"line_number":2590,"context_line":"    The statuses of the health monitors that are"},{"line_number":2591,"context_line":"    associated with the pool."},{"line_number":2592,"context_line":"  in: body"},{"line_number":2593,"context_line":"  required: true"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"df3967d1_a6441b48","line":2590,"range":{"start_line":2590,"start_character":8,"end_line":2590,"end_character":16},"updated":"2017-08-03 16:59:42.000000000","message":"Ditto.","commit_id":"9993dbc51f6a5afb6d6fd88403dfa74745d52661"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"782117afdca0652a271fc75a3307e6bd6035ba4e","unresolved":false,"context_lines":[{"line_number":2591,"context_line":"    associated with the pool."},{"line_number":2592,"context_line":"  in: body"},{"line_number":2593,"context_line":"  required: true"},{"line_number":2594,"context_line":"  type: string"},{"line_number":2595,"context_line":"http_method:"},{"line_number":2596,"context_line":"  description: |"},{"line_number":2597,"context_line":"    The HTTP method that the monitor uses for requests."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"df3967d1_8600f702","line":2594,"range":{"start_line":2594,"start_character":8,"end_line":2594,"end_character":14},"updated":"2017-08-03 16:59:42.000000000","message":"As per the example responses, this isn\u0027t a string tho. Rather its a list of ``healthmonitors_status`` objects, presumably one per healthmonitor.","commit_id":"9993dbc51f6a5afb6d6fd88403dfa74745d52661"},{"author":{"_account_id":5367,"name":"boden","email":"bodenvmw@gmail.com","username":"boden"},"change_message_id":"1d7c3159b122ba25c48a37b9f4bf24f24cca6ffe","unresolved":false,"context_lines":[{"line_number":2587,"context_line":"  type: array"},{"line_number":2588,"context_line":"healthmonitors_status:"},{"line_number":2589,"context_line":"  description: |"},{"line_number":2590,"context_line":"    The statuses of the health monitors that are"},{"line_number":2591,"context_line":"    associated with the pool."},{"line_number":2592,"context_line":"  in: body"},{"line_number":2593,"context_line":"  required: true"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"bf3e73b7_8f7655d8","line":2590,"updated":"2017-08-07 13:16:30.000000000","message":"IMO this should be described in more detail. For example::\n\n   A list of ``healthmonitors_status`` objects, one per healthmonitor that contains a ``monitor_id``, ...","commit_id":"57dfda6b3c696a614d923eac9ab6d7ce52ec20fa"}]}