)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":22860,"name":"Solio Sarabia","email":"solio.sarabia@intel.com","username":"ssarabia"},"change_message_id":"5a2fceab08c82f276a9bde0858e086338e4715d5","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Setup apache template for auto deploy with devstack"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- add user and Ironic bin dir variables to setup with devstack"},{"line_number":10,"context_line":"- add SSL variable to service requests via HTTPS"},{"line_number":11,"context_line":"- use pbr wsgi_scripts to install Ironic wsgi entry point"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"3a461143_24a69f10","line":9,"range":{"start_line":9,"start_character":15,"end_line":9,"end_character":29},"updated":"2017-01-26 18:53:06.000000000","message":"nit: IRONIC_BIN_DIR","commit_id":"8ee041ce2ba80df4859686897c93e38aa4f278b2"},{"author":{"_account_id":13362,"name":"Tan Lin","email":"lin.tan@intel.com","username":"zero"},"change_message_id":"77b32c7f796cd2d5ec4e7bdb19fdc2a3ba33d21b","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Setup apache template for auto deploy with devstack"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- add user and Ironic bin dir variables to setup with devstack"},{"line_number":10,"context_line":"- add SSL variable to service requests via HTTPS"},{"line_number":11,"context_line":"- use pbr wsgi_scripts to install Ironic wsgi entry point"},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"fa69d971_c3321310","line":9,"updated":"2016-01-11 09:41:04.000000000","message":"nit: s/user/\"user/group\"/","commit_id":"8ee041ce2ba80df4859686897c93e38aa4f278b2"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"990fa596d29db453e420fc7dfe6fe036479a1e65","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Setup apache template for auto deploy with devstack"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- add user and Ironic bin dir variables to setup with devstack"},{"line_number":10,"context_line":"- add SSL variable to service requests via HTTPS"},{"line_number":11,"context_line":"- use pbr wsgi_scripts to install Ironic wsgi entry point"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"api/app.wsgi isn\u0027t removed for backward compatibility"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"fa69d971_3aad6684","line":10,"updated":"2016-01-09 23:17:11.000000000","message":"s/variable/variables","commit_id":"8ee041ce2ba80df4859686897c93e38aa4f278b2"}],"etc/apache2/ironic":[{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"f82d04f8b5e7aa9ebb92ca029cd00ca6183f99e5","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"\u003cVirtualHost *:6385\u003e"},{"line_number":20,"context_line":"    WSGIDaemonProcess ironic user\u003d%USER% group\u003d%GROUP% threads\u003d10 display-name\u003d%{GROUP}"},{"line_number":21,"context_line":"    WSGIScriptAlias / %IRONIC_BIN_DIR%/ironic-api-wsgi"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    SetEnv APACHE_RUN_USER stack"},{"line_number":24,"context_line":"    SetEnv APACHE_RUN_GROUP stack"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"fa80f949_36faaebe","line":21,"updated":"2015-11-04 12:06:21.000000000","message":"Where does this file come from?","commit_id":"bc67fb9870289eb77409a816e1e95223bb3831c2"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"04b7cbb97a105b08c887cabc32012408f23a2e27","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"\u003cVirtualHost *:6385\u003e"},{"line_number":20,"context_line":"    WSGIDaemonProcess ironic user\u003d%USER% group\u003d%GROUP% threads\u003d10 display-name\u003d%{GROUP}"},{"line_number":21,"context_line":"    WSGIScriptAlias / %IRONIC_BIN_DIR%/ironic-api-wsgi"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    SetEnv APACHE_RUN_USER stack"},{"line_number":24,"context_line":"    SetEnv APACHE_RUN_GROUP stack"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"fa80f949_f1aaf018","line":21,"in_reply_to":"fa80f949_36faaebe","updated":"2015-11-04 13:06:52.000000000","message":"It\u0027s replaced with correct values by devstack","commit_id":"bc67fb9870289eb77409a816e1e95223bb3831c2"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"f82d04f8b5e7aa9ebb92ca029cd00ca6183f99e5","unresolved":false,"context_lines":[{"line_number":20,"context_line":"    WSGIDaemonProcess ironic user\u003d%USER% group\u003d%GROUP% threads\u003d10 display-name\u003d%{GROUP}"},{"line_number":21,"context_line":"    WSGIScriptAlias / %IRONIC_BIN_DIR%/ironic-api-wsgi"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    SetEnv APACHE_RUN_USER stack"},{"line_number":24,"context_line":"    SetEnv APACHE_RUN_GROUP stack"},{"line_number":25,"context_line":"    WSGIProcessGroup ironic"},{"line_number":26,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":4,"id":"fa80f949_eb58bf58","line":23,"updated":"2015-11-04 12:06:21.000000000","message":"%USER% and %GROUP% here too?","commit_id":"bc67fb9870289eb77409a816e1e95223bb3831c2"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"04b7cbb97a105b08c887cabc32012408f23a2e27","unresolved":false,"context_lines":[{"line_number":20,"context_line":"    WSGIDaemonProcess ironic user\u003d%USER% group\u003d%GROUP% threads\u003d10 display-name\u003d%{GROUP}"},{"line_number":21,"context_line":"    WSGIScriptAlias / %IRONIC_BIN_DIR%/ironic-api-wsgi"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    SetEnv APACHE_RUN_USER stack"},{"line_number":24,"context_line":"    SetEnv APACHE_RUN_GROUP stack"},{"line_number":25,"context_line":"    WSGIProcessGroup ironic"},{"line_number":26,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":4,"id":"fa80f949_317878c4","line":23,"in_reply_to":"fa80f949_eb58bf58","updated":"2015-11-04 13:06:52.000000000","message":"I think about remove this SetEnv, imo no needs to change apache user","commit_id":"bc67fb9870289eb77409a816e1e95223bb3831c2"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"f82d04f8b5e7aa9ebb92ca029cd00ca6183f99e5","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    %SSLCERTFILE%"},{"line_number":32,"context_line":"    %SSLKEYFILE%"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    \u003cDirectory %IRONIC_BIN_DIR%\u003e"},{"line_number":35,"context_line":"        WSGIProcessGroup ironic"},{"line_number":36,"context_line":"        WSGIApplicationGroup %{GLOBAL}"},{"line_number":37,"context_line":"        AllowOverride All"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"fa80f949_f66c269e","line":34,"updated":"2015-11-04 12:06:21.000000000","message":"Looking at devstack patch, this seems to be either /usr/local/bin or /usr/bin, not the directory containing api code. Should this be changed?","commit_id":"bc67fb9870289eb77409a816e1e95223bb3831c2"},{"author":{"_account_id":13636,"name":"aarefiev","display_name":"Anton","email":"arefyevanton@gmail.com","username":"aarefiev"},"change_message_id":"04b7cbb97a105b08c887cabc32012408f23a2e27","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    %SSLCERTFILE%"},{"line_number":32,"context_line":"    %SSLKEYFILE%"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    \u003cDirectory %IRONIC_BIN_DIR%\u003e"},{"line_number":35,"context_line":"        WSGIProcessGroup ironic"},{"line_number":36,"context_line":"        WSGIApplicationGroup %{GLOBAL}"},{"line_number":37,"context_line":"        AllowOverride All"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"fa80f949_431e8f0d","line":34,"in_reply_to":"fa80f949_f66c269e","updated":"2015-11-04 13:06:52.000000000","message":"it\u0027s dir with entry points installation of console scripts and new wsgi_scripts endpoint, so it\u0027s right dir","commit_id":"bc67fb9870289eb77409a816e1e95223bb3831c2"},{"author":{"_account_id":12356,"name":"Vladyslav Drok","email":"vdrok@mirantis.com","username":"vdrok"},"change_message_id":"990fa596d29db453e420fc7dfe6fe036479a1e65","unresolved":false,"context_lines":[{"line_number":21,"context_line":"    WSGIScriptAlias / %IRONIC_BIN_DIR%/ironic-api-wsgi"},{"line_number":22,"context_line":"    WSGIProcessGroup ironic"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"    ErrorLog /var/log/apache2/ir-api.log"},{"line_number":25,"context_line":"    LogLevel info"},{"line_number":26,"context_line":"    CustomLog /var/log/apache2/ir-api_access.log combined"},{"line_number":27,"context_line":"    %SSLENGINE%"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"fa69d971_5ab61afa","line":24,"updated":"2016-01-09 23:17:11.000000000","message":"I\u0027d rather leave error mention in file name.","commit_id":"8ee041ce2ba80df4859686897c93e38aa4f278b2"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"235edda474aba9ce945e7b71278e9fad5a62c910","unresolved":false,"context_lines":[{"line_number":21,"context_line":"    WSGIScriptAlias / %IRONIC_BIN_DIR%/ironic-api-wsgi"},{"line_number":22,"context_line":"    WSGIProcessGroup ironic"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"    ErrorLog /var/log/apache2/ir-api.log"},{"line_number":25,"context_line":"    LogLevel info"},{"line_number":26,"context_line":"    CustomLog /var/log/apache2/ir-api_access.log combined"},{"line_number":27,"context_line":"    %SSLENGINE%"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"3a461143_c6058a7a","line":24,"in_reply_to":"3a461143_8108252f","updated":"2017-01-30 10:49:51.000000000","message":"++ to mention error...\n\nAlso, I think we should call it ironic (instead \"ir-\") cause that\u0027s the name of the module. And honeslty, I don\u0027t see why we are renaming these logs, the current name seems just fine.","commit_id":"8ee041ce2ba80df4859686897c93e38aa4f278b2"},{"author":{"_account_id":22860,"name":"Solio Sarabia","email":"solio.sarabia@intel.com","username":"ssarabia"},"change_message_id":"5a2fceab08c82f276a9bde0858e086338e4715d5","unresolved":false,"context_lines":[{"line_number":21,"context_line":"    WSGIScriptAlias / %IRONIC_BIN_DIR%/ironic-api-wsgi"},{"line_number":22,"context_line":"    WSGIProcessGroup ironic"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"    ErrorLog /var/log/apache2/ir-api.log"},{"line_number":25,"context_line":"    LogLevel info"},{"line_number":26,"context_line":"    CustomLog /var/log/apache2/ir-api_access.log combined"},{"line_number":27,"context_line":"    %SSLENGINE%"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"3a461143_8108252f","line":24,"in_reply_to":"fa69d971_5ab61afa","updated":"2017-01-26 18:53:06.000000000","message":"Or even ir-api_error.log to follow some pattern (ir-api_access.log)","commit_id":"8ee041ce2ba80df4859686897c93e38aa4f278b2"}]}