)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},"change_message_id":"6221874edc4982878e478884fbb6a9dc218a0886","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"nova-libvirt: fix kvm permission issue"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When deploy in AArch64, will meet kvm permission issue."},{"line_number":10,"context_line":"Fix it with Arch specified."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes-bug: #1715356"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7f515b1d_5d755c31","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":22},"updated":"2017-09-27 08:23:05.000000000","message":"Is this using CentOS or Ubuntu based containers?","commit_id":"80249dadba8ba6fa5531e03f3b4fcf6d30247e3e"},{"author":{"_account_id":22076,"name":"Kevin Zhao","email":"kevin.zhao@linaro.org","username":"KevinZhao"},"change_message_id":"dfe35a2a6aaafffa5458879eaefd33d19e3de783","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"nova-libvirt: fix kvm permission issue"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When deploy in AArch64, will meet kvm permission issue."},{"line_number":10,"context_line":"Fix it with Arch specified."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes-bug: #1715356"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"7f515b1d_098789b7","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":22},"in_reply_to":"7f515b1d_5d755c31","updated":"2017-09-28 01:37:56.000000000","message":"Yeah, we use the debian based container.","commit_id":"80249dadba8ba6fa5531e03f3b4fcf6d30247e3e"}],"docker/nova/nova-libvirt/extend_start.sh":[{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"dc42f9da4beb1b1a2675be999b3afa2b436b5874","unresolved":false,"context_lines":[{"line_number":5,"context_line":"# Do not remove unless CentOS has been validated"},{"line_number":6,"context_line":"if [[ -c /dev/kvm ]]; then"},{"line_number":7,"context_line":"    chmod 660 /dev/kvm"},{"line_number":8,"context_line":"    if [[ \"$(uname -m)\" \u003d\u003d \"aarch64\" ]];then"},{"line_number":9,"context_line":"        chown root:kvm /dev/kvm"},{"line_number":10,"context_line":"    else"},{"line_number":11,"context_line":"        chown root:qemu /dev/kvm"}],"source_content_type":"text/x-sh","patch_set":2,"id":"5f2577a7_08b248e7","line":8,"range":{"start_line":8,"start_character":39,"end_line":8,"end_character":41},"updated":"2017-09-12 07:20:09.000000000","message":"nit: add a space here","commit_id":"320a497eee951205369ed89e3a19774930c977ce"},{"author":{"_account_id":22076,"name":"Kevin Zhao","email":"kevin.zhao@linaro.org","username":"KevinZhao"},"change_message_id":"4e602c21f55fcae5bd91fb51625aadc48fa7a999","unresolved":false,"context_lines":[{"line_number":5,"context_line":"# Do not remove unless CentOS has been validated"},{"line_number":6,"context_line":"if [[ -c /dev/kvm ]]; then"},{"line_number":7,"context_line":"    chmod 660 /dev/kvm"},{"line_number":8,"context_line":"    if [[ \"$(uname -m)\" \u003d\u003d \"aarch64\" ]];then"},{"line_number":9,"context_line":"        chown root:kvm /dev/kvm"},{"line_number":10,"context_line":"    else"},{"line_number":11,"context_line":"        chown root:qemu /dev/kvm"}],"source_content_type":"text/x-sh","patch_set":2,"id":"7f515b1d_9727abb7","line":8,"range":{"start_line":8,"start_character":39,"end_line":8,"end_character":41},"in_reply_to":"5f2577a7_08b248e7","updated":"2017-09-22 04:52:01.000000000","message":"Done","commit_id":"320a497eee951205369ed89e3a19774930c977ce"},{"author":{"_account_id":22165,"name":"caoyuan","email":"cao.yingjunz@gmail.com","username":"caoyuan"},"change_message_id":"9705fb31bf1bb3ae5ad686776771ea1264c6e4d1","unresolved":false,"context_lines":[{"line_number":5,"context_line":"# Do not remove unless CentOS has been validated"},{"line_number":6,"context_line":"if [[ -c /dev/kvm ]]; then"},{"line_number":7,"context_line":"    chmod 660 /dev/kvm"},{"line_number":8,"context_line":"    if [[ \"$(uname -m)\" \u003d\u003d \"aarch64\" ]];then"},{"line_number":9,"context_line":"        chown root:kvm /dev/kvm"},{"line_number":10,"context_line":"    else"},{"line_number":11,"context_line":"        chown root:qemu /dev/kvm"}],"source_content_type":"text/x-sh","patch_set":2,"id":"ff5d4b2f_0a94b043","line":8,"range":{"start_line":8,"start_character":39,"end_line":8,"end_character":41},"in_reply_to":"5f2577a7_08b248e7","updated":"2017-09-19 12:09:05.000000000","message":"agree","commit_id":"320a497eee951205369ed89e3a19774930c977ce"},{"author":{"_account_id":22076,"name":"Kevin Zhao","email":"kevin.zhao@linaro.org","username":"KevinZhao"},"change_message_id":"4e602c21f55fcae5bd91fb51625aadc48fa7a999","unresolved":false,"context_lines":[{"line_number":5,"context_line":"# Do not remove unless CentOS has been validated"},{"line_number":6,"context_line":"if [[ -c /dev/kvm ]]; then"},{"line_number":7,"context_line":"    chmod 660 /dev/kvm"},{"line_number":8,"context_line":"    if [[ \"$(uname -m)\" \u003d\u003d \"aarch64\" ]];then"},{"line_number":9,"context_line":"        chown root:kvm /dev/kvm"},{"line_number":10,"context_line":"    else"},{"line_number":11,"context_line":"        chown root:qemu /dev/kvm"}],"source_content_type":"text/x-sh","patch_set":2,"id":"7f515b1d_b7186775","line":8,"range":{"start_line":8,"start_character":39,"end_line":8,"end_character":41},"in_reply_to":"ff5d4b2f_0a94b043","updated":"2017-09-22 04:52:01.000000000","message":"Done","commit_id":"320a497eee951205369ed89e3a19774930c977ce"},{"author":{"_account_id":23811,"name":"Oliver Walsh","email":"owalsh@redhat.com","username":"owalsh"},"change_message_id":"40a2ada1338ce2c626da7042dc099f9927c8a482","unresolved":false,"context_lines":[{"line_number":13,"context_line":"    if [[ \"$(uname -m)\" \u003d\u003d \"aarch64\" ]]; then"},{"line_number":14,"context_line":"        chown root:kvm /dev/kvm"},{"line_number":15,"context_line":"    else"},{"line_number":16,"context_line":"        chown root:qemu /dev/kvm"},{"line_number":17,"context_line":"    fi"},{"line_number":18,"context_line":"fi"},{"line_number":19,"context_line":""}],"source_content_type":"text/x-sh","patch_set":4,"id":"7f515b1d_06cd2b65","line":16,"range":{"start_line":16,"start_character":8,"end_line":16,"end_character":32},"updated":"2017-09-28 10:36:12.000000000","message":"The group doesn\u0027t really matter on CentOS since we have mode 666 but for the sake of consistency it should something like:\n\n    if [[ \"${KOLLA_BASE_DISTRO}\" \u003d~ debian|ubuntu ]]; then\n        chmod 660 /dev/kvm\n        if [[ \"$(uname -m)\" \u003d\u003d \"aarch64\" ]]; then\n            chown root:kvm /dev/kvm\n        else\n            chown root:qemu /dev/kvm\n        fi\n    else\n        chmod 666 /dev/kvm\n        chown root:kvm /dev/kvm\n    fi","commit_id":"80249dadba8ba6fa5531e03f3b4fcf6d30247e3e"}]}