)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},"change_message_id":"0e2e08c4cb21c1bcd710a96e00daecd76741af9a","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Use OS::TripleO::Services::Logging::NovaLibvirt resource to configure"},{"line_number":10,"context_line":"logging in libvirt, so that we can gather logging configuration"},{"line_number":11,"context_line":"into deployments/logging ."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I6db952002b6d5f3d2efa65498db520526614d96b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"3fa7e38b_af6bcbb4","line":11,"range":{"start_line":11,"start_character":24,"end_line":11,"end_character":25},"updated":"2019-09-24 16:37:43.000000000","message":"nit","commit_id":"81ace99e875d6c1e419f492f1872653d6287f234"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"6e71b2c38301e06c901298fb1846e456b696253e","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Use OS::TripleO::Services::Logging::NovaLibvirt resource to configure"},{"line_number":10,"context_line":"logging in libvirt, so that we can gather logging configuration"},{"line_number":11,"context_line":"into deployments/logging ."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I6db952002b6d5f3d2efa65498db520526614d96b"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"3fa7e38b_8fd5afe7","line":11,"range":{"start_line":11,"start_character":24,"end_line":11,"end_character":25},"in_reply_to":"3fa7e38b_af6bcbb4","updated":"2019-09-24 17:00:25.000000000","message":"Done","commit_id":"81ace99e875d6c1e419f492f1872653d6287f234"}],"deployment/logging/files/nova-libvirt.yaml":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"4c41f4b722670cc64ec7b3958217d0c87e74939e","unresolved":false,"context_lines":[{"line_number":29,"context_line":"  docker_config:"},{"line_number":30,"context_line":"    description: Extra containers needed for logging to file in the host."},{"line_number":31,"context_line":"    value: null"},{"line_number":32,"context_line":"  volumes:"},{"line_number":33,"context_line":"    description: The volumes needed to log to files in the host."},{"line_number":34,"context_line":"    value: null"},{"line_number":35,"context_line":"  host_prep_tasks:"},{"line_number":36,"context_line":"    description: Extra ansible tasks needed for logging to files in the host."},{"line_number":37,"context_line":"    value:"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"3fa7e38b_6905d313","line":34,"range":{"start_line":32,"start_character":0,"end_line":34,"end_character":15},"updated":"2019-09-24 15:54:03.000000000","message":"It seems that this overrides volume to null, and this is the reason why we don\u0027t see the log file recorded on host.\nWill submit a patch to fix this.","commit_id":"a1c7370d7b566b8f97c97e34f41a1ec986b6a1e5"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"1d9d0c37178ae5ac90b91706044a680e93c7d34d","unresolved":false,"context_lines":[{"line_number":29,"context_line":"  docker_config:"},{"line_number":30,"context_line":"    description: Extra containers needed for logging to file in the host."},{"line_number":31,"context_line":"    value: null"},{"line_number":32,"context_line":"  volumes:"},{"line_number":33,"context_line":"    description: The volumes needed to log to files in the host."},{"line_number":34,"context_line":"    value: null"},{"line_number":35,"context_line":"  host_prep_tasks:"},{"line_number":36,"context_line":"    description: Extra ansible tasks needed for logging to files in the host."},{"line_number":37,"context_line":"    value:"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"3fa7e38b_4c54a5ff","line":34,"range":{"start_line":32,"start_character":0,"end_line":34,"end_character":15},"in_reply_to":"3fa7e38b_6905d313","updated":"2019-09-24 15:57:01.000000000","message":"Done","commit_id":"a1c7370d7b566b8f97c97e34f41a1ec986b6a1e5"},{"author":{"_account_id":17216,"name":"Martin Schuppert","email":"mschuppert@redhat.com","username":"mcschupp"},"change_message_id":"dece73d868f0ec64a0f4fba2396749f797d53212","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"3fa7e38b_fb13b75a","line":42,"updated":"2019-09-25 09:43:57.000000000","message":"To have it consistent with other services, I think we should\nalso have the usual readme [1] file pointing to the container\nlog location.\n\n[1] https://github.com/openstack/tripleo-heat-templates/blob/master/deployment/logging/files/nova-api.yaml#L57-L63","commit_id":"38e0f8ef0e33e5d1b8c9287bb1812b9c38011d32"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"f68f62209db97000066ec6bb009a8a1057f65d31","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"3fa7e38b_edc3f955","line":42,"in_reply_to":"3fa7e38b_fb13b75a","updated":"2019-09-25 13:45:43.000000000","message":"Just submitted the subsqeuent change to implement this[1] .\n\n[1] https://review.opendev.org/#/c/684750/\n\nThe suggestion makes sense to me, but I\u0027m also a little afraid that that readme can be confusing, because we still have qemu logs recorded under /var/log/libvirt/qemu instead of /var/log/containers/libvirt/qemu.\nIt would be nice if I can get any suggestion about the above thought. (Should I add some additional note describing that qemu logs are still located under /var/log/libvirt?)","commit_id":"38e0f8ef0e33e5d1b8c9287bb1812b9c38011d32"}],"releasenotes/notes/nova_libvirtd_log_level-b1393b582360ad4f.yaml":[{"author":{"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},"change_message_id":"bf3d7523136dfcd494c5c7f1c3d8c618c7895a08","unresolved":false,"context_lines":[{"line_number":6,"context_line":"    logging"},{"line_number":7,"context_line":"deprecations:"},{"line_number":8,"context_line":"  - |"},{"line_number":9,"context_line":"    LibvirtLogOutpjuts option was removed and now has no effect. Use"},{"line_number":10,"context_line":"    LivirtLogLevel to change log level in libvirt."}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3fa7e38b_af3d0bc5","line":9,"range":{"start_line":9,"start_character":18,"end_line":9,"end_character":19},"updated":"2019-09-24 16:33:37.000000000","message":"nit","commit_id":"81ace99e875d6c1e419f492f1872653d6287f234"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"6e71b2c38301e06c901298fb1846e456b696253e","unresolved":false,"context_lines":[{"line_number":6,"context_line":"    logging"},{"line_number":7,"context_line":"deprecations:"},{"line_number":8,"context_line":"  - |"},{"line_number":9,"context_line":"    LibvirtLogOutpjuts option was removed and now has no effect. Use"},{"line_number":10,"context_line":"    LivirtLogLevel to change log level in libvirt."}],"source_content_type":"text/x-yaml","patch_set":10,"id":"3fa7e38b_cfcf2754","line":9,"range":{"start_line":9,"start_character":18,"end_line":9,"end_character":19},"in_reply_to":"3fa7e38b_af3d0bc5","updated":"2019-09-24 17:00:25.000000000","message":"Done","commit_id":"81ace99e875d6c1e419f492f1872653d6287f234"}]}
