)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13325,"name":"Paul Karikh","email":"pkarikh@gmail.com","username":"paul.karikh"},"change_message_id":"dee0faf01522b87650ae9beca87480c30ee6184c","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Closes-Bug: #1332726"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"e On branch bug/1332726"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ib81fc148e1c2c70df2b41309ce83d80a3a28d038"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"baa201ad_269e4e57","line":11,"updated":"2014-10-10 14:57:50.000000000","message":"It looks like there\u0027s a typo, isn\u0027t it?","commit_id":"de3c2bce750624276c85280c6fb076c3b08a18dd"},{"author":{"_account_id":8040,"name":"Timur Sufiev","email":"tsufiev@gmail.com","username":"tsufiev"},"change_message_id":"d0660916b4cacb67cbafb4f5263c9ea9299c8f5b","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Vlad Okhrimenko \u003cvokhrimenko@mirantis.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-10-22 11:09:23 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Switch Region dropdown"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Fixed switch region dropdown,"},{"line_number":10,"context_line":"remove \"hide\" from div-class-name."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"9aa7fdbe_ccabac1c","line":7,"updated":"2014-10-22 09:44:16.000000000","message":"Rewording suggestion:\n\nFix inability to switch region via Switch Region dropdown","commit_id":"313e6eba1244362678d0cefb6f62c327402f1fdd"},{"author":{"_account_id":8040,"name":"Timur Sufiev","email":"tsufiev@gmail.com","username":"tsufiev"},"change_message_id":"d0660916b4cacb67cbafb4f5263c9ea9299c8f5b","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Switch Region dropdown"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Fixed switch region dropdown,"},{"line_number":10,"context_line":"remove \"hide\" from div-class-name."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Related-Bug: #1381413"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"9aa7fdbe_ac7748d3","line":9,"updated":"2014-10-22 09:44:16.000000000","message":"Remove class \u0027hide\u0027 that prevented modal login form from showing.","commit_id":"313e6eba1244362678d0cefb6f62c327402f1fdd"}],"horizon/static/horizon/js/horizon.modals.js":[{"author":{"_account_id":8040,"name":"Timur Sufiev","email":"tsufiev@gmail.com","username":"tsufiev"},"change_message_id":"eec5dc7fbcceb1bacb72c78539230a822df4d13a","unresolved":false,"context_lines":[{"line_number":227,"context_line":"        horizon.modals.modal_spinner(gettext(\"Working\"));"},{"line_number":228,"context_line":"      },"},{"line_number":229,"context_line":"      complete: function () {"},{"line_number":230,"context_line":"        if (redirect_header \u003d\u003d\u003d null) {"},{"line_number":231,"context_line":"            horizon.modals.spinner.modal(\u0027hide\u0027);"},{"line_number":232,"context_line":"        }"},{"line_number":233,"context_line":"        $(\"#modal_wrapper .modal\").last().modal(\"show\");"}],"source_content_type":"text/javascript","patch_set":11,"id":"9a80dd14_ed4361ad","line":230,"updated":"2015-03-16 18:26:00.000000000","message":"Variable redirect_header is not defined in this scope. Once all javascript files had been changed to \u0027strict\u0027 parsing mode, this became obvious.","commit_id":"fc533fbea60748c8a991cc562d8e20dfe25309c0"},{"author":{"_account_id":12355,"name":"Vlad Okhrimenko","email":"vokhrimenko@mirantis.com","username":"vokhrimenko"},"change_message_id":"675b96a87ca8ee38a5b392bbd09acacb371cd500","unresolved":false,"context_lines":[{"line_number":227,"context_line":"        horizon.modals.modal_spinner(gettext(\"Working\"));"},{"line_number":228,"context_line":"      },"},{"line_number":229,"context_line":"      complete: function () {"},{"line_number":230,"context_line":"        if (redirect_header \u003d\u003d\u003d null) {"},{"line_number":231,"context_line":"            horizon.modals.spinner.modal(\u0027hide\u0027);"},{"line_number":232,"context_line":"        }"},{"line_number":233,"context_line":"        $(\"#modal_wrapper .modal\").last().modal(\"show\");"}],"source_content_type":"text/javascript","patch_set":11,"id":"9a80dd14_037f14e1","line":230,"in_reply_to":"9a80dd14_ed4361ad","updated":"2015-03-19 09:48:03.000000000","message":"Done","commit_id":"fc533fbea60748c8a991cc562d8e20dfe25309c0"},{"author":{"_account_id":10442,"name":"Masco Kaliyamoorthy","email":"mkaliyam@redhat.com","username":"masco"},"change_message_id":"268efd82844e6d1f08bedef50b460d4cd1b179df","unresolved":false,"context_lines":[{"line_number":231,"context_line":"      complete: function (jqXHR) {"},{"line_number":232,"context_line":"        var redirect_header \u003d jqXHR.getResponseHeader(\"X-Horizon-Location\");"},{"line_number":233,"context_line":"        if (redirect_header \u003d\u003d\u003d null) {"},{"line_number":234,"context_line":"            horizon.modals.spinner.modal(\u0027hide\u0027);"},{"line_number":235,"context_line":"        }"},{"line_number":236,"context_line":"        $(\"#modal_wrapper .modal\").last().modal(\"show\");"},{"line_number":237,"context_line":"        $button.prop(\"disabled\", false);"}],"source_content_type":"text/javascript","patch_set":12,"id":"da9b358b_34cfce2a","line":234,"updated":"2015-04-01 11:22:14.000000000","message":"nit- we will use only two space indentation in JS","commit_id":"bf85a4dc4dfa08aad7a7c8861728922f285a1fe1"},{"author":{"_account_id":12355,"name":"Vlad Okhrimenko","email":"vokhrimenko@mirantis.com","username":"vokhrimenko"},"change_message_id":"3fc03067be3ad3c862abbedfdcc077ec4018e334","unresolved":false,"context_lines":[{"line_number":231,"context_line":"      complete: function (jqXHR) {"},{"line_number":232,"context_line":"        var redirect_header \u003d jqXHR.getResponseHeader(\"X-Horizon-Location\");"},{"line_number":233,"context_line":"        if (redirect_header \u003d\u003d\u003d null) {"},{"line_number":234,"context_line":"            horizon.modals.spinner.modal(\u0027hide\u0027);"},{"line_number":235,"context_line":"        }"},{"line_number":236,"context_line":"        $(\"#modal_wrapper .modal\").last().modal(\"show\");"},{"line_number":237,"context_line":"        $button.prop(\"disabled\", false);"}],"source_content_type":"text/javascript","patch_set":12,"id":"da9b358b_650b5e8f","line":234,"in_reply_to":"da9b358b_34cfce2a","updated":"2015-04-01 12:34:04.000000000","message":"Done","commit_id":"bf85a4dc4dfa08aad7a7c8861728922f285a1fe1"}],"horizon/templates/horizon/common/_region_selector.html":[{"author":{"_account_id":13325,"name":"Paul Karikh","email":"pkarikh@gmail.com","username":"paul.karikh"},"change_message_id":"dee0faf01522b87650ae9beca87480c30ee6184c","unresolved":false,"context_lines":[{"line_number":8,"context_line":"      \u003cli class\u003d\u0027divider\u0027\u003e\u003c/li\u003e"},{"line_number":9,"context_line":"      {% for region in regions.available %}"},{"line_number":10,"context_line":"        {% if region.name !\u003d regions.current.name %}"},{"line_number":11,"context_line":"{#          \u003cli\u003e\u003ca class\u003d\"ajax-modal\" href\u003d\"{% url \u0027login\u0027 %}?region\u003d{{ region.endpoint|urlencode }}\"\u003e{{ region.name }}\u003c/a\u003e\u003c/li\u003e#}"},{"line_number":12,"context_line":"          \u003cli\u003e\u003ca href\u003d\"{% url \u0027switch_services_region\u0027 %}?region\u003d{{ region.endpoint|urlencode }}\u0026name\u003d{{ region.name }}\"\u003e{{ region.name }}\u003c/a\u003e\u003c/li\u003e"},{"line_number":13,"context_line":"        {% endif %}"},{"line_number":14,"context_line":"      {% endfor %}"}],"source_content_type":"text/html","patch_set":1,"id":"baa201ad_26ec8ec7","line":11,"updated":"2014-10-10 14:57:50.000000000","message":"I think it would be better not to leave commented-out code here. Maybe you forgot to delete it?","commit_id":"de3c2bce750624276c85280c6fb076c3b08a18dd"},{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"b272c8ac01992d70e37abbf26e4d3ff4ef0056a2","unresolved":false,"context_lines":[{"line_number":8,"context_line":"      \u003cli class\u003d\u0027divider\u0027\u003e\u003c/li\u003e"},{"line_number":9,"context_line":"      {% for region in regions.available %}"},{"line_number":10,"context_line":"        {% if region.name !\u003d regions.current.name %}"},{"line_number":11,"context_line":"          \u003cli\u003e\u003ca href\u003d\"{% url \u0027switch_services_region\u0027 \u0027RegionOne\u0027 %}?region\u003d{{ region.endpoint|urlencode }}\"\u003e{{ region.name }}\u003c/a\u003e\u003c/li\u003e"},{"line_number":12,"context_line":"        {% endif %}"},{"line_number":13,"context_line":"      {% endfor %}"},{"line_number":14,"context_line":"    \u003c/ul\u003e"}],"source_content_type":"text/html","patch_set":4,"id":"baa201ad_2fb08542","line":11,"updated":"2014-10-14 22:22:17.000000000","message":"This is hard-codes the region to the RegionOne that is only available in devstack.  Not sure how this supposed to solve the issue.","commit_id":"1304543db84968fe2bad59a74bb35294d518ec99"},{"author":{"_account_id":12355,"name":"Vlad Okhrimenko","email":"vokhrimenko@mirantis.com","username":"vokhrimenko"},"change_message_id":"2a9c3bb064a2a138a994926384d7330d30102351","unresolved":false,"context_lines":[{"line_number":8,"context_line":"      \u003cli class\u003d\u0027divider\u0027\u003e\u003c/li\u003e"},{"line_number":9,"context_line":"      {% for region in regions.available %}"},{"line_number":10,"context_line":"        {% if region.name !\u003d regions.current.name %}"},{"line_number":11,"context_line":"          \u003cli\u003e\u003ca href\u003d\"{% url \u0027switch_services_region\u0027 \u0027RegionOne\u0027 %}?region\u003d{{ region.endpoint|urlencode }}\"\u003e{{ region.name }}\u003c/a\u003e\u003c/li\u003e"},{"line_number":12,"context_line":"        {% endif %}"},{"line_number":13,"context_line":"      {% endfor %}"},{"line_number":14,"context_line":"    \u003c/ul\u003e"}],"source_content_type":"text/html","patch_set":4,"id":"baa201ad_7bd968f2","line":11,"in_reply_to":"baa201ad_2fb08542","updated":"2014-10-15 06:46:24.000000000","message":"Hi! Thanks for the comment. I have a two region --- \u0027Czesc Region\u0027 and \u0027My Fuel Openstak\u0027(they have different region_name), but they always have services_region \u003d\u003d \u0027RegionOne\u0027. I explored problem, but not found where this happens. I saw in the controller-node: keystone endpoint-list ---\u003e all endpoints have a \u0027RegionOne\u0027.\nI use hardcode because i  want show problem. If we will fix this problem (i think we need new bug) this part  https://review.openstack.org/#/c/127547/3/openstack_auth/views.py not need for us","commit_id":"1304543db84968fe2bad59a74bb35294d518ec99"},{"author":{"_account_id":8040,"name":"Timur Sufiev","email":"tsufiev@gmail.com","username":"tsufiev"},"change_message_id":"791ef801f7184fc8d1a7d331b3de32c6048a20b8","unresolved":false,"context_lines":[{"line_number":8,"context_line":"      \u003cli class\u003d\u0027divider\u0027\u003e\u003c/li\u003e"},{"line_number":9,"context_line":"      {% for region in regions.available %}"},{"line_number":10,"context_line":"        {% if region.name !\u003d regions.current.name %}"},{"line_number":11,"context_line":"          \u003cli\u003e\u003ca href\u003d\"{% url \u0027switch_services_region\u0027 request.user.services_region %}?region\u003d{{ region.endpoint|urlencode }}\"\u003e{{ region.name }}\u003c/a\u003e\u003c/li\u003e"},{"line_number":12,"context_line":"        {% endif %}"},{"line_number":13,"context_line":"      {% endfor %}"},{"line_number":14,"context_line":"    \u003c/ul\u003e"}],"source_content_type":"text/html","patch_set":5,"id":"baa201ad_19e9fde5","line":11,"updated":"2014-10-17 11:15:39.000000000","message":"I guess it doesn\u0027t make much sense to pass something contained in a request into an url, because the same value could be extracted from a request in a view that is tied to that url.","commit_id":"1ac0a957ed7f4bdc1cfa1e74c399e2d67a250756"}]}