)]}'
{"tempest/clients.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"766d72119065c12902ada1c5d571195cb818996a","unresolved":false,"context_lines":[{"line_number":668,"context_line":""},{"line_number":669,"context_line":"        auth_url \u003d CONF.identity.uri"},{"line_number":670,"context_line":"        dscv \u003d CONF.identity.disable_ssl_certificate_validation"},{"line_number":671,"context_line":"        region \u003d CONF.identity.region"},{"line_number":672,"context_line":"        endpoint_type \u003d CONF.network.endpoint_type"},{"line_number":673,"context_line":""},{"line_number":674,"context_line":"        return neutronclient.v2_0.client.Client("}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_34790930","line":671,"updated":"2014-07-01 13:38:35.000000000","message":"Should\u0027nt this be CONF.network.region ?","commit_id":"7e4808b8bb8ec90bdcb0d2b1981d474dd6c1f235"},{"author":{"_account_id":9392,"name":"Prashanth Hari","email":"hvprash@gmail.com","username":"hvprash"},"change_message_id":"4db45adc1130b2fd28224301472ba5d2ac8fc3d8","unresolved":false,"context_lines":[{"line_number":668,"context_line":""},{"line_number":669,"context_line":"        auth_url \u003d CONF.identity.uri"},{"line_number":670,"context_line":"        dscv \u003d CONF.identity.disable_ssl_certificate_validation"},{"line_number":671,"context_line":"        region \u003d CONF.identity.region"},{"line_number":672,"context_line":"        endpoint_type \u003d CONF.network.endpoint_type"},{"line_number":673,"context_line":""},{"line_number":674,"context_line":"        return neutronclient.v2_0.client.Client("}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_5c49ed5e","line":671,"in_reply_to":"baada198_2e76c3d7","updated":"2014-07-03 12:02:05.000000000","message":"Region option in all the client configs in scenario tests are broken. Planning to fix these based on this review - https://review.openstack.org/#/c/102827/3/tempest/clients.py","commit_id":"7e4808b8bb8ec90bdcb0d2b1981d474dd6c1f235"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"79181b3fde012d43265289955064ff820e2a7171","unresolved":false,"context_lines":[{"line_number":668,"context_line":""},{"line_number":669,"context_line":"        auth_url \u003d CONF.identity.uri"},{"line_number":670,"context_line":"        dscv \u003d CONF.identity.disable_ssl_certificate_validation"},{"line_number":671,"context_line":"        region \u003d CONF.identity.region"},{"line_number":672,"context_line":"        endpoint_type \u003d CONF.network.endpoint_type"},{"line_number":673,"context_line":""},{"line_number":674,"context_line":"        return neutronclient.v2_0.client.Client("}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_2e76c3d7","line":671,"in_reply_to":"baada198_34790930","updated":"2014-07-03 11:39:45.000000000","message":"Jordan\u0027s point seems reasonable because of the following explanation:\n\ntempest.conf\n [network]\n ...\n # The network region name to use. If empty, the value of\n # identity.region is used instead. If no such region is found\n # in the service catalog, the first found one is used. (string\n # value)\n #region\u003d","commit_id":"7e4808b8bb8ec90bdcb0d2b1981d474dd6c1f235"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"3c0955470c14c4f981b59d5828ffd34185cc89fc","unresolved":false,"context_lines":[{"line_number":668,"context_line":""},{"line_number":669,"context_line":"        auth_url \u003d CONF.identity.uri"},{"line_number":670,"context_line":"        dscv \u003d CONF.identity.disable_ssl_certificate_validation"},{"line_number":671,"context_line":"        region \u003d CONF.identity.region"},{"line_number":672,"context_line":"        endpoint_type \u003d CONF.network.endpoint_type"},{"line_number":673,"context_line":""},{"line_number":674,"context_line":"        return neutronclient.v2_0.client.Client("}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_8f4986b9","line":671,"in_reply_to":"baada198_5c49ed5e","updated":"2014-07-03 15:13:14.000000000","message":"It\u0027s good to have someone to have a look at how the region are handled. Be sure to update the comment in tempest.config (config.py) if you\u0027d like to change the logic.","commit_id":"7e4808b8bb8ec90bdcb0d2b1981d474dd6c1f235"},{"author":{"_account_id":9392,"name":"Prashanth Hari","email":"hvprash@gmail.com","username":"hvprash"},"change_message_id":"8c7c2e054cad0c9badbcff55e6c2de2e69c792d4","unresolved":false,"context_lines":[{"line_number":668,"context_line":""},{"line_number":669,"context_line":"        auth_url \u003d CONF.identity.uri"},{"line_number":670,"context_line":"        dscv \u003d CONF.identity.disable_ssl_certificate_validation"},{"line_number":671,"context_line":"        region \u003d CONF.identity.region"},{"line_number":672,"context_line":"        endpoint_type \u003d CONF.network.endpoint_type"},{"line_number":673,"context_line":""},{"line_number":674,"context_line":"        return neutronclient.v2_0.client.Client("}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_a332daef","line":671,"in_reply_to":"baada198_8f4986b9","updated":"2014-07-03 17:10:42.000000000","message":"Will do.","commit_id":"7e4808b8bb8ec90bdcb0d2b1981d474dd6c1f235"},{"author":{"_account_id":9392,"name":"Prashanth Hari","email":"hvprash@gmail.com","username":"hvprash"},"change_message_id":"c96212947d051937e80a52ff8e2e3bfc7e84af72","unresolved":false,"context_lines":[{"line_number":668,"context_line":""},{"line_number":669,"context_line":"        auth_url \u003d CONF.identity.uri"},{"line_number":670,"context_line":"        dscv \u003d CONF.identity.disable_ssl_certificate_validation"},{"line_number":671,"context_line":"        region \u003d CONF.identity.region"},{"line_number":672,"context_line":"        endpoint_type \u003d CONF.network.endpoint_type"},{"line_number":673,"context_line":""},{"line_number":674,"context_line":"        return neutronclient.v2_0.client.Client("}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_52cd9b85","line":671,"in_reply_to":"baada198_a332daef","updated":"2014-07-09 19:51:17.000000000","message":"Dependency for getting region info. Added \"_get_region\" in clients.py - https://review.openstack.org/#/c/105782/\n\nNeed review","commit_id":"7e4808b8bb8ec90bdcb0d2b1981d474dd6c1f235"}]}
