)]}'
{"docker/nova/nova-base/nova_compute_sudoers":[{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"22cd3b5d93d8efa7c1dc83e81a600b7f73230b18","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"da6ed579_e7e8a7e4","line":2,"updated":"2016-01-13 18:42:00.000000000","message":"did you miss copying this file to the image like\n\nCOPY nova_sudoers /etc/sudoers.d/nova_sudoers","commit_id":"3c1215adf7efbff683f8addcef445934c0e0fb23"}],"docker/nova/nova-compute/extend_start.sh":[{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"f31cb84040cf3afd8406c5f7e16c7fac2a659c26","unresolved":false,"context_lines":[{"line_number":3,"context_line":"# Bootstrap and exit if KOLLA_BOOTSTRAP variable is set. This catches all cases"},{"line_number":4,"context_line":"# of the KOLLA_BOOTSTRAP variable being set, including empty."},{"line_number":5,"context_line":"if [[ \"${!KOLLA_BOOTSTRAP[@]}\" ]]; then"},{"line_number":6,"context_line":"    sudo chown -R nova: /var/lib/nova/instances/"},{"line_number":7,"context_line":"    exit 0"},{"line_number":8,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":1,"id":"da6ed579_aa94b9e1","line":6,"updated":"2016-01-13 17:44:15.000000000","message":"I do not think nova is a sudo user, now.","commit_id":"3c1215adf7efbff683f8addcef445934c0e0fb23"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"862c4160a2bc348ec578ce756d9c83919a71199c","unresolved":false,"context_lines":[{"line_number":3,"context_line":"# Bootstrap and exit if KOLLA_BOOTSTRAP variable is set. This catches all cases"},{"line_number":4,"context_line":"# of the KOLLA_BOOTSTRAP variable being set, including empty."},{"line_number":5,"context_line":"if [[ \"${!KOLLA_BOOTSTRAP[@]}\" ]]; then"},{"line_number":6,"context_line":"    sudo chown -R nova: /var/lib/nova/instances/"},{"line_number":7,"context_line":"    exit 0"},{"line_number":8,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":1,"id":"da6ed579_f57db61c","line":6,"in_reply_to":"da6ed579_aa94b9e1","updated":"2016-01-13 17:52:10.000000000","message":"Does the nova_compute_sudoers file add nova as sudo user?","commit_id":"3c1215adf7efbff683f8addcef445934c0e0fb23"},{"author":{"_account_id":14119,"name":"Sam Yaple","email":"sam+git@yaple.net","username":"SamYaple"},"change_message_id":"a6c03cb7d88bcc3b1b9fed30beee0c4b8f79f852","unresolved":false,"context_lines":[{"line_number":3,"context_line":"# Bootstrap and exit if KOLLA_BOOTSTRAP variable is set. This catches all cases"},{"line_number":4,"context_line":"# of the KOLLA_BOOTSTRAP variable being set, including empty."},{"line_number":5,"context_line":"if [[ \"${!KOLLA_BOOTSTRAP[@]}\" ]]; then"},{"line_number":6,"context_line":"    sudo chown -R nova: /var/lib/nova/instances/"},{"line_number":7,"context_line":"    exit 0"},{"line_number":8,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":2,"id":"da6ed579_47165356","line":6,"updated":"2016-01-13 18:56:54.000000000","message":"this should not be recursive.\n\nAdditionally I liked Hui\u0027s [1] changing of the volume to include /var/lib/nova since there are other things we need from that folder as well.\n\nIf you chown /var/lib/nova with the proper volume you only need to do it once.\n\n[1] https://review.openstack.org/#/c/266575/","commit_id":"fcb7965fa3a8cf97cd436988643851f1ed010719"}],"docker/nova/nova-libvirt/extend_start.sh":[{"author":{"_account_id":14119,"name":"Sam Yaple","email":"sam+git@yaple.net","username":"SamYaple"},"change_message_id":"a6c03cb7d88bcc3b1b9fed30beee0c4b8f79f852","unresolved":false,"context_lines":[{"line_number":11,"context_line":"# Bootstrap and exit if KOLLA_BOOTSTRAP variable is set. This catches all cases"},{"line_number":12,"context_line":"# of the KOLLA_BOOTSTRAP variable being set, including empty."},{"line_number":13,"context_line":"if [[ \"${!KOLLA_BOOTSTRAP[@]}\" ]]; then"},{"line_number":14,"context_line":"    sudo chown -R nova: /var/lib/nova/instances/"},{"line_number":15,"context_line":"    exit 0"},{"line_number":16,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":2,"id":"da6ed579_1a21929f","line":14,"updated":"2016-01-13 18:56:54.000000000","message":"this is not needed and is potentially dangerous to do. if the nova user is different in one containre than the other then youll be changing uids back and forth.","commit_id":"fcb7965fa3a8cf97cd436988643851f1ed010719"}]}