)]}'
{"lib/neutron":[{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"b858ab47ba32ead00d9920d3683593645b8cb7e9","unresolved":false,"context_lines":[{"line_number":262,"context_line":"    if [[ \"$Q_USE_SECGROUP\" \u003d\u003d \"True\" ]]; then"},{"line_number":263,"context_line":"        LIBVIRT_FIREWALL_DRIVER\u003dnova.virt.firewall.NoopFirewallDriver"},{"line_number":264,"context_line":"        iniset $NOVA_CONF DEFAULT security_group_api quantum"},{"line_number":265,"context_line":"    else"},{"line_number":266,"context_line":"        _horizon_config_set $local_settings OPENSTACK_NEUTRON_NETWORK enable_security_group $Q_USE_SECGROUP"},{"line_number":267,"context_line":"    fi"},{"line_number":268,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAQn%2F%2Fg2E%3D","line":265,"updated":"2013-07-30 00:27:53.000000000","message":"if is_service_enabled horizon","commit_id":"4153be3e11f2d8914a6356704086398c261e6566"},{"author":{"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},"change_message_id":"fc19757527f6582e40f201a99484c85b8c68635d","unresolved":false,"context_lines":[{"line_number":262,"context_line":"    if [[ \"$Q_USE_SECGROUP\" \u003d\u003d \"True\" ]]; then"},{"line_number":263,"context_line":"        LIBVIRT_FIREWALL_DRIVER\u003dnova.virt.firewall.NoopFirewallDriver"},{"line_number":264,"context_line":"        iniset $NOVA_CONF DEFAULT security_group_api quantum"},{"line_number":265,"context_line":"    else"},{"line_number":266,"context_line":"        _horizon_config_set $local_settings OPENSTACK_NEUTRON_NETWORK enable_security_group $Q_USE_SECGROUP"},{"line_number":267,"context_line":"    fi"},{"line_number":268,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAQn%2F%2FgLA%3D","line":265,"in_reply_to":"AAAAQn%2F%2Fg2E%3D","updated":"2013-07-30 05:28:27.000000000","message":"I will follow Akihiro\u0027s suggestion, he is the Horizon/neutron master  :-)","commit_id":"4153be3e11f2d8914a6356704086398c261e6566"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"d23d713c38847868332b15c7cfb703abf0c16942","unresolved":false,"context_lines":[{"line_number":263,"context_line":"        LIBVIRT_FIREWALL_DRIVER\u003dnova.virt.firewall.NoopFirewallDriver"},{"line_number":264,"context_line":"        iniset $NOVA_CONF DEFAULT security_group_api quantum"},{"line_number":265,"context_line":"    else"},{"line_number":266,"context_line":"        _horizon_config_set $local_settings OPENSTACK_NEUTRON_NETWORK enable_security_group $Q_USE_SECGROUP"},{"line_number":267,"context_line":"    fi"},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"    # set NOVA_VIF_DRIVER and optionally set options in nova_conf"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAQn%2F%2Fg0U%3D","line":266,"updated":"2013-07-30 01:49:48.000000000","message":"IMO it is better that horizon related configurations are defined in lib/horizon. $local_settings file is created (copied) in init_horizon and it is better to avoid the behavior depends on the order of init_horizon and create\\nova_conf_neutron (though init_horizon is called first at now).\n\nI confirmed it works well after adding the following at init_horizon L.107:\n\n  if is_service_enabled neutron; then\n    _horizon_config_set $local_settings   OPENSTACK_NEUTRON_NETWORK enable_security_group $Q_USE_SECGROUP\n  fi\n\nIf we keep it here, we need to add a check if horizon is enabled (is_service_enabled horizon).","commit_id":"4153be3e11f2d8914a6356704086398c261e6566"},{"author":{"_account_id":704,"name":"Edgar Magana","email":"emagana@gmail.com","username":"emagana"},"change_message_id":"fc19757527f6582e40f201a99484c85b8c68635d","unresolved":false,"context_lines":[{"line_number":263,"context_line":"        LIBVIRT_FIREWALL_DRIVER\u003dnova.virt.firewall.NoopFirewallDriver"},{"line_number":264,"context_line":"        iniset $NOVA_CONF DEFAULT security_group_api quantum"},{"line_number":265,"context_line":"    else"},{"line_number":266,"context_line":"        _horizon_config_set $local_settings OPENSTACK_NEUTRON_NETWORK enable_security_group $Q_USE_SECGROUP"},{"line_number":267,"context_line":"    fi"},{"line_number":268,"context_line":""},{"line_number":269,"context_line":"    # set NOVA_VIF_DRIVER and optionally set options in nova_conf"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAQn%2F%2FgK8%3D","line":266,"in_reply_to":"AAAAQn%2F%2Fg0U%3D","updated":"2013-07-30 05:28:27.000000000","message":"Done","commit_id":"4153be3e11f2d8914a6356704086398c261e6566"}]}