)]}'
{"playbooks/roles/os_horizon/templates/horizon_local_settings.py.j2":[{"author":{"_account_id":7414,"name":"David Wilde","email":"dwilde@redhat.com","username":"d34dh0r53"},"change_message_id":"18177407f934a863e60a7156832245e64a662e2d","unresolved":false,"context_lines":[{"line_number":168,"context_line":"#     (\u0027http://cluster1.example.com:5000/v2.0\u0027, \u0027cluster1\u0027),"},{"line_number":169,"context_line":"#     (\u0027http://cluster2.example.com:5000/v2.0\u0027, \u0027cluster2\u0027),"},{"line_number":170,"context_line":"# ]"},{"line_number":171,"context_line":"{% endif %}"},{"line_number":172,"context_line":""},{"line_number":173,"context_line":"OPENSTACK_HOST \u003d \"{{ internal_lb_vip_address }}\""},{"line_number":174,"context_line":"OPENSTACK_KEYSTONE_URL \u003d \"{{ keystone_service_internalurl }}\""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"3afb71cf_7224fd5c","line":171,"updated":"2015-06-10 14:11:21.000000000","message":"So this is a total nit, but the way this is written you\u0027re going to get an AVAILABLE_REGIONS block and then the comment block describing how to do them, following the conventions of the file the comment block should go first and then the declaration.","commit_id":"a274b2287110e18791e76e77ab71176a225ed315"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"ce1a54c698f62b28955e9d9bced79953d55e0ce5","unresolved":false,"context_lines":[{"line_number":168,"context_line":"#     (\u0027http://cluster1.example.com:5000/v2.0\u0027, \u0027cluster1\u0027),"},{"line_number":169,"context_line":"#     (\u0027http://cluster2.example.com:5000/v2.0\u0027, \u0027cluster2\u0027),"},{"line_number":170,"context_line":"# ]"},{"line_number":171,"context_line":"{% endif %}"},{"line_number":172,"context_line":""},{"line_number":173,"context_line":"OPENSTACK_HOST \u003d \"{{ internal_lb_vip_address }}\""},{"line_number":174,"context_line":"OPENSTACK_KEYSTONE_URL \u003d \"{{ keystone_service_internalurl }}\""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"3afb71cf_8ecab519","line":171,"in_reply_to":"3afb71cf_7224fd5c","updated":"2015-06-10 15:19:09.000000000","message":"Hmm, I\u0027m not sure I understand what you mean. There\u0027s an if..else block - the comment showing the convention only goes in if horizon_available_regions is not defined.","commit_id":"a274b2287110e18791e76e77ab71176a225ed315"},{"author":{"_account_id":7414,"name":"David Wilde","email":"dwilde@redhat.com","username":"d34dh0r53"},"change_message_id":"03e110551b69c35ae74fd145bb3278cd151bc862","unresolved":false,"context_lines":[{"line_number":168,"context_line":"#     (\u0027http://cluster1.example.com:5000/v2.0\u0027, \u0027cluster1\u0027),"},{"line_number":169,"context_line":"#     (\u0027http://cluster2.example.com:5000/v2.0\u0027, \u0027cluster2\u0027),"},{"line_number":170,"context_line":"# ]"},{"line_number":171,"context_line":"{% endif %}"},{"line_number":172,"context_line":""},{"line_number":173,"context_line":"OPENSTACK_HOST \u003d \"{{ internal_lb_vip_address }}\""},{"line_number":174,"context_line":"OPENSTACK_KEYSTONE_URL \u003d \"{{ keystone_service_internalurl }}\""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"3afb71cf_5481eaf9","line":171,"in_reply_to":"3afb71cf_8ecab519","updated":"2015-06-10 15:46:25.000000000","message":"Doh, you\u0027re totally right, I missed the else statement.","commit_id":"a274b2287110e18791e76e77ab71176a225ed315"}]}