)]}'
{"ceilometerclient/client.py":[{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"79ddd5c022bbf03fbad14051ced10b86deda42d1","unresolved":false,"context_lines":[{"line_number":297,"context_line":"            * os_project_domain_id: domain id for the user project"},{"line_number":298,"context_line":"            * os_auth_url: endpoint to authenticate against"},{"line_number":299,"context_line":"            * os_cert|os_cacert: path of CA TLS certificate"},{"line_number":300,"context_line":"            * os_key: SSL private key"},{"line_number":301,"context_line":"            * insecure: allow insecure SSL (no cert verification)"},{"line_number":302,"context_line":"    \"\"\""},{"line_number":303,"context_line":"    endpoint \u003d kwargs.get(\u0027os_endpoint\u0027) or kwargs.get(\u0027ceilometer_url\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"da9b358b_d7c9d279","line":300,"updated":"2015-04-01 02:24:17.000000000","message":"update it to os_insecure\n\nor add it, and update all other parameters such as `username`","commit_id":"ccd3a4a57c2fe6ad836680720fef5b75d584c79a"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"f049e13e25e195b36d020a0b0cf28390e19efe76","unresolved":false,"context_lines":[{"line_number":297,"context_line":"            * os_project_domain_id: domain id for the user project"},{"line_number":298,"context_line":"            * os_auth_url: endpoint to authenticate against"},{"line_number":299,"context_line":"            * os_cert|os_cacert: path of CA TLS certificate"},{"line_number":300,"context_line":"            * os_key: SSL private key"},{"line_number":301,"context_line":"            * insecure: allow insecure SSL (no cert verification)"},{"line_number":302,"context_line":"    \"\"\""},{"line_number":303,"context_line":"    endpoint \u003d kwargs.get(\u0027os_endpoint\u0027) or kwargs.get(\u0027ceilometer_url\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9aa53dc9_e0b7a0f3","line":300,"in_reply_to":"baa041b7_5ab9d52d","updated":"2015-04-07 18:50:11.000000000","message":"could you add this. just so we don\u0027t have to deal with this in the future or possibly forget about it.","commit_id":"ccd3a4a57c2fe6ad836680720fef5b75d584c79a"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"c20949a033c4bb1fa6793c88dbe2a2043c3a974c","unresolved":false,"context_lines":[{"line_number":297,"context_line":"            * os_project_domain_id: domain id for the user project"},{"line_number":298,"context_line":"            * os_auth_url: endpoint to authenticate against"},{"line_number":299,"context_line":"            * os_cert|os_cacert: path of CA TLS certificate"},{"line_number":300,"context_line":"            * os_key: SSL private key"},{"line_number":301,"context_line":"            * insecure: allow insecure SSL (no cert verification)"},{"line_number":302,"context_line":"    \"\"\""},{"line_number":303,"context_line":"    endpoint \u003d kwargs.get(\u0027os_endpoint\u0027) or kwargs.get(\u0027ceilometer_url\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"baa041b7_5ab9d52d","line":300,"in_reply_to":"da9b358b_d7c9d279","updated":"2015-04-01 16:56:20.000000000","message":"sure, I could change this from \"insecure\" to \"os_insecure\"... but that\u0027s a nit, so I probably won\u0027t do that unless there are more significant comments.","commit_id":"ccd3a4a57c2fe6ad836680720fef5b75d584c79a"},{"author":{"_account_id":6676,"name":"zqfan","email":"aji.zqfan@gmail.com","username":"aji-zqfan"},"change_message_id":"79ddd5c022bbf03fbad14051ced10b86deda42d1","unresolved":false,"context_lines":[{"line_number":311,"context_line":"        service_type\u003dkwargs.get(\u0027service_type\u0027),"},{"line_number":312,"context_line":"        token\u003dkwargs.get(\u0027token\u0027),"},{"line_number":313,"context_line":"        endpoint_type\u003dkwargs.get(\u0027endpoint_type\u0027),"},{"line_number":314,"context_line":"        insecure\u003dkwargs.get(\u0027insecure\u0027),"},{"line_number":315,"context_line":"        cacert\u003dkwargs.get(\u0027cacert\u0027),"},{"line_number":316,"context_line":"        tenant_id\u003dkwargs.get(\u0027project_id\u0027) or kwargs.get(\u0027tenant_id\u0027),"},{"line_number":317,"context_line":"        endpoint\u003dendpoint,"}],"source_content_type":"text/x-python","patch_set":2,"id":"da9b358b_d7f2f2d1","line":314,"updated":"2015-04-01 02:24:17.000000000","message":"the v2 client only accept \u0027verify\u0027 but no \u0027insecure\u0027, keystone session only accept \u0027verify\u0027 too, the Deprecated oslo-incubator.apiclient use insecure, which cause a lot problem.\n\nfor compatibility, we can still support \u0027insecure\u0027 parameter, but should also consider scenario only have \u0027verify\u0027","commit_id":"ccd3a4a57c2fe6ad836680720fef5b75d584c79a"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"c20949a033c4bb1fa6793c88dbe2a2043c3a974c","unresolved":false,"context_lines":[{"line_number":311,"context_line":"        service_type\u003dkwargs.get(\u0027service_type\u0027),"},{"line_number":312,"context_line":"        token\u003dkwargs.get(\u0027token\u0027),"},{"line_number":313,"context_line":"        endpoint_type\u003dkwargs.get(\u0027endpoint_type\u0027),"},{"line_number":314,"context_line":"        insecure\u003dkwargs.get(\u0027insecure\u0027),"},{"line_number":315,"context_line":"        cacert\u003dkwargs.get(\u0027cacert\u0027),"},{"line_number":316,"context_line":"        tenant_id\u003dkwargs.get(\u0027project_id\u0027) or kwargs.get(\u0027tenant_id\u0027),"},{"line_number":317,"context_line":"        endpoint\u003dendpoint,"}],"source_content_type":"text/x-python","patch_set":2,"id":"baa041b7_8a794c32","line":314,"in_reply_to":"da9b358b_d7f2f2d1","updated":"2015-04-01 16:56:20.000000000","message":"I tried reworking this to use verify here instead of insecure, but no luck... that breaks the shell use case. It looks like we\u0027re stuck using insecure here until we move away from oslo-incubator\u0027s apiclient, and I don\u0027t want to bite off that big of a change here.","commit_id":"ccd3a4a57c2fe6ad836680720fef5b75d584c79a"}]}
