)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"9c826f54bfd20dc3033b7056c785c34a4a045227","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"f53d4aa1_19d956c7","updated":"2023-11-15 12:45:12.000000000","message":"The logic change looks good. Let me leave one comment about the parameter description format.\n\nAlso, please add a release note which explains the new parameter, because this is a new feature.","commit_id":"d5e5473f9333985644e34ea036bcace3e40602ec"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"2346564bc62ecf18bc306e08288908699ef36b96","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"df93b9f7_4173af34","updated":"2023-11-15 12:41:36.000000000","message":"fyi. the broken unit test cases will be fixed by https://review.opendev.org/c/openstack/puppet-openstacklib/+/901033 .","commit_id":"d5e5473f9333985644e34ea036bcace3e40602ec"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"d398d52e9527dd28a9c8d9cae51a3e2e93a857a7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"16d08aac_6f7696ef","in_reply_to":"f53d4aa1_19d956c7","updated":"2023-11-15 15:38:41.000000000","message":"Done","commit_id":"d5e5473f9333985644e34ea036bcace3e40602ec"}],"manifests/wsgi/apache.pp":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"9c826f54bfd20dc3033b7056c785c34a4a045227","unresolved":true,"context_lines":[{"line_number":219,"context_line":"#   Defaults to undef"},{"line_number":220,"context_line":"#"},{"line_number":221,"context_line":"# [*access_log_env_var*]"},{"line_number":222,"context_line":"#   (Optional) Specifies that only requests with particular environment variables be logged."},{"line_number":223,"context_line":"#"},{"line_number":224,"context_line":"# [*error_log_file*]"},{"line_number":225,"context_line":"#   (Optional) The error log file name for the virtualhost."}],"source_content_type":"text/x-puppet","patch_set":2,"id":"47e65df4_5f2c47f9","line":222,"range":{"start_line":222,"start_character":82,"end_line":222,"end_character":84},"updated":"2023-11-15 12:45:12.000000000","message":"can we wrap the line by 80 chars ? also please add `Defaults to undef` to make the format consistent with the others.","commit_id":"d5e5473f9333985644e34ea036bcace3e40602ec"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"d398d52e9527dd28a9c8d9cae51a3e2e93a857a7","unresolved":false,"context_lines":[{"line_number":219,"context_line":"#   Defaults to undef"},{"line_number":220,"context_line":"#"},{"line_number":221,"context_line":"# [*access_log_env_var*]"},{"line_number":222,"context_line":"#   (Optional) Specifies that only requests with particular environment variables be logged."},{"line_number":223,"context_line":"#"},{"line_number":224,"context_line":"# [*error_log_file*]"},{"line_number":225,"context_line":"#   (Optional) The error log file name for the virtualhost."}],"source_content_type":"text/x-puppet","patch_set":2,"id":"34d8367b_97e9dede","line":222,"range":{"start_line":222,"start_character":82,"end_line":222,"end_character":84},"in_reply_to":"47e65df4_5f2c47f9","updated":"2023-11-15 15:38:41.000000000","message":"Done","commit_id":"d5e5473f9333985644e34ea036bcace3e40602ec"}],"releasenotes/notes/access_log_env_var-87f4bfd3356c6cd2.yaml":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"3706cbd09ecc850788bcf61ba7aea46faff5ac49","unresolved":false,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add access_log_env_var for apache vhost."},{"line_number":5,"context_line":"    Allows that only requests with particular environment "},{"line_number":6,"context_line":"    variables set are logged."},{"line_number":7,"context_line":"    Example: set a dontlog variable for healthchecks,"},{"line_number":8,"context_line":"    so healthchecks are not logged."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"82a4cd70_67084269","line":5,"updated":"2023-11-15 15:43:07.000000000","message":"nit: trailing space","commit_id":"077e95b953c5654e2f194bd3c48f44bf3aed8d7b"},{"author":{"_account_id":7888,"name":"Benedikt Trefzer","email":"benedikt.trefzer@cirrax.com","username":"trefzer"},"change_message_id":"cfc6abfb946124f6cef037e4149136647f032481","unresolved":false,"context_lines":[{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Add access_log_env_var for apache vhost."},{"line_number":5,"context_line":"    Allows that only requests with particular environment "},{"line_number":6,"context_line":"    variables set are logged."},{"line_number":7,"context_line":"    Example: set a dontlog variable for healthchecks,"},{"line_number":8,"context_line":"    so healthchecks are not logged."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"e1e5b396_66b79d8f","line":5,"in_reply_to":"82a4cd70_67084269","updated":"2023-11-15 15:51:38.000000000","message":";)","commit_id":"077e95b953c5654e2f194bd3c48f44bf3aed8d7b"}]}