)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"381a1e9bdb79624bcc9c9b20607e554aba6085b9","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add option to verify certificate authority"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently certificate authority(CA) verification is disabled"},{"line_number":10,"context_line":"for cluster nodes to magnum API communication. This change"},{"line_number":11,"context_line":"introduces a flag that provide an option to enable CA"},{"line_number":12,"context_line":"verification. Default value is set to False to match the"},{"line_number":13,"context_line":"current behavior."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a1ced50_c3bf845e","line":10,"range":{"start_line":10,"start_character":4,"end_line":10,"end_character":18},"updated":"2017-03-20 09:08:06.000000000","message":"coreos and swarm are not covered in this patch. May be you are planning a seperate patch. Can you tell what all patches are pending for this bug.","commit_id":"5b0cdffab588868ba9d769a8282be97374ea2b21"},{"author":{"_account_id":7230,"name":"Vijendar Komalla","email":"vijendar.komalla@rackspace.com","username":"vijendar"},"change_message_id":"fa4f93823d239ce7f53ed474074e2b7374516dc7","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add option to verify certificate authority"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently certificate authority(CA) verification is disabled"},{"line_number":10,"context_line":"for cluster nodes to magnum API communication. This change"},{"line_number":11,"context_line":"introduces a flag that provide an option to enable CA"},{"line_number":12,"context_line":"verification. Default value is set to False to match the"},{"line_number":13,"context_line":"current behavior."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a1ced50_752cb402","line":10,"range":{"start_line":10,"start_character":4,"end_line":10,"end_character":18},"in_reply_to":"1a1ced50_c3bf845e","updated":"2017-03-20 15:26:57.000000000","message":"I am going to submit couple of patches to cover other drivers (coreos and swarm).","commit_id":"5b0cdffab588868ba9d769a8282be97374ea2b21"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"381a1e9bdb79624bcc9c9b20607e554aba6085b9","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently certificate authority(CA) verification is disabled"},{"line_number":10,"context_line":"for cluster nodes to magnum API communication. This change"},{"line_number":11,"context_line":"introduces a flag that provide an option to enable CA"},{"line_number":12,"context_line":"verification. Default value is set to False to match the"},{"line_number":13,"context_line":"current behavior."},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a1ced50_a379a8fe","line":11,"range":{"start_line":11,"start_character":13,"end_line":11,"end_character":17},"updated":"2017-03-20 09:08:06.000000000","message":"where is the flag, do you mean label here. If yes i cannot find it.","commit_id":"5b0cdffab588868ba9d769a8282be97374ea2b21"},{"author":{"_account_id":7230,"name":"Vijendar Komalla","email":"vijendar.komalla@rackspace.com","username":"vijendar"},"change_message_id":"fa4f93823d239ce7f53ed474074e2b7374516dc7","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently certificate authority(CA) verification is disabled"},{"line_number":10,"context_line":"for cluster nodes to magnum API communication. This change"},{"line_number":11,"context_line":"introduces a flag that provide an option to enable CA"},{"line_number":12,"context_line":"verification. Default value is set to False to match the"},{"line_number":13,"context_line":"current behavior."},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a1ced50_7532f411","line":11,"range":{"start_line":11,"start_character":13,"end_line":11,"end_character":17},"in_reply_to":"1a1ced50_a379a8fe","updated":"2017-03-20 15:26:57.000000000","message":"I mean, \u0027heat template parameter\u0027 in each driver.","commit_id":"5b0cdffab588868ba9d769a8282be97374ea2b21"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"381a1e9bdb79624bcc9c9b20607e554aba6085b9","unresolved":false,"context_lines":[{"line_number":13,"context_line":"current behavior."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I0940e76c6f2c32ceea66771916afae1823795eb4"},{"line_number":16,"context_line":"Partially-Implements: #1663757"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a1ced50_439f1450","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":20},"updated":"2017-03-20 09:08:06.000000000","message":"Partial-Bug","commit_id":"5b0cdffab588868ba9d769a8282be97374ea2b21"},{"author":{"_account_id":7230,"name":"Vijendar Komalla","email":"vijendar.komalla@rackspace.com","username":"vijendar"},"change_message_id":"fa4f93823d239ce7f53ed474074e2b7374516dc7","unresolved":false,"context_lines":[{"line_number":13,"context_line":"current behavior."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I0940e76c6f2c32ceea66771916afae1823795eb4"},{"line_number":16,"context_line":"Partially-Implements: #1663757"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a1ced50_b53c6ce3","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":20},"in_reply_to":"1a1ced50_439f1450","updated":"2017-03-20 15:26:57.000000000","message":"Done","commit_id":"5b0cdffab588868ba9d769a8282be97374ea2b21"}],"magnum/drivers/common/templates/kubernetes/fragments/make-cert-client.sh":[{"author":{"_account_id":21660,"name":"Stephen Watson","email":"stephen.watson@intel.com","username":"swatson"},"change_message_id":"79a97d376a6ae67a37cc2f6011ef42ddeb77abfa","unresolved":false,"context_lines":[{"line_number":25,"context_line":"fi"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"VERIFY_CA_ARG\u003d\"-k\""},{"line_number":28,"context_line":"if [ \"$VERIFY_CA\" \u003d\u003d \"True\" ]; then"},{"line_number":29,"context_line":"    VERIFY_CA_ARG\u003d\"\""},{"line_number":30,"context_line":"fi"},{"line_number":31,"context_line":""}],"source_content_type":"text/x-sh","patch_set":6,"id":"ffe62b97_45b17ce1","line":28,"updated":"2017-03-23 20:19:42.000000000","message":"This looks like a swap in logic. Am I misunderstanding something?","commit_id":"b1f8072a0e1340e5c49ca68c7fa4784c9a2b5b68"},{"author":{"_account_id":7230,"name":"Vijendar Komalla","email":"vijendar.komalla@rackspace.com","username":"vijendar"},"change_message_id":"59378fd8254f9cbb56e45b4f5705e14d0c808186","unresolved":false,"context_lines":[{"line_number":25,"context_line":"fi"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"VERIFY_CA_ARG\u003d\"-k\""},{"line_number":28,"context_line":"if [ \"$VERIFY_CA\" \u003d\u003d \"True\" ]; then"},{"line_number":29,"context_line":"    VERIFY_CA_ARG\u003d\"\""},{"line_number":30,"context_line":"fi"},{"line_number":31,"context_line":""}],"source_content_type":"text/x-sh","patch_set":6,"id":"ffe62b97_c57fcca9","line":28,"in_reply_to":"ffe62b97_45b17ce1","updated":"2017-03-23 20:29:44.000000000","message":"curl by default allows secure connection and\u0027-k\u0027 switch is to allow insecure access. If VERIFY_CA is set to \u0027True\u0027, then we should not set \u0027-k\u0027 switch.","commit_id":"b1f8072a0e1340e5c49ca68c7fa4784c9a2b5b68"}]}