)]}'
{"spec/classes/cinder_wsgi_apache_spec.rb":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"f0c199f80551a5798e4c83f223d09ff9c4247287","unresolved":false,"context_lines":[{"line_number":41,"context_line":"          },"},{"line_number":42,"context_line":"          :access_log_file             \u003d\u003e \u0027/var/log/httpd/access_log\u0027,"},{"line_number":43,"context_line":"          :access_log_format           \u003d\u003e \u0027some format\u0027,"},{"line_number":44,"context_line":"          :error_log_syslog            \u003d\u003e \u0027syslog:local0\u0027"},{"line_number":45,"context_line":"        }"},{"line_number":46,"context_line":"      end"},{"line_number":47,"context_line":"      it { is_expected.to contain_class(\u0027cinder::params\u0027) }"}],"source_content_type":"text/x-ruby","patch_set":1,"id":"ff570b3c_d0109652","line":44,"range":{"start_line":44,"start_character":10,"end_line":44,"end_character":57},"updated":"2020-06-11 15:01:37.000000000","message":"error_log_file is still a valid option, so you shouldn\u0027t replace this but add new parameters instead. Also the other parameters should be tested as well.\n\nI think it\u0027s better to add one more context to test syslog logging .","commit_id":"b5ff4b2711a0079f0d97eb7f10347acdc456703a"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"b96aee7c35c1ae37581cd7a4ae29f56ea803afed","unresolved":false,"context_lines":[{"line_number":44,"context_line":"          :custom_wsgi_process_options \u003d\u003e {"},{"line_number":45,"context_line":"            \u0027python_path\u0027 \u003d\u003e \u0027/my/python/admin/path\u0027,"},{"line_number":46,"context_line":"          },"},{"line_number":47,"context_line":"          :access_log_file             \u003d\u003e \u0027/var/log/httpd/access_log\u0027,"},{"line_number":48,"context_line":"          :access_log_format           \u003d\u003e \u0027some format\u0027,"},{"line_number":49,"context_line":"          :error_log_syslog            \u003d\u003e \u0027syslog:local0\u0027"},{"line_number":50,"context_line":"        }"},{"line_number":51,"context_line":"      end"},{"line_number":52,"context_line":"      it { is_expected.to contain_class(\u0027cinder::params\u0027) }"}],"source_content_type":"text/x-ruby","patch_set":2,"id":"bf51134e_69e747e6","line":49,"range":{"start_line":47,"start_character":0,"end_line":49,"end_character":57},"updated":"2020-06-17 02:07:26.000000000","message":"I think we can remove log parameters from this test case to get rid of duplicated tests.","commit_id":"2c9f74f3299d55c205e10e216700ee51509c8582"}]}