)]}'
{"files/apache-ceilometer.template":[{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"dbfdb465923c511294c89210064f6038add23a6b","unresolved":false,"context_lines":[{"line_number":2,"context_line":""},{"line_number":3,"context_line":"\u003cVirtualHost *:%PORT%\u003e"},{"line_number":4,"context_line":"    WSGIDaemonProcess ceilometer-api processes\u003d2 threads\u003d10 user\u003d%USER% display-name\u003d%{GROUP}"},{"line_number":5,"context_line":"    WSGIProcessGroup ceilometer-api"},{"line_number":6,"context_line":"    WSGIScriptAlias / %WSGIAPP%"},{"line_number":7,"context_line":"    WSGIApplicationGroup %{GLOBAL}"},{"line_number":8,"context_line":"    %ERRORLOGFORMAT%"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"fa98f980_f2c17d30","line":5,"updated":"2014-09-16 15:44:45.000000000","message":"Probably an idea to line up the WSGIProcessGroup specified here and in the ceilometer sample:\n\n  https://github.com/openstack/ceilometer/blob/master/etc/apache2/ceilometer#L27","commit_id":"5050de63ef598b025997dbb1ed3ddc8c9728752c"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"773daeb5e182716648f587eab9405470f3b98b6b","unresolved":false,"context_lines":[{"line_number":2,"context_line":""},{"line_number":3,"context_line":"\u003cVirtualHost *:%PORT%\u003e"},{"line_number":4,"context_line":"    WSGIDaemonProcess ceilometer-api processes\u003d2 threads\u003d10 user\u003d%USER% display-name\u003d%{GROUP}"},{"line_number":5,"context_line":"    WSGIProcessGroup ceilometer-api"},{"line_number":6,"context_line":"    WSGIScriptAlias / %WSGIAPP%"},{"line_number":7,"context_line":"    WSGIApplicationGroup %{GLOBAL}"},{"line_number":8,"context_line":"    %ERRORLOGFORMAT%"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"fa98f980_2e3aa034","line":5,"in_reply_to":"fa98f980_f2c17d30","updated":"2014-09-17 15:15:30.000000000","message":"Done","commit_id":"5050de63ef598b025997dbb1ed3ddc8c9728752c"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"340fdb1535965fa06ca00d3431bae106f4fb09c9","unresolved":false,"context_lines":[{"line_number":7,"context_line":"    WSGIApplicationGroup %{GLOBAL}"},{"line_number":8,"context_line":"    %ERRORLOGFORMAT%"},{"line_number":9,"context_line":"    ErrorLog /var/log/%APACHE_NAME%/ceilometer.log"},{"line_number":10,"context_line":"    CustomLog /var/log/%APACHE_NAME%/ceilometer_access.log combined"},{"line_number":11,"context_line":"\u003c/VirtualHost\u003e"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"fa98f980_0e66bcee","line":10,"updated":"2014-09-17 15:04:44.000000000","message":"Should you include these 2 lines ? :\n\n    SetEnv APACHE_RUN_USER %USER%\n    SetEnv APACHE_RUN_GROUP %GROUP%\n\n(see https://github.com/openstack/ceilometer/blob/master/etc/apache2/ceilometer#L25 and https://github.com/openstack-dev/devstack/blob/master/files/apache-horizon.template#L6)","commit_id":"5050de63ef598b025997dbb1ed3ddc8c9728752c"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"773daeb5e182716648f587eab9405470f3b98b6b","unresolved":false,"context_lines":[{"line_number":7,"context_line":"    WSGIApplicationGroup %{GLOBAL}"},{"line_number":8,"context_line":"    %ERRORLOGFORMAT%"},{"line_number":9,"context_line":"    ErrorLog /var/log/%APACHE_NAME%/ceilometer.log"},{"line_number":10,"context_line":"    CustomLog /var/log/%APACHE_NAME%/ceilometer_access.log combined"},{"line_number":11,"context_line":"\u003c/VirtualHost\u003e"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"fa98f980_4e1e04bd","line":10,"in_reply_to":"fa98f980_0e66bcee","updated":"2014-09-17 15:15:30.000000000","message":"I see that people habitually have that in the config but it isn\u0027t clear to me what purpose that serves when using mod_wsgi in daemon mode? The externalized process users as the user indicated in the WSGIDaemonProcess line.\n\nIf there\u0027s something I\u0027m not aware of please say so.","commit_id":"5050de63ef598b025997dbb1ed3ddc8c9728752c"},{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"36e9bb36817846cf2877b96f569673f7e97d7c0b","unresolved":false,"context_lines":[{"line_number":7,"context_line":"    WSGIApplicationGroup %{GLOBAL}"},{"line_number":8,"context_line":"    %ERRORLOGFORMAT%"},{"line_number":9,"context_line":"    ErrorLog /var/log/%APACHE_NAME%/ceilometer.log"},{"line_number":10,"context_line":"    CustomLog /var/log/%APACHE_NAME%/ceilometer_access.log combined"},{"line_number":11,"context_line":"\u003c/VirtualHost\u003e"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"fa98f980_5a9dde65","line":10,"in_reply_to":"fa98f980_4e1e04bd","updated":"2014-09-17 17:15:11.000000000","message":"I have no idea how APACHE_RUN_USER could be used inside an application.\n\nCeilometer documentation just mentions it. Don\u0027t know why though.","commit_id":"5050de63ef598b025997dbb1ed3ddc8c9728752c"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"48fce2cd50855d72d97814288221c51ce8ef0d9f","unresolved":false,"context_lines":[{"line_number":8,"context_line":"    %ERRORLOGFORMAT%"},{"line_number":9,"context_line":"    ErrorLog /var/log/%APACHE_NAME%/ceilometer.log"},{"line_number":10,"context_line":"    CustomLog /var/log/%APACHE_NAME%/ceilometer_access.log combined"},{"line_number":11,"context_line":"\u003c/VirtualHost\u003e"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"fa98f980_03676726","line":11,"updated":"2014-09-18 01:44:57.000000000","message":"Adding WSGISocketPrefix would be good fro rhel6 support -- see If4b74019b6bd389b576fc981154bb1b3aa471c9b","commit_id":"5050de63ef598b025997dbb1ed3ddc8c9728752c"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"cd7120c9cde847e32c1f72af87b72c60776cd544","unresolved":false,"context_lines":[{"line_number":8,"context_line":"    %ERRORLOGFORMAT%"},{"line_number":9,"context_line":"    ErrorLog /var/log/%APACHE_NAME%/ceilometer.log"},{"line_number":10,"context_line":"    CustomLog /var/log/%APACHE_NAME%/ceilometer_access.log combined"},{"line_number":11,"context_line":"\u003c/VirtualHost\u003e"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"fa98f980_6d6019a5","line":11,"in_reply_to":"fa98f980_03676726","updated":"2014-09-18 09:49:28.000000000","message":"I saw that in the keystone setup but wasn\u0027t sure whether to include it or not because duplicating it in each of the configs that _might_ use mod_wsgi seems wrong. I would think it should be in one generic place, but it wasn\u0027t clear where that should be?\n\nAnd if it were to be put in that one place it wouldn\u0027t be in this patch would it?","commit_id":"5050de63ef598b025997dbb1ed3ddc8c9728752c"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"592041de6490a60fa22f5dc710244b4467752ebd","unresolved":false,"context_lines":[{"line_number":10,"context_line":"    \u003c/IfVersion\u003e"},{"line_number":11,"context_line":"    ErrorLog /var/log/%APACHE_NAME%/ceilometer.log"},{"line_number":12,"context_line":"    CustomLog /var/log/%APACHE_NAME%/ceilometer_access.log combined"},{"line_number":13,"context_line":"\u003c/VirtualHost\u003e"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"da9df570_7f732815","line":13,"updated":"2014-09-25 01:34:35.000000000","message":"I\u0027m still worried that this fails if someone tries it on RHEL6, which is still supported at this stage; see in the other files\n\n#  https://bugzilla.redhat.com/show_bug.cgi?id\u003d1121019\nWSGISocketPrefix /var/run/%APACHE_NAME%","commit_id":"2fc6c7b7862a59e22bdbb44803b8cee74b3247a5"}],"lib/apache":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"592041de6490a60fa22f5dc710244b4467752ebd","unresolved":false,"context_lines":[{"line_number":63,"context_line":"    # ensure mod_version enabled for \u003cIfVersion ...\u003e.  This is"},{"line_number":64,"context_line":"    # built-in statically on anything recent, but precise (2.2)"},{"line_number":65,"context_line":"    # doesn\u0027t have it enabled"},{"line_number":66,"context_line":"    sudo a2enmod version || true"},{"line_number":67,"context_line":"}"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"# get_apache_version() - return the version of Apache installed"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"da9df570_df8414d6","line":66,"updated":"2014-09-25 01:34:35.000000000","message":"this is going to conflict with I6478db385fda2fa1c75ced12d3e886b2e1152852 which is in the queue now","commit_id":"2fc6c7b7862a59e22bdbb44803b8cee74b3247a5"}],"lib/ceilometer":[{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"48fce2cd50855d72d97814288221c51ce8ef0d9f","unresolved":false,"context_lines":[{"line_number":134,"context_line":"        # Apache 2.4 supports custom error log formats"},{"line_number":135,"context_line":"        # this should mirror the original log formatting."},{"line_number":136,"context_line":"        local errorlogformat\u003d\u0027ErrorLogFormat \"%{cu}t %M\"\u0027"},{"line_number":137,"context_line":"    fi"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"    # copy proxy vhost and wsgi file"},{"line_number":140,"context_line":"    sudo cp $CEILOMETER_DIR/ceilometer/api/app.wsgi $CEILOMETER_WSGI_DIR/app"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"fa98f980_637593cb","line":137,"updated":"2014-09-18 01:44:57.000000000","message":"not your fault, but i\u0027d rather see this done in the config files, see [1]\n\n[1] https://review.openstack.org/#/c/104125/","commit_id":"5050de63ef598b025997dbb1ed3ddc8c9728752c"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"cd7120c9cde847e32c1f72af87b72c60776cd544","unresolved":false,"context_lines":[{"line_number":134,"context_line":"        # Apache 2.4 supports custom error log formats"},{"line_number":135,"context_line":"        # this should mirror the original log formatting."},{"line_number":136,"context_line":"        local errorlogformat\u003d\u0027ErrorLogFormat \"%{cu}t %M\"\u0027"},{"line_number":137,"context_line":"    fi"},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"    # copy proxy vhost and wsgi file"},{"line_number":140,"context_line":"    sudo cp $CEILOMETER_DIR/ceilometer/api/app.wsgi $CEILOMETER_WSGI_DIR/app"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"fa98f980_6d2e5944","line":137,"in_reply_to":"fa98f980_637593cb","updated":"2014-09-18 09:49:28.000000000","message":"Makes sense, I\u0027ll tune things up. I was cargo culting.","commit_id":"5050de63ef598b025997dbb1ed3ddc8c9728752c"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"826f21748ddba35ac51e9a8edc651b2a8bc74147","unresolved":false,"context_lines":[{"line_number":275,"context_line":"        enable_apache_site ceilometer"},{"line_number":276,"context_line":"        restart_apache_server"},{"line_number":277,"context_line":"        tail_log ceilometer /var/log/$APACHE_NAME/ceilometer.log"},{"line_number":278,"context_line":"        tail_log ceilometer-api /var/log/$APACHE_NAME/ceilometer_access.log"},{"line_number":279,"context_line":"    fi"},{"line_number":280,"context_line":""},{"line_number":281,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":1,"id":"fa98f980_6dca0e1c","line":278,"updated":"2014-09-16 16:21:20.000000000","message":"We started this pattern with Horizon \"because everything else was there\" back when there were maybe 10 screen windows.  Is this still useful?  Especially when it\u0027s also logged to a file by Apache?","commit_id":"5050de63ef598b025997dbb1ed3ddc8c9728752c"}]}